tests: completely mock mercurial tests
Metadata
Current evaluation
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.
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 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes that completely mock mercurial tests to improve execution speed, relying on integration tests for coverage. Successfully merged despite a Codecov base commit reporting issue. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. The changes completely mock Mercurial tests to accelerate execution, with integration tests handling validation. Codecov reported missing base commit coverage, but the pull request was successfully merged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #2716 tests: completely mock subversion tests | snapcraft | merged | 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. | |
| #82 test: improve integration testing | craft-store | merged | Merged improvements to integration testing across eight files. The update works around external fork credential limitations by running full tests only after merging to main. Approved by one reviewer and passed CI. | |
| #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. |