jedarden/pdftract
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test(bf-5dopl): update unmapped glyph assertion message in resolver.rs

Browse source 
Updates the assertion in test_resolve_level2_unmapped_code to use the new
diagnostic message format designed in bf-w1o10.

- Changed bare assert!(result.is_failure()) to assert!(result.is_failure(), "...")
- Added structured message with Expected/Found/Why this matters sections
- Clarifies that codes above 0x7F in StandardEncoding are unmapped per build/unmapped-glyph-names.json

This completes the assertion message update task. All other unmapped glyph
assertions in unmapped.rs, encoding.rs, and test files were already updated
with the new format.

Closes bf-5dopl. All unmapped glyph tests pass (unmapped, encoding, resolver, cmap).
This commit is contained in:
jedarden 2026-07-06 20:39:44 -04:00
parent fd70f0f062
commit 9c0f18c117
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
crates/pdftract-core/src/font/resolver.rs
View file

@ -870,7 +870,13 @@ mod tests {
// Most codes in StandardEncoding are unmapped above 0x7F // Most codes in StandardEncoding are unmapped above 0x7F
let encoding = FontEncoding::new(Some(NamedEncoding::Standard)); let encoding = FontEncoding::new(Some(NamedEncoding::Standard));
let result = resolve_level2(&[0x80], Some(&encoding)); let result = resolve_level2(&[0x80], Some(&encoding));
assert!(result.is_failure()); assert!(
result.is_failure(),
"Level 2 resolution should fail for unmapped character codes. \
Expected: result.is_failure() == true. \
Found: false. \
Why this matters: Most codes in StandardEncoding above 0x7F are unmapped and should fail Level 2 (encoding + AGL) resolution per build/unmapped-glyph-names.json filtering."
);
} }
#[test] #[test]