merge pytest and upload artefact to one step

Author: cylammarco

.github/workflows/python-package.yml

@@ -43,12 +43,14 @@ jobs:
       run: |
         pytest --disable-warnings -p no:warnings test --cov=aspired --cov-report=xml --cov-report=term --cov-config=setup.cfg --cov-branch
         ls -la
-        cp .coverage .coverage.${{ matrix.os }}.${{ matrix.python-version }}
-    - name: Upload coverage data
-      uses: actions/upload-artifact@v4
-      with:
-        name: coverage-data-${{ matrix.os }}-${{ matrix.python-version }}
-        path: .coverage.${{ matrix.os }}.${{ matrix.python-version }}
+        cp .coverage .coverage_${{ matrix.os }}_${{ matrix.python-version }}
+        ls -la
+        - uses: actions/upload-artifact@v4
+          if: always()  # Upload even if tests fail
+          with:
+            name: coverage-data-${{ matrix.os }}-${{ matrix.python-version }}
+            path: .coverage_${{ matrix.os }}_${{ matrix.python-version }}
+
     - name: Coveralls
       if: startsWith(matrix.os, 'ubuntu') && endsWith(matrix.python-version, '3.14')
       uses: coverallsapp/github-action@v2