Reformat test code

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>

Author: pombredanne

.gitignore

@@ -72,3 +72,4 @@ tcl
 
 # Ignore Jupyter Notebook related temp files
 .ipynb_checkpoints/
+/.ruff_cache/

pyproject.toml

@@ -67,16 +67,17 @@ include = [
 [tool.ruff.lint]
 # Rules: https://docs.astral.sh/ruff/rules/
 select = [
-    "E",  # pycodestyle
-    "W",  # pycodestyle warnings
-    "D",  # pydocstyle
-    "F",  # Pyflakes
-    "UP", # pyupgrade
-    "S",  # flake8-bandit
+#    "E",  # pycodestyle
+#    "W",  # pycodestyle warnings
+#    "D",  # pydocstyle
+#    "F",  # Pyflakes
+#    "UP", # pyupgrade
+#    "S",  # flake8-bandit
     "I",  # isort
-    "C9", # McCabe complexity
+#    "C9", # McCabe complexity
 ]
-ignore = ["D1", "D200", "D203", "D205", "D212", "D400", "D415"]
+ignore = ["D1", "D200", "D202", "D203", "D205", "D212", "D400", "D415"]
+
 
 [tool.ruff.lint.isort]
 force-single-line = true
@@ -100,3 +101,5 @@ max-complexity = 10
 
 [tool.ruff.lint.per-file-ignores]
 # Place paths of files to be ignored by ruff here
+"tests/*" = ["S101"]
+"test_*.py" = ["S101"]

tests/test_skeleton_codestyle.py

@@ -7,30 +7,37 @@
 # See https://aboutcode.org for more information about nexB OSS projects.
 #
 
+import configparser
 import subprocess
 import unittest
-import configparser
-
 
 class BaseTests(unittest.TestCase):
     def test_skeleton_codestyle(self):
-        """
-        This test shouldn't run in proliferated repositories.
-        """
+        # This test shouldn't run in proliferated repositories.
+
+        # TODO: update with switch to pyproject.toml
         setup_cfg = configparser.ConfigParser()
         setup_cfg.read("setup.cfg")
         if setup_cfg["metadata"]["name"] != "skeleton":
             return
 
-        args = "venv/bin/black --check -l 100 setup.py etc tests"
+        commands = [
+            ["venv/bin/ruff", "--check"],
+            ["venv/bin/ruff", "format", "--check"],
+        ]
+        command = None
         try:
-            subprocess.check_output(args.split())
+            for command in commands:
+                subprocess.check_output(command)  # noqa: S603
         except subprocess.CalledProcessError as e:
             print("===========================================================")
             print(e.output)
             print("===========================================================")
             raise Exception(
-                "Black style check failed; please format the code using:\n"
-                "  python -m black -l 100 setup.py etc tests",
+                f"Code style and linting command check failed: {' '.join(command)!r}.\n"
+                "You can check and format the code using:\n"
+                "  make valid\n",
+                "OR:\n  ruff format\n",
+                "  ruff check --fix\n",
                 e.output,
             ) from e