doc: add code examples to node test runner by manekinekko · Pull Request #43359 · nodejs/node

manekinekko · 2022-06-09T10:25:59Z

Add a few code examples to the TestContext docs.

cc @benjamingr @fhinkel

doc/api/test.md

benjamingr

Thanks a bunch! cc @cjihrig

cjihrig

LGTM. Thanks!

nodejs-github-bot · 2022-06-14T06:28:49Z

Commit Queue failed

- Loading data for nodejs/node/pull/43359
✔  Done loading data for nodejs/node/pull/43359
----------------------------------- PR info ------------------------------------
Title      doc: add code examples to node test runner (#43359)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     manekinekko:node-test-docs-update -> nodejs:master
Labels     doc, test
Commits    3
 - doc: add code examples to node test runner
 - Update doc/api/test.md
 - Update doc/api/test.md
Committers 2
 - Wassim Chegham 
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 09 Jun 2022 10:25:59 GMT
   ✔  Approvals: 6
   ✔  - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/43359#pullrequestreview-1001843939
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002140928
   ✔  - Akhil Marsonya (@marsonya): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002274416
   ✔  - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002774716
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003208538
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003512636
   ✖  Last GitHub CI failed
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/2493221011

benjamingr · 2022-06-14T07:15:21Z

@manekinekko can you please update info on GitHub so we can land this automatically and (possibly) squash it to one commit :)?

⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!

manekinekko · 2022-06-14T11:56:00Z

I just made my email public. Can you try again and let me know? Thank you ☺️

nodejs-github-bot · 2022-06-14T12:24:18Z

Commit Queue failed

- Loading data for nodejs/node/pull/43359
✔  Done loading data for nodejs/node/pull/43359
----------------------------------- PR info ------------------------------------
Title      doc: add code examples to node test runner (#43359)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     manekinekko:node-test-docs-update -> nodejs:master
Labels     doc, test, commit-queue-squash
Commits    3
 - doc: add code examples to node test runner
 - Update doc/api/test.md
 - Update doc/api/test.md
Committers 2
 - Wassim Chegham 
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 09 Jun 2022 10:25:59 GMT
   ✔  Approvals: 6
   ✔  - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/43359#pullrequestreview-1001843939
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002140928
   ✔  - Akhil Marsonya (@marsonya): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002274416
   ✔  - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002774716
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003208538
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003512636
   ✖  Last GitHub CI failed
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/2495155286

benjamingr · 2022-06-14T20:03:26Z

⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
@manekinekko

manekinekko · 2022-06-14T22:50:54Z

@benjamingr Sorry about that! My email is visible on my profile page though. Am I missing something?

Trott · 2022-06-15T13:41:20Z

@benjamingr Sorry about that! My email is visible on my profile page though. Am I missing something?

The change would be something to make in the first git commit. The author for that commit is:

Wassim Chegham <1699357+manekinekko@users.noreply.github.com>

You'll want to amend that commit to use your email address (or not I suppose if you don't want that public, in which case I can land this manually).

Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

manekinekko · 2022-06-15T13:51:54Z

@Trott I've squashed my commits and my commit should have my email address. Let's see if that fixes the issue.

nodejs-github-bot · 2022-06-15T16:45:16Z

Commit Queue failed

