test(docs-e2e): disable flaky instagram link interaction

cuhacking · Dec 3, 2024 · 3a7b220 · 3a7b220
1 parent 61d7197
commit 3a7b220
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/.github/workflows/TEST_E2E.yml b/.github/workflows/TEST_E2E.yml
@@ -23,22 +23,22 @@ jobs:
       - name: Fetch main branch
         run: git fetch origin main # Explicitly fetch the 'main' branch to ensure it's available
 
-      - name: 📦 Install pnpm
-        uses: pnpm/action-setup@v4
-        with:
-          run_install: true
-
       - name: Install Node.js
         uses: actions/setup-node@v4
         with:
           node-version: 20
           cache: pnpm
 
+      - name: 📦 Install pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          run_install: true
+
       - name: 🎭 Install Playwright Browsers
         run: pnpm playwright install --with-deps
 
       # - name: ⚗️ Run E2E Tests for Affected Projects
-      #   run: pnpm nx affected -t e2e --parallel=2 --base=origin/main --head=HEAD --verbose
+      #   run: pnpm nx affected -t e2e --parallel=2 --reporter=html --base=origin/main --head=HEAD --verbose
 
       - name: 💀 Run Docs E2E Tests
         run: pnpm nx e2e docs-e2e --reporter=html --verbose

diff --git a/apps/docs-e2e/playwright.config.ts b/apps/docs-e2e/playwright.config.ts
@@ -49,14 +49,17 @@ export default defineConfig({
     {
       name: 'firefox (desktop)',
       use: { ...devices['Desktop Firefox'] },
+      testIgnore: ['./src/lighthouse.spec.ts'],
     },
     {
       name: 'webkit (desktop)',
       use: { ...devices['Desktop Safari'] },
+      testIgnore: ['./src/lighthouse.spec.ts'],
     },
     {
       name: 'webkit (tablet)',
       use: { ...devices['iPad Mini'] },
+      testIgnore: ['./src/lighthouse.spec.ts'],
     },
     {
       name: 'chromium (mobile)',
@@ -65,6 +68,7 @@ export default defineConfig({
     {
       name: 'webkit (mobile)',
       use: { ...devices['iPhone 12'] },
+      testIgnore: ['./src/lighthouse.spec.ts'],
     },
   ],
   // Ignore Chromium projects in CI to speed up runs

diff --git a/apps/docs-e2e/src/docs.spec.ts b/apps/docs-e2e/src/docs.spec.ts
@@ -29,7 +29,7 @@ const CUHACKING_2025_PORTAL_URL = 'https://portal.cuhacking.ca/'
 // const CUHACKING_2025_ESLINT_URL = 'https://eslint.cuhacking.ca/rules'
 
 const CUHACKING_2025_DISCORD_URL = 'https://discord.com/invite/h2cQqF9aZf'
-const CUHACKING_2025_INSTAGRAM_URL = 'https://www.instagram.com/cuhacking/'
+// const CUHACKING_2025_INSTAGRAM_URL = 'https://www.instagram.com/cuhacking/'
 
 // TODO: Uncomment when the link works - see test below
 // const CUHACKING_2025_LINKED_IN_URL = 'https://www.linkedin.com/company/cuhacking/'
@@ -131,9 +131,11 @@ test.describe('Common MOBILE, TABLET and DESKTOP Links', {
     await expect(docsLayoutPage.instagramLink).toBeVisible()
   })
 
-  test('should take user to Instagram when Instagram link clicked', async ({ docsLayoutPage }) => {
-    await clickAndGoToPage(docsLayoutPage, docsLayoutPage.instagramLink, CUHACKING_2025_INSTAGRAM_URL)
-  })
+  // TODO: Instagram link requires auth to view
+
+  // test('should take user to Instagram when Instagram link clicked', async ({ docsLayoutPage }) => {
+  //   await clickAndGoToPage(docsLayoutPage, docsLayoutPage.instagramLink, CUHACKING_2025_INSTAGRAM_URL)
+  // })
 
   test('should contain LinkedIn link', async ({ docsLayoutPage }) => {
     await expect(docsLayoutPage.linkedinLink).toBeVisible()