Checking availability of nested parameters in json config files.

[namsonx]

No description provided.

[HolQue]

Handling of
"string_val" : "${not_existing}",
"string_val" : "test string",
works now!

But spelling and wording of error message :
"Unknown exception occurs. Details: Loading of The variable 'not_existing' is not available!"
should be reworked. See proposal in ticket description:
"Unknown exception occurs. Details: The variable 'not_existing' is not available!"

Because of the changes: the version number, the version date and the history should be updated.

[HolQue]

Handling of
"string_val" : "${not_existing}",
"string_val" : "test string",
works now!

But spelling and wording of error message :
"Unknown exception occurs. Details: Loading of The variable 'not_existing' is not available!"
should be reworked. See proposal in ticket description:
"Unknown exception occurs. Details: The variable 'not_existing' is not available!"

Because of the changes: the version number, the version date and the history should be updated.

[namsonx]

Sure I will update the error message and the version of python-jsonpreprocessor package also

[namsonx]

Hello @HolQue ,

"Unknown exception occurs. Details: Loading of The variable 'not_existing' is not available!"
should be reworked. See proposal in ticket description:
"Unknown exception occurs. Details: The variable 'not_existing' is not available!"

I have just checked, this wording issue will be fixed in Testsuites-Management package.

Thank you,
Son

[test-fullautomation]

Hi Son,
I don't understand

I have just checked, this wording issue will be fixed in Testsuites-Management package.

json-preprocessor can run also without testsuites-management. Therefore messages of json-preprocessor must be correct.

json-preprocessor need to raise:
"Unknown exception occurs. Details: The variable 'not_existing' is not available!"

Thank you,
Thomas

[namsonx]

Hello Thomas,

In testsuites-management package, I call jsonload() of jsonpreprocessor package in try: ... Exception: . And in this Exception, I added some more words in error log together with error log of jsonload()
That the reason we have this wording issue in ROBFW-AIO.

Thank you,
Son

[test-fullautomation]

Hi Son,
O.K. very good I got it.
Best is always not to add anything to catched exceptions. The exception message itself should contain all required information.
Thank you,
Thomas

[test-fullautomation]

Hi Son,
I will merge the change.
Thanky you,
Thomas

[namsonx]

Hello Thomas,

Sure, this change work fine.
I implemented and have been testing the enhancement features which we discussed last sync up meeting.
I will create another pull-request tomorrow.

Thank you,
Son