From 2ab01462f48b2d4e0d03ba842c3af8851c67c6f1 Mon Sep 17 00:00:00 2001 From: JamesC Date: Sat, 26 Oct 2019 16:00:02 +0200 Subject: [PATCH] Move assert num_nodes is set into main() This allows a BitcoinTestFramework child class to set test parameters in an overridden setup() rather than in an overridden set_test_params(). --- test/functional/test_framework/test_framework.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/functional/test_framework/test_framework.py b/test/functional/test_framework/test_framework.py index 6c18ef47ab..7375ab1fb7 100755 --- a/test/functional/test_framework/test_framework.py +++ b/test/functional/test_framework/test_framework.py @@ -100,11 +100,11 @@ class BitcoinTestFramework(metaclass=BitcoinTestMetaClass): self.bind_to_localhost_only = True self.set_test_params() - assert hasattr(self, "num_nodes"), "Test must set self.num_nodes in set_test_params()" - def main(self): """Main function. This should not be overridden by the subclass test scripts.""" + assert hasattr(self, "num_nodes"), "Test must set self.num_nodes in set_test_params()" + self.parse_args() try: