Closed HolQue closed 4 weeks ago
Hi @namsonx , can you please look into this together with Holger. Would be for 0.11.0 Thank you, Thomas
Hi Son,
my function require some adaptions. I will change the implementation and make a pull request for the JsonPreprocessor within today. Then it's on you to make a review and check for side effects that possibly I have overlooked.
After this we can decide if this already for 0.11 or not.
Hello Holger,
Your solution looks great especially for end users who will know where they made a mistake in JSON configuration files.
Sure, we will check for side effects then decide if it already for 0.11 or not.
Thank you, Son
Retest successful. Issue can be closed.
integrated in RobotFramework AIO 0.11.0
I always hesitate about JSON errors like that:
Expecting value: line 13 column 15 (char 522)
or
Expecting ',' delimiter: line 4 column 4 (char 76)
How to find the indicated position? What exactly is the JSON code that is used as reference for the counting? Also in case of imported JSON files are involved. Also in case of the entire JSON code is placed in one single line only. Does the counting include code that is commented out?
In my opinion a better support is required here. This is my solution:
It's simple to adapt the JsonPreprocessor to use this function.
Original code:
Adaption:
If this is JSON code containing an error (123 not encapsulated in quotes):
This is the result:
What do you think?