coverage testing blocked by promise should report where that promise came from
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- JavaScript
- Star
- 122k
- Fork
- 37.4k
- Merge trung bình
- 4 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 272
Mô tả
Version
24.8.0
Platform
macos sequioa
Subsystem
test runner
What steps will reproduce the bug?
create any dir and put a test.js file in it with content:
import test, { describe } from "node:test";
import assert from "node:assert/strict";
describe(`project testing`, async () => {
test(`this'll hang`, async () => {
const value = await new Promise((resolve) => {
console.log(`looks like this'll never resolve()`);
});
assert.equal(value, true);
});
});
Then run this with node --test "test.js"
Obviously this'll never finish, so hit ctrl-c, which triggers the final report and shows:
^C✖ test.js (1376.646ms)
ℹ tests 1
ℹ suites 0
ℹ pass 0
ℹ fail 0
ℹ cancelled 1
ℹ skipped 0
ℹ todo 0
ℹ duration_ms 1379.56475
✖ failing tests:
test at test.js:1:1
✖ test.js (1376.646ms)
'Promise resolution is still pending but the event loop has already resolved'
And, sure: that's true... but this message also isn't information, because it tells us nothing that lets us fix the problem.
Which promise is still pending? Which file, line, and column and the promise invocation be found on? Because Node knows where promise call sites are, and it should include that information in the error report.
How often does it reproduce? Is there a required condition?
This is how things work right now.
What is the expected behavior? Why is that the expected behavior?
The error should be:
test at test.js:6:18
✖ test.js (1598.681125ms)
'Promise resolution is still pending but the event loop has already resolved'
explicitly showing the line things went wrong on.
And if there is a promise pending because of some dependency of a dependency, it should show that entire trace: if test.js is awaiting getInfo() imported from ./utils.js and that calls getNetworkDeviceNames() imported from ./utils/network.js and that has a promise that never resolves, then this error should let folks find that problem:
test at test.js:6:18
at utils.js:20:14
at utils/network.js:112:25
✖ test.js (1598.681125ms)
'Promise resolution is still pending but the event loop has already resolved'
Or however traces should be made to look for this functionality. The important part is that it's informative so that we can fix bugs. After all, that's the whole point of using node --test =)
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Tái hiện hành vi bằng test.js được mô tả và node --test "test.js", sau đó bắt đầu từ việc test runner xử lý báo cáo pending-promise. Truy vết cách các vị trí gọi promise và các lệnh gọi async phụ thuộc được biểu diễn, rồi xác định đầu ra để đầu ra bao gồm các vị trí khởi nguồn và mọi trace dependency liên quan.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- javascript
- Lĩnh vực
- testing
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100