tests: remove duplicated unit tests
Metadata
Current evaluation
Merged removal of duplicated unit tests from test_provider.py and test_providers.py, artifacts from a prior LXD and Multipass test consolidation. Approved, passed CI, and reduced two files by 60 lines.
Suggested action: —
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
`test_provider.py` and `test_providers.py` had duplicated tests. This was an artifact of commit c7a49bac29, which combined the same tests from `test_lxd.py` and `test_multipass.py`.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged removal of duplicated unit tests from test_provider.py and test_providers.py, artifacts from a prior LXD and Multipass test consolidation. Approved, passed CI, and reduced two files by 60 lines. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged removal of duplicated unit tests from test_provider.py and test_providers.py. The duplication originated from a prior commit that combined test_lxd.py and test_multipass.py. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a cleanup commit removing duplicated unit tests from test_provider.py and test_providers.py, resolving redundancy introduced by a prior commit that combined LXD and Multipass test suites. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1928 tests: remove duplicate tests | snapcraft | merged | Merged changes that remove duplicate tests and clarify test base class names to prevent future duplication. Approved by two reviewers, passed CI, and reduced unit test execution time. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged after reviewer approval and passing CI checks. Added LXD spread test to verify LXD builds and installs. Modified two files with 29 added lines. | |
| #2631 spread tests: enable LXD build provider tests | snapcraft | merged | Merged into master. Enabled LXD build provider tests by relocating setup to the early prepare phase to reduce overhead and consolidating declarations from task.yaml to spread.yaml. Approved, passed CI, and increased coverage by 0.01%. | |
| #2682 tests: change default spread provider to lxd outside of travis | snapcraft | merged | Merged after approval and CI. Refactored runtests script for clarity and updated Travis config to use google spread provider, removing legacy code and simplifying non-Travis test execution. | |
| #3725 tests: use lxd in clean provider test | snapcraft | merged | Merged into snapcraft/7.0. Enables LXD in provider cleaning tests to align with destructive mode testing. Approved by one reviewer, passed CI checks, and maintained existing code coverage. | |
| #3120 spread: only run in LXD with the google/multipass provider for repo | snapcraft | merged | Merged changes restricting spread tests to LXD with the google/multipass provider for the repo. Switched autopkgtest to destructive mode to bypass dirmngr failures until the environment is verified. Approved and passed CI. | |
| #4160 tests: remove arch_triplet from unit test | snapcraft | merged | Merged a unit test fix removing remaining arch_triplet references. The change resolves file overlap from two prior PRs. Approved by one reviewer, passed CI, and deleted two lines from a single test file. | |
| #1758 tests: move the ppa test trigger to lxd | snapcraft | merged | Merged changes moving the PPA test trigger from Docker to LXD to streamline future debugging. The original failure was valid, but the environment shift improves test maintainability. Approved by reviewers and passed all CI checks. |