b3dkit 0.4.0__tar.gz → 0.5.0__tar.gz

b3dkit-0.5.0/.coveragerc

@@ -0,0 +1,20 @@
+[run]
+source = src/b3dkit
+# `*test*` also matched any source path containing "test", so it was scoped to
+# the test directory instead.
+omit =
+    tests/*
+
+[report]
+show_missing = True
+# CONTRIBUTING.md declares a 90% floor with no ratchet. Enforced here so the
+# number is a gate rather than a suggestion.
+fail_under = 90
+exclude_lines =
+    pragma: no cover
+    # The __main__ blocks are demos, not library code. They were previously
+    # executed by ten tests that asserted nothing, purely so these lines would
+    # count toward coverage; deleting those tests is what this excludes for.
+    if __name__ == .__main__.:
+    # Raised only when the package is not installed, e.g. run from a source tree.
+    except PackageNotFoundError:

{b3dkit-0.4.0 → b3dkit-0.5.0}/CHANGELOG.md

@@ -7,6 +7,41 @@ ship in minor releases.
 
 ## [Unreleased]
 
+## [0.5.0] - 2026-09-09
+
+### Changed
+
+- **`NutCut` now sits on the origin rather than straddling it.** The recess ran
+  from `-head_depth` to `0` while the shaft began at `+head_depth`, so the two
+  never met: the cutter came back as two solids with a `head_depth` gap, and
+  subtracting it left a membrane the bolt could not pass through. It is now one
+  solid spanning `0` to `head_depth + shaft_length`.
+
+  **This moves the cutter.** It previously had to be positioned *below* a
+  surface to work; it can now be placed *on* one. Anything calling `NutCut` and
+  compensating for the old placement needs that compensation removed.
+
+- **`dovetail_subpart` no longer adds each click-fit divot twice.** One copy
+  landed outside the part as a free-floating sliver. Only geometry using
+  `click_fit_radius` is affected; the tail's fused solid is unchanged and the
+  socket's moves by 0.02%.
+
+- **A positive `vertical_offset` no longer cuts the dovetail socket in two.**
+  Tails, `vertical_offset=0` and negative offsets are all unchanged.
+
+### Known issues
+
+Found while adding solid-count assertions, all pre-existing and all invisible to
+the `is_valid` checks that used to stand in for them:
+
+- `TwistSnapConnector` returns five solids: a base and four snapfits that meet
+  only on a shared plane and never fuse.
+- `HexWall` can return partial hexagons clipped at the bounds as separate
+  slivers, depending on how the grid divides the given width.
+- `SNUGTAIL` splits its own tail into disconnected fins on parts wider than
+  about 30mm. Pinned by `TestSnugtailWidthLimit`.
+
+
 ## [0.4.0] - 2026-09-08
 
 ### Removed
@@ -111,6 +146,7 @@ the Python 3.11 floor.
 See the git history. b3dkit began as `fb-library`; the rename accompanied a
 rewrite that reworked names and usage to feel closer to native build123d.
 
-[Unreleased]: https://github.com/x0pherl/b3dkit/compare/v0.4.0...HEAD
+[Unreleased]: https://github.com/x0pherl/b3dkit/compare/v0.5.0...HEAD
+[0.5.0]: https://github.com/x0pherl/b3dkit/releases/tag/v0.5.0
 [0.4.0]: https://github.com/x0pherl/b3dkit/releases/tag/v0.4.0
 [0.3.2]: https://github.com/x0pherl/b3dkit/releases/tag/v0.3.2

{b3dkit-0.4.0 → b3dkit-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: b3dkit
-Version: 0.4.0
+Version: 0.5.0
 Summary: build123d libraries and utilities
 Project-URL: Homepage, https://github.com/x0pherl/b3dkit
 Project-URL: Issues, https://github.com/x0pherl/b3dkit/issues

{b3dkit-0.4.0 → b3dkit-0.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "b3dkit"
-version = "0.4.0"
+version = "0.5.0"
 authors = [
   { name="x0pherl"},
 ]
@@ -63,8 +63,15 @@ build-backend = "hatchling.build"
 [tool.pytest.ini_options]
 pythonpath = ["src"]
 addopts = "--strict-markers"
-markers = [
-    "manual: marks tests that can only be executed manually",
+filterwarnings = [
+    # b3dkit's own warnings are failures; we control those.
+    "error",
+    # Everything below is raised inside the dependency stack, not by b3dkit.
+    # ezdxf, reached through build123d, calls pyparsing's pre-snake_case API.
+    "ignore::DeprecationWarning:ezdxf.*",
+    "ignore::DeprecationWarning:pyparsing.*",
+    "default::DeprecationWarning:build123d.*",
+    "default::PendingDeprecationWarning:build123d.*",
 ]
 
 [tool.black]

{b3dkit-0.4.0 → b3dkit-0.5.0}/src/b3dkit/bolt_fittings.py

@@ -251,13 +251,13 @@ class NutCut(BasePartObject):
         context: BuildPart = BuildPart._get_context()
         validate_inputs(context, self)
 
-        with BuildPart(Location((0, 0, head_depth))) as cut:
+        with BuildPart() as cut:
             with BuildSketch():
                 RegularPolygon(radius=head_radius, side_count=6)
-            extrude(amount=-head_depth)
+            extrude(amount=head_depth)
             Cylinder(
                 radius=shaft_radius,
-                height=shaft_length,
+                height=shaft_length + head_depth,
                 align=(Align.CENTER, Align.CENTER, Align.MIN),
             )
 

{b3dkit-0.4.0 → b3dkit-0.5.0}/src/b3dkit/dovetail.py

@@ -528,18 +528,10 @@ def _traditional_subpart_divots(
             ),
             mode=topmode,
         ):
-            add(
-                Divot(
-                    click_fit_radius,
-                    positive=topmode == Mode.ADD,
-                    extend_base=True,
-                )
-                .rotate(
-                    Axis.X,
-                    (90 * (-1 if vertical_offset < 0 else 1))
-                    + adjusted_top_divot_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                positive=topmode == Mode.ADD,
+                extend_base=True,
             )
         #####################################
         # Bottom divots
@@ -562,29 +554,20 @@ def _traditional_subpart_divots(
             ),
             mode=bottommode,
         ):
-            add(
-                Divot(
-                    click_fit_radius,
-                    positive=bottommode == Mode.ADD,
-                    extend_base=True,
-                )
-                .rotate(
-                    Axis.X,
-                    (90 * (1 if vertical_offset < 0 else -1)) + scarf_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                positive=bottommode == Mode.ADD,
+                extend_base=True,
             )
         with BuildPart(
             Location((end_side.x, end_side.y, click_fit_radius * 2)),
             mode=bottommode,
         ):
-            add(
-                Divot(click_fit_radius, positive=True, extend_base=True)
-                .rotate(
-                    Axis.X,
-                    (90 * (1 if vertical_offset < 0 else -1)) + scarf_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                # was hardcoded True while the sibling divot above derives it
+                positive=bottommode == Mode.ADD,
+                extend_base=True,
             )
 
     return divotedpart.part
@@ -954,7 +937,16 @@ def dovetail_subpart(
         current_floor = (
             part.bounding_box().min.Z
             if vertical_offset <= 0
-            else part.bounding_box().min.Z + abs(vertical_offset) + vertical_tolerance
+            # Must match the straightened slab's top_z above, which uses
+            # vertical_tolerance_adjustment. Using the raw vertical_tolerance
+            # here agreed for the tail, where the adjustment is +tolerance, and
+            # disagreed for the socket, where it is -tolerance: the two slabs
+            # then missed each other by 2 * vertical_tolerance, cutting a slot
+            # clean through the socket and leaving it in two pieces. The
+            # vertical_offset < 0 branch below already uses the adjustment.
+            else part.bounding_box().min.Z
+            + abs(vertical_offset)
+            + vertical_tolerance_adjustment
         )
         add(
             _subpart_slab(

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_antichamfer.py

@@ -1,7 +1,3 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import (
     Align,
@@ -16,7 +12,6 @@ from build123d import (
 )
 
 from b3dkit.antichamfer import anti_chamfer
-from conftest import module_path
 
 
 class TestAntiChamfer:
@@ -122,7 +117,10 @@ class TestAntiChamfer:
             Box(10, 10, 10)
 
         ac_part = anti_chamfer(bp.part.faces().filter_by(Axis.Z)[-1], -1.0, -0.5)
-        assert ac_part.is_valid
+        assert len(ac_part.solids()) == 1
+        # negative lengths still extend the face outward rather than cutting in
+        assert ac_part.volume == pytest.approx(1110.3333, rel=1e-4)
+        assert ac_part.volume > bp.part.volume
 
     def test_anti_chamfer_zero_length_values(self):
         """Test anti_chamfer with negative length values"""
@@ -136,19 +134,6 @@ class TestAntiChamfer:
         assert ac2.is_valid
         assert bp.part.volume == ac1.volume == ac2.volume
 
-    def test_direct_run(self):
-
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("antichamfer"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
-
 
 class TestAntiChamferBuilderContract:
     """anti_chamfer applies to BuildPart only, and must say so.

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_ball_socket.py

@@ -1,13 +1,9 @@
 import math
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
 
 import pytest
 from build123d import Part
 
 from b3dkit.ball_socket import BallMount, BallSocket
-from conftest import module_path
 
 
 # ---------- Helpers ----------
@@ -45,8 +41,11 @@ class TestBallSocket:
     @pytest.mark.parametrize("r,w", [(3, 2), (5, 1), (10, 2), (20, 4), (12.5, 3.5)])
     def test_ball_socket_param_dimensions(self, r, w):
         socket = BallSocket(r, wall_thickness=w)
-        assert socket.is_valid
+        assert len(socket.solids()) == 1
         bbox = socket.bounding_box()
+        # the socket encloses the ball plus a wall on each side
+        assert bbox.size.X == pytest.approx(2 * r + 2 * w)
+        assert socket.volume > 0
 
     def test_ball_socket_custom_wall_thickness(self):
         r, w = 10.0, 3.0
@@ -97,11 +96,3 @@ class TestEdgeCases:
         assert tight.is_valid
         assert loose.is_valid
         assert loose.volume < BallSocket(10.0, tolerance=0.0).volume
-
-
-# ---------- Direct Run ----------
-class TestDirectRun:
-    def test_direct_run(self):
-        with patch("ocp_vscode.show"):
-            loader = SourceFileLoader("__main__", module_path("ball_socket"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_basic_shapes.py

@@ -1,7 +1,3 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import Align, Part
 
@@ -19,7 +15,6 @@ from b3dkit.basic_shapes import (
     opposite_length,
     radius_to_apothem,
 )
-from conftest import module_path
 
 
 class TestApothemConversions:
@@ -265,13 +260,3 @@ class TestDiamondCylinder:
         assert db.Y == pytest.approx(pb.Y)
         assert db.Z == pytest.approx(pb.Z)
         assert diamond.volume == pytest.approx(poly4.volume)
-
-
-class TestBareExecution:
-    def test_bare_execution(self):
-        with (
-            patch("pathlib.Path.mkdir"),
-            patch("ocp_vscode.show"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("basic_shapes"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_bolt_fittings.py

@@ -9,10 +9,6 @@ These tests cover all functions and their parameters to ensure:
 - Mutation testing coverage
 """
 
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import (
     Part,
@@ -26,7 +22,6 @@ from b3dkit.bolt_fittings import (
     SquareNutSinkhole,
     TeardropBoltCutSinkhole,
 )
-from conftest import module_path
 
 
 class TestTeardropBoltCutSinkhole:
@@ -34,19 +29,27 @@ class TestTeardropBoltCutSinkhole:
         """Test teardrop bolt cut with default parameters"""
         sinkhole = TeardropBoltCutSinkhole()
 
-        assert sinkhole.is_valid
+        assert len(sinkhole.solids()) == 1
+        assert sinkhole.volume == pytest.approx(3226.9861, rel=1e-4)
+        assert sinkhole.bounding_box().size.Z == pytest.approx(107.0)
 
     def test_teardrop_bolt_cut_custom_shaft(self):
         """Test teardrop bolt cut with custom shaft dimensions"""
         sinkhole = TeardropBoltCutSinkhole(shaft_radius=2.0, shaft_depth=5.0)
 
-        assert sinkhole.is_valid
+        assert len(sinkhole.solids()) == 1
+        assert sinkhole.volume == pytest.approx(3273.1276, rel=1e-4)
+        # a deeper shaft makes the cutter taller
+        assert sinkhole.bounding_box().size.Z == pytest.approx(110.0)
 
     def test_teardrop_bolt_cut_custom_head(self):
         """Test teardrop bolt cut with custom head dimensions"""
         sinkhole = TeardropBoltCutSinkhole(head_radius=4.0, head_depth=3.0)
 
-        assert sinkhole.is_valid
+        assert len(sinkhole.solids()) == 1
+        assert sinkhole.volume == pytest.approx(5255.4069, rel=1e-4)
+        # a wider head widens the cutter beyond the default 8.2mm
+        assert sinkhole.bounding_box().size.X == pytest.approx(10.0)
 
     def test_teardrop_bolt_cut_with_chamfer(self):
         """Test teardrop bolt cut with various chamfer radii"""
@@ -89,7 +92,11 @@ class TestBoltCutSinkhole:
         """Test bolt cut with default parameters"""
         boltcut = BoltCutSinkhole()
 
-        assert boltcut.is_valid
+        assert len(boltcut.solids()) == 1
+        assert boltcut.volume == pytest.approx(3197.9162, rel=1e-4)
+        # cylindrical, so square in plan unlike the teardrop
+        bbox = boltcut.bounding_box()
+        assert bbox.size.X == pytest.approx(bbox.size.Y)
 
     def test_bolt_cut_with_chamfer(self):
         """Test bolt cut with various chamfer radii"""
@@ -112,7 +119,10 @@ class TestBoltCutSinkhole:
         """Test bolt cut with zero extension (blind hole)"""
         boltcut = BoltCutSinkhole(extension_distance=0)
 
-        assert boltcut.is_valid
+        assert len(boltcut.solids()) == 1
+        assert boltcut.volume == pytest.approx(178.8456, rel=1e-4)
+        # a blind hole is only as deep as the head and shaft
+        assert boltcut.bounding_box().size.Z == pytest.approx(7.0)
 
     def test_bolt_cut_vs_teardrop(self):
         """Test that bolt_cut and TeardropBoltCutSinkhole with default ratio produce different results"""
@@ -158,7 +168,10 @@ class TestSquareNutSinkhole:
         """Test square nut sinkhole with default parameters"""
         sinkhole = SquareNutSinkhole()
 
-        assert sinkhole.is_valid
+        assert len(sinkhole.solids()) == 1
+        assert sinkhole.volume == pytest.approx(1267.7482, rel=1e-4)
+        # the nut trap sets the width: 5.6mm across flats for an M3 nut
+        assert sinkhole.bounding_box().size.X == pytest.approx(5.6)
 
     def test_square_nut_with_extension(self):
         """Test square nut sinkhole with bolt extension"""
@@ -171,7 +184,9 @@ class TestSquareNutSinkhole:
         """Test square nut sinkhole with zero extension"""
         sinkhole = SquareNutSinkhole(bolt_extension=0)
 
-        assert sinkhole.is_valid
+        assert len(sinkhole.solids()) == 1
+        assert sinkhole.volume == pytest.approx(1259.1175, rel=1e-4)
+        assert sinkhole.volume < SquareNutSinkhole().volume
 
     def test_square_nut_different_nut_sizes(self):
         """Test with different nut sizes"""
@@ -193,8 +208,27 @@ class TestScrewCut:
         assert screw.bounding_box().size.Z == pytest.approx(20)
 
     def test_nut_cut(self):
+        """The cutter sits on the origin, in one piece.
+
+        It used to straddle it: the recess ran from -head_depth to 0 while the
+        shaft began at +head_depth, so the two never met and the cutter had to
+        be positioned below a surface rather than on it.
+        """
         nut = NutCut(5, 1, 2, 10)
-        assert nut.is_valid
+        assert len(nut.solids()) == 1
+        bbox = nut.bounding_box()
+        assert bbox.min.Z == pytest.approx(0.0)
+        assert bbox.max.Z == pytest.approx(11.0)
+
+    @pytest.mark.parametrize("head_depth", [0.5, 1, 5])
+    @pytest.mark.parametrize("shaft_length", [6, 10, 20])
+    def test_nut_cut_dimensions(self, head_depth, shaft_length):
+        """The shaft extends shaft_length beyond the recess, from the origin."""
+        nut = NutCut(5, head_depth, 2, shaft_length)
+        assert len(nut.solids()) == 1
+        bbox = nut.bounding_box()
+        assert bbox.min.Z == pytest.approx(0.0)
+        assert bbox.max.Z == pytest.approx(head_depth + shaft_length)
 
     def test_invalid_screw_cut(self):
         with pytest.raises(ValueError):
@@ -206,10 +240,3 @@ class TestScrewCut:
         assert heatsink.bounding_box().size.X == pytest.approx(20)
         assert heatsink.bounding_box().size.Y == pytest.approx(20)
         assert heatsink.bounding_box().size.Z == pytest.approx(11)
-
-
-class TestBareExecution:
-    def test_bare_execution(self):
-        with (patch("ocp_vscode.show"),):
-            loader = SourceFileLoader("__main__", module_path("bolt_fittings"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))

b3dkit-0.5.0/tests/test_click_fit.py

@@ -0,0 +1,19 @@
+import pytest
+
+from b3dkit.click_fit import Divot
+
+
+class TestClickfit:
+    def test_divot(self):
+        hole = Divot(10, False)
+        assert hole.is_valid
+        bump = Divot(10, True)
+        assert bump.is_valid
+        assert hole.volume > bump.volume
+
+    def test_divot_extend_base(self):
+        bump = Divot(10, True, extend_base=True)
+        assert len(bump.solids()) == 1
+        assert bump.volume == pytest.approx(3676.0264, rel=1e-4)
+        # extending the base makes it taller than the plain divot
+        assert bump.bounding_box().size.Z > Divot(10, True).bounding_box().size.Z

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_dovetail.py

@@ -1,7 +1,3 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import Align, Box, BuildPart, Mode, Part, add
 
@@ -14,23 +10,10 @@ from b3dkit.dovetail import (
     dovetail_subpart,
 )
 from b3dkit.point import Point
-from conftest import module_path
 
 
 class TestDovetail:
 
-    def test_direct_run(self):
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("dovetail"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
-
     def test_start_end_match(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
             Box(10, 50, 2, align=(Align.CENTER, Align.CENTER, Align.MIN))
@@ -86,7 +69,10 @@ class TestDovetail:
                     click_fit_radius=0.5,
                 ),
             )
-        assert tail.part.is_valid
+        assert len(tail.part.solids()) == 1
+        assert tail.part.volume == pytest.approx(505.6255, rel=1e-4)
+        # the tail keeps the tongue, so it is longer in Y than the socket
+        assert tail.part.bounding_box().size.Y == pytest.approx(26.8917, rel=1e-4)
 
     def test_valid_socket(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
@@ -103,7 +89,8 @@ class TestDovetail:
                     vertical_offset=-0.5,
                 ),
             )
-        assert socket.part.is_valid
+        assert len(socket.part.solids()) == 1
+        assert socket.part.volume == pytest.approx(650.0268, rel=1e-4)
 
     def test_raises_invalid_style_for_snugtail(self):
         with pytest.raises(ValueError):
@@ -130,7 +117,8 @@ class TestDovetail:
                     vertical_offset=-0.5,
                 ),
             )
-        assert socket.part.is_valid
+        assert len(socket.part.solids()) == 1
+        assert socket.part.volume == pytest.approx(509.5927, rel=1e-4)
 
     def test_valid_tslot_tail(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
@@ -148,7 +136,8 @@ class TestDovetail:
                     vertical_offset=-0.5,
                 ),
             )
-        assert tail.part.is_valid
+        assert len(tail.part.solids()) == 1
+        assert tail.part.volume == pytest.approx(492.0199, rel=1e-4)
 
     def test_valid_snugtail_tail(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
@@ -165,7 +154,10 @@ class TestDovetail:
                     click_fit_radius=1,
                 ),
             )
-        assert tail.part.is_valid
+        assert len(tail.part.solids()) == 1
+        assert tail.part.volume == pytest.approx(346.5381, rel=1e-4)
+        # the click-fit divot stands proud of the 2mm part
+        assert tail.part.bounding_box().max.Z == pytest.approx(2.5)
 
     def test_valid_snugtail_socket(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
@@ -184,7 +176,10 @@ class TestDovetail:
                     click_fit_radius=1,
                 ),
             )
-        assert socket.part.is_valid
+        assert len(socket.part.solids()) == 1
+        assert socket.part.volume == pytest.approx(648.4621, rel=1e-4)
+        # the snugtail socket wraps three sides, so it is much the larger half
+        assert socket.part.volume > 600
 
     def test_snugtail_ratios_exceed_max(self):
         with BuildPart(mode=Mode.PRIVATE) as test:
@@ -377,11 +372,15 @@ class TestStyleConditionalArguments:
         ],
     )
     def test_applicable_argument_accepted(self, style, param, value):
-        assert _subpart(style=style, **{param: value}).is_valid
+        part = _subpart(style=style, **{param: value})
+        assert part.volume > 0
+        assert part.bounding_box().size.Z > 0
 
     @pytest.mark.parametrize("style", list(DovetailStyle))
     def test_defaults_never_raise(self, style):
-        assert _subpart(style=style).is_valid
+        part = _subpart(style=style)
+        assert part.volume > 0
+        assert part.bounding_box().size.Z > 0
 
 
 class TestDovetailSplit:
@@ -449,3 +448,116 @@ class TestDefaultGeometryUnchanged:
     ):
         part = _subpart(style=style, subpart=section)
         assert part.volume == pytest.approx(expected_volume, abs=1e-4)
+
+
+def _narrow_box() -> Part:
+    """A part narrow enough that snugtail's fins meet.
+
+    Snugtail splits its own tail into disconnected fins once the part is about
+    30mm wide -- see TestSnugtailWidthLimit -- so the single-solid tests use a
+    width below that, to guard the defects they are about is_valid rather than
+    that separate one.
+    """
+    with BuildPart(mode=Mode.PRIVATE) as test:
+        Box(20, 50, 10, align=(Align.CENTER, Align.CENTER, Align.MIN))
+    return test.part
+
+
+def _narrow_subpart(**kwargs) -> Part:
+    return dovetail_subpart(_narrow_box(), Point(-10, 0), Point(10, 0), **kwargs)
+
+
+class TestSubpartsAreSingleSolids:
+    """A subpart must come back in one piece, whatever the arguments.
+
+    Two defects hid here for the library's whole history, both invisible to an
+    is_valid assertion because every piece was individually a valid solid.
+
+    A positive vertical_offset built the two Z-slabs with mismatched tolerance
+    terms, cutting a slot clean through the socket and leaving it in two pieces
+    with a tenth of its volume gone.
+
+    click_fit_radius added each divot twice: a Divot is a BasePartObject, so it
+    registers into the enclosing builder on construction, and the code then
+    added a rotated copy as well. Because rotate() turns about the global
+    origin, the builder's Location translated that copy again and it landed
+    outside the part as a free-floating sliver.
+    """
+
+    @pytest.mark.parametrize("style", list(DovetailStyle))
+    @pytest.mark.parametrize("vertical_offset", [0.0, 0.5, -0.5])
+    @pytest.mark.parametrize("click_fit_radius", [0.0, 0.5])
+    @pytest.mark.parametrize("subpart", [DovetailSubpart.TAIL, DovetailSubpart.SOCKET])
+    def test_subpart_is_one_solid(
+        self, style, vertical_offset, click_fit_radius, subpart
+    ):
+        part = _narrow_subpart(
+            style=style,
+            subpart=subpart,
+            vertical_offset=vertical_offset,
+            click_fit_radius=click_fit_radius,
+        )
+        solids = part.solids()
+        assert len(solids) == 1, (
+            f"{style.name} {subpart.name} at vertical_offset={vertical_offset}, "
+            f"click_fit_radius={click_fit_radius} came back as {len(solids)} "
+            f"solids: {sorted(round(s.volume, 4) for s in solids)}"
+        )
+
+    @pytest.mark.parametrize("style", list(DovetailStyle))
+    @pytest.mark.parametrize("vertical_offset", [0.0, 0.5, -0.5])
+    def test_tail_and_socket_account_for_the_whole_part(self, style, vertical_offset):
+        """Together the two subparts should be the original, less tolerance.
+
+        The vertical_offset defect showed up here first: the pair summed to 899
+        of a 1000mm3 box, because the slot cut through the socket removed
+        material that ended up in neither half.
+        """
+        kwargs = dict(style=style, vertical_offset=vertical_offset)
+        total = sum(
+            _narrow_subpart(subpart=s, **kwargs).volume
+            for s in (DovetailSubpart.TAIL, DovetailSubpart.SOCKET)
+        )
+        box_volume = 20 * 50 * 10
+        assert total == pytest.approx(box_volume, rel=0.02), (
+            f"{style.name} at vertical_offset={vertical_offset}: tail and socket "
+            f"sum to {total:.1f}, not the {box_volume} of the part they split"
+        )
+
+
+class TestSnugtailWidthLimit:
+    """Snugtail splits its own tail into fins on parts wider than about 30mm.
+
+    Not a regression: v0.1.5 produces byte-identical geometry. The tail comes
+    back as two or three disconnected pieces with no sliver involved, so it is
+    unrelated to the divot and vertical_offset defects fixed alongside these
+    tests. It lives in the snugtail outline maths, which was tuned against
+    physical prints, so it is recorded here rather than guessed at.
+
+    These tests pin the current boundary. If a fix lands, they should fail and
+    be replaced by an assertion that wide parts work.
+    """
+
+    @staticmethod
+    def _tail_solids(width):
+        with BuildPart(mode=Mode.PRIVATE) as test:
+            Box(width, 60, 30, align=(Align.CENTER, Align.CENTER, Align.MIN))
+        return len(
+            dovetail_subpart(
+                test.part,
+                Point(-width / 2, 0),
+                Point(width / 2, 0),
+                subpart=DovetailSubpart.TAIL,
+                style=DovetailStyle.SNUGTAIL,
+                click_fit_radius=0,
+            ).solids()
+        )
+
+    @pytest.mark.parametrize("width", [20, 24, 28])
+    def test_narrow_parts_give_one_solid(self, width):
+        assert self._tail_solids(width) == 1
+
+    @pytest.mark.parametrize("width", [30, 36, 40])
+    def test_wide_parts_currently_fragment(self, width):
+        """Documents a known defect. Delete this when snugtail is fixed."""
+        assert self._tail_solids(width) > 1

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_hexwall.py

@@ -1,14 +1,10 @@
 from collections import defaultdict
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
 from math import atan2, cos, degrees, sin
-from unittest.mock import patch
 
 import pytest
 from build123d import Align, Box, Cone, Cylinder
 
 from b3dkit.hexwall import HexCylindrical, HexWall
-from conftest import module_path
 
 
 def _cone():
@@ -19,26 +15,21 @@ def _cone():
 class TestHexWall:
     def test_hexwall(self):
         pattern = HexWall(10, 10, 1, 1, 0.2)
-        assert pattern.is_valid
+        assert pattern.volume == pytest.approx(18.5680, rel=1e-4)
+        assert pattern.bounding_box().size.X == pytest.approx(10)
+        assert pattern.bounding_box().size.Y == pytest.approx(10)
+        # 3 solids: the field, plus a 0.1mm3 partial hexagon clipped at each of
+        # the left and right bounds. Suspect, but pinned rather than changed.
+        assert len(pattern.solids()) == 3
 
     def test_hexwall_forces_odd_column_count(self):
         # length=13/apothem=1 yields an even raw column count, exercising the
         # branch that bumps it odd so the hex grid stays centered
         pattern = HexWall(13, 10, 1, 1, 0.2)
-        assert pattern.is_valid
-
-    def test_direct_run(self):
-
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("hexwall"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
+        assert pattern.volume == pytest.approx(23.2053, rel=1e-4)
+        assert pattern.bounding_box().size.X == pytest.approx(13)
+        # the odd column count leaves no clipped hexagons at the bounds
+        assert len(pattern.solids()) == 1
 
 
 class TestHexCylindrical:

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_high_top_slide_box.py

@@ -1,7 +1,3 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import Align, Axis, Box, BuildPart, Compound, Part, fillet
 
@@ -12,7 +8,6 @@ from b3dkit.high_top_slide_box import (
     high_top_slide_box_base,
     high_top_slide_box_lid,
 )
-from conftest import module_path
 
 
 def assert_single_solid(part, name="part"):
@@ -284,19 +279,6 @@ class TestHighTopSlideBox:
         assert_single_solid(result.children[0], "lid")
         assert_single_solid(result.children[1], "base")
 
-    def test_direct_run(self):
-        """Test that the module can be run directly without errors."""
-        with (
-            patch("ocp_vscode.show"),
-            (
-                patch("build123d.export_stl")
-                if hasattr(__import__("build123d"), "export_stl")
-                else patch("builtins.open")
-            ),
-        ):
-            loader = SourceFileLoader("__main__", module_path("high_top_slide_box"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
-
     def test_parameter_validation_edge_cases(self, small_base_part):
         """Test edge cases for parameter validation."""
         # Test with very thin walls

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_slide_box.py

@@ -1,12 +1,7 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import Align, Axis, Box, BuildPart, Plane, fillet, section
 
 from b3dkit.slide_box import _slider_template, slide_box
-from conftest import module_path
 
 
 class TestSlideBox:
@@ -18,21 +13,14 @@ class TestSlideBox:
         sb = slide_box(
             base_box.part, wall_thickness=2, thumb_radius=3.5, divot_radius=0.5
         )
-        assert sb.children[0].is_valid
-        assert sb.children[1].is_valid
-
-    def test_direct_run(self):
-
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("slide_box"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
+        box, lid = sb.children
+        assert (box.label, lid.label) == ("box", "lid")
+        assert len(box.solids()) == 1
+        assert len(lid.solids()) == 1
+        assert box.volume == pytest.approx(4632.0405, rel=1e-4)
+        assert lid.volume == pytest.approx(1498.3503, rel=1e-4)
+        # the lid slides inside the box, so it must be narrower
+        assert lid.bounding_box().size.X < box.bounding_box().size.X
 
 
 class TestSliderDivots:
@@ -47,7 +35,8 @@ class TestSliderDivots:
     """
 
     @pytest.fixture(scope="class")
-    def sketch(self):
+    @classmethod
+    def sketch(cls):
         with BuildPart() as blank:
             Box(60, 100, 20, align=(Align.CENTER, Align.CENTER, Align.MIN))
             fillet(blank.part.edges().filter_by(Axis.Z), radius=1.5)

{b3dkit-0.4.0 → b3dkit-0.5.0}/tests/test_twist_snap.py

@@ -1,29 +1,12 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 
 from b3dkit.twist_snap import (
     TwistSnapConnector,
     TwistSnapSocket,
 )
-from conftest import module_path
 
 
 class TestTwistSnap:
-    def test_bare_execution(self):
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("twist_snap"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
-
     def test_twist_snap_connector(self):
         connector = TwistSnapConnector(
             connector_radius=4.5,
@@ -31,7 +14,12 @@ class TestTwistSnap:
             snapfit_radius_extension=2 * (2 / 3),
             wall_depth=2,
         )
-        assert connector.is_valid
+        assert connector.volume == pytest.approx(275.6677, rel=1e-4)
+        assert connector.bounding_box().size.X == pytest.approx(11.5451, rel=1e-4)
+        # Known issue: the base and its four snapfits meet only on a shared
+        # plane, so they never fuse and this returns five solids. Pinned rather
+        # than asserted as correct; when it is fixed this should read == 1.
+        assert len(connector.solids()) == 5
 
     def test_connector_takes_no_clearance_arguments(self):
         """All fit clearance lives on the socket; the connector is nominal.
@@ -80,4 +68,7 @@ class TestTwistSnap:
             wall_width=2,
             wall_depth=2,
         )
-        assert socket.is_valid
+        assert len(socket.solids()) == 1
+        assert socket.volume == pytest.approx(630.1939, rel=1e-4)
+        # the socket has to swallow the connector, so it is the wider part
+        assert socket.bounding_box().size.X > 11.5451

b3dkit-0.4.0/.coveragerc

@@ -1,6 +0,0 @@
-[run]
-omit =
-    *test*
-    *conftest*
-[report]
-show_missing = True

b3dkit-0.4.0/tests/conftest.py

@@ -1,12 +0,0 @@
-import os
-
-# Absolute path to the package source, so tests that load a module by file path
-# work regardless of the directory pytest was invoked from.
-PACKAGE_DIR = os.path.abspath(
-    os.path.join(os.path.dirname(__file__), "..", "src", "b3dkit")
-)
-
-
-def module_path(module_name: str) -> str:
-    """Absolute path to a b3dkit source file, e.g. module_path("dovetail")."""
-    return os.path.join(PACKAGE_DIR, f"{module_name}.py")

b3dkit-0.4.0/tests/test_click_fit.py

@@ -1,32 +0,0 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
-from b3dkit.click_fit import Divot
-from conftest import module_path
-
-
-class TestClickfit:
-    def test_divot(self):
-        hole = Divot(10, False)
-        assert hole.is_valid
-        bump = Divot(10, True)
-        assert bump.is_valid
-        assert hole.volume > bump.volume
-
-    def test_divot_extend_base(self):
-        bump = Divot(10, True, extend_base=True)
-        assert bump.is_valid
-
-    def test_direct_run(self):
-
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("click_fit"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))