-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
test_runner: do not expose internal loader #54106
Conversation
Review requested:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #54106 +/- ##
========================================
Coverage 87.10% 87.10%
========================================
Files 647 648 +1
Lines 181733 181972 +239
Branches 34886 34919 +33
========================================
+ Hits 158291 158511 +220
- Misses 16746 16760 +14
- Partials 6696 6701 +5
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We already have a lib/internal/test_runner/mock
directory. Could you rename the loader into that directory please.
Would you mind removing this comment as well.
1605d49
to
29a2cf1
Compare
Commit Queue failed- Loading data for nodejs/node/pull/54106 ✔ Done loading data for nodejs/node/pull/54106 ----------------------------------- PR info ------------------------------------ Title test_runner: do not expose internal loader (#54106) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch aduh95:mmock_loader-internal -> nodejs:main Labels author ready, needs-ci, test_runner Commits 1 - test_runner: do not expose internal loader Committers 1 - Antoine du Hamel <duhamelantoine1995@gmail.com> PR-URL: https://github.com/nodejs/node/pull/54106 Fixes: https://github.com/nodejs/node/issues/54071 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/54106 Fixes: https://github.com/nodejs/node/issues/54071 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> -------------------------------------------------------------------------------- ⚠ Commits were pushed since the last approving review: ⚠ - test_runner: do not expose internal loader ℹ This PR was created on Mon, 29 Jul 2024 21:19:22 GMT ✔ Approvals: 2 ✔ - Benjamin Gruenbaum (@benjamingr) (TSC): https://github.com/nodejs/node/pull/54106#pullrequestreview-2206202879 ✔ - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/54106#pullrequestreview-2214824107 ✔ Last GitHub CI successful ℹ Last Full PR CI on 2024-08-13T08:54:19Z: https://ci.nodejs.org/job/node-test-pull-request/61075/ - Querying data for job/node-test-pull-request/61075/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncuhttps://github.com/nodejs/node/actions/runs/10367787099 |
Landed in d0f5943 |
Fixes: #54071