_weld_MultiLineString edge cases

jGaboardi · Dec 4, 2020 · 016b775 · 016b775
1 parent 160da22
commit 016b775
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/tigernet/tests/test_utils.py b/tigernet/tests/test_utils.py
@@ -23,6 +23,14 @@ def test__weld_MultiLineString_2(self):
         observed_weld_wkt = utils._weld_MultiLineString(mls).wkt
         self.assertEqual(observed_weld_wkt, known_weld_wkt)
 
+    def test__weld_MultiLineString_3(self):
+        known_weld_wkt = "LINESTRING (1 0, 1 1, 0 1, 0 0)"
+        mls = MultiLineString(
+            (((0, 1.00000001), (0, 0)), ((1, 1), (0, 1)), ((1, 0), (1, 1)))
+        )
+        observed_weld_wkt = utils._weld_MultiLineString(mls).wkt
+        self.assertEqual(observed_weld_wkt, known_weld_wkt)
+
 
 if __name__ == "__main__":
     unittest.main()
diff --git a/tigernet/utils.py b/tigernet/utils.py
@@ -1037,10 +1037,6 @@ def _weld_MultiLineString(multilinestring, weld_multi=True, skip_restr=True):
                 sp1, ep1 = L1.boundary[0], L1.boundary[1]
                 sp2, ep2 = L2.boundary[0], L2.boundary[1]
 
-                # if equal move along
-                if ep1.equals(sp2) or sp1.equals(ep2):
-                    continue
-
                 # if either sets are almost equal pass along the
                 # altered first line and the original second line
                 if ep1.almost_equals(sp2) or sp1.almost_equals(ep2):