← Back to issue list

Replace subTests with TestScenarios

View original Github issue

Metadata

Project
snapcraft
Number
#943
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

Merged #943, replacing subTests with testtools TestScenarios across 12 test files as the first step toward the LP: #1593079 migration.

Resolution: closed resolved

Reason: PR #943 was merged as commit 9574cbb3f, updating the tests to use TestScenarios and advance the switch to testtools.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This is the first step to switch to testtools, for LP: #1593079

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #943, replacing subTests with testtools TestScenarios across 12 test files as the first step toward the LP: #1593079 migration.
stealth/space-bunny-alpha — closed resolved PR #943 merged as commit 9574cbb3f ("tests: replace subTests with TestScenarios"), converting ~1100 lines across 12 test files from unittest subTests to testtools scenarios as a first step toward testtools (LP: #1593079).
stealth/space-bunny-alpha — closed resolved Merged PR #943 (commit 9574cbb3f) replaced subTests with testtools TestScenarios across 12 test files (+1111/-1064), the first step of the switch to testtools for LP: #1593079; follow-up #948 completed it.
stealth/space-bunny-alpha — closed resolved Merged as commit 9574cbb3f (e3a71c5..5464f38): replaced stdlib subTests with testtools TestScenarios across 12 test files, the first step toward testtools (LP: #1593079).
stealth/space-bunny-alpha — closed resolved Merged PR #943, replacing subTests with testtools TestScenarios across 12 test files to advance LP #1593079.
qwen/qwen3.6-35b-a3b — — Merged following reviewer approval. Replaced subTests with TestScenarios across 12 files to initiate migration to testtools. Changes totaled +1111/-1064 lines while maintaining 97.09% test coverage.
qwen3.6-35b-a3b-mtp-q6 — — Merged the update replacing subTests with TestScenarios as the initial step toward migrating to testtools. Code coverage held at 97.09%. The change was integrated into the main branch by maintainer come-maiz.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#948 Use testtools as the base of all unit tests snapcraft merged PR #948 merged as commit 2eaf421a7, converting all Snapcraft unit tests to base on testtools (LP #1593079); ~50 files changed, coverage held at ~97%. PR #943 (subTests -> TestScenarios) preceded it.
75%