Merge branch 'main' into dev

.github/workflows/dev_fortunatowheels-dev.yml

@@ -2,7 +2,7 @@
 # More GitHub Actions for Azure: https://github.com/Azure/actions
 # More info on Python, GitHub Actions, and Azure App Service: https://aka.ms/python-webapps-actions
 
-name: Build and deploy Python app to Azure Web App - fortunatowheels-dev
+name: Dev - Build/Deploy to Azure-fortunatowheels-dev
 
 on:
   push:

.github/workflows/main_fortunatowheels.yml

@@ -2,7 +2,7 @@
 # More GitHub Actions for Azure: https://github.com/Azure/actions
 # More info on Python, GitHub Actions, and Azure App Service: https://aka.ms/python-webapps-actions
 
-name: Build and deploy Python app to Azure Web App - fortunatowheels
+name: Prod - Build/Deploy to Azure-fortunatowheels
 
 on:
   push:

.github/workflows/testing-coverage.yml

@@ -0,0 +1,46 @@
+# This workflow will install Python dependencies, run tests and lint with a single version of Python
+# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python
+
+name: Testing & Code Coverage
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    branches: [ "main", "dev" ]
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Python 3.10
+      uses: actions/setup-python@v3
+      with:
+        python-version: "3.10"
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install flake8 pytest
+        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+    - name: Lint with flake8
+      run: |
+        # stop the build if there are Python syntax errors or undefined names
+        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+    - name: Test with pytest
+      run: |
+        pytest --cov=src --cov-report=xml
+    - name: Upload coverage reports to Codecov
+      uses: codecov/codecov-action@v3
+      env: 
+        CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+      with:
+          files: ./coverage.xml   # coverage report

README.md

@@ -1,5 +1,12 @@
+[![Contributors][contributors-shield]][contributors-url]
+[![Forks][forks-shield]][forks-url]
+[![Stargazers][stars-shield]][stars-url]
+[![Issues][issues-shield]][issues-url]
+[![MIT License][license-shield]][license-url]
+[![codecov][codecov-shield]][codecov-url]
+
 ![](assets/fortunato-wheels-logo-blue-white.png)
-(Fortunato: Latin word for "lucky")
+(Fortunato: Latin for "lucky")
 
 Fortunato wheels takes the guesswork out of buying a used car. It does this by compiling datasets of used cars from multiple websites, analyzing the data to identify trends and baselines for car prices, then make this information available to users in an interactive tool to evaluate car prices.
 
@@ -31,3 +38,16 @@ To start using Fortunato wheels you can browse through our database of used car
 ## References
 
 - Craigslist Used Cars Dataset, Austin Reese, https://www.kaggle.com/datasets/austinreese/craigslist-carstrucks-data
+
+[contributors-shield]: https://img.shields.io/github/contributors/tieandrews/fortunato-wheels.svg?style=for-the-badge
+[contributors-url]: https://github.com/tieandrews/fortunato-wheels/graphs/contributors
+[forks-shield]: https://img.shields.io/github/forks/tieandrews/fortunato-wheels.svg?style=for-the-badge
+[forks-url]: https://github.com/tieandrews/fortunato-wheels/network/members
+[stars-shield]: https://img.shields.io/github/stars/tieandrews/fortunato-wheels.svg?style=for-the-badge
+[stars-url]: https://github.com/tieandrews/fortunato-wheels/stargazers
+[issues-shield]: https://img.shields.io/github/issues/tieandrews/fortunato-wheels.svg?style=for-the-badge
+[issues-url]: https://github.com/tieandrews/fortunato-wheels/issues
+[license-shield]: https://img.shields.io/github/license/tieandrews/fortunato-wheels.svg?style=for-the-badge
+[license-url]: https://github.com/tieandrews/fortunato-wheels/blob/master/LICENSE.txt
+[codecov-shield]: https://img.shields.io/codecov/c/github/tieandrews/fortunato-wheels?style=for-the-badge
+[codecov-url]: https://codecov.io/gh/tieandrews/fortunato-wheels

codecov.yml

@@ -0,0 +1,2 @@
+ignore:
+  - "*/tests/*”

requirements.txt

@@ -14,4 +14,8 @@ dash-loading-spinners~=1.0
 azure-storage-blob~=12.16
 azure-identity~=1.12
 python-dotenv~=1.0
-python-frontmatter~=1.0
+python-frontmatter~=1.0
+pytest-cov~=4.1
+pytest~=6.2
+coverage~=7.2
+codecov~=2.1

src/pages/explore_ads.py

@@ -423,14 +423,19 @@ def update_filter_options(
     ],
     State("num-ads-summary-store", "data"),
     State("makes-models-store", "data"),
+    State("price-summary-store", "data"),
 )
 def update_ad_filter_count(
-    age_range, price_range, models, makes, num_ads_summary_dict, makes_models_dict
+    age_range, price_range, models, makes, num_ads_summary_dict, makes_models_dict, price_summary_store
 ):
     start_time = time.time()
 
     num_ads_summary_df = pd.DataFrame.from_dict(num_ads_summary_dict, orient="columns")
     makes_models_df = pd.DataFrame.from_dict(makes_models_dict, orient="columns")
+    price_summary_df = pd.DataFrame.from_dict(price_summary_store, orient="columns")
+
+    if price_range is None:
+        price_range = price_summary_df.max().max()
 
     if (len(models) == 0) & (len(makes) == 0):
         # remove some models that are not really models

tests/visualizations/test_explore_ads_plots.py

@@ -2,15 +2,9 @@
 # Date: 2023-05-08
 import os, sys
 import pytest
-
-# on launch ensure src is in path
-cur_dir = os.getcwd()
-try:
-    SRC_PATH = cur_dir[: cur_dir.index("fortunato-wheels") + len("fortunato-wheels")]
-except ValueError:
-    # deal with Azure app service not working with relative imports
-    SRC_PATH = ""
-    pass
+
+# ensure that the parent directory is on the path for relative imports
+SRC_PATH = sys.path.append(os.path.join(os.path.dirname(__file__), "..", ".."))
 if SRC_PATH not in sys.path:
     sys.path.append(SRC_PATH)