Skip to content

test(converter): add feature verification tests for resolved gaps#88

Merged
JSv4 merged 1 commit intomainfrom
feature/verify-resolved-gaps
Dec 21, 2025
Merged

test(converter): add feature verification tests for resolved gaps#88
JSv4 merged 1 commit intomainfrom
feature/verify-resolved-gaps

Conversation

@JSv4
Copy link
Copy Markdown
Owner

@JSv4 JSv4 commented Dec 21, 2025

Summary

  • Add comprehensive test suite (FeatureVerificationTests.cs) that verifies all 8 resolved features in the WmlToHtmlConverter gaps document
  • Update gaps documentation with verification summary table

Features Verified (All 15 tests passing)

Feature Test(s) Status
@page CSS rule generation FV001
Table DXA width conversion FV002
Borderless table detection FV003
Theme color resolution FV004, FV005
Document language on <html> FV006, FV007
Foreign text language spans FV008, FV009
Font fallback improvements FV010, FV011, FV012
CJK font-family fallback FV013, FV014
All features together FV099

Test plan

  • All 15 new tests pass
  • Existing tests still pass (996 total)
  • HTML output manually verified for comprehensive test

Add comprehensive test suite (FeatureVerificationTests.cs) that verifies
all 8 resolved features in the WmlToHtmlConverter gaps document:

- @page CSS rule generation (GeneratePageCss setting)
- Table DXA width conversion (twips to points)
- Borderless table detection (data-borderless attribute)
- Theme color resolution (ResolveThemeColors setting)
- Document language on <html> element
- Foreign text language spans (lang attributes)
- Font fallback improvements (serif/sans-serif/monospace)
- CJK font-family fallback chains (Japanese, Chinese, Korean)

All 15 tests pass, confirming the features work correctly.

Also updates gaps document with verification summary table.
@JSv4 JSv4 merged commit b9df194 into main Dec 21, 2025
6 checks passed
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.

1 participant