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: https://github.com/nodejs/node/pull/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>
This commit is contained in:
parent
b54d4a68e3
commit
a361b94b78
@ -78,13 +78,13 @@ class SimpleTestCase(test.TestCase):
|
|||||||
# failure so such tests are also skipped.
|
# failure so such tests are also skipped.
|
||||||
if (any(flag.startswith('--inspect') for flag in flags) and
|
if (any(flag.startswith('--inspect') for flag in flags) and
|
||||||
not self.context.v8_enable_inspector):
|
not self.context.v8_enable_inspector):
|
||||||
print('Skipping as node was compiled without inspector support')
|
print(': Skipping as node was compiled without inspector support')
|
||||||
elif (('--use-bundled-ca' in flags or
|
elif (('--use-bundled-ca' in flags or
|
||||||
'--use-openssl-ca' in flags or
|
'--use-openssl-ca' in flags or
|
||||||
'--tls-v1.0' in flags or
|
'--tls-v1.0' in flags or
|
||||||
'--tls-v1.1' in flags) and
|
'--tls-v1.1' in flags) and
|
||||||
not self.context.node_has_crypto):
|
not self.context.node_has_crypto):
|
||||||
print('Skipping as node was compiled without crypto support')
|
print(': Skipping as node was compiled without crypto support')
|
||||||
else:
|
else:
|
||||||
result += flags
|
result += flags
|
||||||
files_match = FILES_PATTERN.search(source);
|
files_match = FILES_PATTERN.search(source);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user