Nightly e2e_bun_lockb red: offline takeover test predates #1648

Đã đóng
  • Có gì không ổn?
#1,770 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Một pull request liên quan đã được merge.

  • #1771 của @mikolalysenko — đã merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
github-actions, rust
Lĩnh vực
testing

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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

Ngôn ngữ chính
Rust
Star
9
Fork
2
Merge trung bình
18 giờ 47 phút
Pull request đã merge (30 ngày)
637

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của SocketDev/socket-patch

Tất cả issue của SocketDev/socket-patch

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.