Revise what pages are build on CI. Make all demo pages available.

dnbexperience · Sep 21, 2023 · 37efa39 · 37efa39
1 parent 47f0018
commit 37efa39
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 16 deletions.
diff --git a/packages/dnb-design-system-portal/gatsby-node.js b/packages/dnb-design-system-portal/gatsby-node.js
@@ -139,20 +139,26 @@ exports.onCreatePage = ({ page, actions }) => {
   // Only build pages without "'/uilib'" when building for visual tests
   if (process.env.IS_VISUAL_TEST === '1') {
     if (
-      (page.path !== '/' &&
-        !existsInPages(page.path, [
-          '/404',
-          '/500',
-          '/uilib',
-          '/quickguide-designer/colors',
-          '/quickguide-designer/fonts',
-          '/contribute/getting-started',
-        ])) ||
-      existsInPages(page.path, ['/forms/']) ||
-      existsInPages(page.componentPath, [
-        'properties.mdx',
-        'events.mdx',
-        'info.mdx',
+      page.path !== '/' &&
+      !existsInPages(page.path, [
+        // General pages
+        '/404',
+        '/500',
+
+        // Playwright e2e tests
+        '/uilib',
+        '/uilib/components/button',
+        '/uilib/components',
+        '/uilib/extensions',
+        '/uilib/elements',
+        '/quickguide-designer/colors',
+        '/quickguide-designer/fonts',
+        '/contribute/getting-started',
+      ]) &&
+      !existsInPages(page.componentPath, [
+        // Visual e2e tests
+        'visual-tests',
+        'demos.mdx',
       ])
     ) {
       deletedPages.push(page.path)

diff --git a/packages/dnb-design-system-portal/src/e2e/fullscreen.spec.ts b/packages/dnb-design-system-portal/src/e2e/fullscreen.spec.ts
@@ -2,7 +2,7 @@ import { test, expect } from '@playwright/test'
 
 test.describe('Fullscreen', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto('/uilib/components/button')
+    await page.goto('/uilib/components/button/demos')
   })
 
   test('click on fullscreen button should navigate to the fullscreen page', async ({
@@ -17,6 +17,8 @@ test.describe('Fullscreen', () => {
     })
 
     const currentURL = page.url()
-    expect(currentURL).toContain('/uilib/components/button/?fullscreen')
+    expect(currentURL).toContain(
+      '/uilib/components/button/demos/?fullscreen',
+    )
   })
 })