← Back to issue list

ci: separate unit and integration tests

View original Github issue

Metadata

Project
starbase
Number
#107
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

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.

Resolution: closed resolved

Reason: The PR was approved and merged into main as commit 3244cc2 ("ci: separate unit and integration tests (#107)"), confirmed in the commit history of .github/workflows/tests.yaml with a +46/-7 diff across 2 files.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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.
stealth/space-bunny-alpha — closed resolved Merged #107 via commit 3244cc2, separating unit and integration test CI jobs in two files (+46/-7). Codecov reported coverage remained at 100%.
qwen/qwen3.6-35b-a3b — — Merged into main after approval and passing CI checks. The PR separated unit and integration tests in the CI pipeline, modifying two files (+46/-7 lines). Code coverage remained unchanged at 100%.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main. Separates unit and integration tests in the CI configuration. Codecov confirms zero coverage impact upon integration.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/starbase#7 (confidence 72%)

    Prior commit 63fc646 'tools: Split unit and integration tests (#7)' laid the groundwork this CI PR built on.

  • Superseded By: canonical/starbase#272 (confidence 65%)

    Commit a411e24 'ci: use starflow CI actions (#272)' later replaced this hand-written workflow.

  • Related To: canonical/starbase#310 (confidence 55%)

    Commit 6f78e1a 'feat: remove tox.ini (#310)' eventually removed the tox config this PR's test environments relied on.

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.
71%
#1638 tests: reorganize unit and integration suites to make them easier to split for travis snapcraft merged Merged #1638 as commit 30f62685c, reorganizing unit and integration test suites for easier splitting across Travis CI jobs.
71%
#3373 ci: migrate static & unit tests from travis to github actions snapcraft merged Merged PR #3373 as e9d0d1789, moving Snapcraft static and unit-test CI from Travis CI to GitHub Actions after prerequisite #3372.
71%
#267 ci: reorganize github CI tests craft-providers merged Merged PR #267 reorganized CI into lint, unit, and integration jobs, expanded OS/Python unit coverage, and added a macOS workaround in commit e77f54e.
70%
#4005 Testing: Lint and test using Tox snapcraft merged Merged PR #4005 (4526909), adding Tox-based linting and unit-test GitHub Actions while retaining legacy checks for comparison.
70%