Commit a41cfd1
test: fix parsing test flags
This removes replacing `_` with `-` in the flags defined.
Signed-off-by: Daeyeon Jeong <daeyeon.dev@gmail.com>
PR-URL: #48012
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 35c9615 commit a41cfd1
1 file changed
Lines 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'))) { | |
0 commit comments