Skip to content

Commit d2e5b05

Browse files
committed
OP#10323: Add #Requirement links to test docstrings for traceability
1 parent 47de7d3 commit d2e5b05

3 files changed

Lines changed: 7 additions & 2 deletions

File tree

‎tests/unit/OTAnalytics/plugin_parser/test_otconfig_parser.py‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,8 @@ def test_serialize_config_omits_file_references_in_s3_mode(
155155
references have no reader in s3 mode. Recording paths into
156156
`user_source` would only leave stale references after the startup
157157
wipe. See OP#10323's decision log.
158+
159+
#Requirement https://openproject.platomo.de/wp/10323
158160
"""
159161
video_parser = Mock(spec=VideoParser)
160162
flow_parser = Mock(spec=FlowParser)
@@ -204,6 +206,8 @@ def test_convert_agrees_across_calls_in_s3_mode_despite_live_repo_drift(
204206
the two calls and report themselves as permanently unsaved. Putting
205207
the omission inside `convert`, keyed only on `s3_key_prefix`, makes
206208
both calls agree regardless. See decision 4 in OP#10323.
209+
210+
#Requirement https://openproject.platomo.de/wp/10323
207211
"""
208212
video_parser = Mock(spec=VideoParser)
209213
flow_parser = Mock(spec=FlowParser)

‎tests/unit/OTAnalytics/plugin_s3/test_otconfig_upload.py‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ def create_target(given: Given) -> S3OtconfigUpload:
2424

2525
class TestS3OtconfigUpload:
2626
async def test_uploads_under_the_prefix_by_file_name(self, tmp_path: Path) -> None:
27+
"""#Requirement https://openproject.platomo.de/wp/10323"""
2728
given = create_given()
2829
target = create_target(given)
2930
file = tmp_path / "my_project.otconfig"

‎tests/unit/OTAnalytics/plugin_ui/test_nicegui_application.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,14 +100,14 @@ def test_s3_mode_wipes_the_configured_user_source(
100100

101101

102102
class TestOtconfigUpload:
103-
"""#Requirement https://openproject.platomo.de/wp/10323"""
104-
105103
def test_local_mode_never_uploads(self, local_mode: None) -> None:
104+
"""#Requirement https://openproject.platomo.de/wp/10323"""
106105
target = create_target(create_given())
107106

108107
assert isinstance(target.otconfig_upload, NoOtconfigUpload)
109108

110109
def test_s3_mode_uploads_beside_the_project_data(self, s3_mode: None) -> None:
110+
"""#Requirement https://openproject.platomo.de/wp/10323"""
111111
target = create_target(create_given())
112112

113113
assert isinstance(target.otconfig_upload, S3OtconfigUpload)

0 commit comments

Comments
 (0)