From f618c58b75ba365ef851089ba31017273d52f777 Mon Sep 17 00:00:00 2001 From: Ben Woosley Date: Thu, 24 Jan 2019 16:57:41 -0800 Subject: [PATCH] Docs: Update python docs to reflect that wildcard imports are disallowed --- test/functional/README.md | 2 +- test/functional/example_test.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/functional/README.md b/test/functional/README.md index bce0d5db2..628c77eb1 100644 --- a/test/functional/README.md +++ b/test/functional/README.md @@ -26,7 +26,7 @@ don't have test cases for. The Travis linter also checks this, but [possibly not in all cases](https://github.com/bitcoin/bitcoin/pull/14884#discussion_r239585126). - See [the python lint script](/test/lint/lint-python.sh) that checks for violations that could lead to bugs and issues in the test code. -- Avoid wildcard imports where possible +- Avoid wildcard imports - Use a module-level docstring to describe what the test is testing, and how it is testing it. - When subclassing the BitcoinTestFramwork, place overrides for the diff --git a/test/functional/example_test.py b/test/functional/example_test.py index be3544ee7..f367e4fca 100755 --- a/test/functional/example_test.py +++ b/test/functional/example_test.py @@ -13,7 +13,7 @@ is testing and *how* it's being tested # libraries then local imports). from collections import defaultdict -# Avoid wildcard * imports if possible +# Avoid wildcard * imports from test_framework.blocktools import (create_block, create_coinbase) from test_framework.messages import CInv from test_framework.mininode import (