fixup! Update CI and dependencies after 1.5 years

maximz · Feb 24, 2024 · 3eeec85 · 3eeec85
1 parent dbac839
commit 3eeec85
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -89,7 +89,7 @@ jobs:
         run: echo "$IS_PR_TARGETING_MASTER" "$MASTER_PUSH"
       - name: Run tests
         # use temporary directory cleaned up after every job
-        run: pytest --basetemp=${{ runner.temp }} --cov=./ --cov-report xml --mpl --mpl-results-path=tests/results
+        run: pytest --basetemp=${{ runner.temp }} --cov=./ --cov-report xml --run-snapshots --mpl --mpl-results-path=tests/results
         env:
           MPLBACKEND: Agg
       - name: Upload pytest test result artifacts on failure

diff --git a/tests/test_plots.py b/tests/test_plots.py
@@ -374,6 +374,7 @@ def test_add_sample_size_to_legend(categorical_df):
     return fig
 
 
+# Marking as a manual snapshot test so it is skipped when not running in Docker or in Github Actions (controlled by --run-snapshots pytest flag)
 @pytest.mark.snapshot_custom
 def test_pdf_deterministic_output(tmp_path, snapshot):
     # Can't use snapshot_image here because pytest-mpl doesn't support PDF