spread: move legacy wiki tests to spread
Metadata
Current evaluation
Merged following reviewer approval and passing CI. Relocated legacy wiki tests from integration tests to spread, updating 12 files with a net reduction of 42 lines. Addresses LP: #1792717.
Suggested action: —
No scores available.
Issue body
This includes all wiki tests from the integration tests that lived
under general that do not require a fake server.
LP: #1792717
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/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] 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 following reviewer approval and passing CI. Relocated legacy wiki tests from integration tests to spread, updating 12 files with a net reduction of 42 lines. Addresses LP: #1792717. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Migrated legacy wiki tests from the integration suite to the spread framework. Authored by sergiusens, the change addresses LP: #1792717 and consolidates test execution without a fake server. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged migration of legacy wiki tests from integration tests to the spread framework. Resolves LP: #1792717 by consolidating tests that do not require a fake server into the spread suite. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3432 spread tests: remove legacy plugin tests | snapcraft | merged | Merged removal of legacy plugin tests from the spread test suite. Deleted 665 lines across 29 files. Approved by reviewer, passed CI, and maintained coverage. Unrelated test failures were addressed in a separate PR. | |
| #1792717 move non-mocked wiki tests to spread | snapcraft (launchpad) | closed | Legacy tests were migrated to spread. The request was fulfilled and the issue closed. | |
| #143 docs: make docs spread-testable | craft-store | merged | Merged documentation spread-testing infrastructure setup. Approved by two reviewers and passing CI, the change modifies six files (+136/-41 lines) to enable local spread test execution for documentation. | |
| #4248 spread: separate legacy tests | snapcraft | closed | Merged into main. The pull request separated legacy spread tests into dedicated GitHub Actions workflows to reduce redundant CI execution time and improve pipeline efficiency. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| #4247 spread: separate legacy tests | snapcraft | closed | Merged changes to split legacy spread tests into separate GitHub Actions workflows, reducing redundant execution time and improving CI efficiency. | |
| #3242 tests: migrate legacy classic patchelf tests to spread | snapcraft | merged | Merged after reviewer approval and passing CI. The change migrated legacy classic patchelf tests to the spread framework, updating 13 files with a net reduction of 186 lines. | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #3090 ci: move staging store tests to spread | snapcraft | merged | Merged into master. The change migrated staging store tests to the spread testing framework, removing 1434 lines across 27 files. Approved by two reviewers, passed CI, and achieved 100% diff coverage prior to integration. | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration. |