tests: reorganize unit and integration suites to make them easier to split for travis
Metadata
Current evaluation
Merged #1638 as commit 30f62685c, reorganizing unit and integration test suites for easier splitting across Travis CI jobs.
Resolution: closed resolved
Reason: PR #1638 was approved and merged into the main branch as commit 30f62685c, delivering the planned test-suite reorganization.
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`?
-----
requires #1630
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #1638 as commit 30f62685c, reorganizing unit and integration test suites for easier splitting across Travis CI jobs. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #1638 (commit 30f62685c), reorganizing unit and integration test suites to enable easier Travis CI splitting. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged following two approvals. Reorganized unit and integration test suites to facilitate splitting for Travis CI. A transient CI network error was observed but did not prevent the merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged reorganization of unit and integration test suites to facilitate splitting for Travis CI. A transient DNS timeout triggered a single CI failure on xenial-armhf, but the changes were accepted and merged. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#1630
(confidence 100%)
Required prerequisite that added suite selection to autopkgtest runs.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1252 ci: split plugin integration tests out. | snapcraft | merged | Merged PR #1252 in commit 1387c0aa9, splitting plugin integration tests into separate Travis CI runs to stay within time limits. | |
| #7 tools: Split unit and integration tests | starbase | merged | Merged #7 as commit 63fc646, splitting tool tests into separate unit and integration directories. | |
| #1318 tests: refactor the travis jobs using stages | snapcraft | merged | Merged PR #1318 (commit 59902915d) by come-maiz reorganized Travis CI jobs into stages, touching 7 files (+201/-40) to streamline the test pipeline; approved by 2 reviewers. | |
| #2217 Travis test | snapcraft | merged | Merged in snapcraft#2217 via commit 281defaef, improving Travis test speed and simplifying CI across 26 files. | |
| #2301 tests: move most tests to spread and reorder travis.yaml | snapcraft | merged | Merged PR #2301 moved most Python tests into spread, retained the store test, and reordered Travis jobs to improve test speed (commit 88005edf4). | |
| #1015 tests: reorganize command tests into subdirectory. | snapcraft | merged | Merged PR #1015 as commit 90b63c94, moving command unit tests into a tests/unit/commands/ subdirectory (30 files, +70/-49) to shrink the unit suite and close LP #1651908. | |
| #2614 [WIP] travis-ci: split spread tests to reduce job times | snapcraft | closed | Closed unmerged in favor of snapcraft#2615, which increased Spread worker parallelism and merged as commit 5c91f9681 to reduce job times. | |
| #107 ci: separate unit and integration tests | starbase | merged | Merged PR #107 as commit 3244cc2, splitting the GitHub Actions test workflow into separate unit and integration test jobs; repo now has tests/unit and tests/integration trees plus a test-integration tox env. | |
| #2523 ci: improve travis integration conditionals | snapcraft | merged | Merged as commit 4900f7323: .travis.yml spread/integration test conditionals fixed, unused cron trigger removed, integration script indentation normalized. | |
| #1282 [refactor] Separate unit and integration tests | charmcraft | open | Unit and integration test separation is already implemented, with dedicated tests/unit and tests/integration directories; the craft-application rebase appears to have delivered it. |