Update test case polygons so some should fail when lat / long is used… #4608
Conversation
|
This PR updates some of the Polygons used in the test spec, which should trigger tests to fail. Polygons were almost all squares that were mirrored over an x=y line, so it did not expose issues where the lat/long values either were or were not swapped appropriately. |
| @@ -129,7 +129,7 @@ describe('Parse.Polygon testing', () => { | |||
| }); | |||
|
|
|||
| it('polygonContain query', (done) => { | |||
| const points1 = [[0,0],[0,1],[1,1],[1,0]]; | |||
There was a problem hiding this comment.
This is the most important change. If the coordinates of either the polygon or the coordinate we are seeing if is contained within the polygon are reversed from what we intend, the check will fail. To be frank, many of the other changes may be completely moot, as they seem to be just validating data that exists and doesn't criss / cross. In all likelihood, everything but this line change could be scrapped to highlight the same results
|
The appropriate test was moved to #4609 |
… improperly