Commit fdb6512
committed
fix(testdata): normalize diff artifacts to 813x1626 canvas
Several diff artifacts in testdata/diffs/ were stale or mismatched:
- gb_normal/gb_0_width/gb_0_height, postnl_qr, qr_code_offset: leftover
400x640 diff images from an old canvas era; these labels are now
PERFECT so the pipeline never regenerated them
- cf_font_designator, cf_font_no_orientation, code128_mode_d_fnc1,
cp850_hex_chars, fo_lenient_coord, maxicode_default_mode2,
maxicode_mode4: Labelary reference PNGs were 812x1624, so every
diff run produced 1629x1626 comparisons and 812x1624 diffs
Fix: pad the 7 unit references to 813x1626 (same normalization the
bootstrap already applies to new fetches), drop the stale diff images,
and make the diff report self-healing — when a label has no diff,
remove any leftover diff image so old canvases can't linger.
Diff percentages drop for the 7 labels since the spurious 1px column
no longer counts (e.g. fo_lenient_coord 0.27% -> 0.02%, maxicode
labels 0.84%/0.83% -> 0.59%/0.58%). All e2e_golden tests still pass.1 parent 9b7b93a commit fdb6512
38 files changed
Lines changed: 16 additions & 12 deletions
File tree
- testdata
- diffs
- unit
- tests/common
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
42 | | - | |
43 | | - | |
| 42 | + | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| |||
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
72 | | - | |
| 72 | + | |
73 | 73 | | |
0 commit comments