Skip to content

Commit e039bde

Browse files
committed
tools: fix docs and run known_issues by default
- Updates `test/README.md` with new suites - Fixes some outdated `IGNORED_SUITES` listings - Allows for `test/known_issues` suite to be run by default
1 parent b675fe3 commit e039bde

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

test/README.md

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,13 @@ GitHub with the `autocrlf` git config flag set to true.
1616
|Directory |Runs on CI |Purpose |
1717
|-------------------|---------------|---------------|
1818
|abort |Yes |Tests for when the ``` --abort-on-uncaught-exception ``` flag is used.|
19-
|addons |Yes |Tests for [addon](https://nodejs.org/api/addons.html) functionality along with some tests that require an addon to function properly.|
20-
|cctest |Yes |C++ test that is run as part of the build process.|
19+
|addons |Yes |Tests for [addon](https://nodejs.org/api/addons.html) functionality along with some tests that require an addon to function properly.|
20+
|addons-napi |Yes |Tests for [n-api](https://nodejs.org/api/n-api.html) functionality.|
21+
|async-hooks |Yes |Tests for [async_hooks](https://nodejs.org/api/async_hooks.html) functionality.|
22+
|cctest |Yes |C++ tests that are run as part of the build process.|
23+
|code-cache |No |Tests for a Node.js binary compiled with V8 code cache.|
2124
|common | |Common modules shared among many tests. [Documentation](./common/README.md)|
25+
|doctool |Yes |Tests for the documentation generator.|
2226
|es-module |Yes |Test ESM module loading.|
2327
|fixtures | |Test fixtures used in various tests throughout the test suite.|
2428
|gc |No |Tests for garbage collection related functionality.|
@@ -31,7 +35,7 @@ GitHub with the `autocrlf` git config flag set to true.
3135
|sequential |Yes |Various tests that are run sequentially.|
3236
|testpy | |Test configuration utility used by various test suites.|
3337
|tick-processor |No |Tests for the V8 tick processor integration. The tests are for the logic in ```lib/internal/v8_prof_processor.js``` and ```lib/internal/v8_prof_polyfill.js```. The tests confirm that the profile processor packages the correct set of scripts from V8 and introduces the correct platform specific logic.|
34-
|timers |No |Tests for [timing utilities](https://nodejs.org/api/timers.html) (```setTimeout``` and ```setInterval```).|
38+
|v8-updates |No |Tests for V8 performance integration.|
3539

3640
_When a new test directory is added, make sure to update the `CI_JS_SUITES`
3741
variable in the `Makefile` and the `js_test_suites` variable in

tools/test.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1554,9 +1554,7 @@ def PrintCrashed(code):
15541554
'gc',
15551555
'internet',
15561556
'pummel',
1557-
'test-known-issues',
15581557
'tick-processor',
1559-
'timers',
15601558
'v8-updates'
15611559
]
15621560

0 commit comments

Comments
 (0)