From 5760e419d7a516b9320ea00eb2aa03ffb60d147e Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Thu, 13 Dec 2018 12:10:34 +0100 Subject: [PATCH] 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 Reviewed-By: Richard Lau Reviewed-By: Ruben Bridgewater Reviewed-By: Luigi Pinca --- test/testpy/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/testpy/__init__.py b/test/testpy/__init__.py index 7d23c47214eedb..2bd976bbbdca5c 100644 --- a/test/testpy/__init__.py +++ b/test/testpy/__init__.py @@ -78,13 +78,13 @@ def GetCommand(self): # failure so such tests are also skipped. if (any(flag.startswith('--inspect') for flag in flags) and 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 '--use-openssl-ca' in flags or '--tls-v1.0' in flags or '--tls-v1.1' in flags) and 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: result += flags files_match = FILES_PATTERN.search(source);