Remove multipolygon is simple test

nyalldawson · nyalldawson · commit 1c0f696f6d4a · 2021-02-04T12:58:29.000+10:00
The behavior in GEOS relating to this was changed in GEOS 3.8 (see libgeos/geos@dddda85) In any case, we have enough other tests for this method to trust that we're correctly delegating to GEOS to do the actual test.
diff --git a/tests/src/core/testqgsgeometry.cpp b/tests/src/core/testqgsgeometry.cpp
@@ -17666,7 +17666,6 @@ void TestQgsGeometry::isSimple_data()
   QTest::newRow( "multiline string simple" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 0), (0 1, 1 1))" ) << true;
   QTest::newRow( "may be touching at endpoints" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 0), (0 0, 1 0))" ) << true;
   QTest::newRow( "must not intersect each other" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 1), (0 1, 1 0))" ) << false;
-  QTest::newRow( "multi-polygons are always simple" ) << QStringLiteral( "MULTIPOLYGON(((0 0, 1 1, 1 1, 0 0)),((0 0, 1 1, 1 1, 0 0)))" ) << true;
 }
 
 void TestQgsGeometry::isSimple()

Original file line number	Diff line number	Diff line change
`@@ -17666,7 +17666,6 @@ void TestQgsGeometry::isSimple_data()`
`17666`	`17666`	`QTest::newRow( "multiline string simple" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 0), (0 1, 1 1))" ) << true;`
`17667`	`17667`	`QTest::newRow( "may be touching at endpoints" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 0), (0 0, 1 0))" ) << true;`
`17668`	`17668`	`QTest::newRow( "must not intersect each other" ) << QStringLiteral( "MULTILINESTRING((0 0, 1 1), (0 1, 1 0))" ) << false;`
`17669`		`- QTest::newRow( "multi-polygons are always simple" ) << QStringLiteral( "MULTIPOLYGON(((0 0, 1 1, 1 1, 0 0)),((0 0, 1 1, 1 1, 0 0)))" ) << true;`
`17670`	`17669`	`}`
`17671`	`17670`
`17672`	`17671`	`void TestQgsGeometry::isSimple()`