Skip to content

Commit

Permalink
Only update placeholder if pics is enabled (#24795)
Browse files Browse the repository at this point in the history
  • Loading branch information
tehampson authored and pull[bot] committed Sep 22, 2023
1 parent 50ff0ef commit 3532668
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions scripts/py_matter_yamltests/matter_yamltests/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -401,12 +401,13 @@ def __init__(self, test: _TestStepWithPlaceholders, runtime_config_variable_stor
self._runtime_config_variable_storage = runtime_config_variable_storage
self.arguments = copy.deepcopy(test.arguments_with_placeholders)
self.response = copy.deepcopy(test.response_with_placeholders)
self._update_placeholder_values(self.arguments)
self._update_placeholder_values(self.response)
self._test.node_id = self._config_variable_substitution(
self._test.node_id)
test.update_arguments(self.arguments)
test.update_response(self.response)
if test.is_pics_enabled:
self._update_placeholder_values(self.arguments)
self._update_placeholder_values(self.response)
self._test.node_id = self._config_variable_substitution(
self._test.node_id)
test.update_arguments(self.arguments)
test.update_response(self.response)

@property
def is_enabled(self):
Expand Down

0 comments on commit 3532668

Please sign in to comment.