Nightly e2e_bun_lockb red: offline takeover test predates #1648

Geschlossen
  • Stimmt etwas nicht?
#1,770 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.

  • #1771 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
Veraltet
Tech-Stack
github-actions, rust
Bereich
testing

Rechercherichtung

Start at crates/socket-patch-cli/tests/e2e_bun_lockb.rs:928 in native_binary_hosted_vendored_takeover_roundtrip, where the assertion still expects redirect_revert_failed. Run that test with bun 1.0.36 on main to see vendor --offline --dry-run exit 1 with the top-level offline_eject_unavailable error, then update the step to assert that error, exit code 1, and no files written. Check the remaining roundtrip assertions still hold. Note PR #1771 is already open against this, so coordinate before starting.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

agent:claimed agent:triaged priority:p3

[agent] The nightly e2e-extended legs e2e_bun_lockb (bun 1.0.36 and 1.1.45) have been red on main since #1648 merged.

Evidence

  • Run https://github.com/SocketDev/socket-patch/actions/runs/40639505834 (schedule, main b787454, 2026-10-28), jobs 122654524514 and 122654524565.
  • native_binary_hosted_vendored_takeover_roundtrip panics at crates/socket-patch-cli/tests/e2e_bun_lockb.rs:928: expected redirect_revert_failed, but vendor --offline --dry-run now returns the top-level error offline_eject_unavailable ("ejecting 1 hosted package needs 1 patch record(s) from the Socket API, and this run is offline; re-run without --offline").
  • Reproduces locally with bun 1.0.36 on origin/main.

Cause
#1648 (vendor hosted patches directly from lockfiles) made vendor --offline refuse up front whenever a hosted pin is in scope, which its description records as the chosen behaviour. This test still expects the older per-pin redirect_revert_failed event. It only runs on the bun 1.0/1.1 legs, which are nightly-only, so the PR and merge-queue runs never exercised it.

Fix
Update the test's offline-refusal step to the new contract (exit 1, offline_eject_unavailable, nothing written), and check that the rest of the roundtrip still holds.

Generated by Claude Code

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

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.