Add edge case tests (null, negative coords, zero dimensions) #13

New Issue

2026-03-27T15:01:29Z

thib8956 commented

2026-03-27 15:01:29 +00:00

Priority: P2 — Lower

Context: A testing audit identified that existing shape tests have no edge case coverage. Specific issues found:

No null handling — no tests verify behavior with null arguments
No negative coordinates — untested territory
No zero dimensions — shapes with width/height 0 untested
Reflection usage — SCircleTest uses reflection to check color (fragile, should be replaced)

Scope: All shape classes + SCollection + Selection

Goal: Harden existing tests with:

Null arguments handling
Negative coordinates
Zero-dimension shapes
Shapes at Integer.MAX_VALUE boundaries

Estimate: 2 hours
Dependencies: Benefits from #8 (ShapeContractTest) being done first for the parameterized pattern

**Priority:** P2 — Lower **Context:** A testing audit identified that existing shape tests have no edge case coverage. Specific issues found: - **No null handling** — no tests verify behavior with null arguments - **No negative coordinates** — untested territory - **No zero dimensions** — shapes with width/height 0 untested - **Reflection usage** — SCircleTest uses reflection to check color (fragile, should be replaced) **Scope:** All shape classes + SCollection + Selection **Goal:** Harden existing tests with: - Null arguments handling - Negative coordinates - Zero-dimension shapes - Shapes at Integer.MAX_VALUE boundaries **Estimate:** 2 hours **Dependencies:** Benefits from #8 (ShapeContractTest) being done first for the parameterized pattern

thib8956 referenced this issue from a commit

2026-03-27 15:37:48 +00:00

test: add edge case tests for shapes

thib8956 referenced a pull request that will close this issue

2026-03-27 15:38:04 +00:00

test: add edge case tests (null, negative coords, zero dimensions) #23

thib8956 referenced this issue from a commit

2026-03-27 23:23:18 +00:00

test: add edge case tests for shapes

thib8956 closed this issue

2026-03-27 23:24:12 +00:00

Sign in to join this conversation.