Merge pull request #48 from predict-idlab/dev-jonas

🙏 update testing

Author: jonasvdd

.github/workflows/test.yml

@@ -6,39 +6,54 @@ name: Unit tests
 on:
   push:
     branches: [ main ]
+    paths:
+      - "powershap/**"
+      - "tests/**"
+      - "poetry.lock"
+      - ".github/workflows/test.yml"
   pull_request:
     branches: [ main ]
+    paths:
+      - "powershap/**"
+      - "tests/**"
+      - "poetry.lock"
+      - ".github/workflows/test.yml"
 
 jobs:
-  build:
+  test:
 
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
-        os: ['ubuntu-latest']
+        os: ['ubuntu-latest']  # TODO - do we add other OSes?
         python-version: ["3.7", "3.8", "3.9", "3.10"]
+    defaults:
+      run:
+        shell: bash
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v4
       with:
         python-version: ${{ matrix.python-version }}
 
     - name: Install Poetry
       uses: snok/install-poetry@v1
+      with:
+        version: 1.5.1
     - name: Cache poetry
       id: cached-poetry-dependencies
-      uses: actions/cache@v2
+      uses: actions/cache@v3
       with:
         path: ~/.cache/pypoetry/virtualenvs
         key: ${{ runner.os }}-poetry-${{ hashFiles('poetry.lock') }}-python-${{ matrix.python-version }}
     - run: poetry --version
 
     - run: sudo apt-get install llvm  # Because https://github.com/slundberg/shap/issues/1854
     - name: Install dependencies
-      run: poetry install
+      run: poetry install --all-extras
       # Do not use caching (anymore)
 #      if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
 
@@ -49,16 +64,16 @@ jobs:
 #         # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
 #         flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
     - name: Test with pytest
+
       run: |
         poetry run pytest --cov=powershap --junitxml=junit/test-results-${{ matrix.python-version }}.xml --cov-report=xml tests
-
     - name: Upload pytest test results
-      uses: actions/upload-artifact@v2
+      uses: actions/upload-artifact@v3
       with:
         name: pytest-results-${{ matrix.python-version }}
         path: junit/test-results-${{ matrix.python-version }}.xml
         # Use always() to always run this step to publish test results when there are test failures
       if: ${{ always() }}
 
     - name: Upload coverage to Codecov
-      uses: codecov/codecov-action@v1
+      uses: codecov/codecov-action@v3