♻️ do not run regression tests by default; update test and lint packages (#213)

Author: ianardee

.github/workflows/test-regression.yml

@@ -40,16 +40,16 @@ jobs:
       uses: actions/cache@v3
       with:
         path: ~/.cache/pip
-        key: ${{ runner.os }}-test-${{ hashFiles('**/setup.cfg') }}
+        key: ${{ runner.os }}-dev-${{ hashFiles('setup.cfg') }}
         restore-keys: |
-          ${{ runner.os }}-test-
+          ${{ runner.os }}-dev-
 
     - name: Install dependencies
       run: |
         python -m pip install pip
-        pip install -e .[test]
+        pip install -e .[dev]
     - name: Run Regression Testing
       env:
           MINDEE_API_KEY: ${{ secrets.MINDEE_API_KEY_SE_TESTS }}
       run: |
-        pytest -m regression tests
+        pytest -m regression

.github/workflows/unit-test.yml

@@ -37,17 +37,17 @@ jobs:
       uses: actions/cache@v3
       with:
         path: ~/.cache/pip
-        key: ${{ runner.os }}-test-${{ hashFiles('**/setup.cfg') }}
+        key: ${{ runner.os }}-dev-${{ hashFiles('setup.cfg') }}
         restore-keys: |
-          ${{ runner.os }}-test-
+          ${{ runner.os }}-dev-
 
     - name: Install dependencies
       run: |
         python -m pip install pip
-        pip install -e .[test]
+        pip install -e .[dev]
 
     - name: Unit testing with pytest
       env:
         MINDEE_API_KEY: ${{ secrets.MINDEE_API_KEY_SE_TESTS }}
       run: |
-        pytest -m "not regression" tests
+        pytest --cov-fail-under 87

.pre-commit-config.yaml

@@ -1,11 +1,11 @@
 repos:
   - repo: https://github.com/ambv/black
-    rev: 23.10.0
+    rev: 24.1.1
     hooks:
       - id: black
 
   - repo: https://github.com/pycqa/isort
-    rev: 5.11.5
+    rev: 5.13.2
     hooks:
       - id: isort
         args: [ "--profile", "black" ]

mindee/cli.py

@@ -293,9 +293,9 @@ def __init__(
             client
             if client
             else Client(
-                api_key=self.parsed_args.api_key
-                if "api_key" in self.parsed_args
-                else None
+                api_key=(
+                    self.parsed_args.api_key if "api_key" in self.parsed_args else None
+                )
             )
         )
         self._set_input()

pyproject.toml

@@ -32,8 +32,11 @@ safe_licenses = [
 ]
 
 [tool.pytest.ini_options]
+addopts = "--pyargs --cov mindee --cov-report term:skip-covered --cov-report term-missing -m 'not regression'"
+python_files = "test*.py"
+junit_family = "xunit2"
 markers = [
-  "regression: marks tests as regression tests (deselect with '-m \"not regression\"')",
+  "regression: marks tests as regression tests - select with '-m regression'",
   "lineitems: debug line items"
 ]
 testpaths = [

setup.cfg

@@ -48,18 +48,13 @@ mindee =
 
 [options.extras_require]
 dev =
-    black==23.10.0
-    mypy==1.6.1
     pylint==2.17.7
     setuptools==51.3.3
-    isort==5.11.5
     pre-commit~=2.21.0
-    pydocstyle==6.3.0
     types-pytz~=2023.3
     types-requests~=2.31
-test =
-    pytest~=7.1
-    pytest-cov~=4.0
+    pytest~=7.4
+    pytest-cov~=4.1
 docs =
     sphinx~=5.3
     sphinx_rtd_theme~=1.1
@@ -70,8 +65,3 @@ build =
 
 [options.packages.find]
 exclude = tests
-
-[tool:pytest]
-addopts = --pyargs --cov-fail-under 85 --cov-config=.coveragerc --cov mindee --cov-report term:skip-covered --cov-report term-missing
-python_files = test*.py
-junit_family = xunit2

tests/test_pkg_versions.py

@@ -30,5 +30,5 @@ def test_style_pkg_versions():
                 pkg_version = lines[idx + 1].strip().split(":")[-1].strip()
                 pre_commit_versions[pkg_name] = pkg_version
 
-    for req in ("black", "pylint", "isort", "pydocstyle", "mypy"):
+    for req in ("pylint",):
         _test_version(requirements_versions, pre_commit_versions, req)