add roundtrip tests

Author: willwade

src/processors/gridsetProcessor.ts

@@ -1780,14 +1780,22 @@ class GridsetProcessor extends BaseProcessor {
       '?xml': { '@_version': '1.0', '@_encoding': 'UTF-8' },
       GridSetSettings: {
         '@_xmlns:xsi': 'http://www.w3.org/2001/XMLSchema-instance',
+        Name: tree.metadata?.name || '',
+        Description: tree.metadata?.description || '',
+        Author: tree.metadata?.author || '',
+        PrimaryLanguage: tree.metadata?.locale || 'en-US',
         StartGrid: startGrid,
         // Add other common Grid3 settings
+        Thumbnail: (tree.metadata as any)?.thumbnail || '',
+        ThumbnailBackground: (tree.metadata as any)?.thumbnailBackground || '',
+        DocumentationUrl: tree.metadata?.homepageUrl || tree.metadata?.url || '',
+        DocumentationSlug: (tree.metadata as any)?.documentationSlug || '',
         ScanEnabled: 'false',
         ScanTimeoutMs: '2000',
         HoverEnabled: 'false',
         HoverTimeoutMs: '1000',
         MouseclickEnabled: 'true',
-        Language: 'en-US',
+        Language: tree.metadata?.locale || 'en-US',
       },
     };
 

src/processors/snapProcessor.ts

@@ -820,6 +820,18 @@ class SnapProcessor extends BaseProcessor {
           Data BLOB,
           RefCount INTEGER DEFAULT 1
         );
+
+        CREATE TABLE IF NOT EXISTS PageSetProperties (
+          Id INTEGER PRIMARY KEY,
+          Name TEXT,
+          Description TEXT,
+          Author TEXT,
+          Locale TEXT,
+          DefaultHomePageUniqueId TEXT,
+          DefaultKeyboardPageUniqueId TEXT,
+          DashboardUniqueId TEXT,
+          ToolBarUniqueId TEXT
+        );
       `);
 
       // Insert pages
@@ -978,6 +990,27 @@ class SnapProcessor extends BaseProcessor {
           insertPlacement.run(placementIdCounter++, elementRefId, gridPosition);
         });
       });
+
+      // Insert PageSetProperties metadata
+      const insertProps = db.prepare(`
+        INSERT INTO PageSetProperties (
+          Id, Name, Description, Author, Locale, 
+          DefaultHomePageUniqueId, DefaultKeyboardPageUniqueId, 
+          DashboardUniqueId, ToolBarUniqueId
+        ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
+      `);
+
+      insertProps.run(
+        1,
+        tree.metadata?.name || null,
+        tree.metadata?.description || null,
+        tree.metadata?.author || null,
+        tree.metadata?.locale || null,
+        tree.metadata?.defaultHomePageId || tree.rootId || null,
+        tree.metadata?.defaultKeyboardPageId || null,
+        tree.metadata?.dashboardId || null,
+        tree.metadata?.hasGlobalToolbar ? tree.metadata.toolbarId || null : null
+      );
     } finally {
       db.close();
     }

test/gridsetProcessor.roundtrip.test.ts

@@ -32,6 +32,13 @@ describe('GridsetProcessor round-trip', () => {
     expect(Object.keys(tree2.pages).length).toBeGreaterThan(0);
     expect(Object.keys(tree1.pages).length).toBe(Object.keys(tree2.pages).length);
 
+    // Compare metadata
+    expect(tree2.metadata.name).toBe(tree1.metadata.name);
+    expect(tree2.metadata.description?.trim()).toBe(tree1.metadata.description?.trim());
+    if (tree1.metadata.locale) {
+      expect(tree2.metadata.locale).toBe(tree1.metadata.locale);
+    }
+
     // Compare page names and button counts
     for (const pageId in tree1.pages) {
       expect(tree2.pages).toHaveProperty(pageId);

test/obfProcessor.roundtrip.test.ts

@@ -33,6 +33,12 @@ describe('OBFProcessor round-trip', () => {
     // Compare basic structure
     expect(Object.keys(tree1.pages).length).toBe(Object.keys(tree2.pages).length);
 
+    // Compare metadata
+    expect(tree2.metadata.name).toBe(tree1.metadata.name);
+    expect(tree2.metadata.description).toBe(tree1.metadata.description);
+    expect(tree2.metadata.locale).toBe(tree1.metadata.locale);
+    if (tree1.metadata.url) expect(tree2.metadata.url).toBe(tree1.metadata.url);
+
     // Compare page content
     for (const pageId in tree1.pages) {
       expect(tree2.pages).toHaveProperty(pageId);
@@ -66,6 +72,10 @@ describe('OBFProcessor round-trip', () => {
     // Compare structure
     expect(Object.keys(tree2.pages).length).toBeGreaterThan(0);
     expect(Object.keys(tree1.pages).length).toBe(Object.keys(tree2.pages).length);
+
+    // Compare metadata from root board
+    expect(tree2.metadata.name).toBe(tree1.metadata.name);
+    expect(tree2.metadata.locale).toBe(tree1.metadata.locale);
   });
 
   it('can save and load a simple constructed tree', () => {

test/snapProcessor.roundtrip.test.js

@@ -24,6 +24,10 @@ describe("SnapProcessor round-trip", () => {
       const btnLabels2 = tree2.pages[pid].buttons.map((b) => b.label).sort();
       expect(btnLabels1).toEqual(btnLabels2);
     }
+
+    // Compare metadata
+    expect(tree2.metadata.name).toBe(tree1.metadata.name);
+    expect(tree2.metadata.locale).toBe(tree1.metadata.locale);
   });
 
   it.skip("round-trips .sps file without losing pages (saveFromTree not implemented)", () => {