Skip to content

Commit de6a0e2

Browse files
squirrelscLiliDeng
authored andcommitted
doc: allow duplicate test case names in different test suites.
1 parent 88e93c0 commit de6a0e2

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

docs/tools/test_spec_gen.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ def update_file() -> None:
4848
for suite in extract_metadata(cls_visitor.get_suites()):
4949
_write_suite(test_spec, suite)
5050
for case in extract_metadata(func_visitor.get_cases()):
51-
_write_case(test_spec, case)
51+
_write_case(test_spec, case, suite["name"])
5252

5353

5454
def _write_title(file: TextIO) -> None:
@@ -83,15 +83,18 @@ def _write_suite(file: TextIO, metadata: Dict[str, str]) -> None:
8383
file.write(f" :category: ``{metadata['category']}``\n\n") # Category
8484

8585

86-
def _write_case(file: TextIO, metadata: Dict[str, str]) -> None:
86+
def _write_case(file: TextIO, metadata: Dict[str, str], suite_name: str = "") -> None:
8787
"""
8888
Writes info of a test case.
8989
9090
Args:
9191
file (TextIO): test spec file
9292
metadata (Dict[str, str]): test case metadata
93+
suite_name (str): name of the parent test suite
9394
"""
94-
file.write(f".. _{metadata['name']}:\n\n") # custom anchor
95+
# Create unique anchor by combining suite and case name if suite is provided
96+
anchor_name = f"{suite_name}_{metadata['name']}" if suite_name else metadata["name"]
97+
file.write(f".. _{anchor_name}:\n\n") # custom anchor
9598
file.write(f" .. method:: {metadata['name']}\n") # Test Case Name
9699
file.write(" :noindex:\n\n")
97100

docs/tools/test_summary_gen.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,12 +85,15 @@ def _update_line(file: TextIO, metadata: Dict[str, str], index: int) -> None:
8585
metadata (Dict[str, str]): test case metadata
8686
index (int): no.# of test case
8787
"""
88+
# Create unique anchor by combining suite and case name
89+
anchor_name = f"{metadata['suite_name']}_{metadata['case_name']}"
90+
8891
file.write(f" * - {index}\n") # noqa: E221
8992
file.write(
9093
f" - :ref:`{metadata['suite_name']} <{metadata['suite_name']}>`\n" # noqa: E221,E501
9194
) # Test Suite Name
9295
file.write(
93-
f" - :ref:`{metadata['case_name']} <{metadata['case_name']}>`\n" # noqa: E221,E501
96+
f" - :ref:`{metadata['case_name']} <{anchor_name}>`\n" # noqa: E221,E501
9497
) # Test Case Name
9598
file.write(f" - {metadata.get('priority', 2)}\n") # noqa: E221
9699
file.write(" - Azure, Ready\n") # Platform - defaults to both

0 commit comments

Comments
 (0)