simplify a bit

Author: Zeroto521

.github/workflows/docs-preview.yml

@@ -36,7 +36,7 @@ jobs:
           path: site/
 
       - name: Test built result
-        run: pytest .\tests\test_documentation_build.py
+        run: pytest -m "documentation"
 
       - name: Deploy docs preview to Netlify
         uses: nwtgck/[email protected]

pytest.ini

@@ -12,4 +12,5 @@ markers =
     spark_functions: tests for pyspark functions
     xarray: tests for xarray functions
     timeseries: tests for timeseries
+    documentation: tests for documentation
     turtle: tests that take more than 5 seconds to execute

tests/test_documentation_build.py

@@ -2,16 +2,12 @@
 
 import pytest
 
-try:
-    import mkdocs
 
-    # Even if 'mkdocs' is installed
-    # 'mkdocs' is could be imported as <module 'mkdocs' (namespace)>
-    # Need to check if 'mkdocs' has '__version__' attribute
-
-    mkdocs_installed = hasattr(mkdocs, "__version__")
-except (ImportError, AttributeError):
-    mkdocs_installed = False
+# Even if 'mkdocs' is installed
+# 'mkdocs' is could be imported as <module 'mkdocs' (namespace)>
+# Need to check if 'mkdocs' has '__version__' attribute
+mkdocs = pytest.importorskip("mkdocs")
+mkdocs_installed = hasattr(mkdocs, "__version__")
 
 
 @pytest.mark.skipif(
@@ -21,6 +17,7 @@
         "And only test documentation in documentation building CI."
     ),
 )
+@pytest.mark.documentation
 def test_docs_general_functions_present():
     """Test that all docs pages build correctly.