Merge pull request #641 from n3world/issue_362_unconfigure

Issue #362: Only unconfigure if configured

Author: youtux

src/pytest_bdd/plugin.py

@@ -72,7 +72,8 @@ def pytest_configure(config: Config) -> None:
 
 def pytest_unconfigure(config: Config) -> None:
     """Unconfigure all subplugins."""
-    CONFIG_STACK.pop()
+    if CONFIG_STACK:
+        CONFIG_STACK.pop()
     cucumber_json.unconfigure(config)
 
 

tests/test_hooks.py

@@ -135,3 +135,21 @@ def pytest_bdd_after_scenario(request, feature, scenario):
 
     assert scenario_1.name == "Scenario 1"
     assert scenario_2.name == "Scenario 2"
+
+
+def test_pytest_unconfigure_without_configure(pytester):
+    """
+    Simulate a plugin forcing an exit during configuration before bdd is configured
+    https://github.com/pytest-dev/pytest-bdd/issues/362
+    """
+    pytester.makeconftest(
+        """
+    import pytest
+
+    def pytest_configure(config):
+        pytest.exit("Exit during configure", 0)
+        """
+    )
+
+    result = pytester.runpytest()
+    assert result.ret == 0