Skip to content

Commit

Permalink
doc: add directory structure in writing-tests.md
Browse files Browse the repository at this point in the history
PR-URL: #18802
Fixes: #18774
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Franziska Hinkelmann <[email protected]>
Reviewed-By: Shingo Inoue <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
  • Loading branch information
juggernaut451 authored and targos committed Mar 27, 2018
1 parent 157fc28 commit 43f24c0
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions doc/guides/writing-tests.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,13 @@ Add tests when:
- Fixing regressions and bugs.
- Expanding test coverage.

## Test directory structure

See [directory structure overview][] for outline of existing test & locations.
When deciding on whether to expand an existing test file or create a new one,
consider going through the files related to the subsystem.
For example, look for `test-streams` when writing a test for `lib/streams.js`.

## Test structure

Let's analyze this basic test from the Node.js test suite:
Expand Down Expand Up @@ -380,3 +387,4 @@ will depend on what is being tested if this is required or not.
[all maintained branches]: https://github.com/nodejs/lts
[node.green]: http://node.green/
[test fixture]: https://github.com/google/googletest/blob/master/googletest/docs/Primer.md#test-fixtures-using-the-same-data-configuration-for-multiple-tests
[directory structure overview]: https://github.com/nodejs/node/blob/master/test/README.md#test-directories

0 comments on commit 43f24c0

Please sign in to comment.