Migrate from Playwright to Node.js built-in test runner

- Convert all test files to use node:test and node:assert/strict
- Create copy-tests.js script in each epicshop/ to propagate tests from solution to problem directories
- Add test scripts to all exercise package.json files
- Remove Playwright config and test files
- Update GitHub Actions workflow to use Node test runner

Test files now use top-level await with test() calls.

Author: kentcdodds

.github/workflows/validate.yml

@@ -56,21 +56,9 @@ jobs:
       - name: 📦 Install dependencies
         run: npm ci
 
-      - name: 🎭 Install Playwright Browsers
-        run: npx playwright install --with-deps chromium
-        working-directory: ./epicshop
-
-      - name: 🎭 Run Playwright tests
-        run: npx playwright test
-        working-directory: ./epicshop
-
-      - name: 📤 Upload Playwright Report
-        uses: actions/upload-artifact@v4
-        if: ${{ !cancelled() }}
-        with:
-          name: playwright-report
-          path: epicshop/playwright-report/
-          retention-days: 30
+      - name: 🧪 Run tests
+        id: run_tests
+        run: node ./epicshop/test.js ..s
 
   deploy:
     name: 🚀 Deploy

SETUP_STEPS.md

@@ -0,0 +1,54 @@
+import fs from 'node:fs'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const exercisesDir = path.join(__dirname, '..', 'exercises')
+
+function findSolutionDirs(dir) {
+	const solutionDirs = []
+
+	for (const exerciseDir of fs.readdirSync(dir)) {
+		const exercisePath = path.join(dir, exerciseDir)
+		if (!fs.statSync(exercisePath).isDirectory()) continue
+
+		for (const stepDir of fs.readdirSync(exercisePath)) {
+			if (stepDir.includes('.solution.')) {
+				solutionDirs.push(path.join(exercisePath, stepDir))
+			}
+		}
+	}
+
+	return solutionDirs
+}
+
+function copyTestFiles() {
+	const solutionDirs = findSolutionDirs(exercisesDir)
+	let copiedCount = 0
+
+	for (const solutionDir of solutionDirs) {
+		const problemDir = solutionDir.replace('.solution.', '.problem.')
+
+		if (!fs.existsSync(problemDir)) {
+			console.log(`⚠️  No problem dir for: ${path.basename(solutionDir)}`)
+			continue
+		}
+
+		const testFiles = fs
+			.readdirSync(solutionDir)
+			.filter((f) => f.endsWith('.test.ts'))
+
+		for (const testFile of testFiles) {
+			const src = path.join(solutionDir, testFile)
+			const dest = path.join(problemDir, testFile)
+
+			fs.copyFileSync(src, dest)
+			console.log(`✅ Copied ${testFile} to ${path.basename(problemDir)}`)
+			copiedCount++
+		}
+	}
+
+	console.log(`\n📋 Copied ${copiedCount} test file(s)`)
+}
+
+copyTestFiles()

epicshop/copy-tests.js

