Skip to content

Commit 8f9f7c1

Browse files
chore: ADDON-80802 Resolve minor review comments
1 parent e751d5d commit 8f9f7c1

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

pytest_splunk_addon/fields_tests/test_templates.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -186,12 +186,12 @@ def test_requirements_fields(
186186
basic_search += f" {param}={param_value}"
187187

188188
if splunk_searchtime_fields_requirements.get("unique_identifier"):
189-
record_property(
190-
"Event_with", splunk_searchtime_fields_requirements["unique_identifier"]
191-
)
192189
unique_identifier = splunk_searchtime_fields_requirements[
193190
"unique_identifier"
194191
]
192+
record_property(
193+
"Event_with", unique_identifier
194+
)
195195

196196
search = f'search {index_list} {basic_search} unique_identifier="{unique_identifier}" | fields *'
197197
else:
@@ -420,10 +420,10 @@ def test_datamodels(
420420
)
421421

422422
if splunk_searchtime_fields_datamodels.get("unique_identifier"):
423+
unique_identifier = splunk_searchtime_fields_datamodels["unique_identifier"]
423424
record_property(
424-
"Event_with", splunk_searchtime_fields_datamodels["unique_identifier"]
425+
"Event_with", unique_identifier
425426
)
426-
unique_identifier = splunk_searchtime_fields_datamodels["unique_identifier"]
427427

428428
self.logger.info(
429429
f'Testing for tag {datamodels} with unique_identifier="{unique_identifier}"'

tests/unit/tests_standard_lib/test_fields_tests/test_test_generator.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -574,8 +574,6 @@ def test_generate_requirement_tests_with_uuid(mock_uuid4):
574574
]
575575

576576
with patch.object(
577-
xml_event_parser, "strip_syslog_header", return_value="escaped_event"
578-
), patch.object(
579577
xml_event_parser, "escape_char_event", return_value="escaped_event"
580578
), patch.object(
581579
pytest, "param", side_effect=lambda x, id: (x, id)
@@ -694,8 +692,6 @@ def test_generate_requirement_datamodel_tests_with_uuid(mock_uuid4):
694692
]
695693

696694
with patch.object(
697-
xml_event_parser, "strip_syslog_header", return_value="escaped_event"
698-
), patch.object(
699695
xml_event_parser, "escape_char_event", return_value="escaped_event"
700696
), patch.object(
701697
pytest, "param", side_effect=lambda x, id: (x, id)

0 commit comments

Comments
 (0)