tests: remove dependency of internal repo from integration suite
Metadata
Current evaluation
Merged. Removed the internal repository dependency from the integration test suite, closing issue #1973. Approved by one reviewer, the change modified four files with 66 additions and 10 deletions.
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`?
-----
closes #1973
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Removed the internal repository dependency from the integration test suite, closing issue #1973. Approved by one reviewer, the change modified four files with 66 additions and 10 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Removes the internal repository dependency from the integration test suite, resolving issue #1973. This update streamlines the testing workflow by decoupling integration tests from internal project builds. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to remove the internal repository dependency from the integration test suite, resolving issue #1973. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1965 tests: remove ProjectOptions dependency from integration suite | snapcraft | merged | Merged changes removing the ProjectOptions dependency from the integration test suite, closing issue #1953. Approved by one reviewer, passed CI, and modified 12 files (+90/-33 lines). | |
| #1972 tests: remove _options dependency from integration suite | snapcraft | merged | Merged, removing the _options dependency from the integration test suite and closing issue #1953. The commit initially omitted the Closes: keyword but was approved and merged. | |
| #1981 tests: remove the internal os_release dependency | snapcraft | merged | Merged after approval. Removes the internal os_release dependency from tests, closing issue #1952. Two files changed with 52 additions and 10 deletions. | |
| #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. | |
| #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. | |
| #488 Remove the integration tests coverage. | snapcraft | merged | Merged changes removing integration tests coverage, resolving LP #1575383. The update modifies four files, deleting 62 lines and adding 3, with no review comments or CI checks required. | |
| #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. | |
| #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. | |
| #1266 tests(spread): requirements dependencies | charmcraft | merged | Merged following approval by two reviewers. Introduces regression tests for requirements file and dependency combinations, resolving issue #1179. The change modifies five files, adding 149 lines and removing one. |