GeezerVison is showing github.com — test: fix parsing test flags · nodejs/node@a41cfd1
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: test/common/index.js +1-3Lines changed: 1 addition & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -82,7 +82,6 @@ function parseTestFlags(filename = process.argv[1]) { |
82 | 82 | } |
83 | 83 | return source |
84 | 84 | .substring(flagStart, flagEnd) |
85 | | - .replace(/_/g, '-') |
86 | 85 | .split(/\s+/) |
87 | 86 | .filter(Boolean); |
88 | 87 | } |
@@ -98,9 +97,8 @@ if (process.argv.length === 2 && |
98 | 97 | require('cluster').isPrimary && |
99 | 98 | fs.existsSync(process.argv[1])) { |
100 | 99 | const flags = parseTestFlags(); |
101 | | - const args = process.execArgv.map((arg) => arg.replace(/_/g, '-')); |
102 | 100 | for (const flag of flags) { |
103 | | - if (!args.includes(flag) && |
| 101 | + if (!process.execArgv.includes(flag) && |
104 | 102 | // If the binary is build without `intl` the inspect option is |
105 | 103 | // invalid. The test itself should handle this case. |
106 | 104 | (process.features.inspector || !flag.startsWith('--inspect'))) { |
|