Add testing, linting, and format checks

.github/workflows/ci.yml

@@ -0,0 +1,153 @@
+name: CI
+
+on:
+  push:
+    branches:
+    - main
+  pull_request:
+    branches:
+    - main
+
+permissions:
+  contents: read        # For checkout and comparing commits
+  pull-requests: write  # For creating/updating PR comments
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.14'
+
+      - name: Install Poetry
+        uses: abatilo/actions-poetry@v4
+        with:
+          poetry-version: 2.2.1
+
+      - name: Load cached venv
+        id: cached-poetry-dependencies
+        uses: actions/cache/restore@v5
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}
+
+      - name: Install dependencies
+        run: poetry install
+        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
+
+      - name: Cache venv
+        id: cache-poetry-dependencies
+        uses: actions/cache/save@v5
+        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}
+
+  lint:
+    runs-on: ubuntu-latest
+    needs: build
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.14'
+
+      - name: Install Poetry
+        uses: abatilo/actions-poetry@v4
+        with:
+          poetry-version: 2.2.1
+
+      - name: Load cached venv
+        id: cached-poetry-dependencies
+        uses: actions/cache/restore@v5
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}
+
+      - name: Install dependencies
+        run: poetry install
+        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
+
+      - name: Run linter
+        run: |
+          source ./.venv/bin/activate
+          poetry run ruff check --config=pyproject.toml
+
+  format:
+    runs-on: ubuntu-latest
+    needs: build
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.14'
+
+      - name: Install Poetry
+        uses: abatilo/actions-poetry@v4
+        with:
+          poetry-version: 2.2.1
+
+      - name: Load cached venv
+        id: cached-poetry-dependencies
+        uses: actions/cache/restore@v5
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}
+
+      - name: Install dependencies
+        run: poetry install
+        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
+
+      - name: Run linter
+        run: |
+          source ./.venv/bin/activate
+          poetry run ruff format --check --config=pyproject.toml
+
+  test:
+    runs-on: ubuntu-latest
+    needs: build
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.14'
+
+      - name: Install Poetry
+        uses: abatilo/actions-poetry@v4
+        with:
+          poetry-version: 2.2.1
+
+      - name: Load cached venv
+        id: cached-poetry-dependencies
+        uses: actions/cache/restore@v5
+        with:
+          path: .venv
+          key: venv-${{ runner.os }}-${{ hashFiles('**/poetry.lock') }}
+
+      - name: Install dependencies
+        run: poetry install
+        if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
+
+      - name: Run tests with coverage
+        run: | 
+          source ./.venv/bin/activate
+          set -o pipefail
+          poetry run pytest ./tests --junitxml=pytest.xml --cov-report=term-missing:skip-covered --cov=f1p  | tee pytest-coverage.txt
+
+      - name: Pytest Coverage Comment
+        uses: MishaKav/pytest-coverage-comment@v1.2.0
+        with:
+          pytest-coverage-path: ./pytest-coverage.txt
+          junitxml-path: ./pytest.xml
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          create-new-comment: true

.gitignore

@@ -3,4 +3,6 @@ __pycache__/
 *.py[codz]
 *$py.class
 .idea
-.fastf1-cache
+.fastf1-cache
+.coverage
+coverage.xml

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "F1 Player"
-version = "0.0.1"
+version = "0.0.4"
 description = "A tool for watching F1 races from a data perspective."
 authors = [
     "Mitko Toshev <mr.sir88@gmail.com>"
@@ -9,15 +9,15 @@ readme = "README.md"
 packages = [{include = "f1p", from = "src"}]
 
 [tool.poetry.dependencies]
-python = ">=3.13,<4.0"
+python = ">=3.14,<4.0"
 fastf1 = "^3.7.0"
 panda3d = "^1.10.15"
 pandas-stubs = "^2.3.3.251219"
 
-
 [tool.poetry.group.dev.dependencies]
 ruff = "^0.12.8"
 pytest = "8.4.1"
+coverage = "^7.13.1"
 pytest-mock = "3.14.1"
 pytest-cov = "6.2.1"
 pytest-asyncio = "1.1.0"
@@ -30,13 +30,24 @@ requires = ["poetry-core"]
 build-backend = "poetry.core.masonry.api"
 
 [tool.poetry.scripts]
-f1p = "f1p.main:app"
+f1p = "f1p:app"
+
+[tool.coverage.run]
+branch = true
+relative_files = true
+omit = [
+    "tests/*",
+    "src/procedural3d",
+]
 
 [tool.ruff]
-target-version = "py313"
+target-version = "py314"
 line-length = 120
 src = ["src", "tests"]
 preview = true
+exclude = [
+    "src/procedural3d"
+]
 
 [tool.ruff.lint]
 select = ["E", "F", "I", "W", "ARG", "A", "COM", "C4", "PIE", "T20", "PT"]

src/f1p/main.py

@@ -66,7 +66,7 @@ def register_ui_components(self) -> Self:
             30,
             self.symbols_font,
             self.text_font,
-            self.data_extractor
+            self.data_extractor,
         )
 
         circuit_map = Map(self.render, self.data_extractor)
@@ -76,7 +76,7 @@ def register_ui_components(self) -> Self:
             self.symbols_font,
             self.text_font,
             circuit_map,
-            self.data_extractor
+            self.data_extractor,
         )
 
         self.ui_components = [
@@ -90,9 +90,4 @@ def register_ui_components(self) -> Self:
 
 app = F1PlayerApp()
 app.disableMouse()  # disable camera controls
-(
-    app.configure_window()
-    .draw_menu()
-    .register_ui_components()
-    .run()
-)
+(app.configure_window().draw_menu().register_ui_components().run())