chore: Include 'test' dir in style check

Author: bow

justfile

@@ -86,7 +86,7 @@ lint-types:
 
 # Lint style conventions.
 lint-style:
-    ruff check {{src-dir}}
+    ruff check
     black -t py312 --check {{src-dir}} {{test-dir}}
 
 # Lint various metrics.

tests/_import_test.py

@@ -19,7 +19,7 @@ def test_import_file_ok(tmpdir):
         mod = import_file(mod_fp, "volt.test.custom")
 
         assert hasattr(mod, "Test")
-        cls = getattr(mod, "Test")
+        cls = mod.Test
         inst = cls()
         assert inst.val == 1
 

tests/cli_test.py

@@ -176,17 +176,19 @@ def test_build_ok_e2e(
     runner = u.CommandRunner()
     toks = ["build"]
 
-    with runner.isolated_filesystem() as ifs:
-        with isolated_project_dir(ifs, "ok_minimal") as project_dir:
-            output_dir = project_dir / constants.PROJECT_OUTPUT_DIR_NAME
-            assert not output_dir.exists()
+    with (
+        runner.isolated_filesystem() as ifs,
+        isolated_project_dir(ifs, "ok_minimal") as project_dir,
+    ):
+        output_dir = project_dir / constants.PROJECT_OUTPUT_DIR_NAME
+        assert not output_dir.exists()
 
-            res = runner.invoke(cli.root, toks)
-            assert res.exit_code == 0, res.output
+        res = runner.invoke(cli.root, toks)
+        assert res.exit_code == 0, res.output
 
-            assert output_dir.exists()
-            u.assert_dir_contains_only(output_dir, ["assets", "index.html"])
-            u.assert_dir_contains_only(output_dir / "assets", ["style.css"])
+        assert output_dir.exists()
+        u.assert_dir_contains_only(output_dir, ["assets", "index.html"])
+        u.assert_dir_contains_only(output_dir / "assets", ["style.css"])
 
     return None
 
@@ -478,18 +480,20 @@ def test_help_with_xcmd(
 ) -> None:
     runner = u.CommandRunner()
 
-    with runner.isolated_filesystem() as ifs:
-        with isolated_project_dir(ifs, "ok_extended") as project_dir:
-            assert (project_dir / "extension" / "cli.py").exists()
+    with (
+        runner.isolated_filesystem() as ifs,
+        isolated_project_dir(ifs, "ok_extended") as project_dir,
+    ):
+        assert (project_dir / "extension" / "cli.py").exists()
 
-            res0 = runner.invoke(cli.root, [])
-            assert res0.exit_code == 0, res0.output
-            assert " xcmd " in res0.output, res0.output
+        res0 = runner.invoke(cli.root, [])
+        assert res0.exit_code == 0, res0.output
+        assert " xcmd " in res0.output, res0.output
 
-            res1 = runner.invoke(cli.root, ["xcmd"])
-            assert res1.exit_code == 0, res1.output
-            assert " hello-ext " in res1.output, res1.output
+        res1 = runner.invoke(cli.root, ["xcmd"])
+        assert res1.exit_code == 0, res1.output
+        assert " hello-ext " in res1.output, res1.output
 
-            res2 = runner.invoke(cli.root, ["xcmd", "hello-ext"])
-            assert res2.exit_code == 0, res2.output
-            assert "FooBar!" in res2.output, res2.output
+        res2 = runner.invoke(cli.root, ["xcmd", "hello-ext"])
+        assert res2.exit_code == 0, res2.output
+        assert "FooBar!" in res2.output, res2.output

tests/utils.py

@@ -138,13 +138,15 @@ def serve() -> None:
         runner = CommandRunner()
         toks = args or ["serve", "-h", host, "-p", f"{port}", "--no-sig-handlers"]
 
-        with runner.isolated_filesystem() as ifs:
-            with isolation_func(ifs, project_fixture_name) as pd:
-                project_dir = pd
-                sentinel_file = pd / sentinel_project_file
-                assert not sentinel_file.exists()
-
-                runner.invoke(cli.root, toks)
+        with (
+            runner.isolated_filesystem() as ifs,
+            isolation_func(ifs, project_fixture_name) as pd,
+        ):
+            project_dir = pd
+            sentinel_file = pd / sentinel_project_file
+            assert not sentinel_file.exists()
+
+            runner.invoke(cli.root, toks)
 
     def start() -> None:
         nonlocal sentinel_file