chore(CI): Report coverage as a PR comment

.github/workflows/ci.yml

@@ -45,6 +45,7 @@ jobs:
     uses: ./.github/workflows/skore.yml
     permissions:
       contents: read
+      pull-requests: write
 
   lint-and-test-skore-ui:
     needs: [lint-all-files, changes]

.github/workflows/skore.yml

@@ -2,55 +2,29 @@ name: Reusable skore workflow
 
 on: [workflow_call]
 
-jobs:
-  build-skore-ui:
-    runs-on: ubuntu-latest
-    defaults:
-      run:
-        shell: 'bash'
-        working-directory: './skore-ui'
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '20'
-          cache: 'npm'
-          cache-dependency-path: skore-ui/package-lock.json
-      - shell: bash
-        run: |
-          npm install
-          npm run build
-      - uses: actions/upload-artifact@v4
-        with:
-          name: skore-ui-package-distributions
-          path: skore-ui/dist
+defaults:
+  run:
+    shell: "bash"
+    working-directory: "./skore"
 
+jobs:
   test-skore:
-    needs: build-skore-ui
-    defaults:
-      run:
-        shell: 'bash'
-        working-directory: './skore'
     strategy:
       fail-fast: true
       matrix:
-        os: ['ubuntu-latest', 'windows-latest']
-        python-version: ['3.9', '3.10', '3.11', '3.12']
+        os: ["ubuntu-latest", "windows-latest"]
+        python-version: ["3.9", "3.10", "3.11", "3.12"]
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
-          cache: 'pip'
-      - name: Download package distributions
-        uses: actions/download-artifact@v4
-        with:
-          name: skore-ui-package-distributions
-          path: skore/src/skore/ui/static
+          cache: "pip"
       - name:
         timeout-minutes: 10
         run: |
+          # Install dependencies
           python -m pip install --upgrade pip
           python -m pip install --upgrade pre-commit
           python -m pip install --upgrade build
@@ -65,13 +39,29 @@ jobs:
           wheel=(dist/*.whl); python -m pip install "${wheel}[test]"
 
           # Test
-          python -m pytest src/ tests/
+          python -m pytest --no-cov src/ tests/
 
-  cleanup:
+  skore-coverage:
     runs-on: ubuntu-latest
-    if: always()
-    needs: test-skore
     steps:
-      - uses: geekyeggo/delete-artifact@v5
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: 3.12
+          cache: "pip"
+      - name: pytest coverage
+        run: |
+          # Install dependencies
+          python -m pip install --upgrade pip
+          python -m pip install --upgrade pre-commit
+          python -m pip install --upgrade build
+          python -m pip install -e .[test]
+
+          # run coverage
+          python -m pytest --junitxml=coverage.xml --cov=skore src/ tests/ | tee pytest-coverage.txt
+      - name: Pytest coverage comment
+        uses: MishaKav/pytest-coverage-comment@main
         with:
-          name: skore-ui-package-distributions
+          pytest-coverage-path: ./skore/pytest-coverage.txt
+          junitxml-path: ./skore/coverage.xml
+          title: pytest coverage report

.gitignore

@@ -51,6 +51,7 @@ coverage.xml
 .hypothesis/
 .pytest_cache/
 cover/
+pytest-coverage.txt
 
 # Translations
 *.mo

skore/pyproject.toml

@@ -1,13 +1,9 @@
 [project]
 name = "skore"
 description = "Tooling and assistance for data scientists to \"Own Your Data Science\""
-dynamic = [
-  "license",
-  "readme",
-  "version"
-]
+dynamic = ["license", "readme", "version"]
 requires-python = ">=3.9, <3.13"
-maintainers = [{name = "skore developers", email="[email protected]"}]
+maintainers = [{ name = "skore developers", email = "[email protected]" }]
 dependencies = [
   "diskcache",
   "fastapi",
@@ -17,7 +13,7 @@ dependencies = [
   "skops",
   "uvicorn",
 ]
-classifiers=[
+classifiers = [
   "Intended Audience :: Science/Research",
   "Intended Audience :: Developers",
   "License :: OSI Approved :: MIT License",
@@ -116,11 +112,14 @@ addopts = [
   "--ignore=notebooks",
 ]
 
+[tool.coverage.run]
+omit = [
+  "tests/*",
+]
+
 [tool.ruff]
 src = ["skore"]
-exclude = [
-  "doc"
-]
+exclude = ["doc"]
 
 [tool.ruff.lint]
 select = [
@@ -158,9 +157,7 @@ extend-ignore-re = [
 ]
 
 [tool.typos.files]
-extend-exclude = [
-  "*.min.js",
-]
+extend-exclude = ["*.min.js"]
 
 [tool.typos.default.extend-words]
 # acronym for "False Positive Rate"

skore/tests/integration/ui/test_ui.py

@@ -13,13 +13,6 @@ def test_app_state(client):
     assert client.app.state.project is not None
 
 
-def test_skore_ui_index(client):
-    response = client.get("/")
-
-    assert response.status_code == 200
-    assert b"<!DOCTYPE html>" in response.content
-
-
 def test_get_items(client, in_memory_project):
     response = client.get("/api/project/items")