agnaldo4j / kanban_vision_api_iex / 3ea7ca7f0877da2ea9e40baeea3c1d856076787f

88%

Build Type

push

github

Committed by web-flow

Commit Message

fix: Code review quality improvements across all layers (#13)

* fix: Code review quality improvements across all layers

- Fix test tag typo :domain_smulations → :domain_simulations
- Add 29 domain entity unit tests for all 13 entities (kanban_domain)
- Add delete operations for Simulations and Boards (port, agent, command, use case, GenServer)
- Add get_by_id to Boards agent and port for API consistency
- Add BoardRepository contract test
- Fix Step.new() parameter order (required params before optional)
- Fix Audit.new() to use single DateTime call (prevents microsecond drift)
- Add telemetry events to GetAll use cases for observability consistency
- Add board_id/board_name to Logger metadata config
- Update coverage thresholds (kanban_domain: 70%, usecase: 50%)
- Fix CLAUDE.md test path reference

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* test: Achieve 100% domain entity coverage by exercising all arities

Cover all generated default-parameter clauses for Board, Simulation,
Project, Step, and Task factory functions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* test: Add missing persistence tests for delete and get_by_id operations

Cover Boards.get_by_id/2, Boards.delete/2, and Simulations.delete/2
in persistence unit tests to restore 100% coverage threshold.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Run Details

29 of 68 new or added lines in 11 files covered. (42.65%)

5 existing lines in 2 files now uncovered.

280 of 335 relevant lines covered (83.58%)

3.59 hits per line