tests: reenable the cleanbuild integration test
Metadata
Current evaluation
Merged the cleanbuild integration test reenablement. Approved by one reviewer with zero unresolved comments. Modified seven files, adding 69 lines and removing 15.
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 unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged the cleanbuild integration test reenablement. Approved by one reviewer with zero unresolved comments. Modified seven files, adding 69 lines and removing 15. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the pull request to re-enable the cleanbuild integration test, restoring it to the continuous integration pipeline. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to re-enable the cleanbuild integration test, restoring it to the CI pipeline for ongoing validation of clean build processes. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1387 [WIP] tests: reenable the cleanbuild integration test | snapcraft | closed | Closed as stale after remaining work-in-progress with unresolved review comments and CI failures. The cleanbuild integration test reenablement was abandoned and never merged. | |
| #1245 tests: add parallel build test | charmcraft | merged | Merged after three approvals. Added a parallel build test to resolve issue #1138, modifying three files with 44 additions and 2 deletions. All checks passed and the change was integrated. | |
| #576 Allow to run a subset of integration tests. | snapcraft | merged | Merged to allow running a subset of integration tests. The change updates four files with minor adjustments (+7/-2 lines). The author requested retests before final merge approval. | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration. | |
| #2474 build providers: remove dead code | snapcraft | merged | Merged removal of dead code for legacy cleanbuild support in build providers. Approved by one reviewer, passed CI, and modified three files (+54/-165 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. | |
| #755 Enable the static checks for the integration tests | snapcraft | merged | Merged changes enabling static checks for integration tests across 15 files (+103/-84 lines). The update was integrated without reviewer comments or CI checks. | |
| #1290 tests: refactor tests with build and stage packages | snapcraft | merged | Merged after approval. Refactored tests across 12 files to use build and stage packages, changing 115 lines added and 110 removed. Code coverage remained unchanged at 95.27% upon integration into master. | |
| #863 Fix 1.25 build | craft-parts | merged | Merged to resolve 1.25 build failures. Approved by two reviewers with zero unresolved comments. The commit modified 11 files, adding 17 and removing 16 lines. | |
| #82 test: improve integration testing | craft-store | merged | Merged improvements to integration testing across eight files. The update works around external fork credential limitations by running full tests only after merging to main. Approved by one reviewer and passed CI. |