@flighthq/bitmapfont-formats 0.3.1-next.847.2e6f8bf → 0.3.1-next.887.9f59a84

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/bitmapfont-formats",
-  "version": "0.3.1-next.847.2e6f8bf",
+  "version": "0.3.1-next.887.9f59a84",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -38,10 +38,10 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/bitmapfont": "0.3.1-next.847.2e6f8bf",
-    "@flighthq/importdiagnostics": "0.3.1-next.847.2e6f8bf",
-    "@flighthq/types": "0.3.1-next.847.2e6f8bf",
-    "@flighthq/xml": "0.3.1-next.847.2e6f8bf"
+    "@flighthq/bitmapfont": "0.3.1-next.887.9f59a84",
+    "@flighthq/importdiagnostics": "0.3.1-next.887.9f59a84",
+    "@flighthq/types": "0.3.1-next.887.9f59a84",
+    "@flighthq/xml": "0.3.1-next.887.9f59a84"
   },
   "devDependencies": {
     "@flighthq/textureatlas": "*",

package/src/bitmapFontFnt.test.ts

@@ -79,6 +79,20 @@ describe('formatBitmapFontFnt', () => {
 });
 
 describe('parseBitmapFontFnt', () => {
+  // A clean parse is two claims: the values are right AND THE PARSER IS NOT COMPLAINING. Every other test
+  // here checks the first. This checks the second — the one that catches a walk that desynchronised and
+  // still left the asserted fields looking plausible. Asserted as an EMPTY list rather than a filter over
+  // truncation-shaped kind names: a pattern built from expected vocabulary silently exempts every kind
+  // whose name nobody guessed, and this importer has kinds like that.
+  it('raises no diagnostic at all for a well-formed file', () => {
+    const diagnostics: ImportDiagnostic[] = [];
+
+    parseBitmapFontFnt(FNT_TEXT, { resolvePage: () => createTextureAtlas() }, diagnostics);
+
+    const complaints = diagnostics.map((diagnostic) => diagnostic.kind);
+    expect(complaints, `a good fnt file made the parser complain: ${complaints.join(', ')}`).toEqual([]);
+  });
+
   it('parses glyphs, kerning, and line metrics with the resolved atlas', () => {
     const atlas = createTextureAtlas();
     const font = parseBitmapFontFnt(FNT_TEXT, { resolvePage: () => atlas });

package/src/bitmapFontJson.test.ts

@@ -5,6 +5,7 @@ import {
   getBitmapFontPage,
 } from '@flighthq/bitmapfont/contract';
 import { createTextureAtlas } from '@flighthq/textureatlas/contract';
+import type { ImportDiagnostic } from '@flighthq/types/contract';
 import type { BitmapFont } from '@flighthq/types/contract';
 import { describe, expect, it } from 'vitest';
 
@@ -46,6 +47,20 @@ const FNT_XML = [
 ].join('\n');
 
 describe('parseBitmapFontJson', () => {
+  // A clean parse is two claims: the values are right AND THE PARSER IS NOT COMPLAINING. Every other test
+  // here checks the first. This checks the second — the one that catches a walk that desynchronised and
+  // still left the asserted fields looking plausible. Asserted as an EMPTY list rather than a filter over
+  // truncation-shaped kind names: a pattern built from expected vocabulary silently exempts every kind
+  // whose name nobody guessed, and this importer has kinds like that.
+  it('raises no diagnostic at all for a well-formed file', () => {
+    const diagnostics: ImportDiagnostic[] = [];
+
+    parseBitmapFontJson(FNT_JSON, { resolvePage: () => createTextureAtlas() }, diagnostics);
+
+    const complaints = diagnostics.map((diagnostic) => diagnostic.kind);
+    expect(complaints, `a good fnt-json file made the parser complain: ${complaints.join(', ')}`).toEqual([]);
+  });
+
   it('parses glyphs, kerning, and line metrics with the resolved atlas', () => {
     const atlas = createTextureAtlas();
     const font = parseBitmapFontJson(FNT_JSON, { resolvePage: () => atlas });

package/src/bitmapFontXml.test.ts

@@ -5,6 +5,7 @@ import {
   getBitmapFontPage,
 } from '@flighthq/bitmapfont/contract';
 import { createTextureAtlas } from '@flighthq/textureatlas/contract';
+import type { ImportDiagnostic } from '@flighthq/types/contract';
 import { describe, expect, it } from 'vitest';
 
 import { parseBitmapFontXml } from './bitmapFontXml';
@@ -26,6 +27,20 @@ const FNT_XML = [
 ].join('\n');
 
 describe('parseBitmapFontXml', () => {
+  // A clean parse is two claims: the values are right AND THE PARSER IS NOT COMPLAINING. Every other test
+  // here checks the first. This checks the second — the one that catches a walk that desynchronised and
+  // still left the asserted fields looking plausible. Asserted as an EMPTY list rather than a filter over
+  // truncation-shaped kind names: a pattern built from expected vocabulary silently exempts every kind
+  // whose name nobody guessed, and this importer has kinds like that.
+  it('raises no diagnostic at all for a well-formed file', () => {
+    const diagnostics: ImportDiagnostic[] = [];
+
+    parseBitmapFontXml(FNT_XML, { resolvePage: () => createTextureAtlas() }, diagnostics);
+
+    const complaints = diagnostics.map((diagnostic) => diagnostic.kind);
+    expect(complaints, `a good fnt-xml file made the parser complain: ${complaints.join(', ')}`).toEqual([]);
+  });
+
   it('parses glyphs, kerning, and line metrics with the resolved atlas', () => {
     const atlas = createTextureAtlas();
     const font = parseBitmapFontXml(FNT_XML, { resolvePage: () => atlas });