CI: Add coverage diff reporting (#9)

Author: thekaveman

.github/workflows/pytest.yml

@@ -1,12 +1,22 @@
 name: Pytest
 
 on:
-  - push
-  - pull_request
+  pull_request:
+  push:
+    branches:
+      - "main"
 
 jobs:
   pytest:
     runs-on: ubuntu-latest
+    permissions:
+      # Gives the action the necessary permissions for publishing new
+      # comments in pull requests.
+      pull-requests: write
+      # Gives the action the necessary permissions for pushing data to the
+      # python-coverage-comment-action branch, and for editing existing
+      # comments (to avoid publishing multiple comments in the same PR)
+      contents: write
     steps:
       - name: Check out code
         uses: actions/checkout@v3
@@ -28,8 +38,9 @@ jobs:
       - name: Run tests
         run: ./tests/run.sh
 
-      - name: Upload coverage report
-        uses: actions/upload-artifact@v3
+      - name: Coverage comment
+        uses: py-cov-action/python-coverage-comment-action@v3
         with:
-          name: coverage-report
-          path: ./tests/coverage
+          GITHUB_TOKEN: ${{ github.token }}
+          MINIMUM_GREEN: 90
+          MINIMUM_ORANGE: 80

pyproject.toml

@@ -9,7 +9,7 @@ keywords = ["gam", "google"]
 authors = [
     { name = "Compiler LLC", email = "dev@compiler.la" }
 ]
-requires-python = ">=3.9"
+requires-python = ">=3.11"
 dependencies = [
     "advanced-gam-for-google-workspace @ git+https://github.com/taers232c/GAMADV-XTD3.git@v6.61.14#subdirectory=src"
 ]
@@ -26,8 +26,8 @@ dev = [
     "pre-commit"
 ]
 test = [
+    "coverage",
     "pytest",
-    "pytest-cov",
     "pytest-mock"
 ]
 
@@ -43,6 +43,11 @@ line-length = 127
 target-version = ['py311']
 include = '\.pyi?$'
 
+[tool.coverage.run]
+branch = true
+relative_files = true
+source = ["compiler_admin"]
+
 [tool.pyright]
 include = ["compiler_admin", "tests"]
 

tests/run.sh

@@ -2,7 +2,7 @@
 set -eu
 
 # run normal pytests, skip e2e tests
-pytest -m "not e2e" --cov=compiler_admin --cov-branch --import-mode=importlib
+coverage run -m pytest -m "not e2e"
 
 # clean out old coverage results
 rm -rf ./tests/coverage

tests/web.sh

@@ -0,0 +1,4 @@
+#!/usr/bin/env bash
+set -eu
+
+python -m http.server -d ./tests/coverage