Pull Request Test Coverage Report for Build 10910242816

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 87.555%

Totals
Change from base Build 10899958577:	0.0%
Covered Lines:	2603
Relevant Lines:	2973

💛 - Coveralls

Just to clarify why the test was written as it is/was. I won't loose any sleep on this being changed ;-).

Even though I agree in general, I wrote this test as it is/was to have a clear overview in the parameters how each input WKT is treated depending on the different possible values of on_invalid.
When there is a new relevant case to be added to the test in the future it is also very clear and easy to add this for the different cases, without risk that one of the possibilities gets forgotten.

With the test split up you need to scroll through the different tests and it is quite a bit more difficult to get this overview.

@theroggy I agree, I did lose the synoptic overview, that's fair. I'll improve this. I bet we can have it all.