test: conditional e2e discovery

When pytest discovers test files to run it executes the files (or something like it). This means that imports must be available at that point in time. Let's add an import check to `conftest.py` to ignore discovery in the e2e subdirectory when playwright is not available. This closes #284. Signed-off-by: Simon de Vlieger <[email protected]>
supakeen · Nov 15, 2024 · c2f346b · c2f346b
1 parent b383e77
commit c2f346b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -4,6 +4,11 @@ Changelog
 ``pinnwand`` is Python pastebin software that tried to keep it simple but got
 a little more complex.
 
+v1.6.1 (unreleased)
+*******************
+* Conditionally discover e2e testcases only when playwright is available in
+  the environment. Closes #284.
+
 v1.6.0 (20241101)
 *******************
 * Added a browser test suite by nekhvoya_.

diff --git a/test/conftest.py b/test/conftest.py
@@ -5,6 +5,12 @@
 from slugify import slugify
 
 
+try:
+    import playwright
+except ImportError:
+    collect_ignore = ["e2e/"]
+
+
 def pytest_addoption(parser):
     parser.addoption(
         "--e2e",