jblu/qbit-maid
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 6 Wiki Activity

test: audit suite for meaningfulness, reach 100% coverage #66

Merged
trex Bot merged 2 commits from dev-test-suite-audit into main 2026-11-03 16:47:04 -06:00
Conversation 0 Commits 2 Files Changed 10
+1647 -461
2 Commits
Author SHA1 Message Date
trex Bot ab46ea1fb2 ci: fix unit-tests job on the Debian-packaged runner pip
ci / docker (push) Successful in 23s
Details
test / unit-tests (pull_request) Successful in 5s
Details
test / komodo-deploy-script (pull_request) Successful in 16s
Details
test / unit-tests (push) Successful in 6s
Details
test / komodo-deploy-script (push) Successful in 17s
Details 
The runner image ships pip 24.0 from the Debian package, so
`pip install --upgrade pip` aborts with "Cannot uninstall pip 24.0,
RECORD file not found" and the job failed after 4s. Drop the upgrade
and install with --break-system-packages.
 2026-11-03 15:35:39 -06:00
trex Bot 925f9619be test: audit suite for meaningfulness, reach 100% coverage
test / unit-tests (push) Failing after 4s
Details
test / komodo-deploy-script (push) Successful in 16s
Details
test / unit-tests (pull_request) Failing after 3s
Details
test / komodo-deploy-script (pull_request) Successful in 16s
Details
ci / docker (push) Successful in 1m37s
Details 
The suite had 8 empty `pass` tests, two tests that made live network
calls to a production healthcheck endpoint, and a write_csv test that
only asserted the file existed. CI ran only test_komodo_deploy, so the
main suite never executed at all.

- Split tests by module: test_qlist, test_qprocess, test_qlogging_extra,
  test_apprise, test_qbt; rewrote test_qbitmaid and test_write_csv.
- qbt_fakes.py: shared fakes (FakeQbt, FakeQbtClient, FakeAPIError,
  StubPushover, FailingPushover, torrent factory).
- test_qbt.py loads the hyphenated qbit-maid.py via importlib and covers
  the __main__ entry point.
- Replaced the live-network tests with a RecordingRequests stub
  (suite time 14s -> 0.035s).
- .coveragerc + CI unit-tests job running the full suite with
  --fail-under=100.

127 unit tests, 100% statement coverage on all 5 first-party modules
(440 stmts), 15/15 mutations caught.

Seven pre-existing bugs are pinned by tests with KNOWN BUG docstrings
and left unfixed pending a decision.
 2026-11-03 15:34:57 -06:00