Linting

Author: domdfcoding

domdf_python_tools/import_tools.py

@@ -68,12 +68,12 @@ def discover(
 	matched_classes = list()
 
 	for _, module_name, _ in pkgutil.walk_packages(
-			# https://github.com/python/mypy/issues/1422
-			# Stalled PRs: https://github.com/python/mypy/pull/3527
-			#              https://github.com/python/mypy/pull/5212
-			package.__path__,  # type: ignore
-			prefix=package.__name__ + '.',
-			):
+		# https://github.com/python/mypy/issues/1422
+		# Stalled PRs: https://github.com/python/mypy/pull/3527
+		#              https://github.com/python/mypy/pull/5212
+		package.__path__,  # type: ignore
+		prefix=package.__name__ + '.',
+		):
 		module = __import__(module_name, fromlist=['__trash'], level=0)
 
 		# Check all the functions in that module

tests/discover_demo_module/submodule_a.py

@@ -3,4 +3,4 @@ def foo():
 
 
 def bar():
-	pass
+	pass

tests/discover_demo_module/submodule_b.py

@@ -1,9 +1,11 @@
-from string import ascii_letters
+# stdlib
 from math import ceil
+from string import ascii_letters
+
 
 class Bob():
 	pass
 
 
 class Alice():
-	pass
+	pass

tests/test_import_tools.py

@@ -12,7 +12,7 @@
 sys.path.append(".")
 sys.path.append("tests")
 
-# 3rd partys
+# 3rd party
 import discover_demo_module
 
 
@@ -28,18 +28,22 @@ def test_discover():
 
 def test_discover_function_only():
 	# Alphabetical order regardless of order in the module.
-	assert discover(discover_demo_module, match_func=inspect.isfunction) == [
-			discover_demo_module.submodule_a.bar,
-			discover_demo_module.submodule_a.foo,
-			]
+	assert discover(
+			discover_demo_module, match_func=inspect.isfunction
+			) == [
+					discover_demo_module.submodule_a.bar,
+					discover_demo_module.submodule_a.foo,
+					]
 
 
 def test_discover_class_only():
 	# Alphabetical order regardless of order in the module.
-	assert discover(discover_demo_module, match_func=inspect.isclass) == [
-			discover_demo_module.submodule_b.Alice,
-			discover_demo_module.submodule_b.Bob,
-			]
+	assert discover(
+			discover_demo_module, match_func=inspect.isclass
+			) == [
+					discover_demo_module.submodule_b.Alice,
+					discover_demo_module.submodule_b.Bob,
+					]
 
 
 def test_discover_hasattr():
@@ -74,16 +78,19 @@ def raises_attribute_error(obj, **kwargs):
 			)
 
 
-@pytest.mark.parametrize("obj, expects", [
-		raises_attribute_error("abc", id="string"),
-		raises_attribute_error(123, id="int"),
-		raises_attribute_error(12.34, id="float"),
-		raises_attribute_error([1, 2, 3], id="list"),
-		raises_attribute_error((1, 2, 3), id="tuple"),
-		raises_attribute_error({1, 2, 3}, id="set"),
-		raises_attribute_error({"a": 1, "b": 2, "c": 3}, id="dictionary"),
-		pytest.param(HasPath, haspath_error, id="HasPath"),
-		])
+@pytest.mark.parametrize(
+		"obj, expects",
+		[
+				raises_attribute_error("abc", id="string"),
+				raises_attribute_error(123, id="int"),
+				raises_attribute_error(12.34, id="float"),
+				raises_attribute_error([1, 2, 3], id="list"),
+				raises_attribute_error((1, 2, 3), id="tuple"),
+				raises_attribute_error({1, 2, 3}, id="set"),
+				raises_attribute_error({"a": 1, "b": 2, "c": 3}, id="dictionary"),
+				pytest.param(HasPath, haspath_error, id="HasPath"),
+				]
+		)
 def test_discover_errors(obj, expects):
 	with expects:
 		discover(obj)