Merge pull request #634 from NLeSC/580-ruff-check

Author: abelsiqueira

template/src/{{package_name}}/my_module.py.jinja

@@ -24,7 +24,7 @@ def hello(name: str) -> str:
         'Hello Jane Smith!'
 
     """
-    if name == 'nobody':
-        msg = 'Can not say hello to nobody'
+    if name == "nobody":
+        msg = "Can not say hello to nobody"
         raise ValueError(msg)
-    return f'Hello {name}!'
+    return f"Hello {name}!"

template/tests/test_my_module.py.jinja

@@ -5,22 +5,22 @@ from {{ package_name }}.my_module import hello
 
 def test_hello():
     """Example using assert."""
-    assert hello('nlesc') == 'Hello nlesc!'
+    assert hello("nlesc") == "Hello nlesc!"
 
 
 def test_hello_with_error():
     """Example of testing for raised errors."""
     with pytest.raises(ValueError) as excinfo:
-        hello('nobody')
-    assert 'Can not say hello to nobody' in str(excinfo.value)
+        hello("nobody")
+    assert "Can not say hello to nobody" in str(excinfo.value)
 
 
 @pytest.fixture
 def some_name():
     """Example fixture."""
-    return 'Jane Smith'
+    return "Jane Smith"
 
 
 def test_hello_with_fixture(some_name: str):
     """Example using a fixture."""
-    assert hello(some_name) == 'Hello Jane Smith!'
+    assert hello(some_name) == "Hello Jane Smith!"

tests/test_project.py

@@ -191,7 +191,7 @@ def test_ruff_check(baked_with_development_dependencies, project_env_bin_dir):
     project_dir = baked_with_development_dependencies
     bin_dir = project_env_bin_dir
 
-    result = run([f'{bin_dir}ruff', 'check', '--fix'], project_dir)
+    result = run([f'{bin_dir}ruff', 'check'], project_dir)
     assert result.returncode == 0
     assert '' in result.stdout