Skip to content

Commit 2205a09

Browse files
ralfstxclaude
andcommitted
🧪 Fix langSysTag test expectations for padded tags
Commit 6c28834 added space-padding to `langSysTag` values to work around pdf-core expecting 4-character tags, but the tests were not updated to match. Update 4 test expectations in `text.test.ts` to expect `'DEU '` instead of `'DEU'`. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 473b605 commit 2205a09

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

‎src/text.test.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ describe('text', () => {
198198

199199
expect(shapeTextSpy).toHaveBeenCalledWith('foo', {
200200
scriptTag: 'latn',
201-
langSysTag: 'DEU',
201+
langSysTag: 'DEU ',
202202
});
203203
});
204204

@@ -301,19 +301,19 @@ describe('text', () => {
301301
});
302302

303303
it('includes langSysTag when language is set', () => {
304-
expect(buildShapeOptions({ language: 'de' })).toEqual({ langSysTag: 'DEU' });
304+
expect(buildShapeOptions({ language: 'de' })).toEqual({ langSysTag: 'DEU ' });
305305
});
306306

307307
it('combines langSysTag with scriptTag', () => {
308308
expect(buildShapeOptions({ language: 'de' }, 'latn')).toEqual({
309309
scriptTag: 'latn',
310-
langSysTag: 'DEU',
310+
langSysTag: 'DEU ',
311311
});
312312
});
313313

314314
it('combines langSysTag with features', () => {
315315
expect(buildShapeOptions({ language: 'de', fontKerning: 'none' })).toEqual({
316-
langSysTag: 'DEU',
316+
langSysTag: 'DEU ',
317317
features: { kern: false },
318318
});
319319
});

0 commit comments

Comments
 (0)