tools: Split unit and integration tests
Metadata
Current evaluation
Merged changes splitting unit and integration tests into separate directories. Approved by one reviewer. The update modifies six files, adding 33 net lines. The integration folder was created solely for structural separation.
Suggested action: —
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes splitting unit and integration tests into separate directories. Approved by one reviewer. The update modifies six files, adding 33 net lines. The integration folder was created solely for structural separation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged test reorganization that separates unit and integration tests into distinct directories. The author acknowledged the integration test was a placeholder to avoid modifying the main module. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1282 [refactor] Separate unit and integration tests | charmcraft | open | Refactor to separate unit and integration tests in the tests directory. The repo now has distinct tests/unit (60 files) and tests/integration (107 files) directories, indicating the separation described in the issue has been achieved. | |
| #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. | |
| #948 Use testtools as the base of all unit tests | snapcraft | merged | Merged changes to adopt testtools as the base framework for all unit tests. Approved by a maintainer, the update modified 50 files, reduced code by 236 lines, and maintained 97% coverage. | |
| #131 build: separate unit and integration tests in CI | craft-application | merged | Merged into main. Separates unit and integration tests in CI to improve readability and fix log rotation conflicts. Scope was narrowed from parallel execution to simply splitting test categories. Approved and passed checks. | |
| #107 ci: separate unit and integration tests | starbase | merged | 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%. | |
| #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. | |
| #172 tools: update external testing tools | rockcraft | merged | Merged after approval by two reviewers. The update refreshes the external testing tools submodule, modifying two files with a minimal diff of two additions and one deletion. No unresolved comments or CI issues were present. | |
| #1716 tests: split the integration autopkgtests | snapcraft | merged | Merged after splitting integration autopkgtests to eliminate timeout failures. The change adds minor per-test setup overhead but resolves recurring execution timeouts. Approved by two reviewers with passing CI checks. | |
| #1638 tests: reorganize unit and integration suites to make them easier to split for travis | snapcraft | merged | 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. | |
| #1464 ci: split spread tests by system | charmcraft | closed | Merged after approval to split spread tests by system, reducing repetition and failure rates. The change modified two files with a minimal diff. |