Skip to content

test_runner: emit start event when subtest starts #47797

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sankalp1999 wants to merge 8 commits into from
Closed

test_runner: emit start event when subtest starts #47797

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a24f544
test_runner: emit start event when subtest starts
sankalp1999 May 1, 2023
7158d9d
test_runner: implement start event in tests_stream.js
sankalp1999 May 2, 2023
a500692
test_runner: revert the first commit
sankalp1999 May 12, 2023
b2e8085
test_runner: add lifecycle:run call and make tests pass
sankalp1999 May 25, 2023
f4e13b5
test_runner:add test:lifecycle:run event at run()
sankalp1999 May 25, 2023
d4b38f6
test_runner: rebase and refactor
sankalp1999 May 25, 2023
9033855
test_runner: move method to start(), fix tests
sankalp1999 May 29, 2023
f7dbe8a
test_runner: remove comments and extra line
sankalp1999 May 30, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test_runner: revert the first commit
  • Loading branch information
@sankalp1999
sankalp1999 committed May 25, 2023
commit a500692233ab61dd1e44fbd9ccec94182d61191e
1 change: 0 additions & 1 deletion lib/internal/test_runner/test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -515,7 +515,6 @@ class Test extends AsyncResource {
}

async run() {

if (this.parent !== null) {
this.parent.activeSubtests++;
}
Expand Down