microsoft · scwilbanks · Nov 7, 2021 · Nov 7, 2021 · Nov 8, 2021
diff --git a/news/2 Fixes/17954.md b/news/2 Fixes/17954.md
@@ -0,0 +1,2 @@
+Pytest test discovery no longer fails when there are not the same number of open and closed brackets in testcase name
+(thanks [Smith Wilbanks](https://github.com/scwilbanks))
diff --git a/pythonFiles/testing_tools/adapter/pytest/_pytest_item.py b/pythonFiles/testing_tools/adapter/pytest/_pytest_item.py
@@ -448,16 +448,16 @@ def _find_left_bracket(nodeid):
     """
     if not nodeid.endswith("]"):
         return nodeid, "", ""
-    bracketcount = 0
-    for index, char in enumerate(nodeid[::-1]):
-        if char == "]":
-            bracketcount += 1
-        elif char == "[":
-            bracketcount -= 1
-        if bracketcount == 0:
-            n = len(nodeid) - 1 - index
-            return nodeid[:n], nodeid[n], nodeid[n + 1 :]
-    return nodeid, "", ""
+
+    testname_testcase = nodeid.split("::")[-1]
+    nodeid_base = nodeid[: -len(testname_testcase)]
+    open_bracket_n = testname_testcase.find("[")
+
+    if open_bracket_n == -1:
+        return nodeid, "", ""
+
+    n = open_bracket_n + len(nodeid_base)
+    return nodeid[:n], nodeid[n], nodeid[n + 1 :]
 
 
 def _iter_nodes(
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Pytest test discovery no longer fails when there are not the same number of open and closed brackets in testcase name
		(thanks [Smith Wilbanks](https://github.com/scwilbanks))