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

[Feature] Update the VSIX path used by smoke tests

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#3,002 0 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

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

Từ ngày 14/10/2026.

  • #3001 của @EmmaYuan1015 — đang mở

Đá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
76/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ệ
typescript
Lĩnh vực
testing

Hướng nghiên cứu

Đường dẫn sai là một lệnh gọi path.resolve(__dirname, "..", "..", "resources", "extension") trong mã kiểm thử khói VSIX nằm dưới test/smoke. Issue không nêu tên tệp, vì vậy hãy tìm nó bằng cách tìm lệnh gọi đó. Ghi log thư mục đã phân giải từ bài kiểm thử đã biên dịch trong test/smoke/out để xác nhận nó trỏ tới test/resources/extension. Công việc hoàn thành khi nó phân giải thành test/smoke/resources/extension, các bài kiểm thử VSIX tìm được tệp của chúng thay vì bị bỏ qua, và kiểm tra hồi quy sẽ thất bại nếu đường dẫn lại bị thay đổi.

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

Mô tả

enhancement

Description

The VSIX smoke tests resolve the extension resource directory using two parent-directory segments:

path.resolve(__dirname, "..", "..", "resources", "extension");

At runtime, the compiled test is located under:

test/smoke/out

As a result, the current path resolves to:

test/resources/extension

Instead of the intended directory:

test/smoke/resources/extension

This is not detected because the existing tests are skipped when the resolved directory or VSIX does not exist.

Expected behavior

The smoke tests should resolve and validate VSIX files from:

test/smoke/resources/extension

Proposed fix

  • Remove the extra parent-directory segment:
path.resolve(__dirname, "..", "resources", "extension");
  • Ensure the smoke tests use the intended VSIX resource directory.
  • Add a regression check to prevent the path from resolving to the wrong directory.
Ngôn ngữ chính
TypeScript
Star
2.7k
Fork
298
Merge trung bình
14 giờ 54 phút
Pull request đã merge (30 ngày)
28

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 microsoft/vscode-react-native

Tất cả issue của microsoft/vscode-react-native

Issue tương tự

Thêm issue về TypeScript

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.