← Back to issue list

tools: Split unit and integration tests

View original Github issue

Metadata

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

Current evaluation

Merged #7 as commit 63fc646, splitting tool tests into separate unit and integration directories.

Resolution: closed resolved

Reason: PR #7 was approved and merged as commit 63fc646. The author clarified that the integration directory was for structural separation, despite the test not being a true integration test.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #7 as commit 63fc646, splitting tool tests into separate unit and integration directories.
stealth/space-bunny-alpha — closed resolved Merged in #7, splitting tools tests into separate unit and integration directories. The maintainer clarified that the integration test was only categorized for structural separation.
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
#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.
75%
#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%