Nightly mode_migration_vlt red: vendor tests predate #1648 eject

Geschlossen
  • Stimmt etwas nicht?
#1,774 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #1775 von @mikolalysenko — gemerged

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
35/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
rust
Bereich
testing

Rechercherichtung

Start in crates/socket-patch-cli/tests/mode_migration_vlt.rs at the three assertion blocks named: lines 310-313, 360-363, and 385-388. Compare them with the eject contract already asserted in mode_migration_bun.rs after #1648, then update the vlt migration tests to expect eject_planned and no vendor_takeover_reverted_redirect/vendor_would_revert_redirect. Run the vlt_pinned_matrix tests or the nightly e2e-extended leg for mode_migration_vlt to confirm the stale advisory assertions are gone.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

agent:claimed agent:triaged ci-janitor priority:p3

[agent] Filed by the CI-lane fixer.

The nightly CI run on main (40639505834, head b787454) fails e2e-extended (mode_migration_vlt, *, --include-ignored vlt_pinned_matrix) on every vlt leg (vlt 0.0.0-32, 1.0.0-rc.14, 1.2.0 on Linux, plus the Windows/macOS legs):

thread 'vlt_pinned_matrix_migration_hosted_then_vendored' panicked at crates/socket-patch-cli/tests/mode_migration_vlt.rs:312:9:
vendor: exit 0
  ... "errorCode": "vendor_prebuilt_downloaded" ... "vendor_vlt_reinstall_required" ...
failures:
    vlt_pinned_matrix_migration_dry_run_parity
    vlt_pinned_matrix_migration_hosted_then_vendored

Root cause: #1648 made plain vendor with a hosted pin in scope an eject (dry run reports eject_planned, the wet run no longer emits vendor_takeover_reverted_redirect). It updated the PR-tier migration suites (mode_migration_bun.rs, _npm, _cargo, _pypi) but missed mode_migration_vlt.rs, whose tests are #[ignore] (real vlt) and run only in the nightly e2e-extended legs. The vendor runs themselves succeed; only the stale advisory assertions fail:

  • mode_migration_vlt.rs:310-313 (hosted_then_vendored, vendor driver) expects vendor_takeover_reverted_redirect.
  • mode_migration_vlt.rs:360-363 (dry_run_parity) expects vendor_would_revert_redirect from vendor --dry-run.
  • mode_migration_vlt.rs:385-388 (dry_run_parity) expects vendor_takeover_reverted_redirect from the wet vendor.

Fix (test-only): assert the eject contract the bun suite now asserts. Sibling of #1770 (same #1648 fallout in e2e_bun_lockb.rs).

Generated by Claude Code

Vorherrschende Sprache
Rust
Sterne
8
Forks
2
Ø Merge
19 Std. 24 Min.
Gemergte PRs (30 T.)
669

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus SocketDev/socket-patch

Alle Issues in SocketDev/socket-patch

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.