test-fullautomation / python-jsonpreprocessor

A preprocessor for json files
Apache License 2.0
2 stars 2 forks source link

Invalid nested parameter format #117

Closed HolQue closed 4 months ago

HolQue commented 10 months ago

The JSON code

"param1" : [1,2,3],
"param2" : {"${param1}" : "value"}

causes the following error message:

'Invalid nested parameter format: {"${param1}"'

What is an invalid format in this case? This error message makes no sense. This does not help.

Expected is an error message telling, that it's not allowed to substitute parameters of composite data types in dictionary key names.

namsonx commented 10 months ago

Hello Holger,

Thank you for your finding, I will update and handle an error message for this case.

Thank you, Son

HolQue commented 8 months ago

Retest failed.

Error message changed to:

'Could not set variable '[1, 2, 3]' with value 'value'! Reason: cannot assign to literal (, line 1)'!

This makes no sense.

Expected is an error message telling, that it's not allowed to substitute parameters of composite data types in dictionary key names.

namsonx commented 8 months ago

Hello Holger,

Thank you for your finding, I will update the error message.

test-fullautomation commented 8 months ago

removed "fix in review" state.

test-fullautomation commented 8 months ago

Hi @namsonx , what is the status of this ticket? Thank you, Thomas

test-fullautomation commented 8 months ago

Hi @namsonx , what is the status of this ticket? Thank you, Thomas

namsonx commented 8 months ago

Hello Thomas,

I created new commit 0ec343e007 to update the error message log as Holger mentioned.

Thank you, Son

namsonx commented 7 months ago

Hello Holger,

This ticket was fixed with the latest commit on stabi branch.

Thank you, Son

HolQue commented 4 months ago

Retest successful. Issue can be closed.

namsonx commented 4 months ago

merged to develop branch

test-fullautomation commented 4 months ago

solved with version 0.10.0