Skip to content

Commit 4513516

Browse files
danbevMylesBorins
authored andcommitted
build: add a space to clarify skipping crypto msg
This commit adds a space to the message that is displayed for tests that are skipped when node was built --without-ssl. For example, this is what is currently displayed: "release test-https-agent-additional-optionsSkipping as node was compiled without crypto support" After this change this will be: "release test-https-agent-additional-options: Skipping as node was compiled without crypto support" PR-URL: #25011 Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent 30b6155 commit 4513516

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/testpy/__init__.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -78,13 +78,13 @@ def GetCommand(self):
7878
# failure so such tests are also skipped.
7979
if (any(flag.startswith('--inspect') for flag in flags) and
8080
not self.context.v8_enable_inspector):
81-
print('Skipping as node was compiled without inspector support')
81+
print(': Skipping as node was compiled without inspector support')
8282
elif (('--use-bundled-ca' in flags or
8383
'--use-openssl-ca' in flags or
8484
'--tls-v1.0' in flags or
8585
'--tls-v1.1' in flags) and
8686
not self.context.node_has_crypto):
87-
print('Skipping as node was compiled without crypto support')
87+
print(': Skipping as node was compiled without crypto support')
8888
else:
8989
result += flags
9090
files_match = FILES_PATTERN.search(source);

0 commit comments

Comments
 (0)