Removed screenshot testing from complex pages (#624)

* Strip screenshot tests for nondeterministic page endpoints

Some acmcsuf.com pages give the Playwright testing some trouble, being capable of generating non-deterministic screenshots. This is a problem and should be stripped out of the main branch for now so that the nondeterministic screenshot tests may be rewritten on a case-by-case basis.

src/routes/(site)/1st/page.test.ts

@@ -1,58 +1,8 @@
-import type { TestInfo } from '@playwright/test';
 import { expect, test } from '@playwright/test';
-import path from 'node:path';
 
 test.describe.configure({ mode: 'parallel' });
 
-// setupSnapshot is a helper that creates handles the necessary boilerplate
-// for snapshot testing. It returns a function that can be called after the
-// snapshot test has executed.
-//
-// See:
-// https://github.com/microsoft/playwright/issues/14527#issuecomment-1143352769
-export function setupSnapshot(testInfo: TestInfo) {
-  const snapshotDir = testInfo.snapshotDir;
-  const snapshotSuffix = testInfo.snapshotSuffix;
-  const projectName = testInfo.project.name;
-
-  testInfo.snapshotDir = path.resolve(snapshotDir, '..', 'screenshots');
-  testInfo.snapshotSuffix = '';
-  testInfo.project.name = '';
-
-  return {
-    data: { projectName },
-    cleanupSnapshot() {
-      testInfo.snapshotDir = snapshotDir;
-      testInfo.snapshotSuffix = snapshotSuffix;
-      testInfo.project.name = projectName;
-    },
-  };
-}
-
 test('first contributions page has expected h1', async ({ page }) => {
   await page.goto('/1st');
   expect(await page.textContent('h1')).toBe('First Contributions');
 });
-
-test('first contributions page matches screenshot', async ({ page }, testInfo) => {
-  const { cleanupSnapshot, data } = setupSnapshot(testInfo);
-
-  if (testInfo.project.use.viewport) {
-    await page.setViewportSize(testInfo.project.use.viewport);
-  }
-
-  if (testInfo.project.use.colorScheme) {
-    await page.emulateMedia({
-      colorScheme: testInfo.project.use.colorScheme,
-    });
-  }
-
-  await page.goto('/1st', { waitUntil: 'networkidle' });
-  expect(await page.screenshot({ fullPage: true })).toMatchSnapshot({
-    name: `page-${data.projectName}.png`,
-    maxDiffPixelRatio: 0.1,
-    threshold: 1,
-  });
-
-  cleanupSnapshot();
-});

src/routes/(site)/blog/[id]/page.test.ts

@@ -33,26 +33,3 @@ test('blog post h1 matches page title', async ({ page }) => {
   await page.goto('/blog/272');
   expect(await page.title()).toContain(await page.textContent('h1'));
 });
-
-test('blog post page matches screenshot', async ({ page }, testInfo) => {
-  const { cleanupSnapshot, data } = setupSnapshot(testInfo);
-
-  if (testInfo.project.use.viewport) {
-    await page.setViewportSize(testInfo.project.use.viewport);
-  }
-
-  if (testInfo.project.use.colorScheme) {
-    await page.emulateMedia({
-      colorScheme: testInfo.project.use.colorScheme,
-    });
-  }
-
-  await page.goto('/blog/272', { waitUntil: 'networkidle' });
-  expect(await page.screenshot({ fullPage: true, scale: 'css' })).toMatchSnapshot({
-    name: `page-${data.projectName}.png`,
-    maxDiffPixelRatio: 0.1,
-    threshold: 1,
-  });
-
-  cleanupSnapshot();
-});

src/routes/(site)/events/page.test.ts

@@ -1,58 +1,6 @@
-import type { TestInfo } from '@playwright/test';
 import { expect, test } from '@playwright/test';
-import path from 'node:path';
-
-test.describe.configure({ mode: 'parallel' });
-
-// setupSnapshot is a helper that creates handles the necessary boilerplate
-// for snapshot testing. It returns a function that can be called after the
-// snapshot test has executed.
-//
-// See:
-// https://github.com/microsoft/playwright/issues/14527#issuecomment-1143352769
-export function setupSnapshot(testInfo: TestInfo) {
-  const snapshotDir = testInfo.snapshotDir;
-  const snapshotSuffix = testInfo.snapshotSuffix;
-  const projectName = testInfo.project.name;
-
-  testInfo.snapshotDir = path.resolve(snapshotDir, '..', 'screenshots');
-  testInfo.snapshotSuffix = '';
-  testInfo.project.name = '';
-
-  return {
-    data: { projectName },
-    cleanupSnapshot() {
-      testInfo.snapshotDir = snapshotDir;
-      testInfo.snapshotSuffix = snapshotSuffix;
-      testInfo.project.name = projectName;
-    },
-  };
-}
 
 test('events page has expected h2', async ({ page }) => {
   await page.goto('/events');
   expect(await page.textContent('h2')).toBe('Curated events for growth and success');
 });
-
-test('events page matches screenshot', async ({ page }, testInfo) => {
-  const { cleanupSnapshot, data } = setupSnapshot(testInfo);
-
-  if (testInfo.project.use.viewport) {
-    await page.setViewportSize(testInfo.project.use.viewport);
-  }
-
-  if (testInfo.project.use.colorScheme) {
-    await page.emulateMedia({
-      colorScheme: testInfo.project.use.colorScheme,
-    });
-  }
-
-  await page.goto('/events', { waitUntil: 'networkidle' });
-  expect(await page.screenshot({ fullPage: true })).toMatchSnapshot({
-    name: `page-${data.projectName}.png`,
-    maxDiffPixelRatio: 0.1,
-    threshold: 1,
-  });
-
-  cleanupSnapshot();
-});