Nightly Windows: 3 Go workspace unit tests fail (Unix-only GOWORK paths)

Đã đóng
#1,815 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

@mikolalysenko đang làm issue này rồi.

Từ ngày 29/10/2026.

  • #1816 của @mikolalysenko — đang mở

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
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, go, rust
Lĩnh vực
ci-cd, devtools, testing

Hướng nghiên cứu

Read the failing tests in go_workspace.rs at lines 1141 and 1479 and golang.rs at line 1848, then inspect the Windows nightly run linked in the issue. Run the affected tests on Windows through the CI workflow_dispatch path. Done means all three assertions pass on Windows; issue #1816 is already an open pull request against this work.

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

Nightly CI on main (run 40824880386, job test (windows-latest, 1)) fails -p socket-patch-core --lib with 3 of 6536 tests failing. These are deterministic assertion failures, not flakes. They came in with #1763 / #1764 (merged 2026-10-28). PR CI and the merge queue don't run the Windows test matrix, so the queue never saw them.

vendor::go_workspace::tests::gowork_resolves_as_go_does  (go_workspace.rs:1141)
  left:  Invalid("GOWORK=/elsewhere/go.work is not an absolute path, so go refuses to build …")
  right: File("/elsewhere/go.work")

vex::discover::golang::tests::gowork_naming_another_workspace_attests_nothing  (golang.rs:1848)
  /nonexistent/go.work: [Diag { code: "patched_ref_invalid", … GOWORK=/nonexistent/go.work is not an absolute path … }]

vendor::go_workspace::tests::a_workspace_module_is_never_replaced  (go_workspace.rs:1479)
  MainModule { work_file: "\\\\?\\C:\\Users\\runneradmin\\AppData\\Local\\Temp\\.tmpSk8aez\\ws\\go.work", member: "\\\\?\\C:\\…\\ws\\dep" }

Likely fix (test side): build the GOWORK fixtures from a tempdir or another absolute path that works on each platform instead of /elsewhere and /nonexistent. For the third test, compare paths after stripping the verbatim prefix (or canonicalize both sides), or find out where \\?\ comes from (a canonicalize on Windows). Open PRs #1812 and #1800 touch both files, so the janitor didn't change them this run. Whoever lands next should check these tests on Windows (workflow_dispatch of CI).

The cargo-home half of the same nightly failure is fixed in #1814.

Ngôn ngữ chính
Rust
Star
8
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.