refactor(tests): Move constants to tests.constants

Author: tony

tests/__init__.py

@@ -1,11 +0,0 @@
-"""Tests for tmuxp.
-
-tmuxp.tests
-~~~~~~~~~~~
-
-"""
-import pathlib
-
-TESTS_PATH = pathlib.Path(__file__).parent
-EXAMPLE_PATH = TESTS_PATH.parent / "examples"
-FIXTURES_PATH = TESTS_PATH / "fixtures"

tests/constants.py

@@ -0,0 +1,5 @@
+import pathlib
+
+TESTS_PATH = pathlib.Path(__file__).parent
+EXAMPLE_PATH = TESTS_PATH.parent / "examples"
+FIXTURE_PATH = TESTS_PATH / "fixtures"

tests/fixtures/utils.py

@@ -1,6 +1,6 @@
 import pathlib
 
-FIXTURE_PATH = pathlib.Path(__file__).parent
+from ..constants import FIXTURE_PATH
 
 
 def read_config_file(_file):  # return fixture data, relative to __file__

tests/test_cli.py

@@ -28,8 +28,8 @@
 )
 from tmuxp.workspacebuilder import WorkspaceBuilder
 
+from .constants import FIXTURE_PATH
 from .fixtures import utils as test_utils
-from .fixtures._util import FIXTURE_PATH
 
 
 def test_creates_config_dir_not_exists(tmp_path: pathlib.Path):

tests/test_config.py

@@ -10,7 +10,7 @@
 
 from tmuxp import config, exc
 
-from . import EXAMPLE_PATH
+from .constants import EXAMPLE_PATH
 
 if typing.TYPE_CHECKING:
     from .fixtures import config as ConfigFixture

tests/test_util.py

@@ -5,11 +5,11 @@
 from tmuxp.exc import BeforeLoadScriptError, BeforeLoadScriptNotExists
 from tmuxp.util import get_session, run_before_script
 
-from . import FIXTURES_PATH
+from .constants import FIXTURE_PATH
 
 
 def test_raise_BeforeLoadScriptNotExists_if_not_exists():
-    script_file = FIXTURES_PATH / "script_noexists.sh"
+    script_file = FIXTURE_PATH / "script_noexists.sh"
 
     with pytest.raises(BeforeLoadScriptNotExists):
         run_before_script(script_file)
@@ -19,30 +19,30 @@ def test_raise_BeforeLoadScriptNotExists_if_not_exists():
 
 
 def test_raise_BeforeLoadScriptError_if_retcode():
-    script_file = FIXTURES_PATH / "script_failed.sh"
+    script_file = FIXTURE_PATH / "script_failed.sh"
 
     with pytest.raises(BeforeLoadScriptError):
         run_before_script(script_file)
 
 
 def test_return_stdout_if_ok(capsys):
-    script_file = FIXTURES_PATH / "script_complete.sh"
+    script_file = FIXTURE_PATH / "script_complete.sh"
 
     run_before_script(script_file)
     out, err = capsys.readouterr()
     assert "hello" in out
 
 
 def test_beforeload_returncode():
-    script_file = FIXTURES_PATH / "script_failed.sh"
+    script_file = FIXTURE_PATH / "script_failed.sh"
 
     with pytest.raises(exc.BeforeLoadScriptError) as excinfo:
         run_before_script(script_file)
         assert excinfo.match(r"113")
 
 
 def test_beforeload_returns_stderr_messages():
-    script_file = FIXTURES_PATH / "script_failed.sh"
+    script_file = FIXTURE_PATH / "script_failed.sh"
 
     with pytest.raises(exc.BeforeLoadScriptError) as excinfo:
         run_before_script(script_file)

tests/test_workspacebuilder.py

@@ -16,7 +16,7 @@
 from tmuxp.cli import load_plugins
 from tmuxp.workspacebuilder import WorkspaceBuilder
 
-from . import EXAMPLE_PATH, FIXTURES_PATH
+from .constants import EXAMPLE_PATH, FIXTURE_PATH
 from .fixtures import utils as test_utils
 
 
@@ -597,7 +597,7 @@ def test_before_load_throw_error_if_retcode_error(server):
     )
     sconfig = kaptan.Kaptan(handler="yaml")
     yaml = config_script_fails.format(
-        script_failed=FIXTURES_PATH / "script_failed.sh",
+        script_failed=FIXTURE_PATH / "script_failed.sh",
     )
 
     sconfig = sconfig.import_config(yaml).get()
@@ -622,7 +622,7 @@ def test_before_load_throw_error_if_file_not_exists(server):
     )
     sconfig = kaptan.Kaptan(handler="yaml")
     yaml = config_script_not_exists.format(
-        script_not_exists=FIXTURES_PATH / "script_not_exists.sh",
+        script_not_exists=FIXTURE_PATH / "script_not_exists.sh",
     )
     sconfig = sconfig.import_config(yaml).get()
     sconfig = config.expand(sconfig)
@@ -645,7 +645,7 @@ def test_before_load_true_if_test_passes(server):
     config_script_completes = test_utils.read_config_file(
         "workspacebuilder/config_script_completes.yaml"
     )
-    script_complete_sh = FIXTURES_PATH / "script_complete.sh"
+    script_complete_sh = FIXTURE_PATH / "script_complete.sh"
     assert script_complete_sh.exists()
     sconfig = kaptan.Kaptan(handler="yaml")
     yaml = config_script_completes.format(
@@ -666,7 +666,7 @@ def test_before_load_true_if_test_passes_with_args(server):
     config_script_completes = test_utils.read_config_file(
         "workspacebuilder/config_script_completes.yaml"
     )
-    script_complete_sh = FIXTURES_PATH / "script_complete.sh"
+    script_complete_sh = FIXTURE_PATH / "script_complete.sh"
     assert script_complete_sh.exists()
     sconfig = kaptan.Kaptan(handler="yaml")
     yaml = config_script_completes.format(script_complete=script_complete_sh)