From 3eeec85b4ea340ac5eb0bc963e676faef25dbd5d Mon Sep 17 00:00:00 2001
From: Maxim Zaslavsky <maxim@maximzaslavsky.com>
Date: Fri, 23 Feb 2024 23:36:55 -0800
Subject: [PATCH] fixup! Update CI and dependencies after 1.5 years

---
 .github/workflows/ci.yaml | 2 +-
 tests/test_plots.py       | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index 28a015c..c3ca9a1 100644
--- 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
index f70c38d..e8fa88b 100644
--- 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