@@ -61,14 +61,6 @@ function printTestSummary(results) {
 
 async function main() {
 	const allApps = await getApps()
-	const allAppsWithTests = allApps.filter((app) => app.test?.type === 'script')
-
-	if (allAppsWithTests.length === 0) {
-		console.error(
-			'❌ No apps with tests were found. Ensure your apps have a test script defined in the package.json. Exiting.',
-		)
-		process.exit(1)
-	}
 
 	let selectedApps
 	let additionalArgs = []
@@ -82,7 +74,7 @@ async function main() {
 
 	if (process.argv[2]) {
 		const patterns = process.argv[2].toLowerCase().split(',')
-		selectedApps = allAppsWithTests.filter((app) => {
+		selectedApps = allApps.filter((app) => {
 			const { exerciseNumber, stepNumber, type } = app
 
 			return patterns.some((pattern) => {
@@ -103,10 +95,7 @@ async function main() {
 		})
 	} else {
 		const displayNameMap = new Map(
-			allAppsWithTests.map((app) => [
-				getAppDisplayName(app, allAppsWithTests),
-				app,
-			]),
+			allApps.map((app) => [getAppDisplayName(app, allApps), app]),
 		)
 		const choices = displayNameMap.keys()
 
@@ -122,14 +111,14 @@ async function main() {
 		})
 
 		selectedApps = response.appDisplayNames.includes('All')
-			? allAppsWithTests
+			? allApps
 			: response.appDisplayNames.map((appDisplayName) =>
 					displayNameMap.get(appDisplayName),
 				)
 
 		// Update this block to use process.argv
 		const appPattern =
-			selectedApps.length === allAppsWithTests.length
+			selectedApps.length === allApps.length
 				? '*'
 				: selectedApps
 						.map((app) => `${app.exerciseNumber}.${app.stepNumber}.${app.type}`)

epicshop/playwright.config.ts

@@ -0,0 +1,55 @@
+import assert from 'node:assert/strict'
+import { test } from 'node:test'
+import { price, productName, quantity, description } from './index.ts'
+
+await test('price should be defined correctly', () => {
+	assert.strictEqual(
+		price,
+		29.99,
+		'🚨 price should be set to 29.99 - check that you declared it as a number',
+	)
+	assert.strictEqual(
+		typeof price,
+		'number',
+		'🚨 price should be a number type - make sure you used a number literal, not a string',
+	)
+})
+
+await test('productName should be defined correctly', () => {
+	assert.strictEqual(
+		productName,
+		'TypeScript Guide',
+		'🚨 productName should be set to "TypeScript Guide" - check that you used the exact string',
+	)
+	assert.strictEqual(
+		typeof productName,
+		'string',
+		'🚨 productName should be a string type - make sure you used quotes around the value',
+	)
+})
+
+await test('quantity should be defined correctly', () => {
+	assert.strictEqual(
+		quantity,
+		100,
+		'🚨 quantity should be set to 100 - check that you declared it as a number',
+	)
+	assert.strictEqual(
+		typeof quantity,
+		'number',
+		'🚨 quantity should be a number type - make sure you used a number literal, not a string',
+	)
+})
+
+await test('description should be defined correctly', () => {
+	assert.strictEqual(
+		description,
+		'Product: TypeScript Guide | Price: $29.99 | In Stock: 100',
+		'🚨 description should combine productName, price, and quantity into a formatted string - use template literals or string concatenation',
+	)
+	assert.strictEqual(
+		typeof description,
+		'string',
+		'🚨 description should be a string type - make sure you created a string value',
+	)
+})

epicshop/test.js

@@ -2,6 +2,7 @@
   "name": "exercises_01.primitive-types_01.problem.numbers-and-strings",
   "type": "module",
   "scripts": {
-    "start": "node index.ts"
+    "start": "node index.ts",
+    "test": "node --test index.test.ts"
   }
 }

epicshop/workshop.test.ts

@@ -1,46 +1,55 @@
-import { testStep, expect } from '@epic-web/workshop-utils/test'
+import assert from 'node:assert/strict'
+import { test } from 'node:test'
 import { price, productName, quantity, description } from './index.ts'
 
-await testStep('price should be defined correctly', async () => {
-	expect(
+await test('price should be defined correctly', () => {
+	assert.strictEqual(
 		price,
+		29.99,
 		'🚨 price should be set to 29.99 - check that you declared it as a number',
-	).toBe(29.99)
-	expect(
+	)
+	assert.strictEqual(
 		typeof price,
+		'number',
 		'🚨 price should be a number type - make sure you used a number literal, not a string',
-	).toBe('number')
+	)
 })
 
-await testStep('productName should be defined correctly', async () => {
-	expect(
+await test('productName should be defined correctly', () => {
+	assert.strictEqual(
 		productName,
+		'TypeScript Guide',
 		'🚨 productName should be set to "TypeScript Guide" - check that you used the exact string',
-	).toBe('TypeScript Guide')
-	expect(
+	)
+	assert.strictEqual(
 		typeof productName,
+		'string',
 		'🚨 productName should be a string type - make sure you used quotes around the value',
-	).toBe('string')
+	)
 })
 
-await testStep('quantity should be defined correctly', async () => {
-	expect(
+await test('quantity should be defined correctly', () => {
+	assert.strictEqual(
 		quantity,
+		100,
 		'🚨 quantity should be set to 100 - check that you declared it as a number',
-	).toBe(100)
-	expect(
+	)
+	assert.strictEqual(
 		typeof quantity,
+		'number',
 		'🚨 quantity should be a number type - make sure you used a number literal, not a string',
-	).toBe('number')
+	)
 })
 
-await testStep('description should be defined correctly', async () => {
-	expect(
+await test('description should be defined correctly', () => {
+	assert.strictEqual(
 		description,
+		'Product: TypeScript Guide | Price: $29.99 | In Stock: 100',
 		'🚨 description should combine productName, price, and quantity into a formatted string - use template literals or string concatenation',
-	).toBe('Product: TypeScript Guide | Price: $29.99 | In Stock: 100')
-	expect(
+	)
+	assert.strictEqual(
 		typeof description,
+		'string',
 		'🚨 description should be a string type - make sure you created a string value',
-	).toBe('string')
+	)
 })

exercises/01.primitive-types/01.problem.numbers-and-strings/index.test.ts

@@ -2,6 +2,7 @@
   "name": "exercises_01.primitive-types_01.solution.numbers-and-strings",
   "type": "module",
   "scripts": {
-    "start": "node index.ts"
+    "start": "node index.ts",
+    "test": "node --test index.test.ts"
   }
 }