✅ Add needs_bash test fixture (#888)

svlandeg · pre-commit-ci[bot] · web-flow · commit a5b7557140ed · 2024-08-24T16:08:27.000-05:00
Co-authored-by: pre-commit-ci[bot] &lt;66853113+pre-commit-ci[bot]@users.noreply.github.com&gt;
diff --git a/tests/test_completion/test_completion.py b/tests/test_completion/test_completion.py
@@ -5,9 +5,10 @@
 
 from docs_src.commands.index import tutorial001 as mod
 
-from ..utils import needs_linux
+from ..utils import needs_bash, needs_linux
 
 
+@needs_bash
 @needs_linux
 def test_show_completion():
     result = subprocess.run(
@@ -23,6 +24,7 @@ def test_show_completion():
     assert "_TUTORIAL001.PY_COMPLETE=complete_bash" in result.stdout
 
 
+@needs_bash
 @needs_linux
 def test_install_completion():
     bash_completion_path: Path = Path.home() / ".bashrc"
diff --git a/tests/utils.py b/tests/utils.py
@@ -2,10 +2,26 @@
 
 import pytest
 
+try:
+    import shellingham
+    from shellingham import ShellDetectionFailure
+
+    shell = shellingham.detect_shell()[0]
+except ImportError:  # pragma: no cover
+    shellingham = None
+    shell = None
+except ShellDetectionFailure:  # pragma: no cover
+    shell = None
+
+
 needs_py310 = pytest.mark.skipif(
     sys.version_info < (3, 10), reason="requires python3.10+"
 )
 
 needs_linux = pytest.mark.skipif(
     not sys.platform.startswith("linux"), reason="Test requires Linux"
 )
+
+needs_bash = pytest.mark.skipif(
+    not shellingham or not shell or "bash" not in shell, reason="Test requires Bash"
+)
