[Sketcher] add null case to constraint change on deletion checks

AjinkyaDahale · Sep 23, 2024 · 79803cd · 79803cd
1 parent 81c14cd
commit 79803cd
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/tests/src/Mod/Sketcher/App/SketchObject.cpp b/tests/src/Mod/Sketcher/App/SketchObject.cpp
@@ -560,6 +560,8 @@ TEST_F(SketchObjectTest, testConstraintAfterDeletingGeo)
     // Arrange
     int geoId1 = 42, geoId2 = 10, geoId3 = 0, geoId4 = -8;
 
+    Sketcher::Constraint* nullConstr = nullptr;
+
     Sketcher::Constraint constr1;
     constr1.Type = Sketcher::ConstraintType::Coincident;
     constr1.First = geoId1;
@@ -576,6 +578,18 @@ TEST_F(SketchObjectTest, testConstraintAfterDeletingGeo)
     constr2.Third = geoId1;
     constr2.ThirdPos = Sketcher::PointPos::start;
 
+    // Act
+    auto nullConstrAfter = getObject()->getConstraintAfterDeletingGeo(nullConstr, 5);
+
+    // Assert
+    EXPECT_EQ(nullConstrAfter, nullptr);
+
+    // Act
+    getObject()->changeConstraintAfterDeletingGeo(nullConstr, 5);
+
+    // Assert
+    EXPECT_EQ(nullConstr, nullptr);
+
     // Act
     // delete typical in-sketch geo
     auto constr1PtrAfter1 = getObject()->getConstraintAfterDeletingGeo(&constr1, 5);