Skip to content

docs: Add comprehensive test coverage analysis - #31

Merged
marevol merged 2 commits into
masterfrom
claude/analyze-test-coverage-ppOip
Jan 22, 2026
Merged

marevol merged 2 commits into
masterfrom
claude/analyze-test-coverage-ppOip

Conversation

@marevol

@marevol marevol commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

Analyze existing test coverage and identify gaps including:

  • DWG file reading (zero coverage for major feature)
  • Dimension entities (7 types completely untested)
  • Leader and Hatch entities
  • Entity attribute verification (color, layer, lineweight)
  • Table objects (DimStyle, TextStyle, Vport)
  • Header variable validation
  • Character encoding handling

Provides prioritized recommendations and estimated effort for
improving test coverage.

Analyze existing test coverage and identify gaps including:

- DWG file reading (zero coverage for major feature)
- Dimension entities (7 types completely untested)
- Leader and Hatch entities
- Entity attribute verification (color, layer, lineweight)
- Table objects (DimStyle, TextStyle, Vport)
- Header variable validation
- Character encoding handling

Provides prioritized recommendations and estimated effort for
improving test coverage.
…utes

Add new test files to improve test coverage:

- test_dimensions.cpp: Tests for all 7 dimension types (Aligned, Linear,
  Radial, Diametric, Angular, Angular3P, Ordinate)
- test_annotations.cpp: Tests for Leader and Hatch entities including
  solid hatches, pattern hatches, and arc boundaries
- test_attributes.cpp: Tests for entity attributes including color,
  layer assignment, line weight, line type, coordinates, and visibility

Update test infrastructure:
- Extend TestInterface with counters and storage for new entity types
- Add dimension, leader, viewport, image, and block counters
- Store last entity of each type for property verification

Update build configuration:
- Add new test executables to CMakeLists.txt
- Add new test targets to tests/Makefile.am

All 11 tests pass successfully.
@marevol
marevol merged commit 6814a4c into master Jan 22, 2026
4 checks passed
@marevol
marevol deleted the claude/analyze-test-coverage-ppOip branch February 5, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants