Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 780144c

Browse filesBrowse files
cjihrigdanielleadams
authored andcommitted
doc: remove unnecessary leading commas
There are several locations in the test_runner API docs where the optional first argument to a function was written with a leading comma. Since these are first arguments, the commas can be removed. PR-URL: #44854 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 781ad96 commit 780144c
Copy full SHA for 780144c

File tree

Expand file treeCollapse file tree

1 file changed

+6
-6
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+6
-6
lines changed
Open diff view settings
Collapse file

‎doc/api/test.md‎

Copy file name to clipboardExpand all lines: doc/api/test.md
+6-6Lines changed: 6 additions & 6 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -480,7 +480,7 @@ same as [`it([name], { skip: true }[, fn])`][it options].
480480
Shorthand for marking a test as `TODO`,
481481
same as [`it([name], { todo: true }[, fn])`][it options].
482482

483-
## `before([, fn][, options])`
483+
## `before([fn][, options])`
484484

485485
<!-- YAML
486486
added: v18.8.0
@@ -508,7 +508,7 @@ describe('tests', async () => {
508508
});
509509
```
510510

511-
## `after([, fn][, options])`
511+
## `after([fn][, options])`
512512

513513
<!-- YAML
514514
added: v18.8.0
@@ -536,7 +536,7 @@ describe('tests', async () => {
536536
});
537537
```
538538

539-
## `beforeEach([, fn][, options])`
539+
## `beforeEach([fn][, options])`
540540

541541
<!-- YAML
542542
added: v18.8.0
@@ -565,7 +565,7 @@ describe('tests', async () => {
565565
});
566566
```
567567

568-
## `afterEach([, fn][, options])`
568+
## `afterEach([fn][, options])`
569569

570570
<!-- YAML
571571
added: v18.8.0
@@ -645,7 +645,7 @@ An instance of `TestContext` is passed to each test function in order to
645645
interact with the test runner. However, the `TestContext` constructor is not
646646
exposed as part of the API.
647647

648-
### `context.beforeEach([, fn][, options])`
648+
### `context.beforeEach([fn][, options])`
649649

650650
<!-- YAML
651651
added: v18.8.0
@@ -677,7 +677,7 @@ test('top level test', async (t) => {
677677
});
678678
```
679679

680-
### `context.afterEach([, fn][, options])`
680+
### `context.afterEach([fn][, options])`
681681

682682
<!-- YAML
683683
added: v18.8.0

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.