Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Nightly mode_migration_vlt red: vendor tests predate #1648 eject

Đã đóng
  • Có gì không ổn?
#1,774 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.

  • #1775 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
Sôi nổi
Công nghệ
rust
Lĩnh vực
testing

Hướng nghiên cứu

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.

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 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

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

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.