integration tests: disable tests using snaps in bionic container
Metadata
Current evaluation
Merged changes disabling integration tests using snaps in bionic containers to address reported failures. The +7/-1 line update across two files passed CI and received reviewer approval.
Suggested action: —
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 tests/unit`?
-----
See https://forum.snapcraft.io/t/7339 for more information.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes disabling integration tests using snaps in bionic containers to address reported failures. The +7/-1 line update across two files passed CI and received reviewer approval. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Disables integration tests that rely on snaps in bionic containers to resolve compatibility issues. Minor code review feedback regarding setUp decoration was provided but did not block the merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Disables integration tests using snaps in bionic containers to address reported failures. Despite a Codecov warning and a reviewer comment on setUp decoration, the changes were accepted and merged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2353 tests: remove dependency on snapcraft for integration tests | snapcraft | merged | Merged after reviewer approval and passing CI. The change removes the snapcraft dependency from integration tests to improve test independence, modifying 27 files with 956 additions and 733 deletions. | |
| #2014 integration tests: snap tests shouldn't be arch-specific | snapcraft | merged | Merged a change removing architecture-specific constraints from snap integration tests. Approved by one reviewer with a two-line modification across a single file. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | Merged integration tests for build snaps. Approved by two reviewers and passed CI. Two test scenarios omitted due to known bugs. | |
| #307 tests: disable broken integration test | craft-application | merged | Merged a temporary workaround disabling a broken integration test to restore CI functionality, pending a permanent fix in issue #306. Approved by two reviewers with passing checks. | |
| #990 tests: fix snaps tests in armhf | snapcraft | merged | Merged into master to fix failing snap tests on the armhf architecture. Approved by a reviewer, the change updated 11 files and resolved CI failures without altering overall code coverage. |