Add test for regression #45153

qgis · Sep 27, 2021 · bee364e · bee364e
1 parent 157c489
commit bee364e
Showing 1 changed file with 31 additions and 0 deletions.
diff --git a/tests/src/app/maptooladdfeaturepoint/testqgsmaptooladdfeaturepoint.cpp b/tests/src/app/maptooladdfeaturepoint/testqgsmaptooladdfeaturepoint.cpp
@@ -31,6 +31,8 @@
 #include "qgsmapmouseevent.h"
 #include "testqgsmaptoolutils.h"
 
+#include <QSignalSpy>
+
 
 /**
  * \ingroup UnitTests
@@ -47,6 +49,7 @@ class TestQgsMapToolAddFeaturePoint : public QObject
     void cleanupTestCase();// will be called after the last testfunction was executed.
 
     void testPoint();
+    void testMultiPoint(); //!< Test crash from regression GH #45153
 
   private:
     QgisApp *mQgisApp = nullptr;
@@ -139,5 +142,33 @@ void TestQgsMapToolAddFeaturePoint::testPoint()
   mLayerPoint->undoStack()->undo();
 }
 
+void TestQgsMapToolAddFeaturePoint::testMultiPoint()
+{
+
+  QgsAdvancedDigitizingDockWidget dw{mCanvas};
+
+  QgsVectorLayer layerMultiPoint { QStringLiteral( "MultiPoint?crs=EPSG:27700" ), QStringLiteral( "layer multi point" ), QStringLiteral( "memory" ) };
+  layerMultiPoint.startEditing();
+  mCanvas->setCurrentLayer( &layerMultiPoint );
+
+  QgsMapToolDigitizeFeature tool{mCanvas, &dw};
+  QgsFeature feature;
+
+  connect( &tool, &QgsMapToolDigitizeFeature::digitizingCompleted, this, [ & ]( const QgsFeature & f )
+  {
+    feature = f;
+  } );
+
+  QSignalSpy spy( &tool, SIGNAL( digitizingCompleted( const QgsFeature & ) ) );
+  QMouseEvent mouseEvt{QMouseEvent::Type::MouseButtonRelease, QPointF{128, 128}, Qt::MouseButton::LeftButton, Qt::NoButton, Qt::NoModifier};
+  QgsMapMouseEvent evt( mCanvas, &mouseEvt );
+
+  tool.cadCanvasReleaseEvent( &evt );
+
+  QCOMPARE( spy.count(), 1 );
+  QCOMPARE( feature.geometry().asWkt( 1 ).toUpper(), QString( "MULTIPOINT ((2 6))" ) );
+
+}
+
 QGSTEST_MAIN( TestQgsMapToolAddFeaturePoint )
 #include "testqgsmaptooladdfeaturepoint.moc"