- Loading data for nodejs/node/pull/43359
✔  Done loading data for nodejs/node/pull/43359
----------------------------------- PR info ------------------------------------
Title      doc: add code examples to node test runner (#43359)
Author     Wassim Chegham  (@manekinekko)
Branch     manekinekko:node-test-docs-update -> nodejs:main
Labels     doc, test
Commits    1
 - doc: add code examples to node test runner
Committers 1
 - Wassim Chegham 
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/43359
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Colin Ihrig 
Reviewed-By: Akhil Marsonya 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last review:
   ⚠  - doc: add code examples to node test runner
   ℹ  This PR was created on Thu, 09 Jun 2022 10:25:59 GMT
   ✔  Approvals: 8
   ✔  - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/43359#pullrequestreview-1001843939
   ✔  - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002140928
   ✔  - Akhil Marsonya (@marsonya): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002274416
   ✔  - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002774716
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003208538
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003512636
   ✔  - Franziska Hinkelmann (@fhinkel) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1006639788
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1007529851
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/2503782670

nodejs-github-bot · 2022-06-16T04:02:16Z

Commit Queue failed

- Loading data for nodejs/node/pull/43359 ✔ Done loading data for nodejs/node/pull/43359 ----------------------------------- PR info ------------------------------------ Title doc: add code examples to node test runner (#43359) Author Wassim Chegham (@manekinekko) Branch manekinekko:node-test-docs-update -> nodejs:main Labels doc, test Commits 1 - doc: add code examples to node test runner Committers 1 - Wassim Chegham PR-URL: https://github.com/nodejs/node/pull/43359 Reviewed-By: Benjamin Gruenbaum Reviewed-By: Colin Ihrig Reviewed-By: Akhil Marsonya Reviewed-By: Harshitha K P Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Franziska Hinkelmann Reviewed-By: Rich Trott ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/43359 Reviewed-By: Benjamin Gruenbaum Reviewed-By: Colin Ihrig Reviewed-By: Akhil Marsonya Reviewed-By: Harshitha K P Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Franziska Hinkelmann Reviewed-By: Rich Trott -------------------------------------------------------------------------------- ℹ This PR was created on Thu, 09 Jun 2022 10:25:59 GMT ✔ Approvals: 8 ✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/43359#pullrequestreview-1001843939 ✔ - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002140928 ✔ - Akhil Marsonya (@marsonya): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002274416 ✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/43359#pullrequestreview-1002774716 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003208538 ✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1003512636 ✔ - Franziska Hinkelmann (@fhinkel) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1006639788 ✔ - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/43359#pullrequestreview-1008458724 ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/main up to date... From https://github.com/nodejs/node * branch main -> FETCH_HEAD 358008f473..72f9597d3e main -> origin/main ✔ origin/main is now up-to-date main is out of sync with origin/main. Mismatched commits: - 0c7231265a tools: update lint-md-dependencies to rollup@2.75.6 - 72f9597d3e tools: update lint-md-dependencies to rollup@2.75.6 -------------------------------------------------------------------------------- HEAD is now at 72f9597d3e tools: update lint-md-dependencies to rollup@2.75.6 ✔ Reset to origin/main - Downloading patch for 43359 From https://github.com/nodejs/node * branch refs/pull/43359/merge -> FETCH_HEAD ✔ Fetched commits as 72f9597d3e8f..ed16e2897c85 -------------------------------------------------------------------------------- [main 3cab79599a] doc: add code examples to node test runner Author: Wassim Chegham Date: Thu Jun 9 12:23:40 2022 +0200 1 file changed, 43 insertions(+) ✔ Patches applied -------------------------------------------------------------------------------- --------------------------------- New Message ---------------------------------- doc: add code examples to node test runner

Co-authored-by: Antoine du Hamel duhamelantoine1995@gmail.com

PR-URL: #43359
Reviewed-By: Benjamin Gruenbaum benjamingr@gmail.com
Reviewed-By: Colin Ihrig cjihrig@gmail.com
Reviewed-By: Akhil Marsonya akhil.marsonya27@gmail.com
Reviewed-By: Harshitha K P harshitha014@gmail.com
Reviewed-By: Luigi Pinca luigipinca@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Franziska Hinkelmann franziska.hinkelmann@gmail.com
Reviewed-By: Rich Trott rtrott@gmail.com

[main b9034e7eb3] doc: add code examples to node test runner
Author: Wassim Chegham github@wassim.dev
Date: Thu Jun 9 12:23:40 2022 +0200
1 file changed, 43 insertions(+)
✖ b9034e7eb3697444d38881970f8d27fefb7bcf40
✖ 1:0 Co-authored-by must be a trailer co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 3:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

ℹ Please fix the commit message and try again.

https://github.com/nodejs/node/actions/runs/2506616458

manekinekko · 2022-06-16T05:50:37Z

It looks like the automatic merge is still failing. @Trott maybe you can merge this manually then (if that's possible). I don't know how to fix it. Sorry about the inconvenience 😃

lpinca · 2022-06-16T06:01:46Z

Landed in 6975dd1.

PR-URL: #43359 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com> Reviewed-By: Harshitha K P <harshitha014@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

benjamingr · 2022-06-16T07:04:23Z

@manekinekko what happened is that you merged a suggestion by @aduh95 into this PR, when you squashed it included the Co-Authored-By part which node-core-utils doesn't currently (fully) support automatically so a member (in this case Luigi) fixed the formatting and landed it.

manekinekko · 2022-06-16T07:55:26Z

Thank you @benjamingr for the explanation. And thank you @lpinca for landing this PR 💖

PR-URL: #43359 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com> Reviewed-By: Harshitha K P <harshitha014@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

targos · 2022-07-18T14:37:45Z

Depends on #42658

PR-URL: #43359 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com> Reviewed-By: Harshitha K P <harshitha014@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: nodejs/node#43359 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com> Reviewed-By: Harshitha K P <harshitha014@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

nodejs-github-bot added doc Issues and PRs related to the documentations. test Issues and PRs related to the tests. labels Jun 9, 2022

aduh95 reviewed Jun 9, 2022

View reviewed changes

doc/api/test.md Outdated Show resolved Hide resolved

aduh95 reviewed Jun 9, 2022

View reviewed changes

doc/api/test.md Outdated Show resolved Hide resolved

benjamingr approved these changes Jun 9, 2022

View reviewed changes

cjihrig approved these changes Jun 10, 2022

View reviewed changes

marsonya approved these changes Jun 10, 2022

View reviewed changes

HarshithaKP approved these changes Jun 10, 2022

View reviewed changes

lpinca approved these changes Jun 10, 2022

View reviewed changes

jasnell approved these changes Jun 12, 2022

View reviewed changes

lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 14, 2022

nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 14, 2022

fhinkel approved these changes Jun 14, 2022

View reviewed changes

Trott approved these changes Jun 15, 2022

View reviewed changes

doc: add code examples to node test runner

ed16e28

Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

manekinekko force-pushed the node-test-docs-update branch from 0ea9a19 to ed16e28 Compare June 15, 2022 13:48

nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 15, 2022

Trott approved these changes Jun 16, 2022

View reviewed changes

Trott added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Jun 16, 2022

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 16, 2022

nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Jun 16, 2022

lpinca closed this Jun 16, 2022

targos mentioned this pull request Jul 12, 2022

v18.6.0 release proposal #43789

Merged

targos added backport-blocked-v16.x and removed backport-blocked-v16.x labels Jul 18, 2022

targos removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. dont-land-on-v16.x labels Jul 31, 2022

targos mentioned this pull request Aug 3, 2022

v16.17.0 release proposal #44098

Merged

Uh oh!

Conversation

manekinekko commented Jun 9, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

benjamingr left a comment

Choose a reason for hiding this comment

Uh oh!

cjihrig left a comment

Choose a reason for hiding this comment

Uh oh!

nodejs-github-bot commented Jun 14, 2022

Uh oh!

benjamingr commented Jun 14, 2022

Uh oh!

manekinekko commented Jun 14, 2022

Uh oh!

nodejs-github-bot commented Jun 14, 2022

Uh oh!

benjamingr commented Jun 14, 2022

Uh oh!

manekinekko commented Jun 14, 2022

Uh oh!

Trott commented Jun 15, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

manekinekko commented Jun 15, 2022

Uh oh!

nodejs-github-bot commented Jun 15, 2022

Uh oh!

nodejs-github-bot commented Jun 16, 2022

Uh oh!

manekinekko commented Jun 16, 2022

Uh oh!

lpinca commented Jun 16, 2022

Uh oh!

benjamingr commented Jun 16, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

manekinekko commented Jun 16, 2022

Uh oh!

targos commented Jul 18, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

manekinekko commented Jun 9, 2022 •

edited

Loading

Trott commented Jun 15, 2022 •

edited

Loading

benjamingr commented Jun 16, 2022 •

edited

Loading