upgrade/fix tests

Author: rmorshea

Diff for: .github/workflows/test.yaml

@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.7, 3.8, 3.9]
+        python-version: ["3.7", "3.8", "3.9", "3.10"]
 
     steps:
       - uses: actions/checkout@v2
@@ -16,6 +16,6 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install Python Dependencies
-        run: pip install -r requirements/ci.txt
+        run: pip install -r requirements/nox-deps.txt
       - name: Run Tests
-        run: tox
+        run: nox -s test

Diff for: flake8_idom_hooks/__init__.py

@@ -1,7 +1,5 @@
-from pkg_resources import (
-    get_distribution as _get_distribution,
-    DistributionNotFound as _DistributionNotFound,
-)
+from pkg_resources import DistributionNotFound as _DistributionNotFound
+from pkg_resources import get_distribution as _get_distribution
 
 try:
     __version__: str = _get_distribution(__name__).version

Diff for: flake8_idom_hooks/exhaustive_deps.py

@@ -1,8 +1,7 @@
 import ast
-from typing import Optional, Union, Set
-
-from .utils import is_hook_def, is_component_def, ErrorVisitor, set_current
+from typing import Optional, Set, Union
 
+from .utils import ErrorVisitor, is_component_def, is_hook_def, set_current
 
 HOOKS_WITH_DEPS = ("use_effect", "use_callback", "use_memo")
 

Diff for: flake8_idom_hooks/rules_of_hooks.py

@@ -1,10 +1,10 @@
 import ast
-from typing import Union, Optional
+from typing import Optional, Union
 
 from .utils import (
-    is_hook_def,
-    is_component_def,
     ErrorVisitor,
+    is_component_def,
+    is_hook_def,
     is_hook_function_name,
     set_current,
 )

Diff for: flake8_idom_hooks/run.py

@@ -2,9 +2,9 @@
 
 import ast
 
-from .utils import ErrorVisitor
 from .exhaustive_deps import ExhaustiveDepsVisitor
 from .rules_of_hooks import RulesOfHooksVisitor
+from .utils import ErrorVisitor
 
 
 def run_checks(

Diff for: flake8_idom_hooks/utils.py

@@ -1,6 +1,6 @@
 import ast
 from contextlib import contextmanager
-from typing import List, Tuple, Iterator, Any
+from typing import Any, Iterator, List, Tuple
 
 
 @contextmanager

Diff for: noxfile.py

@@ -0,0 +1,36 @@
+from pathlib import Path
+
+from nox import Session, session
+
+ROOT = Path(".")
+REQUIREMENTS_DIR = ROOT / "requirements"
+
+
+@session
+def test(session: Session) -> None:
+    session.notify("test_style")
+    session.notify("test_types")
+    session.notify("test_suite")
+
+
+@session
+def test_style(session: Session) -> None:
+    install_requirements(session, "style")
+    session.run("isort", "--check", ".")
+    session.run("flake8", ".")
+
+
+@session
+def test_types(session: Session) -> None:
+    install_requirements(session, "types")
+    session.run("mypy", "--strict", "flake8_idom_hooks")
+
+
+@session
+def test_suite(session: Session) -> None:
+    install_requirements(session, "test-env")
+    session.run("pytest", "tests")
+
+
+def install_requirements(session: Session, name: str) -> None:
+    session.install("-r", str(REQUIREMENTS_DIR / f"{name}.txt"))

Diff for: pyproject.toml

@@ -0,0 +1,6 @@
+[build-system]
+requires = ["setuptools>=42", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[tool.isort]
+profile = "black"

Diff for: requirements.txt

@@ -1,4 +1,5 @@
--r requirements/dev.txt
--r requirements/prod.txt
--r requirements/test.txt
--r requirements/lint.txt
+-r requirements/nox-deps.txt
+-r requirements/pkg-deps.txt
+-r requirements/style.txt
+-r requirements/test-env.txt
+-r requirements/types.txt

Diff for: requirements/ci.txt

@@ -0,0 +1 @@
+nox

Diff for: requirements/dev.txt

@@ -1 +1,2 @@
 flake8 >=3.7
+black

Diff for: requirements/nox-deps.txt

@@ -0,0 +1,4 @@
+flake8 >=3.7
+black
+isort
+flake8-tidy-imports

Diff for: requirements/prod.txt renamed to requirements/pkg-deps.txt

@@ -1,4 +1,2 @@
-flake8 >=3.7
-black
 mypy
 types-setuptools

Diff for: requirements/style.txt

@@ -7,10 +7,10 @@ warn_unused_ignores = True
 [flake8]
 ignore = E203, E266, E501, W503, F811, N802
 max-line-length = 88
-max-complexity = 18
-exclude =
-    .eggs/*
-    .tox/*
+extend-exclude =
+    .nox
+    venv
+    .venv
     tests/hook_usage_test_cases.py
 
 [coverage:report]

Diff for: requirements/test.txt renamed to requirements/test-env.txt

@@ -1,4 +1,5 @@
 import os
+
 import setuptools
 
 # the name of the project
@@ -46,7 +47,7 @@
 
 
 requirements = []
-with open(os.path.join(here, "requirements", "prod.txt"), "r") as f:
+with open(os.path.join(here, "requirements", "pkg-deps.txt"), "r") as f:
     for line in map(str.strip, f):
         if not line.startswith("#"):
             requirements.append(line)

Diff for: requirements/lint.txt renamed to requirements/types.txt

@@ -5,7 +5,6 @@
 
 from flake8_idom_hooks import Plugin
 
-
 options_manager = OptionManager("test", "0.0.0")
 Plugin.add_options(options_manager)