[GH-ISSUE #89] Add unit tests for amazon_q, continue_dev, and copilot_cli adapters #82

New issue

Closed

opened 2026-08-13 19:11:09 +02:00 by zaph0d · 1 comment

zaph0d commented

2026-08-13 19:11:09 +02:00

Owner

Originally created by @niels-emmer on GitHub (Aug 13, 2026).
Original GitHub issue: https://github.com/niels-emmer/myace/issues/89

Every other adapter in `backend/app/adapters/` has a dedicated `TestXAdapter` class in `backend/tests/test_adapters.py` (see `TestClaudeCodeAdapter`, `TestCodexCliAdapter`, `TestWindsurfAdapter` for the pattern) — but three shipped adapters currently have zero test coverage:

`amazon_q.py`
`continue_dev.py`
`copilot_cli.py`

Task: add a `TestXAdapter` class for each, following the existing pattern — feed a small set of `CanonicalArtifact`s through `translate()` and assert on the returned `{filename: content}` dict (file paths, frontmatter/structure, and any target-specific quirks the adapter handles).

Good first issue: no new abstractions needed, just following an established pattern three times. See `docs/extending.md` for adapter/testing conventions.

Originally created by @niels-emmer on GitHub (Aug 13, 2026). Original GitHub issue: https://github.com/niels-emmer/myace/issues/89 Every other adapter in \`backend/app/adapters/\` has a dedicated \`TestXAdapter\` class in \`backend/tests/test_adapters.py\` (see \`TestClaudeCodeAdapter\`, \`TestCodexCliAdapter\`, \`TestWindsurfAdapter\` for the pattern) — but three shipped adapters currently have zero test coverage: - \`amazon_q.py\` - \`continue_dev.py\` - \`copilot_cli.py\` **Task:** add a \`TestXAdapter\` class for each, following the existing pattern — feed a small set of \`CanonicalArtifact\`s through \`translate()\` and assert on the returned \`{filename: content}\` dict (file paths, frontmatter/structure, and any target-specific quirks the adapter handles). Good first issue: no new abstractions needed, just following an established pattern three times. See \`docs/extending.md\` for adapter/testing conventions.

zaph0d added the

good first issue

label

2026-08-13 19:11:09 +02:00

zaph0d referenced this issue

2026-08-13 19:11:27 +02:00

[PR #82] [MERGED] feat: add automated Postgres backup sidecar with daily pg_dump and retention #84

zaph0d closed this issue

2026-08-18 07:30:43 +02:00

zaph0d commented

2026-08-18 07:30:43 +02:00

Author

Owner

@niels-emmer commented on GitHub (Aug 17, 2026):

Checked before starting work on this: `TestAmazonQAdapter`, `TestContinueAdapter`, and `TestCopilotCliAdapter` already exist in `backend/tests/test_adapters.py`, added in #22 (2026-08-09, Copilot CLI) and #74 (2026-08-12, Continue + Amazon Q) — both merged before this issue was filed (2026-08-13). All three cover the pattern this issue asks for (translate() output per artifact type, file paths, frontmatter/structure, target-specific quirks like Amazon Q's no-frontmatter rule files).

Ran the suite to confirm: 50/50 adapter tests pass on `main` as of this comment, including all three classes.

Closing as already resolved — no code change needed.

@niels-emmer commented on GitHub (Aug 17, 2026): Checked before starting work on this: \`TestAmazonQAdapter\`, \`TestContinueAdapter\`, and \`TestCopilotCliAdapter\` already exist in \`backend/tests/test_adapters.py\`, added in #22 (2026-08-09, Copilot CLI) and #74 (2026-08-12, Continue + Amazon Q) — both merged before this issue was filed (2026-08-13). All three cover the pattern this issue asks for (translate() output per artifact type, file paths, frontmatter/structure, target-specific quirks like Amazon Q's no-frontmatter rule files). Ran the suite to confirm: 50/50 adapter tests pass on \`main\` as of this comment, including all three classes. Closing as already resolved — no code change needed.