[refactor] Separate unit and integration tests
Metadata
Current evaluation
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.
Suggested action: keep open
Reason: The issue is a maintainer-filed refactor with no labels, no comments, and no assignee, showing no sign of triage or categorization. While the current repo layout (tests/unit and tests/integration both populated) suggests the separation goal has largely been achieved, I could not confirm a specific PR/commit that closed it, so it should be triaged to verify whether the remaining work is done or if it can be closed as completed.
Impact:
25
Quick Win:
15.0
Staleness:
70
Complexity:
40
Confidence:
60
Support Request:
0
Issue body
### What needs to get done
The tests directory is mostly a single thing, with only some new craft-application stuff separated. This should be a straightforward refactor. Any updates to the tests (parametrizing, etc.) is explicitly out of scope.
### Why it needs to get done
A more organized test structure will help use more quickly iterate.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.8-27b |
Impact:
25
Quick Win:
15.0
Staleness:
70
Complexity:
40
Confidence:
60
Support Request:
0
|
keep 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. | |
| qwen/qwen3.6-35b-a3b |
Staleness:
95
Complexity:
10
Confidence:
85
Support Request:
0
|
close stale | Maintainer-requested refactor to separate unit and integration tests for better organization. Unassigned, zero comments, and inactive for over 2.5 years. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
80
Complexity:
15
Confidence:
80
Support Request:
5
|
needs triage | Maintainer-requested refactor to separate unit and integration tests into distinct directories. Currently unlabelled, inactive for over ten months, and has received zero comments or maintainer assessment. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
98
Complexity:
15
Confidence:
85
Support Request:
5
|
close stale | Refactor to separate unit and integration tests into distinct directories. Open for over 2.5 years with no maintainer comments or labels, indicating it remains untriaged and inactive. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
85
Complexity:
20
Confidence:
75
Support Request:
5
|
close stale | Separate unit and integration tests to improve project organization and iteration speed. Currently unassigned, lacks labels or comments, and has been inactive for over a year. Requires triage and assignment to proceed. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #7 tools: Split unit and integration tests | starbase | merged | 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. | |
| #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%. | |
| #275 Refactor integration tests to split data and code | craft-parts | open | Maintainer-proposed refactor of tests/integration to move inline temp-file creation into test-data directories with a datadir-like fixture. Open since 2021, last human comment 2022, no implementation landed; tests still create fixtures inline. | |
| #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. |