Update CI cov

- Combines reports
- Adds additional markers for cloud vs not cloud tests

Author: kaitj

.github/workflows/ci.yaml

@@ -1,27 +1,28 @@
 name: CI
 
+permissions:
+  contents: read
+  pull-requests: write
+
 on:
   push:
-    branches: ["main"]
+    branches: [ "main" ]
   pull_request:
-    branches: ["main"]
+    branches: [ "main" ]
 
 env:
   UV_FROZEN: true
+  COVERAGE_PYTHON: "3.12"
 
 jobs:
   format:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
         with:
           submodules: "true"
-      - name: Install uv
-        uses: astral-sh/setup-uv@v5
-        with:
-          pyproject-file: pyproject.toml
-      - name: Install the project
-        run: uv sync --all-extras
+      - uses: astral-sh/setup-uv@v8.1.0
+      - run: uv sync --all-extras
       - name: Check quality
         run: |
           uv run ruff check bids2table tests
@@ -32,26 +33,45 @@ jobs:
     needs: format
     strategy:
       matrix:
-        python-version: ["3.11", "3.12", "3.13"]
+        python-version: [ "3.11", "3.12", "3.13", "3.14" ]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
         with:
           submodules: "true"
-      - name: Install uv with python version
-        uses: astral-sh/setup-uv@v6
+      - uses: astral-sh/setup-uv@v8.1.0
         with:
           python-version: ${{ matrix.python-version }}
 
       - name: Run tests without cloudpathlib
         run: |
-          uv run pytest tests
+          uv run pytest \
+            -m "not cloud" \
+            --junitxml=pytest-cloudless.xml \
+            --cov-report=xml:coverage.xml \
+            --cov bids2table \
+            tests
+
       - name: Run tests with cloudpathlib
         run: |
-          uv run --all-extras pytest \
-            --junitxml=pytest.xml \
+          uv run --extra cloud pytest \
+            -m "cloud" \
+            --junitxml=pytest-cloud.xml \
             --cov-report=xml:coverage.xml \
-            --cov=bids2table tests
-      - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v4
+            --cov=bids2table \
+            --cov-append \
+            tests
+
+      - name: Merge JUnit XML reports
+        run: |
+          uvx junitparser merge \
+            pytest-cloudless.xml \
+            pytest-cloud.xml \
+            pytest.xml
+
+      - name: Pytest coverage comment
+        if: github.event_name == 'pull_request' && matrix.python-version ==
+          env.COVERAGE_PYTHON
+        uses: MishaKav/pytest-coverage-comment@v1
         with:
-          token: ${{ secrets.CODECOV_TOKEN }}
+          pytest-xml-coverage-path: ./coverage.xml
+          junitxml-path: ./pytest.xml

tests/test_indexing.py

@@ -52,6 +52,7 @@ def test_find_bids_datasets():
     assert datasets_no_derivatives == expected_datasets_no_derivatives
 
 
+@pytest.mark.cloud
 @pytest.mark.skipif(
     not cloudpathlib_is_available(), reason="cloudpathlib not installed"
 )
@@ -80,6 +81,7 @@ def test_index_dataset(root: str, expected_count: int):
     assert len(table) == expected_count
 
 
+@pytest.mark.cloud
 @pytest.mark.skipif(
     not cloudpathlib_is_available(), reason="cloudpathlib not installed"
 )

tests/test_metadata.py

@@ -24,6 +24,7 @@ def test_load_bids_metadata(inherit: bool):
     assert metadata == expected_metadata
 
 
+@pytest.mark.cloud
 @pytest.mark.skipif(
     not cloudpathlib_is_available(), reason="cloudpathlib not installed"
 )