split nojs tests

concurrency may explain failures

Author: gordonwoodhull

tests/integration/playwright/tests/html-dark-mode-nojs.spec.ts

@@ -0,0 +1,22 @@
+import { test, expect } from '@playwright/test';
+
+test.use({ javaScriptEnabled: false });
+
+test('Dark brand default, no JS', async ({ page }) => {
+  // This document use a custom theme file that change the background color of the title banner
+  // Same user defined color should be used in both dark and light theme
+  await page.goto('./html/dark-brand/brand-after-theme.html');
+  const locatr = await page.locator('body').first();
+  await expect(locatr).toHaveClass('fullcontent quarto-dark');
+  await expect(locatr).toHaveCSS('background-color', 'rgb(66, 7, 11)');
+});
+
+
+test('Light brand default, no JS', async ({ page }) => {
+  // This document use a custom theme file that change the background color of the title banner
+  // Same user defined color should be used in both dark and light theme
+  await page.goto('./html/dark-brand/brand-before-theme.html');
+  const locatr = await page.locator('body').first();
+  await expect(locatr).toHaveClass('fullcontent quarto-light');
+  await expect(locatr).toHaveCSS('background-color', 'rgb(252, 252, 252)');
+});

tests/integration/playwright/tests/html-dark-mode.spec.ts

@@ -1,5 +1,4 @@
 import { test, expect } from '@playwright/test';
-import { asRGB, checkBorderProperties, checkColor, getCSSProperty } from '../src/utils';
 
 test('Dark and light brand after user themes', async ({ page }) => {
   // This document use a custom theme file that change the background color of the title banner
@@ -25,24 +24,3 @@ test('Dark and light brand before user themes', async ({ page }) => {
   const locatr2 = await page.locator('body').first();
   await expect(locatr2).toHaveCSS('background-color', 'rgb(6, 6, 6)');
 });
-
-test.use({ javaScriptEnabled: false });
-
-test('Dark brand default, no JS', async ({ page }) => {
-  // This document use a custom theme file that change the background color of the title banner
-  // Same user defined color should be used in both dark and light theme
-  await page.goto('./html/dark-brand/brand-after-theme.html');
-  const locatr = await page.locator('body').first();
-  await expect(locatr).toHaveClass('fullcontent quarto-dark');
-  await expect(locatr).toHaveCSS('background-color', 'rgb(66, 7, 11)');
-});
-
-
-test('Light brand default, no JS', async ({ page }) => {
-  // This document use a custom theme file that change the background color of the title banner
-  // Same user defined color should be used in both dark and light theme
-  await page.goto('./html/dark-brand/brand-before-theme.html');
-  const locatr = await page.locator('body').first();
-  await expect(locatr).toHaveClass('fullcontent quarto-light');
-  await expect(locatr).toHaveCSS('background-color', 'rgb(252, 252, 252)');
-});