search

test-fullautomation / python-jsonpreprocessor

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

Namsonx/task/stabi branch #181

Closed namsonx closed 5 months ago

test-fullautomation commented 5 months ago

Hi Son,

please consider my comments in corresponding ticket:

#153 (comment) #153 (comment) #153 (comment) #153 (comment)

Hi Son,

please consider the four above linked issues as prio1 bugs. Please provide as soon as possible a pull-request for this.

Thank you, Thomas

namsonx commented 5 months ago

Hi Son, please consider my comments in corresponding ticket: #153 (comment) #153 (comment) #153 (comment) #153 (comment)

Hi Son,

please consider the four above linked issues as prio1 bugs. Please provide as soon as possible a pull-request for this.

Thank you, Thomas

Hello Thomas, Hello Holger,

Thank you for your findings! I resolved all 4 issues based on your comment in the latest commit 34a35aa697fc2 in this pull-request.

Thank you, Son

HolQue commented 5 months ago

Hi Son,

restest mostly successful.

But one curious observation left: JPP_1057 executed separately, causes:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: name 'C' is not defined'

That is OK.

But executed together with other test cases, the error message changes to:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: 2'

Seems that there are still interdependencies.

namsonx commented 5 months ago

Hi Son,

restest mostly successful.

But one curious observation left: JPP_1057 executed separately, causes:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: name 'C' is not defined'

That is OK.

But executed together with other test cases, the error message changes to:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: 2'

Seems that there are still interdependencies.

Hello Holger,

I will check the test case ID JPP_1057 and analyze the root cause.

Thank you, Son

namsonx commented 5 months ago

Hi Son,

restest mostly successful.

But one curious observation left: JPP_1057 executed separately, causes:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: name 'C' is not defined'

That is OK.

But executed together with other test cases, the error message changes to:

'Could not set variable 'testdict[C][C]' with value '4'! Reason: 2'

Seems that there are still interdependencies.

Hello Holger,

I have just pushed new commit a95c86f271780 to fix the issue you mentioned in test case ID JPP_1057

Thank you, Son