tests: add SNAPCRAFT_KEEP_DATA to debug
Metadata
Current evaluation
Merged PR #1996 added SNAPCRAFT_KEEP_DATA_PATH support to retain integration-test data for debugging; landed as commit 0024c35ff.
Resolution: closed resolved
Reason: PR #1996 merged with two approvals. Commit 0024c35ff added the integration-test data preservation path for debugging.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Farewell gift for @sergiusens .
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1996 added SNAPCRAFT_KEEP_DATA_PATH support to retain integration-test data for debugging; landed as commit 0024c35ff. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #1996 was merged with two approvals, adding SNAPCRAFT_KEEP_DATA to preserve test data for debugging; the change updated two files (+14/-2). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged test updates adding the SNAPCRAFT_KEEP_DATA environment variable for debugging. Approved by two reviewers, the change modifies two files with a net addition of 14 lines. Submitted as a farewell gift to @sergiusens. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged adding the SNAPCRAFT_KEEP_DATA environment variable to tests for debugging. Integrated directly without comments or revisions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2130 tests: remove obsolete env var | snapcraft | merged | Merged PR #2130 as commit 5305ac01c, removing obsolete SNAPCRAFT_SEND_ERROR_DATA test configuration after the related error-data logic was removed. | |
| #2793 tests: enable SNAPCRAFT_BUILD_INFO for spread | snapcraft | merged | Merged as commit 12370aae1, enabling SNAPCRAFT_BUILD_INFO across Spread tests to exercise manifest generation and updating nil-basic for the resulting manifest. | |
| #2295 tests: use SNAPCRAFT_PACKAGE_TYPE everywhere | snapcraft | merged | PR #2295 merged as commit ffbad69ea: unit tests now use SNAPCRAFT_PACKAGE_TYPE instead of legacy SNAPCRAFT_FROM_* env vars, and unused container tests were removed to ease migration to spread. (LP: #1794379) | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. |