tests: fix lifecycle tests to run on new directory
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Fix lifecycle tests to not run on the current directory.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into snapcraft/7.0, updating lifecycle tests to execute in a new directory rather than the current one. Accepted after maintainer review despite a minor 0.03% coverage decrease. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into snapcraft/7.0. Fixed lifecycle tests to run in a new directory rather than the current one. The merge resulted in a 0.03% coverage decrease. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2163 tests: add lifecycle ordering tests | snapcraft | merged | Merged changes adding lifecycle ordering tests and refactoring them into a dedicated package. Approved by a reviewer, passed CI checks, and increased code coverage by 0.02%. | |
| #4452 test(spread): check for new lifecycle message | snapcraft | merged | Merged a test update to verify a new lifecycle message, resolving a regression from #4431. Approved by two reviewers with passing CI and stable coverage. The fix unblocks PR #4450. | |
| #3305 unit tests: fix runtests.sh not filtering tests when passed a subdirectory | snapcraft | merged | Merged a one-line fix correcting runtests.sh to properly filter tests when a subdirectory is passed. Previously, the script ran the entire suite. Approved by two reviewers, passed CI, and merged. | |
| #4584 lifecycle commands | snapcraft | merged | Merged implementation of lifecycle commands. Approved by two reviewers, passed all CI checks, and modified 15 files with full test coverage. | |
| #766 fix(lifecycle): don't rebuild on test file changes | craft-application | merged | Merged fix that adds spread files to the lifecycle manager's ignore list, preventing unnecessary repulls when test files change. Approved by two reviewers, passed all CI checks, and resolves issue #756. | |
| #3024 tests: remove usage of FakeApt fixtures in lifecycle | snapcraft | merged | Merged changes replacing FakeApt fixtures in lifecycle tests with targeted mock.patch calls and related cleanup. Approved by one reviewer and passed CI checks. | |
| #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. | |
| #2637 unit tests: make lifecycle tests more robust | snapcraft | merged | Merged updates to improve lifecycle unit test robustness. Replaced strict message assertions with flexible checks to prevent fake webserver output from causing false failures. Approved and passed CI. | |
| #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. | |
| #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. |