-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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.
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
nested test counting is wrong #47696
Labels
confirmed-bug
Issues with confirmed bugs.
test_runner
Issues and PRs related to the test runner subsystem.
Comments
MoLow
added
test_runner
Issues and PRs related to the test runner subsystem.
confirmed-bug
Issues with confirmed bugs.
labels
Apr 24, 2023
This was referenced Apr 24, 2023
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 28, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 28, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 28, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 28, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 29, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
yjl9903
pushed a commit
to yjl9903/node
that referenced
this issue
Apr 29, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MoLow
added a commit
to MoLow/node
that referenced
this issue
Jul 6, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
MoLow
added a commit
to MoLow/node
that referenced
this issue
Jul 6, 2023
PR-URL: nodejs#47675 Fixes: nodejs#47365 Fixes: nodejs#47696 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
confirmed-bug
Issues with confirmed bugs.
test_runner
Issues and PRs related to the test runner subsystem.
Version
v18.16.0, v21.0.0-pre
Platform
Darwin Moshes-MacBook-Pro.local 22.4.0 Darwin Kernel Version 22.4.0: Mon Mar 6 20:59:28 PST 2023; root:xnu-8796.101.5~3/RELEASE_ARM64_T6000 arm64
Subsystem
test_runner
What steps will reproduce the bug?
How often does it reproduce? Is there a required condition?
allways
What is the expected behavior? Why is that the expected behavior?
each test should have its index
What do you see instead?
tests are
ok 1
ok 2
ok 3
Additional information
No response
The text was updated successfully, but these errors were encountered: