Move ansiblelint.testing.RunFromText import into test code (#1827)

ansible · Jan 24, 2022 · 8ec0519 · 8ec0519
1 parent adf3c2f
commit 8ec0519
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/src/ansiblelint/rules/ComparisonToEmptyStringRule.py b/src/ansiblelint/rules/ComparisonToEmptyStringRule.py
@@ -6,7 +6,6 @@
 from typing import TYPE_CHECKING, Any, Dict, Union
 
 from ansiblelint.rules import AnsibleLintRule
-from ansiblelint.testing import RunFromText
 from ansiblelint.yaml_utils import nested_items_path
 
 if TYPE_CHECKING:
@@ -53,6 +52,8 @@ def matchtask(
 
     import pytest
 
+    from ansiblelint.testing import RunFromText  # pylint: disable=ungrouped-imports
+
     SUCCESS_PLAY = '''
 - hosts: all
   tasks:

diff --git a/src/ansiblelint/rules/FQCNBuiltinsRule.py b/src/ansiblelint/rules/FQCNBuiltinsRule.py
@@ -4,7 +4,6 @@
 
 from ansiblelint.file_utils import Lintable
 from ansiblelint.rules import AnsibleLintRule
-from ansiblelint.testing import RunFromText
 
 builtins = [
     "add_host",
@@ -98,6 +97,8 @@ def matchtask(
 
     import pytest
 
+    from ansiblelint.testing import RunFromText  # pylint: disable=ungrouped-imports
+
     SUCCESS_PLAY = '''
 - hosts: localhost
   tasks:

diff --git a/src/ansiblelint/rules/IgnoreErrorsRule.py b/src/ansiblelint/rules/IgnoreErrorsRule.py
@@ -3,7 +3,6 @@
 from typing import TYPE_CHECKING, Any, Dict, Union
 
 from ansiblelint.rules import AnsibleLintRule
-from ansiblelint.testing import RunFromText
 
 if TYPE_CHECKING:
     from typing import Optional
@@ -44,6 +43,8 @@ def matchtask(
 if "pytest" in sys.modules:
     import pytest
 
+    from ansiblelint.testing import RunFromText  # pylint: disable=ungrouped-imports
+
     IGNORE_ERRORS_TRUE = '''
 - hosts: all
   tasks:

diff --git a/src/ansiblelint/rules/MissingFilePermissionsRule.py b/src/ansiblelint/rules/MissingFilePermissionsRule.py
@@ -22,7 +22,6 @@
 from typing import TYPE_CHECKING, Any, Dict, Set, Union
 
 from ansiblelint.rules import AnsibleLintRule
-from ansiblelint.testing import RunFromText
 
 if TYPE_CHECKING:
     from typing import Optional
@@ -126,6 +125,8 @@ def matchtask(
 if "pytest" in sys.modules:  # noqa: C901
     import pytest
 
+    from ansiblelint.testing import RunFromText  # pylint: disable=ungrouped-imports
+
     SUCCESS_PERMISSIONS_PRESENT = '''
 - hosts: all
   tasks:

diff --git a/src/ansiblelint/rules/NoLogPasswordsRule.py b/src/ansiblelint/rules/NoLogPasswordsRule.py
@@ -17,7 +17,6 @@
 from typing import TYPE_CHECKING, Any, Dict, Union
 
 from ansiblelint.rules import AnsibleLintRule
-from ansiblelint.testing import RunFromText
 from ansiblelint.utils import convert_to_boolean
 
 if TYPE_CHECKING:
@@ -73,6 +72,8 @@ def matchtask(
 if "pytest" in sys.modules:
     import pytest
 
+    from ansiblelint.testing import RunFromText  # pylint: disable=ungrouped-imports
+
     NO_LOG_UNUSED = '''
 - hosts: all
   tasks: