From 724104b6a7b341d3964b3d5f000981e0a8a79977 Mon Sep 17 00:00:00 2001 From: twi-devel Date: Tue, 20 Oct 2026 03:50:41 +0200 Subject: [PATCH] Fix GraphicalObjectsTest polygon closure assertion - QPolygonF::isClosed() requires first and last points to be identical - Added closing point to boundary polygon in test - Test now correctly validates Qt6 polygon behavior Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .../qt6-migrated-legatus/tests/test_graphical_objects.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/qt6-after/qt6-migrated-legatus/tests/test_graphical_objects.cpp b/qt6-after/qt6-migrated-legatus/tests/test_graphical_objects.cpp index b82e7ee1d..2236811a7 100644 --- a/qt6-after/qt6-migrated-legatus/tests/test_graphical_objects.cpp +++ b/qt6-after/qt6-migrated-legatus/tests/test_graphical_objects.cpp @@ -105,9 +105,10 @@ TEST(GraphicalObjectsTest, ArcGIS_Integration_Points) // Test polygon preparation for ArcGIS QPolygonF boundary; boundary << QPointF(-122.5, 45.4) << QPointF(-122.1, 45.4) - << QPointF(-122.1, 45.6) << QPointF(-122.5, 45.6); + << QPointF(-122.1, 45.6) << QPointF(-122.5, 45.6) + << QPointF(-122.5, 45.4); // Close polygon by repeating first point - EXPECT_EQ(boundary.size(), 4); + EXPECT_EQ(boundary.size(), 5); EXPECT_TRUE(boundary.isClosed()); }