fix: normalize line endings in structure snapshots for cross-platform compatibility

Author: SabotageAndi

tests/all-pages.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from '@playwright/test';
-import { verifyPageLoads, createPageStructureSnapshot } from './test-utils';
+import { verifyPageLoads, createPageStructureSnapshot, toNormalizedSnapshot } from './test-utils';
 
 /**
  * Comprehensive tests that verify ALL collection items render correctly
@@ -182,30 +182,30 @@ test.describe('Page Content Snapshots', () => {
     await page.waitForLoadState('networkidle');
 
     const structure = await createPageStructureSnapshot(page);
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('all-home-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('all-home-structure.txt');
   });
 
   test('events page content snapshot', async ({ page }) => {
     await page.goto('/events/');
     await page.waitForLoadState('networkidle');
 
     const structure = await createPageStructureSnapshot(page);
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('all-events-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('all-events-structure.txt');
   });
 
   test('sponsors page content snapshot', async ({ page }) => {
     await page.goto('/sponsors/');
     await page.waitForLoadState('networkidle');
 
     const structure = await createPageStructureSnapshot(page);
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('all-sponsors-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('all-sponsors-structure.txt');
   });
 
   test('speakers page content snapshot', async ({ page }) => {
     await page.goto('/speakers/');
     await page.waitForLoadState('networkidle');
 
     const structure = await createPageStructureSnapshot(page);
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('all-speakers-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('all-speakers-structure.txt');
   });
 });

tests/events.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from '@playwright/test';
-import { verifyPageLoads, createPageStructureSnapshot } from './test-utils';
+import { verifyPageLoads, createPageStructureSnapshot, toNormalizedSnapshot } from './test-utils';
 
 test.describe('Events Page', () => {
   test('should load successfully', async ({ page }) => {
@@ -44,7 +44,7 @@ test.describe('Events Page', () => {
     await page.goto('/events/');
     const structure = await createPageStructureSnapshot(page);
 
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('events-page-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('events-page-structure.txt');
   });
 
   test('visual snapshot', async ({ page }) => {

tests/home.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from '@playwright/test';
-import { verifyPageLoads, createPageStructureSnapshot, extractTextContent } from './test-utils';
+import { verifyPageLoads, createPageStructureSnapshot, extractTextContent, toNormalizedSnapshot } from './test-utils';
 
 test.describe('Home Page', () => {
   test('should load successfully', async ({ page }) => {
@@ -51,7 +51,7 @@ test.describe('Home Page', () => {
     await page.goto('/');
     const structure = await createPageStructureSnapshot(page);
 
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('home-page-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('home-page-structure.txt');
   });
 
   test('visual snapshot', async ({ page }) => {

tests/speakers.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from '@playwright/test';
-import { verifyPageLoads, createPageStructureSnapshot, extractTextContent } from './test-utils';
+import { verifyPageLoads, createPageStructureSnapshot, extractTextContent, toNormalizedSnapshot } from './test-utils';
 
 test.describe('Speakers Page', () => {
   test('should load successfully', async ({ page }) => {
@@ -46,7 +46,7 @@ test.describe('Speakers Page', () => {
     await page.goto('/speakers/');
     const structure = await createPageStructureSnapshot(page);
 
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('speakers-page-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('speakers-page-structure.txt');
   });
 
   test('visual snapshot', async ({ page }) => {

tests/sponsors.spec.ts

@@ -1,5 +1,5 @@
 import { test, expect } from '@playwright/test';
-import { verifyPageLoads, createPageStructureSnapshot, extractTextContent } from './test-utils';
+import { verifyPageLoads, createPageStructureSnapshot, extractTextContent, toNormalizedSnapshot } from './test-utils';
 
 test.describe('Sponsors Page', () => {
   test('should load successfully', async ({ page }) => {
@@ -36,7 +36,7 @@ test.describe('Sponsors Page', () => {
     await page.goto('/sponsors/');
     const structure = await createPageStructureSnapshot(page);
 
-    expect(JSON.stringify(structure, null, 2)).toMatchSnapshot('sponsors-page-structure.txt');
+    expect(toNormalizedSnapshot(structure)).toMatchSnapshot('sponsors-page-structure.txt');
   });
 
   test('visual snapshot', async ({ page }) => {

tests/test-utils.ts

@@ -4,6 +4,24 @@ import { expect, Page } from '@playwright/test';
  * Utility functions for approval testing
  */
 
+/**
+ * Normalizes line endings to LF and removes empty lines for cross-platform snapshot comparison
+ */
+export function normalizeLineEndings(text: string): string {
+  return text
+    .replace(/\r\n/g, '\n')  // Convert CRLF to LF
+    .replace(/\r/g, '\n')     // Convert remaining CR to LF
+    .replace(/\n\s*\n/g, '\n') // Remove empty lines
+    .trim();
+}
+
+/**
+ * Creates a normalized JSON string for snapshot comparison
+ */
+export function toNormalizedSnapshot(obj: object): string {
+  return normalizeLineEndings(JSON.stringify(obj, null, 2));
+}
+
 /**
  * Normalizes HTML content for approval testing by removing dynamic content
  * like timestamps, session IDs, etc.