(Actually) skip VCR.py related tests if VCRPY_ENCRYPTION_KEY is not…

… available
pawelad · Sep 10, 2024 · 634eaaa · 634eaaa
1 parent 5fe1d90
commit 634eaaa
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/tests/pymonzo/test_accounts.py b/tests/pymonzo/test_accounts.py
@@ -172,7 +172,7 @@ def test_list_respx(
 
     @pytest.mark.vcr()
     @pytest.mark.skipif(
-        os.getenv("VCRPY_ENCRYPTION_KEY") is None,
+        not bool(os.getenv("VCRPY_ENCRYPTION_KEY")),
         reason="`VCRPY_ENCRYPTION_KEY` is not available on GitHub PRs.",
     )
     def test_list_vcr(

diff --git a/tests/pymonzo/test_balance.py b/tests/pymonzo/test_balance.py
@@ -89,7 +89,7 @@ def test_get_respx(
 
     @pytest.mark.vcr()
     @pytest.mark.skipif(
-        os.getenv("VCRPY_ENCRYPTION_KEY") is None,
+        not bool(os.getenv("VCRPY_ENCRYPTION_KEY")),
         reason="`VCRPY_ENCRYPTION_KEY` is not available on GitHub PRs.",
     )
     def test_get_vcr(self, balance_resource: BalanceResource) -> None:

diff --git a/tests/pymonzo/test_pots.py b/tests/pymonzo/test_pots.py
@@ -134,7 +134,7 @@ def test_get_default_pot(
 
     @pytest.mark.vcr()
     @pytest.mark.skipif(
-        os.getenv("VCRPY_ENCRYPTION_KEY") is None,
+        not bool(os.getenv("VCRPY_ENCRYPTION_KEY")),
         reason="`VCRPY_ENCRYPTION_KEY` is not available on GitHub PRs.",
     )
     def test_list_vcr(self, pots_resource: PotsResource) -> None:

diff --git a/tests/pymonzo/test_whoami.py b/tests/pymonzo/test_whoami.py
@@ -19,7 +19,7 @@ class TestWhoAmIResource:
 
     @pytest.mark.vcr()
     @pytest.mark.skipif(
-        os.getenv("VCRPY_ENCRYPTION_KEY") is None,
+        not bool(os.getenv("VCRPY_ENCRYPTION_KEY")),
         reason="`VCRPY_ENCRYPTION_KEY` is not available on GitHub PRs.",
     )
     def test_whoami_vcr(self, whoami_resource: WhoAmIResource) -> None: