Skip to content

engine/tester: report ambiguous argument specification#61

Open
okt-yurijp wants to merge 1 commit into
oktetlabs:mainfrom
okt-yurijp:ambig_arg_error
Open

engine/tester: report ambiguous argument specification#61
okt-yurijp wants to merge 1 commit into
oktetlabs:mainfrom
okt-yurijp:ambig_arg_error

Conversation

@okt-yurijp

Copy link
Copy Markdown
Contributor

Report the error when argument with the same name defined in run item and in parent session.

The following code in package.xml will now cause the error:
<run>
<session>
<arg name="max_len"/>
<run>
<script name="alloc"/>
<arg name="n_iterations">
<value>1000</value>
</arg>
<arg name="min_len">
<value>1</value>
</arg>
<arg name="max_len">
<value>65536</value>
</arg>
</run>
</session>
<arg name="max_len">
<value>65534</value>
<value>65535</value>
</arg>
</run>

This could be a very big problem, because we think, that this could break many suites.

@ol-andrewr ol-andrewr self-assigned this May 13, 2025
Comment thread engine/tester/config_parse.c Outdated
Report the error when argument with the same name defined in run item
and in parent session.

Signed-off-by: Yurij Plotnikov <yurij.plotnikov@arknetworks.am>
Acked-by: Denis Pryazhennikov <denis.pryazhennikov@arknetworks.am>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants