test: add CadQuery executor and CNC validator integration tests

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

tests/test_executor.py

@@ -0,0 +1,104 @@
+"""Tests for core/executor.py — CadQuery code execution and export.
+
+These tests require CadQuery to be installed.
+"""
+
+import pytest
+from pathlib import Path
+from core.executor import sanitize_code, execute_cadquery, export_step, export_stl, export_all
+
+pytestmark = pytest.mark.requires_cadquery
+
+
+class TestSanitizeCode:
+    def test_strips_markdown_fences(self):
+        code = "```python\nresult = 1\n```"
+        assert "```" not in sanitize_code(code)
+
+    def test_strips_plain_fences(self):
+        code = "```\nresult = 1\n```"
+        assert "```" not in sanitize_code(code)
+
+    def test_removes_cadquery_imports(self):
+        code = "import cadquery as cq\nresult = cq.Workplane('XY').box(10,10,10)"
+        cleaned = sanitize_code(code)
+        assert "import cadquery" not in cleaned
+        assert "result" in cleaned
+
+    def test_removes_math_import(self):
+        code = "import math\nresult = cq.Workplane('XY').box(10,10,10)"
+        cleaned = sanitize_code(code)
+        assert "import math" not in cleaned
+
+    def test_preserves_valid_code(self):
+        code = "result = cq.Workplane('XY').box(10, 20, 30)"
+        assert sanitize_code(code) == code
+
+
+class TestExecuteCadquery:
+    def test_simple_box(self):
+        result = execute_cadquery("result = cq.Workplane('XY').box(10, 20, 30)")
+        assert result.success is True
+        assert result.volume > 0
+        assert result.face_count == 6
+        assert result.edge_count == 12
+        assert len(result.bounding_box) == 3
+
+    def test_cylinder(self):
+        result = execute_cadquery("result = cq.Workplane('XY').cylinder(20, 10)")
+        assert result.success is True
+        assert result.volume > 0
+
+    def test_missing_result_variable(self):
+        result = execute_cadquery("x = cq.Workplane('XY').box(10,10,10)")
+        assert result.success is False
+        assert "result" in result.error
+
+    def test_syntax_error(self):
+        result = execute_cadquery("result = cq.Workplane('XY').box(10, 10,")
+        assert result.success is False
+        assert result.error is not None
+
+    def test_wrong_type(self):
+        result = execute_cadquery("result = 42")
+        assert result.success is False
+        assert "Workplane" in result.error
+
+    def test_code_with_markdown_fences(self):
+        code = "```python\nimport cadquery as cq\nresult = cq.Workplane('XY').box(5,5,5)\n```"
+        result = execute_cadquery(code)
+        assert result.success is True
+
+    def test_summary_on_success(self):
+        result = execute_cadquery("result = cq.Workplane('XY').box(10, 20, 30)")
+        summary = result.summary()
+        assert "OK" in summary
+        assert "Volume" in summary
+
+    def test_summary_on_failure(self):
+        result = execute_cadquery("result = bad_code")
+        summary = result.summary()
+        assert "FAILED" in summary
+
+
+class TestExport:
+    def test_export_step(self, tmp_path):
+        exec_result = execute_cadquery("result = cq.Workplane('XY').box(10,10,10)")
+        assert exec_result.success
+        path = export_step(exec_result.result, tmp_path / "test.step")
+        assert path.exists()
+        assert path.suffix == ".step"
+
+    def test_export_stl(self, tmp_path):
+        exec_result = execute_cadquery("result = cq.Workplane('XY').box(10,10,10)")
+        assert exec_result.success
+        path = export_stl(exec_result.result, tmp_path / "test.stl")
+        assert path.exists()
+        assert path.suffix == ".stl"
+
+    def test_export_all(self, tmp_path):
+        exec_result = execute_cadquery("result = cq.Workplane('XY').box(10,10,10)")
+        assert exec_result.success
+        files = export_all(exec_result.result, tmp_path / "part")
+        assert files["step"].exists()
+        assert files["stl"].exists()

tests/test_validator.py

@@ -0,0 +1,76 @@
+"""Tests for core/validator.py — CNC manufacturability validation.
+
+These tests require CadQuery to be installed.
+"""
+
+import pytest
+from core.executor import execute_cadquery
+from core.validator import validate_for_cnc, CNCValidationResult, CNCIssue
+
+pytestmark = pytest.mark.requires_cadquery
+
+
+def _make_solid(code: str):
+    """Helper to create a CadQuery Workplane from code."""
+    result = execute_cadquery(code)
+    assert result.success, f"Code failed: {result.error}"
+    return result.result
+
+
+class TestValidateForCnc:
+    def test_simple_box_is_machinable(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid, "test_box")
+        assert val.machinable is True
+        assert val.error_count == 0
+
+    def test_result_has_part_name(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid, "my_part")
+        assert val.part_name == "my_part"
+
+    def test_axis_recommendation_default_3axis(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid)
+        assert "3-axis" in val.axis_recommendation or "3" in val.axis_recommendation
+
+    def test_complex_part_gets_higher_axis(self):
+        code = '''
+result = cq.Workplane('XY').box(50, 50, 50)
+for i in range(5):
+    result = result.faces('>Z').workplane().pushPoints([(i*8-16, 0)]).hole(3)
+for i in range(5):
+    result = result.faces('>X').workplane().pushPoints([(i*8-16, 0)]).hole(3)
+'''
+        solid = _make_solid(code)
+        val = validate_for_cnc(solid)
+        assert val.part_name is not None
+
+    def test_oversized_part_flagged(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(600, 600, 600)")
+        val = validate_for_cnc(solid, config={"max_part_size_mm": 500.0})
+        assert any(i.category == "Size" for i in val.issues)
+
+    def test_tiny_part_flagged(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(0.5, 0.5, 0.5)")
+        val = validate_for_cnc(solid, config={"min_part_size_mm": 1.0})
+        assert any(i.category == "Size" for i in val.issues)
+
+    def test_summary_format(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid, "test")
+        summary = val.summary()
+        assert isinstance(summary, str)
+        assert "test" in summary
+
+    def test_custom_config(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid, config={"min_wall_thickness_mm": 0.5})
+        assert isinstance(val, CNCValidationResult)
+
+    def test_error_and_warning_counts(self):
+        solid = _make_solid("result = cq.Workplane('XY').box(50, 30, 10)")
+        val = validate_for_cnc(solid)
+        assert val.error_count >= 0
+        assert val.warning_count >= 0
+        assert val.error_count + val.warning_count <= len(val.issues)