tests: move test data to a common location
Metadata
Current evaluation
Merged following reviewer approval and passing CI. Relocates test data to a common directory, resolving issue #39. Modifies 13 files with zero coverage impact.
Suggested action: —
No scores available.
Issue body
fixes #39
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged following reviewer approval and passing CI. Relocates test data to a common directory, resolving issue #39. Modifies 13 files with zero coverage impact. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main, resolving issue #39 by relocating test data to a shared directory. Code coverage remained stable at 88.07%. All CI checks passed, successfully centralizing test data. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #278 tests: additional tests to improve coverage | fetch-service | merged | Merged after two reviews and passing CI checks. Adds additional tests across ten files to improve code coverage, changing +386/-60 lines. | |
| #461 test: improve test coverage | fetch-service | merged | Merged to improve test coverage, adding 270 lines across three files. Approved by two reviewers with all CI checks passing. | |
| #19 tests: update test environments for coverage | craft-archives | merged | Merged into main after approval. Updated test environments across four files, removing 55 lines and adding 3. Overall coverage decreased by 0.27%, though the diff achieved 100% coverage. | |
| #458 test: improve test coverage | fetch-service | merged | Merged after approval from two reviewers and passing CI checks. Added 370 lines of test code across five files to improve overall test coverage. | |
| #1787 tests: move refresh to unit folder | snapcraft | merged | Merged a fix moving a misplaced refresh test into the unit folder. The test was previously excluded from regular runs due to the new directory structure, causing false passes. Approved by one reviewer, passed CI, and merged. | |
| #896 test: don't leave behind test files | rockcraft | merged | Merged a fix preventing OCI tests from leaving residual files in the project root. The change redirects test output to a temporary directory. Approved by three reviewers with all CI checks passing. | |
| #1928 tests: remove duplicate tests | snapcraft | merged | Merged changes that remove duplicate tests and clarify test base class names to prevent future duplication. Approved by two reviewers, passed CI, and reduced unit test execution time. | |
| #5375 test: fix command tests | snapcraft | merged | Merged after updating test fixtures to resolve failing command unit tests. Approved by two reviewers, passed CI checks, and modified two files with 17 additions and 9 deletions. | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #3719 tests: fix lifecycle tests to run on new directory | snapcraft | merged | Merged a fix for lifecycle tests to run in a new directory instead of the current one. Approved by a reviewer, passed CI, and changed three lines. |