Skip to content

Commit 18d4059

Browse files
committedNov 4, 2018
Correct test comparisons
Previously QCOMPARE( QgsGeometry, QgsGeometry ) was actually only testing for QCOMPARE( bool(QgsGeometry), bool(QgsGeometry) )...!
1 parent 7be2925 commit 18d4059

File tree

1 file changed

+13
-16
lines changed

1 file changed

+13
-16
lines changed
 

‎tests/src/core/testqgsgeometry.cpp

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -16933,7 +16933,7 @@ void TestQgsGeometry::minimalEnclosingCircle()
1693316933
QCOMPARE( radius, 0.0 );
1693416934
QVERIFY( result.isNull() );
1693516935

16936-
// caase 1
16936+
// case 1
1693716937
geomTest = QgsGeometry::fromPointXY( QgsPointXY( 5, 5 ) );
1693816938
result = geomTest.minimalEnclosingCircle( center, radius );
1693916939
QCOMPARE( center, QgsPointXY( 5, 5 ) );
@@ -16943,47 +16943,44 @@ void TestQgsGeometry::minimalEnclosingCircle()
1694316943

1694416944
// case 2
1694516945
geomTest = QgsGeometry::fromWkt( QStringLiteral( "MULTIPOINT( 3 8, 7 4 )" ) );
16946-
result = geomTest.minimalEnclosingCircle( center, radius );
16946+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1694716947
QGSCOMPARENEARPOINT( center, QgsPointXY( 5, 6 ), 0.0001 );
1694816948
QGSCOMPARENEAR( radius, sqrt( 2 ) * 2, 0.0001 );
16949-
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16950-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16949+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((7 4, 4.3 3.3, 2.3 5.3, 3 8, 5.7 8.7, 7.7 6.7, 7 4))" ) );
1695116950

1695216951
geomTest = QgsGeometry::fromWkt( QStringLiteral( "LINESTRING( 0 5, 2 2, 0 -5, -1 -1 )" ) );
16953-
result = geomTest.minimalEnclosingCircle( center, radius );
16952+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1695416953
QGSCOMPARENEARPOINT( center, QgsPointXY( 0, 0 ), 0.0001 );
1695516954
QGSCOMPARENEAR( radius, 5, 0.0001 );
16956-
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16957-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16955+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((0 5, 4.3 2.5, 4.3 -2.5, -0 -5, -4.3 -2.5, -4.3 2.5, 0 5))" ) );
1695816956

1695916957
geomTest = QgsGeometry::fromWkt( QStringLiteral( "MULTIPOINT( 0 5, 2 2, 0 -5, -1 -1 )" ) );
16960-
result = geomTest.minimalEnclosingCircle( center, radius );
16958+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1696116959
QGSCOMPARENEARPOINT( center, QgsPointXY( 0, 0 ), 0.0001 );
1696216960
QGSCOMPARENEAR( radius, 5, 0.0001 );
16963-
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16964-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16961+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((0 5, 4.3 2.5, 4.3 -2.5, -0 -5, -4.3 -2.5, -4.3 2.5, 0 5))" ) );
1696516962

1696616963
geomTest = QgsGeometry::fromWkt( QStringLiteral( "POLYGON(( 0 5, 2 2, 0 -5, -1 -1 ))" ) );
16967-
result = geomTest.minimalEnclosingCircle( center, radius );
16964+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1696816965
QGSCOMPARENEARPOINT( center, QgsPointXY( 0, 0 ), 0.0001 );
1696916966
QGSCOMPARENEAR( radius, 5, 0.0001 );
1697016967
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16971-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16968+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((0 5, 4.3 2.5, 4.3 -2.5, -0 -5, -4.3 -2.5, -4.3 2.5, 0 5))" ) );
1697216969

1697316970
geomTest = QgsGeometry::fromWkt( QStringLiteral( "MULTIPOINT( 0 5, 0 -5, 0 0 )" ) );
16974-
result = geomTest.minimalEnclosingCircle( center, radius );
16971+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1697516972
QGSCOMPARENEARPOINT( center, QgsPointXY( 0, 0 ), 0.0001 );
1697616973
QGSCOMPARENEAR( radius, 5, 0.0001 );
1697716974
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16978-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16975+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((0 5, 4.3 2.5, 4.3 -2.5, -0 -5, -4.3 -2.5, -4.3 2.5, 0 5))" ) );
1697916976

1698016977
// case 3
1698116978
geomTest = QgsGeometry::fromWkt( QStringLiteral( "MULTIPOINT((0 0), (5 5), (0 -5), (0 5), (-5 0))" ) );
16982-
result = geomTest.minimalEnclosingCircle( center, radius );
16979+
result = geomTest.minimalEnclosingCircle( center, radius, 6 );
1698316980
QGSCOMPARENEARPOINT( center, QgsPointXY( 0.8333, 0.8333 ), 0.0001 );
1698416981
QGSCOMPARENEAR( radius, 5.8926, 0.0001 );
1698516982
resultTest.set( QgsCircle( QgsPoint( center ), radius ).toPolygon( 36 ) );
16986-
QCOMPARE( result.asWkt(), resultTest.asWkt() );
16983+
QCOMPARE( result.asWkt( 1 ), QStringLiteral( "Polygon ((0.8 6.7, 5.9 3.8, 5.9 -2.1, 0.8 -5.1, -4.3 -2.1, -4.3 3.8, 0.8 6.7))" ) );
1698716984

1698816985
}
1698916986

0 commit comments

Comments
 (0)
Please sign in to comment.