← Back to issue list

tests: remove the reusable parts tour test

View original Github issue

Metadata

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

Current evaluation

PR #1321 removed the reusable-parts tour test, but #1324 reverted it; the tour was later fully removed by merged PR #1520 (commit 9696c57e9).

Resolution: closed superseded

Reason: The change merged as 2bd87c5af but was reverted by #1324 (fbb228aa8) after the failure was attributed to Snapcraft rather than the tour. PR #1520 later completed the intended tour removal.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The tour is being removed from the docs because it's now replaced by the tutorials. This test is blocking our release, and it's not serving any good purpose. As soon as the docs are updated, we will remove all the tour snaps, commands and tests.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded PR #1321 removed the reusable-parts tour test, but #1324 reverted it; the tour was later fully removed by merged PR #1520 (commit 9696c57e9).
stealth/space-bunny-alpha — closed resolved Merged #1321, removing 28 lines across two test files because the reusable parts tour was replaced by tutorials and its test blocked the release; tour snaps and commands were slated for removal.
qwen/qwen3.6-35b-a3b — — Merged following approval. Removed tests for the reusable parts tour, which was replaced by tutorials and blocking the release. The commit deletes 28 lines across 2 files.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the reusable parts tour test. The tour was replaced by tutorials, and the test was blocking the release. All related tour snaps, commands, and tests were removed to unblock the release.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#1324 (confidence 100%)

    Merged commit fbb228aa8 reverted PR #1321 and restored the test.

  • Superseded By: snapcraft#1520 (confidence 100%)

    Merged commit 9696c57e9 later removed the tour assets after replacement by tutorials.

Related issues

Issue Project State Summary Similarity
#1324 Revert "tests: remove the reusable parts tour test (#1321)" snapcraft merged Merged PR #1324 (commit fbb228aa8) restored the reusable-parts tour test by reverting #1321 after the failure was traced to Snapcraft rather than the tested code.
84%
#549 Skip the other test tour, temporarily. snapcraft merged Merged PR #549 as commit d4209d10f, temporarily skipping the other test tour with a three-line change.
72%
#1520 tour: remove the tour assets snapcraft merged Merged PR #1520 (commit 9696c57e9), removing 39 tour asset files after sunsetting the tour in favor of Snapcraft tutorials.
70%