Skip to content

Commit e4a1cff

Browse files
codebytereMylesBorins
authored andcommittedDec 17, 2019
test: handle undefined default_configuration
PR-URL: #30465 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent ec5fe99 commit e4a1cff

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed
 

‎test/common/index.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,9 @@ const enoughTestCpu = Array.isArray(cpus) &&
118118

119119
const rootDir = isWindows ? 'c:\\' : '/';
120120

121-
const buildType = process.config.target_defaults.default_configuration;
122-
121+
const buildType = process.config.target_defaults ?
122+
process.config.target_defaults.default_configuration :
123+
'Release';
123124

124125
// If env var is set then enable async_hook hooks for all tests.
125126
if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) {

0 commit comments

Comments
 (0)