Skip to content

Commit bfaeb84

Browse files
author
MarcoFalke
committed
Merge #14381: test: Add missing call to skip_if_no_cli()
ff94da7 tests: Make appveyor run with --usecli (practicalswift) db01839 test: Add missing call to skip_if_no_cli() (practicalswift) Pull request description: Add missing call to `skip_if_no_cli()` as suggested by @MarcoFalke in #14365. Tree-SHA512: b0a2ddfad0f81cc9544f63c4e490fb983d833a47c23522549d1200ea6a8a132b2cd4bf0d66b862ef3a548d8471128b80aea3525fb5dec65221e23f32a8d46746
2 parents 1f59c6f + ff94da7 commit bfaeb84

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

.appveyor.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ after_build:
4646
before_test:
4747
- ps: ${conf_ini} = (Get-Content([IO.Path]::Combine(${env:APPVEYOR_BUILD_FOLDER}, "test", "config.ini.in")))
4848
- ps: ${conf_ini} = $conf_ini.Replace("@abs_top_srcdir@", ${env:APPVEYOR_BUILD_FOLDER}).Replace("@abs_top_builddir@", ${env:APPVEYOR_BUILD_FOLDER}).Replace("@EXEEXT@", ".exe")
49-
- ps: ${conf_ini} = $conf_ini.Replace("@ENABLE_WALLET_TRUE@", "").Replace("@BUILD_BITCOIN_UTILS_TRUE@", "").Replace("@BUILD_BITCOIND_TRUE@", "").Replace("@ENABLE_ZMQ_TRUE@", "")
49+
- ps: ${conf_ini} = $conf_ini.Replace("@ENABLE_WALLET_TRUE@", "").Replace("@BUILD_BITCOIN_CLI_TRUE@", "").Replace("@BUILD_BITCOIND_TRUE@", "").Replace("@ENABLE_ZMQ_TRUE@", "")
5050
- ps: ${utf8} = New-Object System.Text.UTF8Encoding ${false}
5151
- ps: '[IO.File]::WriteAllLines([IO.Path]::Combine(${env:APPVEYOR_BUILD_FOLDER}, "test", "config.ini"), $conf_ini, ${utf8})'
5252
- ps: move "build_msvc\${env:PLATFORM}\${env:CONFIGURATION}\*.exe" src

test/functional/test_framework/test_framework.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,10 @@ def main(self):
161161
success = TestStatus.FAILED
162162

163163
try:
164-
if self.options.usecli and not self.supports_cli:
165-
raise SkipTest("--usecli specified but test does not support using CLI")
164+
if self.options.usecli:
165+
if not self.supports_cli:
166+
raise SkipTest("--usecli specified but test does not support using CLI")
167+
self.skip_if_no_cli()
166168
self.skip_test_if_missing_module()
167169
self.setup_chain()
168170
self.setup_network()

0 commit comments

Comments
 (0)