scores#79: Add E2E tests to workflow and ignore notebooks with nc deps

.github/workflows/python-app.yml

@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.9", "3.10", "3.11"]    
+        python-version: ["3.9", "3.10", "3.11"]
 
     steps:
     - uses: actions/checkout@v3
@@ -38,3 +38,6 @@ jobs:
     - name: Report test coverage
       run: |
         coverage report -m
+    - name: Test tutorial notebooks with pytest
+      run: |
+        coverage run -m pytest --nbmake tutorials/

pyproject.toml

@@ -85,16 +85,13 @@ ignore_missing_imports = true
 source = ["/src/"]
 
 [tool.pytest.ini_options]
-addopts = [
-    '--cov-fail-under=98',
-    '--cov-report=html',
-    '--cov-report=term-missing',
-    '--junitxml=report.xml'
-]
+# Ignore notebooks which depend on external data for E2E tests
+addopts = "--cov-fail-under=98 --cov-report=html --cov-report=term-missing --junitxml=report.xml --ignore='tutorials/First - Data Fetching.ipynb' --ignore='tutorials/Mean Squared Error.ipynb' --ignore='tutorials/Root Mean Squared Error.ipynb' --ignore='tutorials/Weighting Results.ipynb' --ignore='tutorials/Mean Absolute Error.ipynb'"
+
 [tool.pylint.master]
 # https://github.com/PyCQA/pylint/issues/4081#issuecomment-778242554
 init-hook = 'import sys; sys.setrecursionlimit(3 * sys.getrecursionlimit())'
-ignore-paths = [ 
+ignore-paths = [
     'docs/'
 ]