Merge pull request #258 from NLeSC/full-coverage

Full coverage

Author: fdiblen

.github/workflows/tests.yml

@@ -32,4 +32,4 @@ jobs:
         python3 -m pip install .[dev]
     - name: Run pytest
       run: |
-        pytest
+        pytest -v

tests/test_project.py

@@ -40,7 +40,7 @@ def test_pytest(baked_with_development_dependencies):
     project_dir, env_bin = baked_with_development_dependencies
     pytest_output = run([f'{env_bin}pytest'], project_dir)
     assert pytest_output.returncode == 0
-    assert '== 3 passed in' in pytest_output.stdout
+    assert '== 4 passed in' in pytest_output.stdout
     assert (project_dir / 'coverage.xml').exists()
     assert (project_dir / 'htmlcov/index.html').exists()
 

{{cookiecutter.project_name}}/tests/test_my_module.py

@@ -6,6 +6,10 @@
 from {{ cookiecutter.package_name }} import my_module
 
 
+def test_hello():
+    assert my_module.hello('nlesc') == 'Hello nlesc!'
+
+
 def test_something():
     assert True
 

{{cookiecutter.project_name}}/{{cookiecutter.package_name}}/my_module.py

@@ -1,9 +1,5 @@
 """Documentation about the {{ cookiecutter.package_name }} module."""
-import logging
-
-
-logger = logging.getLogger(__name__)
 
 # FIXME: put actual code here
-def example():
-    logger.info("Providing information about the excecution of the function.")
+def hello(name):
+  return f'Hello {name}!'