Changed stack level and added test for code coverage.

gumyr · Jan 5, 2025 · 3a10788 · 3a10788
1 parent 9841df9
commit 3a10788
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/src/build123d/exporters.py b/src/build123d/exporters.py
@@ -1191,7 +1191,7 @@ def _circle_segments(self, edge: Edge, reverse: bool) -> list[PathSegment]:
         if edge.length < 1e-6:
             warnings.warn(
                 "Skipping arc that is too small to export safely (length < 1e-6).",
-                stacklevel=2,
+                stacklevel=7,
             )
             return []
         curve = edge._geom_adaptor()
@@ -1243,7 +1243,7 @@ def _ellipse_segments(self, edge: Edge, reverse: bool) -> list[PathSegment]:
         if edge.length < 1e-6:
             warnings.warn(
                 "Skipping arc that is too small to export safely (length < 1e-6).",
-                stacklevel=2,
+                stacklevel=7,
             )
             return []
         curve = edge._geom_adaptor()

diff --git a/tests/test_exporters.py b/tests/test_exporters.py
@@ -178,13 +178,19 @@ def test_svg_small_arc(self):
         pnts = ((0, 0), (0, 0.000001), (0.000001, 0))
         small_arc = ThreePointArc(pnts).scale(0.01)
         with self.assertWarns(UserWarning):
-            self.basic_svg_export(small_arc, "test_svg_small_arc.svg")
+            svg_exporter = ExportSVG()
+            segments = svg_exporter._circle_segments(small_arc.edges()[0], False)
+            self.assertEqual(len(segments), 0, "Small arc should produce no segments")
 
     def test_svg_small_ellipse(self):
         pnts = ((0, 0), (0, 0.000001), (0.000002, 0))
         small_ellipse = ThreePointArc(pnts).scale(0.01)
         with self.assertWarns(UserWarning):
-            self.basic_svg_export(small_ellipse, "test_svg_small_ellipse.svg")
+            svg_exporter = ExportSVG()
+            segments = svg_exporter._ellipse_segments(small_ellipse.edges()[0], False)
+            self.assertEqual(
+                len(segments), 0, "Small ellipse should produce no segments"
+            )
 
 
 @pytest.mark.parametrize(