Remove dependency to molecule.test (#206)

Author: cristianonicolai

conftest.py

@@ -1,6 +1,7 @@
 import contextlib
 import os
 import random
+import shutil
 import string
 
 import pytest
@@ -84,3 +85,24 @@ def molecule_ephemeral_directory(_fixture_uuid):
     return ephemeral_directory(
         os.path.join("molecule_test", project_directory, scenario_name),
     )
+
+
+def metadata_lint_update(role_directory: str) -> None:
+    # By default, ansible-lint will fail on newly-created roles because the
+    # fields in this file have not been changed from their defaults. This is
+    # good because molecule should create this file using the defaults, and
+    # users should receive feedback to change these defaults. However, this
+    # blocks the testing of 'molecule init' itself, so ansible-lint should
+    # be configured to ignore these metadata lint errors.
+    dirname = os.path.dirname(os.path.abspath(__file__))
+    ansible_lint_src = os.path.join(dirname, ".ansible-lint")
+    shutil.copy(ansible_lint_src, role_directory)
+
+    # Explicitly lint here to catch any unexpected lint errors before
+    # continuining functional testing. Ansible lint is run at the root
+    # of the role directory and pointed at the role directory to ensure
+    # the customize ansible-lint config is used.
+    with change_dir_to(role_directory):
+        cmd = ["ansible-lint", "."]
+        result = run_command(cmd)
+    assert result.returncode == 0

test/azure/functional/conftest.py

@@ -20,4 +20,4 @@
 #  DEALINGS IN THE SOFTWARE.
 
 
-from molecule.test.conftest import *  # noqa
+from conftest import *  # noqa

test/azure/functional/test_azure.py

@@ -23,8 +23,8 @@
 
 import pytest
 
+from conftest import change_dir_to
 from molecule import logger
-from molecule.test.conftest import change_dir_to
 from molecule.util import run_command
 
 LOG = logger.get_logger(__name__)

test/containers/functional/conftest.py

@@ -20,4 +20,4 @@
 #  DEALINGS IN THE SOFTWARE.
 """PyTest Config."""
 
-from molecule.test.conftest import *  # noqa pylint: disable=wildcard-import,unused-wildcard-import
+from conftest import *  # noqa pylint: disable=wildcard-import,unused-wildcard-import

test/containers/functional/test_containers.py

@@ -21,8 +21,8 @@
 """Functional Tests."""
 import os
 
+from conftest import change_dir_to, molecule_directory
 from molecule import logger
-from molecule.test.conftest import change_dir_to, molecule_directory
 from molecule.util import run_command
 
 LOG = logger.get_logger(__name__)

test/docker/conftest.py

@@ -1,7 +1,7 @@
 """Pytest Fixtures."""
-import pytest
+from conftest import random_string, temp_dir  # noqa
 
-from molecule.test.conftest import random_string, temp_dir  # noqa
+import pytest
 
 
 @pytest.fixture()

test/docker/test_func.py

@@ -6,8 +6,8 @@
 
 import pytest
 
+from conftest import change_dir_to
 from molecule import logger
-from molecule.test.conftest import change_dir_to
 from molecule.util import run_command
 
 LOG = logger.get_logger(__name__)

test/ec2/functional/conftest.py

@@ -20,4 +20,4 @@
 #  DEALINGS IN THE SOFTWARE.
 
 
-from molecule.test.conftest import *  # noqa
+from conftest import *  # noqa

test/ec2/functional/test_ec2.py

@@ -23,9 +23,8 @@
 
 import pytest
 
+from conftest import change_dir_to, metadata_lint_update
 from molecule import logger
-from molecule.test.b_functional.conftest import metadata_lint_update
-from molecule.test.conftest import change_dir_to
 from molecule.util import run_command
 
 LOG = logger.get_logger(__name__)

test/gce/functional/conftest.py

@@ -20,4 +20,4 @@
 #  DEALINGS IN THE SOFTWARE.
 
 
-from molecule.test.conftest import *  # noqa
+from conftest import *  # noqa