Fix issue 418: Detect pytest-bdd scenario module path on Windows

Author: drcrazy

patches/0001-fix-detect-pytest-bdd-scenario-module-path-on-Window.patch

@@ -0,0 +1,99 @@
+From 08fad6c5527c5c3a79c0cd5f689569a796f1bdbe Mon Sep 17 00:00:00 2001
+From: drcrazy <hcb@wowhcb.ru>
+Date: Tue, 7 Apr 2026 22:09:10 +0300
+Subject: [PATCH] fix: detect pytest-bdd scenario module path on Windows (#418)
+
+Item.location[0] uses backslashes on Windows, so the suffix check for
+pytest_bdd/scenario.py never matched and RP received duplicate items.
+
+Normalize path separators before comparing the suffix.
+
+Closes #418
+
+Made-with: Cursor
+---
+ pytest_reportportal/service.py | 16 +++++++++++++---
+ tests/unit/test_service.py     | 19 +++++++++++++++++++
+ 2 files changed, 32 insertions(+), 3 deletions(-)
+
+diff --git a/pytest_reportportal/service.py b/pytest_reportportal/service.py
+index 685d8b5..39e42b2 100644
+--- a/pytest_reportportal/service.py
++++ b/pytest_reportportal/service.py
+@@ -93,6 +93,16 @@ ALPHA_REGEX = re.compile(r"^\d+_*")
+ BACKGROUND_STEP_NAME = "Background"
+ 
+ 
++def _is_pytest_bdd_scenario_location(location_path: str) -> bool:
++    """
++    Return True if the pytest collection path points at pytest-bdd's scenario module.
++
++    ``Item.location[0]`` uses OS-native separators (backslashes on Windows), so a
++    plain suffix check with ``/`` is not portable. See #418.
++    """
++    return location_path.replace("\\", "/").endswith("/pytest_bdd/scenario.py")
++
++
+ def trim_docstring(docstring: str) -> str:
+     """
+     Convert docstring.
+@@ -907,7 +917,7 @@ class PyTestService:
+         if not self.__started():
+             self.start()
+ 
+-        if PYTEST_BDD and test_item.location[0].endswith("/pytest_bdd/scenario.py"):
++        if PYTEST_BDD and _is_pytest_bdd_scenario_location(test_item.location[0]):
+             self._bdd_item_by_name[test_item.name] = test_item
+             return
+ 
+@@ -928,7 +938,7 @@ class PyTestService:
+         if report.longrepr:
+             self.post_log(test_item, report.longreprtext, log_level="ERROR")
+ 
+-        if PYTEST_BDD and test_item.location[0].endswith("/pytest_bdd/scenario.py"):
++        if PYTEST_BDD and _is_pytest_bdd_scenario_location(test_item.location[0]):
+             return
+ 
+         leaf = self._tree_path[test_item][-1]
+@@ -1011,7 +1021,7 @@ class PyTestService:
+         leaf = self._tree_path[test_item][-1]
+         self._process_metadata_item_finish(leaf)
+ 
+-        if PYTEST_BDD and test_item.location[0].endswith("/pytest_bdd/scenario.py"):
++        if PYTEST_BDD and _is_pytest_bdd_scenario_location(test_item.location[0]):
+             del self._bdd_item_by_name[test_item.name]
+             return
+ 
+diff --git a/tests/unit/test_service.py b/tests/unit/test_service.py
+index b914cde..69e646a 100644
+--- a/tests/unit/test_service.py
++++ b/tests/unit/test_service.py
+@@ -15,6 +15,25 @@
+ 
+ from delayed_assert import assert_expectations, expect
+ 
++from pytest_reportportal.service import _is_pytest_bdd_scenario_location
++
++
++def test_is_pytest_bdd_scenario_location_posix_path():
++    """pytest-bdd scenario items use forward slashes in location on POSIX."""
++    path = "/usr/lib/python3.12/site-packages/pytest_bdd/scenario.py"
++    assert _is_pytest_bdd_scenario_location(path) is True
++
++
++def test_is_pytest_bdd_scenario_location_windows_path():
++    """pytest-bdd scenario items use backslashes in location on Windows (#418)."""
++    path = r"C:\Python312\Lib\site-packages\pytest_bdd\scenario.py"
++    assert _is_pytest_bdd_scenario_location(path) is True
++
++
++def test_is_pytest_bdd_scenario_location_regular_test_module():
++    """Regular tests must not be treated as pytest-bdd scenario glue."""
++    assert _is_pytest_bdd_scenario_location("/project/tests/test_foo.py") is False
++
+ 
+ def test_get_item_parameters(mocked_item, rp_service):
+     """Test that parameters are returned in a way supported by the client."""
+-- 
+2.53.0.windows.2
+