Commit 95842db
test: refactor test/parallel/test-bootstrap-modules.js
This simplifies the test a bit by removing duplicated code and by
focusing the reader on the passed through module.
Signed-off-by: Ruben Bridgewater <ruben@bridgewater.de>
PR-URL: #33282
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>1 parent 69061dc commit 95842db
1 file changed
Lines changed: 22 additions & 20 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -86,26 +86,28 @@ const expectedModules = new Set([ | |||
| 86 | 86 | ]); | |
| 87 | 87 | ||
| 88 | 88 | if (!common.isMainThread) { | |
| 89 | - expectedModules.add('Internal Binding messaging'); | ||
| 90 | - expectedModules.add('Internal Binding symbols'); | ||
| 91 | - expectedModules.add('Internal Binding worker'); | ||
| 92 | - expectedModules.add('NativeModule _stream_duplex'); | ||
| 93 | - expectedModules.add('NativeModule _stream_passthrough'); | ||
| 94 | - expectedModules.add('NativeModule _stream_readable'); | ||
| 95 | - expectedModules.add('NativeModule _stream_transform'); | ||
| 96 | - expectedModules.add('NativeModule _stream_writable'); | ||
| 97 | - expectedModules.add('NativeModule internal/error-serdes'); | ||
| 98 | - expectedModules.add('NativeModule internal/process/worker_thread_only'); | ||
| 99 | - expectedModules.add('NativeModule internal/streams/buffer_list'); | ||
| 100 | - expectedModules.add('NativeModule internal/streams/destroy'); | ||
| 101 | - expectedModules.add('NativeModule internal/streams/end-of-stream'); | ||
| 102 | - expectedModules.add('NativeModule internal/streams/legacy'); | ||
| 103 | - expectedModules.add('NativeModule internal/streams/pipeline'); | ||
| 104 | - expectedModules.add('NativeModule internal/streams/state'); | ||
| 105 | - expectedModules.add('NativeModule internal/worker'); | ||
| 106 | - expectedModules.add('NativeModule internal/worker/io'); | ||
| 107 | - expectedModules.add('NativeModule stream'); | ||
| 108 | - expectedModules.add('NativeModule worker_threads'); | ||
| 89 | + [ | ||
| 90 | + 'Internal Binding messaging', | ||
| 91 | + 'Internal Binding symbols', | ||
| 92 | + 'Internal Binding worker', | ||
| 93 | + 'NativeModule _stream_duplex', | ||
| 94 | + 'NativeModule _stream_passthrough', | ||
| 95 | + 'NativeModule _stream_readable', | ||
| 96 | + 'NativeModule _stream_transform', | ||
| 97 | + 'NativeModule _stream_writable', | ||
| 98 | + 'NativeModule internal/error-serdes', | ||
| 99 | + 'NativeModule internal/process/worker_thread_only', | ||
| 100 | + 'NativeModule internal/streams/buffer_list', | ||
| 101 | + 'NativeModule internal/streams/destroy', | ||
| 102 | + 'NativeModule internal/streams/end-of-stream', | ||
| 103 | + 'NativeModule internal/streams/legacy', | ||
| 104 | + 'NativeModule internal/streams/pipeline', | ||
| 105 | + 'NativeModule internal/streams/state', | ||
| 106 | + 'NativeModule internal/worker', | ||
| 107 | + 'NativeModule internal/worker/io', | ||
| 108 | + 'NativeModule stream', | ||
| 109 | + 'NativeModule worker_threads', | ||
| 110 | + ].forEach(expectedModules.add.bind(expectedModules)); | ||
| 109 | 111 | } | |
| 110 | 112 | ||
| 111 | 113 | if (common.hasIntl) { | |
0 commit comments