@design-core/core 0.0.6 → 50.1.0

package/test/tools/chamfer.test.js

@@ -1,6 +1,7 @@
 import { Core } from '../../core/core/core.js';
 import { Point } from '../../core/entities/point.js';
 import { Chamfer } from '../../core/tools/chamfer.js';
+import { DesignCore } from '../../core/designCore.js';
 import { Strings } from '../../core/lib/strings.js';
 import { SingleSelection } from '../../core/lib/selectionManager.js';
 import { expect, jest } from '@jest/globals';
@@ -20,7 +21,7 @@ test('Chamfer.register returns correct command object', () => {
 
 test('Chamfer.action does nothing when entities are not set', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(10, 0)] });
   const countBefore = core.scene.entities.count();
 
   const chamfer = new Chamfer();
@@ -31,8 +32,8 @@ test('Chamfer.action does nothing when entities are not set', () => {
 
 test('Chamfer.action notifies when first entity type is not supported', () => {
   core.scene.clear();
-  core.scene.addItem('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(10, 0)] });
 
   const notifySpy = jest.spyOn(core, 'notify');
 
@@ -49,8 +50,8 @@ test('Chamfer.action notifies when first entity type is not supported', () => {
 
 test('Chamfer.action notifies when second entity type is not supported', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(10, 0)] });
-  core.scene.addItem('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
 
   const notifySpy = jest.spyOn(core, 'notify');
 
@@ -67,8 +68,8 @@ test('Chamfer.action notifies when second entity type is not supported', () => {
 
 test('Chamfer.action notifies for parallel lines', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(10, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 5), new Point(10, 5)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 5), new Point(10, 5)] });
 
   const notifySpy = jest.spyOn(core, 'notify');
 
@@ -89,8 +90,8 @@ test('Chamfer.action dist=0 trimMode=true trims both lines to intersection', ()
   // Lines only touching when extended: horizontal (0,0)→(8,0), vertical (10,-10)→(10,-2)
   // Infinite extensions meet at (10, 0).
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(8, 0)] });
-  core.scene.addItem('Line', { points: [new Point(10, -10), new Point(10, -2)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(8, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(10, -10), new Point(10, -2)] });
 
   core.scene.headers.chamferDistanceA = 0;
   core.scene.headers.chamferDistanceB = 0;
@@ -121,8 +122,8 @@ test('Chamfer.action dist=0 trimMode=true trims both lines to intersection', ()
 
 test('Chamfer.action dist=0 trimMode=false makes no changes', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(8, 0)] });
-  core.scene.addItem('Line', { points: [new Point(10, -10), new Point(10, -2)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(8, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(10, -10), new Point(10, -2)] });
 
   core.scene.headers.chamferDistanceA = 0;
   core.scene.headers.chamferDistanceB = 0;
@@ -145,8 +146,8 @@ test('Chamfer.action distance method trimMode=true adds chamfer line and trims',
   // Perpendicular L-corner at origin: horizontal (-10,0)→(0,0), vertical (0,0)→(0,10)
   // Equal distances of 2 → chamfer endpoints at (-2,0) and (0,2)
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -190,8 +191,8 @@ test('Chamfer.action distance method trimMode=true adds chamfer line and trims',
 test('Chamfer.action distance method asymmetric distances', () => {
   // L-corner at origin, distA=3 along horizontal, distB=1 along vertical
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 3;
   core.scene.headers.chamferDistanceB = 1;
@@ -215,8 +216,8 @@ test('Chamfer.action distance method asymmetric distances', () => {
 
 test('Chamfer.action distance method trimMode=false adds chamfer line only', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -244,8 +245,8 @@ test('Chamfer.action distance method trimMode=false adds chamfer line only', ()
 test('Chamfer.action notifies when distance is too large for the segments', () => {
   // Lines only 5 units long from intersection — distance 10 won't fit
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-5, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 5)] });
+  core.scene.addEntity('Line', { points: [new Point(-5, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 5)] });
 
   core.scene.headers.chamferDistanceA = 10;
   core.scene.headers.chamferDistanceB = 10;
@@ -272,8 +273,8 @@ test('Chamfer.action angle method 45° on perpendicular lines matches distance m
   // With perpendicular lines and 45°, the chamfer endpoint on line2 should also be 2 units
   // from the intersection — matching a symmetric distance chamfer.
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferLength = 2;
   core.scene.headers.chamferAngle = 45; // degrees
@@ -306,8 +307,8 @@ test('Chamfer.action notifies when click coincides with intersection', () => {
   // L-corner at origin. Clicking firstClickPoint exactly at (0,0) makes
   // firstClickDistance = 0, triggering the "Selected corner cannot be chamfered" guard.
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -332,8 +333,8 @@ test('Chamfer.action notifies when click coincides with intersection', () => {
 
 test('Chamfer.action angle method notifies INVALIDNUMBER when angle is zero', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferLength = 2;
   core.scene.headers.chamferAngle = 0; // alpha = 0 * π/180 = 0, fails alpha <= 0 guard
@@ -361,8 +362,8 @@ test('Chamfer.action angle method notifies PARALLELLINES when chamfer direction
   // candidate1 = rotate(-1,0) by +π/2 = (0,-1); dot with secondClickUnit (0,1) = -1 < 0.
   // candidate2 = rotate(-1,0) by -π/2 = (0, 1) → parallel to secondLineDirection (0,1) → cross = 0.
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferLength = 2;
   core.scene.headers.chamferAngle = 90; // degrees
@@ -396,8 +397,8 @@ test('Chamfer.execute returns early when first input is undefined', async () =>
 
 test('Chamfer.execute selects two lines and calls executeCommand', async () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   const executeCommandSpy = jest.fn();
 
@@ -543,7 +544,7 @@ test('Chamfer.execute sets chamferMode to true via Method=Angle option', async (
 
 test('Chamfer.execute re-prompts and notifies when first selected entity is not a Line', async () => {
   core.scene.clear();
-  core.scene.addItem('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
+  core.scene.addEntity('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
 
   const notifySpy = jest.spyOn(core, 'notify');
 
@@ -562,8 +563,8 @@ test('Chamfer.execute re-prompts and notifies when first selected entity is not
 
 test('Chamfer.execute re-prompts and notifies when second selected entity is not a Line', async () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(-10, 0), new Point(10, 0)] });
-  core.scene.addItem('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(10, 0)] });
+  core.scene.addEntity('Circle', { points: [new Point(0, 0), new Point(5, 0)] });
 
   const notifySpy = jest.spyOn(core, 'notify');
 
@@ -593,8 +594,8 @@ test('Chamfer.execute re-prompts and notifies when second selected entity is not
 test('Chamfer.action dist>0 trimMode=true Line first + Lwpolyline second keepStart: chamfer embedded in polyline', () => {
   // Click polyline near (-15,0) → segment 1. keepStart keeps the start portion.
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
-  core.scene.addItem('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -634,8 +635,8 @@ test('Chamfer.action dist>0 trimMode=true Line first + Lwpolyline second keepSta
 
 test('Chamfer.action dist>0 trimMode=true Lwpolyline first + Line second keepStart: same result with entities reversed', () => {
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -675,8 +676,8 @@ test('Chamfer.action dist>0 trimMode=true Line + Lwpolyline keepEnd: keeps end p
   // Click polyline near (8,0) → segment 3 (right of intersection). keepEnd.
   // polyChamfer = (2,0), lineChamfer = (0,2).
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
-  core.scene.addItem('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -719,7 +720,7 @@ test('Chamfer.action dist>0 trimMode=true same Lwpolyline consecutive segments:
   // Segment 1: (-10,0)→(0,0), Segment 2: (0,0)→(0,10). Corner at (0,0).
   // dist=2 → chamfer at (-2,0) and (0,2).
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(-10, 0), new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-10, 0), new Point(0, 0), new Point(0, 10)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -762,7 +763,7 @@ test('Chamfer.action dist>0 trimMode=true same Lwpolyline open ends: separate ch
   // dist=2, clicks at (-3,0) and (1,3):
   //   firstChamfer = (-1,0), secondChamfer = (1,2).
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(-5, 0), new Point(0, 0), new Point(1, 0), new Point(1, 5)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-5, 0), new Point(0, 0), new Point(1, 0), new Point(1, 5)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -808,7 +809,7 @@ test('Chamfer.action dist>0 trimMode=true closed Lwpolyline seg3+closing-seg: co
   // Closing segment (idx=4) goes (0,10)→(0,0); seg 3 goes (10,10)→(0,10).
   // Corner at (0,10) — chamfer dist=2 trims to (2,10) on seg3 and (0,8) on closing seg.
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
   const polyEntity = core.scene.entities.get(0);
   polyEntity.flags.setFlagValue(1);
 
@@ -846,7 +847,7 @@ test('Chamfer.action dist>0 trimMode=true closed Lwpolyline closing-wrap (seg4+s
   // Bug: old code had no closed-poly guard, so closing-wrap was treated as open-ends.
   // Fix: closed poly must NOT enter the open-ends path → corner-splice at index 0.
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
   const polyEntity = core.scene.entities.get(0);
   polyEntity.flags.setFlagValue(1);
 
@@ -872,17 +873,26 @@ test('Chamfer.action dist>0 trimMode=true closed Lwpolyline closing-wrap (seg4+s
   const poly = core.scene.entities.get(0);
   // Shared vertex (0,0) at index 0 replaced by two chamfer points → total 5 points
   expect(poly.points.length).toBe(5);
-  // Endpoints must NOT have been moved to the intersection (open-ends symptom)
-  // The first two new points should be the chamfer points, not (0,0)/(0,0)
-  const allAtOrigin = poly.points[0].x === 0 && poly.points[0].y === 0 &&
-    poly.points[1].x === 0 && poly.points[1].y === 0;
-  expect(allAtOrigin).toBe(false);
+  // Verify the actual chamfer point coordinates and traversal order.
+  // The closing segment (0,10)→(0,0) is chamfered at (0,2),
+  // seg 1 (0,0)→(10,0) is chamfered at (2,0).
+  // After splice: [(0,2), (2,0), (10,0), (10,10), (0,10)]
+  expect(poly.points[0].x).toBeCloseTo(0);
+  expect(poly.points[0].y).toBeCloseTo(2);
+  expect(poly.points[1].x).toBeCloseTo(2);
+  expect(poly.points[1].y).toBeCloseTo(0);
+  expect(poly.points[2].x).toBeCloseTo(10);
+  expect(poly.points[2].y).toBeCloseTo(0);
+  expect(poly.points[3].x).toBeCloseTo(10);
+  expect(poly.points[3].y).toBeCloseTo(10);
+  expect(poly.points[4].x).toBeCloseTo(0);
+  expect(poly.points[4].y).toBeCloseTo(10);
 });
 
 test('Chamfer.action dist>0 trimMode=false closed Lwpolyline closing-wrap: standalone chamfer Line only', () => {
   // Same square, closed, trimMode=false. No polyline mutation; only chamfer Line added.
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(0, 0), new Point(10, 0), new Point(10, 10), new Point(0, 10)] });
   const polyEntity = core.scene.entities.get(0);
   polyEntity.flags.setFlagValue(1);
 
@@ -912,7 +922,7 @@ test('Chamfer.action dist>0 trimMode=true open Lwpolyline seg1+lastIdx still tak
   // Open 4-point polyline (no flags). Seg 1 and seg 3 (lastIdx=3) → open-ends path still fires.
   // Regression guard: the closed-poly fix must not break the open-poly open-ends case.
   core.scene.clear();
-  core.scene.addItem('Lwpolyline', { points: [new Point(-5, 0), new Point(0, 0), new Point(1, 0), new Point(1, 5)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-5, 0), new Point(0, 0), new Point(1, 0), new Point(1, 5)] });
   const polyEntity = core.scene.entities.get(0);
   // No flags set — open polyline
 
@@ -939,8 +949,8 @@ test('Chamfer.action dist>0 trimMode=true open Lwpolyline seg1+lastIdx still tak
 
 test('Chamfer.action dist>0 trimMode=false Line + Lwpolyline: standalone chamfer Line added, entities unchanged', () => {
   core.scene.clear();
-  core.scene.addItem('Line', { points: [new Point(0, 0), new Point(0, 10)] });
-  core.scene.addItem('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0)] });
 
   core.scene.headers.chamferDistanceA = 2;
   core.scene.headers.chamferDistanceB = 2;
@@ -968,3 +978,215 @@ test('Chamfer.action dist>0 trimMode=false Line + Lwpolyline: standalone chamfer
   expect(core.scene.entities.get(0).points.length).toBe(2);
   expect(core.scene.entities.get(1).points.length).toBe(4);
 });
+
+// ─── action: entity property inheritance ─────────────────────────────────────
+
+test('Chamfer.action chamfer line inherits layer, lineWidth, lineType from first entity', () => {
+  // Perpendicular L-corner at origin; first entity on a non-default layer.
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)], layer: 'walls', lineWidth: 5, lineType: 'DASHED' });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+
+  core.scene.headers.chamferDistanceA = 2;
+  core.scene.headers.chamferDistanceB = 2;
+  core.scene.headers.chamferMode = false;
+  core.scene.headers.trimMode = false;
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.secondPick.entity = core.scene.entities.get(1);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+  chamfer.secondPick.clickPoint = new Point(0, 5);
+  chamfer.action();
+
+  expect(core.scene.entities.count()).toBe(3);
+  const chamferLine = core.scene.entities.get(2);
+  expect(chamferLine.type).toBe('Line');
+  expect(chamferLine.getProperty('layer')).toBe('walls');
+  expect(chamferLine.getProperty('lineWidth')).toBe(5);
+  expect(chamferLine.getProperty('lineType')).toBe('DASHED');
+});
+
+test('Chamfer.preview does not throw', () => {
+  const chamfer = new Chamfer();
+  expect(() => chamfer.preview()).not.toThrow();
+});
+
+test('Chamfer.preview returns early when findClosestItem returns undefined', () => {
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  DesignCore.Scene.previewEntities.clear();
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+
+  const findSpy = jest.spyOn(core.scene.selectionManager, 'findClosestItem').mockReturnValue(undefined);
+  DesignCore.Mouse.pointOnScene = () => new Point(0, 5);
+  chamfer.preview();
+  findSpy.mockRestore();
+
+  expect(DesignCore.Scene.previewEntities.count()).toBe(0);
+});
+
+test('Chamfer.preview adds chamfer line to previewEntities in no-trim mode', () => {
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  DesignCore.Scene.previewEntities.clear();
+
+  core.scene.headers.chamferDistanceA = 2;
+  core.scene.headers.chamferDistanceB = 2;
+  core.scene.headers.chamferMode = false;
+  core.scene.headers.trimMode = false;
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+
+  const findSpy = jest.spyOn(core.scene.selectionManager, 'findClosestItem').mockReturnValue(1);
+  DesignCore.Mouse.pointOnScene = () => new Point(0, 5);
+  chamfer.preview();
+  findSpy.mockRestore();
+
+  expect(DesignCore.Scene.previewEntities.count()).toBeGreaterThanOrEqual(1);
+});
+
+test('Chamfer.preview skips chamfer line in no-trim mode when distances are both 0 (sharp corner)', () => {
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  DesignCore.Scene.previewEntities.clear();
+
+  core.scene.headers.chamferDistanceA = 0;
+  core.scene.headers.chamferDistanceB = 0;
+  core.scene.headers.chamferMode = false;
+  core.scene.headers.trimMode = false;
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+
+  const findSpy = jest.spyOn(core.scene.selectionManager, 'findClosestItem').mockReturnValue(1);
+  DesignCore.Mouse.pointOnScene = () => new Point(0, 5);
+  chamfer.preview();
+  findSpy.mockRestore();
+
+  expect(DesignCore.Scene.previewEntities.count()).toBe(0);
+});
+
+test('Chamfer.preview adds 5 entities to previewEntities in trim mode', () => {
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+  DesignCore.Scene.previewEntities.clear();
+
+  core.scene.headers.chamferDistanceA = 2;
+  core.scene.headers.chamferDistanceB = 2;
+  core.scene.headers.chamferMode = false;
+  core.scene.headers.trimMode = true;
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+
+  const findSpy = jest.spyOn(core.scene.selectionManager, 'findClosestItem').mockReturnValue(1);
+  DesignCore.Mouse.pointOnScene = () => new Point(0, 5);
+  chamfer.preview();
+  findSpy.mockRestore();
+
+  // dulled firstSeg + dulled secondSeg + firstTrimLine + secondTrimLine + chamferLine = 5
+  expect(DesignCore.Scene.previewEntities.count()).toBeGreaterThanOrEqual(5);
+});
+
+test('Chamfer.execute returns early when second-entity input is undefined (inner loop)', async () => {
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+
+  const executeCommandSpy = jest.fn();
+  await withMockInput(
+      core.scene,
+      [new SingleSelection(0, new Point(-5, 0))],
+      async () => {
+        const chamfer = new Chamfer();
+        await chamfer.execute();
+        expect(executeCommandSpy).not.toHaveBeenCalled();
+      },
+      { extraMethods: { executeCommand: executeCommandSpy } },
+  );
+});
+
+test('Chamfer.execute notifies NONCONSECUTIVESEGMENTS for non-adjacent polyline segments', async () => {
+  // 5-point polyline → 4 segments. Pick segment 1 (near -15,0) then segment 3 (near 5,0).
+  // Neither consecutive nor open-ends → NONCONSECUTIVESEGMENTS, then undefined to exit.
+  core.scene.clear();
+  core.scene.addEntity('Lwpolyline', { points: [new Point(-20, 0), new Point(-10, 0), new Point(0, 0), new Point(10, 0), new Point(20, 0)] });
+
+  const notifySpy = jest.spyOn(core, 'notify');
+  await withMockInput(
+      core.scene,
+      [new SingleSelection(0, new Point(-15, 0)), new SingleSelection(0, new Point(5, 0)), undefined],
+      async () => {
+        const chamfer = new Chamfer();
+        await chamfer.execute();
+      },
+  );
+  expect(notifySpy).toHaveBeenCalledWith(expect.stringContaining(Strings.Message.NONCONSECUTIVESEGMENTS));
+  notifySpy.mockRestore();
+});
+
+test('Chamfer.execute Method→Distance sets chamferMode to false', async () => {
+  core.scene.headers.chamferMode = true;
+
+  await withMockInput(
+      core.scene,
+      ['Method', 'Distance', undefined],
+      async () => {
+        const chamfer = new Chamfer();
+        await chamfer.execute();
+      },
+  );
+
+  expect(core.scene.headers.chamferMode).toBe(false);
+});
+
+test('Chamfer.execute Trim→Trim keeps trimMode=true', async () => {
+  core.scene.headers.trimMode = true;
+
+  await withMockInput(
+      core.scene,
+      ['Trim', 'Trim', undefined],
+      async () => {
+        const chamfer = new Chamfer();
+        await chamfer.execute();
+      },
+  );
+
+  expect(core.scene.headers.trimMode).toBe(true);
+});
+
+test('Chamfer.action angle method angle=180 notifies INVALIDNUMBER', () => {
+  // alpha = 180 * π/180 = π; checks alpha >= Math.PI → INVALIDNUMBER
+  core.scene.clear();
+  core.scene.addEntity('Line', { points: [new Point(-10, 0), new Point(0, 0)] });
+  core.scene.addEntity('Line', { points: [new Point(0, 0), new Point(0, 10)] });
+
+  core.scene.headers.chamferLength = 2;
+  core.scene.headers.chamferAngle = 180;
+  core.scene.headers.chamferMode = true;
+  core.scene.headers.trimMode = true;
+
+  const notifySpy = jest.spyOn(core, 'notify');
+
+  const chamfer = new Chamfer();
+  chamfer.firstPick.entity = core.scene.entities.get(0);
+  chamfer.secondPick.entity = core.scene.entities.get(1);
+  chamfer.firstPick.clickPoint = new Point(-5, 0);
+  chamfer.secondPick.clickPoint = new Point(0, 5);
+  chamfer.action();
+
+  expect(notifySpy).toHaveBeenCalledWith(expect.stringContaining(Strings.Error.INVALIDNUMBER));
+  expect(core.scene.entities.count()).toBe(2);
+  notifySpy.mockRestore();
+});