tests: do not rely on project_dir
Metadata
Current evaluation
Merged to master, refactoring tests to remove dependency on project_dir. Approved despite 56 unresolved review comments and acknowledged minor errors. Code coverage unchanged. Resolves LP #1659356.
Suggested action: —
No scores available.
Issue body
LP: [#1659356](https://bugs.launchpad.net/snapcraft/+bug/1659356)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to master, refactoring tests to remove dependency on project_dir. Approved despite 56 unresolved review comments and acknowledged minor errors. Code coverage unchanged. Resolves LP #1659356. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Tests updated to remove reliance on project_dir, addressing Launchpad bug #1659356. Merged despite minor potential errors due to time constraints, with code coverage remaining stable at 96.38%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Tests refactored to remove project_dir dependency, resolving Launchpad bug #1659356. Merged with unchanged code coverage, despite author noting potential minor errors. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1027 tests: fix broken unit test in master | snapcraft | merged | Merged a one-line fix resolving a broken unit test in master. Approved by one reviewer with no unresolved comments. Changes passed CI and maintained 96.37% code coverage. Integrated directly into the main branch. | |
| #1015 tests: reorganize command tests into subdirectory. | snapcraft | merged | Reorganized command tests into a subdirectory to improve manageability. Merged following approval and CI checks, resolving Launchpad bug #1651908 and slightly increasing code coverage. | |
| #950 tests: Use a more stable ftp source | snapcraft | merged | Merged into master after reviewer approval and passing code coverage checks. Updates test configuration to use a more stable FTP source, resolving LP: #1647751 with a minimal two-line change. | |
| #60 tests: move test data to a common location | craft-archives | merged | Merged following reviewer approval and passing CI. Relocates test data to a common directory, resolving issue #39. Modifies 13 files with zero coverage impact. | |
| #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. | |
| #4660 refactor(test): versioned remote-build tests | snapcraft | merged | Merged test refactoring for versioned remote-build tests with no logic changes. Approved by two reviewers, zero unresolved comments, and passing CI. Codecov confirmed full coverage on modified lines. Successfully integrated. | |
| #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. | |
| #1082 project: new plugin directory location | snapcraft | merged | Merged into master after maintainer approval. The change relocates the plugin directory across 12 files, adding 74 lines and removing 12. Code coverage impact is negligible, decreasing by less than 0.01%. Resolves LP #1659085. |