test-fullautomation / python-jsonpreprocessor

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

Tiny wording adaption wanted #175

Closed HolQue closed 5 months ago

HolQue commented 7 months ago

The following error message is for sure correct, but also difficult to capture:

''Could not substitute parameter '${param1}_${param2}_suffix'! Composite data types are not allowed. The value of parameter '${param2}' is [1, 2, 3]''!

I would prefer an alternative version like that:

Found expression '${param1}_${param2}_suffix' with at least one parameter of composite data type ('${param2}' is of type 'list'). Because of this expression is the name of a parameter, only simple data types are allowed to be substituted inside.

namsonx commented 5 months ago

Hello Holger,

I pushed the change to stabi branch to update the error message log in this case, I also revised the expected exception of selftest based on the update error message log.

Thank you, Son

HolQue commented 5 months ago

Retest successful. Issue can be closed.

namsonx commented 5 months ago

Merged to develop branch

test-fullautomation commented 5 months ago

solved with version 0.10.0