tests: completely mock subversion tests
Metadata
Current evaluation
Merged into master to accelerate test execution by fully mocking subversion tests. Approved by reviewers, passed CI, and caused a negligible 0.02% decrease in overall test coverage.
Suggested action: —
No scores available.
Issue body
This should speed them up, there are integration tests for ensuring
the rest.
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into master to accelerate test execution by fully mocking subversion tests. Approved by reviewers, passed CI, and caused a negligible 0.02% decrease in overall test coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master, this change completely mocks subversion tests to improve execution speed. The update was accepted despite a negligible 0.03% overall coverage decrease reported by Codecov. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Subversion tests are now fully mocked to improve execution speed, with validation handled by integration tests. Codecov noted a 0.03% coverage decrease upon merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2714 tests: completely mock mercurial tests | snapcraft | merged | Merged changes that completely mock Mercurial tests to improve execution speed, relying on integration tests for validation. Approved by reviewer, passed CI, and modified two files. | |
| #2713 tests: completely mock bzr tests | snapcraft | merged | Merged changes completely mocking bzr tests to improve execution speed. Approved by a reviewer, passed CI checks, and modified two files (+68/-20). Integration tests remain for validation. | |
| #1027 tests: fix broken unit test in master | snapcraft | merged | Merged a one-line fix resolving a broken unit test in master. Approved by one reviewer with no unresolved comments. Changes passed CI and maintained 96.37% code coverage. Integrated directly into the main branch. |