49b413bf07
* feat: implement comprehensive testing infrastructure - Fix image dataURL truncation bug in security.ts with configurable size limits - Add backend integration tests (22 tests) with Vitest for API validation - Add frontend unit tests (11 tests) for JSON serialization - Implement browser-based E2E tests (8 tests) with Playwright - Create Docker setup for repeatable E2E testing environment - Add GitHub Actions CI workflow for automated testing - Update .gitignore for test artifacts and temporary files Testing Infrastructure: - Backend: Vitest + Supertest for API integration tests - Frontend: Vitest + Testing Library for component tests - E2E: Playwright with Chromium for full browser automation - CI/CD: GitHub Actions with parallel test execution Security Improvements: - Make dataURL size limit configurable (default: 10MB) - Enhanced validation for image dataURLs - Block malicious content (javascript:, script tags) All tests pass: 41 total (22 backend + 11 frontend + 8 E2E) * feat(tests): add comprehensive E2E tests for dashboard workflows and image persistence chore(env): update environment variables for consistent API URL usage fix(api): centralize API request helpers for drawing and collection management style(DrawingCard): enhance accessibility with ARIA attributes and data-testid for testing * cleanup/revise documentation * cleanup/revise documentation * Add end-to-end tests for drawing CRUD, export/import, search/sort, and theme toggle functionalities - Implemented E2E tests for drawing creation, editing, and deletion in `drawing-crud.spec.ts`. - Added tests for export and import features, including JSON and SQLite formats in `export-import.spec.ts`. - Created tests for searching and sorting drawings by name and date in `search-and-sort.spec.ts`. - Developed tests for theme toggle functionality to ensure persistence across sessions in `theme-toggle.spec.ts`. * fix: exclude test files from production build to fix Docker build * feat: implement comprehensive testing infrastructure (#19) * bump version 0.1.7 * feat: implement comprehensive testing infrastructure - Fix image dataURL truncation bug in security.ts with configurable size limits - Add backend integration tests (22 tests) with Vitest for API validation - Add frontend unit tests (11 tests) for JSON serialization - Implement browser-based E2E tests (8 tests) with Playwright - Create Docker setup for repeatable E2E testing environment - Add GitHub Actions CI workflow for automated testing - Update .gitignore for test artifacts and temporary files Testing Infrastructure: - Backend: Vitest + Supertest for API integration tests - Frontend: Vitest + Testing Library for component tests - E2E: Playwright with Chromium for full browser automation - CI/CD: GitHub Actions with parallel test execution Security Improvements: - Make dataURL size limit configurable (default: 10MB) - Enhanced validation for image dataURLs - Block malicious content (javascript:, script tags) All tests pass: 41 total (22 backend + 11 frontend + 8 E2E) * feat(tests): add comprehensive E2E tests for dashboard workflows and image persistence chore(env): update environment variables for consistent API URL usage fix(api): centralize API request helpers for drawing and collection management style(DrawingCard): enhance accessibility with ARIA attributes and data-testid for testing * Add end-to-end tests for drawing CRUD, export/import, search/sort, and theme toggle functionalities - Implemented E2E tests for drawing creation, editing, and deletion in `drawing-crud.spec.ts`. - Added tests for export and import features, including JSON and SQLite formats in `export-import.spec.ts`. - Created tests for searching and sorting drawings by name and date in `search-and-sort.spec.ts`. - Developed tests for theme toggle functionality to ensure persistence across sessions in `theme-toggle.spec.ts`. * Update backend/src/__tests__/testUtils.ts --------- Co-authored-by: Zimeng Xiong <zxzimeng@gmail.com> * version bump 0.1.8 * fix(ci): consolidate E2E server startup to prevent shell isolation issues Background processes started with & in separate GitHub Actions run steps can terminate when those steps complete because each step creates a new shell. This caused the backend and frontend servers to die before the E2E tests could run. Fixed by consolidating server startup and test execution into a single shell step with: - Proper PID tracking for cleanup - Health check loops instead of fixed sleep times - All processes run in the same shell session * fix(ci): use absolute database path for E2E tests * fix(backend): use resolved DATABASE_URL path for export/import endpoints --------- Co-authored-by: Adrian Acala <adrianacala017@gmail.com>
153 lines
4.7 KiB
TypeScript
153 lines
4.7 KiB
TypeScript
import { test, expect } from "@playwright/test";
|
|
|
|
/**
|
|
* E2E Tests for Theme Toggle functionality
|
|
*
|
|
* Tests the dark/light theme feature:
|
|
* - Toggle theme via Settings page
|
|
* - Theme persists across page reloads
|
|
* - Theme applies to all pages
|
|
*/
|
|
|
|
test.describe("Theme Toggle", () => {
|
|
test("should toggle theme from Settings page", async ({ page }) => {
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Find the theme toggle button
|
|
const themeButton = page.getByRole("button", { name: /Dark Mode|Light Mode/i });
|
|
await expect(themeButton).toBeVisible();
|
|
|
|
// Get initial theme state from html element
|
|
const html = page.locator("html");
|
|
const initialDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
|
|
// Click to toggle theme
|
|
await themeButton.click();
|
|
await page.waitForTimeout(500);
|
|
|
|
// Verify theme changed
|
|
const newDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
expect(newDark).toBe(!initialDark);
|
|
|
|
// Button text should also change
|
|
if (initialDark) {
|
|
await expect(themeButton).toContainText("Dark Mode");
|
|
} else {
|
|
await expect(themeButton).toContainText("Light Mode");
|
|
}
|
|
});
|
|
|
|
test("should persist theme across page navigation", async ({ page }) => {
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
const html = page.locator("html");
|
|
const themeButton = page.getByRole("button", { name: /Dark Mode|Light Mode/i });
|
|
|
|
// Set to dark mode first
|
|
const isDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
if (!isDark) {
|
|
await themeButton.click();
|
|
await page.waitForTimeout(500);
|
|
}
|
|
|
|
// Verify dark mode is set
|
|
await expect(html).toHaveClass(/dark/);
|
|
|
|
// Navigate to dashboard
|
|
await page.goto("/");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Theme should persist
|
|
await expect(html).toHaveClass(/dark/);
|
|
|
|
// Navigate back to settings
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Theme should still be dark
|
|
await expect(html).toHaveClass(/dark/);
|
|
|
|
// Toggle back to light for cleanup
|
|
const lightButton = page.getByRole("button", { name: /Light Mode/i });
|
|
if (await lightButton.isVisible()) {
|
|
await lightButton.click();
|
|
}
|
|
});
|
|
|
|
test("should persist theme across page reload", async ({ page }) => {
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
const html = page.locator("html");
|
|
const themeButton = page.getByRole("button", { name: /Dark Mode|Light Mode/i });
|
|
|
|
// Toggle to dark mode
|
|
const initialDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
if (!initialDark) {
|
|
await themeButton.click();
|
|
await page.waitForTimeout(500);
|
|
}
|
|
|
|
// Reload the page
|
|
await page.reload();
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Theme should persist after reload
|
|
await expect(html).toHaveClass(/dark/);
|
|
});
|
|
|
|
test("should apply dark theme styling to dashboard", async ({ page }) => {
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
const html = page.locator("html");
|
|
const themeButton = page.getByRole("button", { name: /Dark Mode|Light Mode/i });
|
|
|
|
// Ensure dark mode
|
|
const isDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
if (!isDark) {
|
|
await themeButton.click();
|
|
await page.waitForTimeout(500);
|
|
}
|
|
|
|
// Navigate to dashboard
|
|
await page.goto("/");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Check that dark theme styles are applied
|
|
// The body should have dark background colors
|
|
const body = page.locator("body");
|
|
const bodyBgColor = await body.evaluate((el) => {
|
|
return window.getComputedStyle(el).backgroundColor;
|
|
});
|
|
|
|
// Dark mode typically has dark backgrounds (low RGB values)
|
|
// This is a basic check - adjust based on actual theme colors
|
|
expect(bodyBgColor).toBeTruthy();
|
|
});
|
|
|
|
test("should apply light theme styling to dashboard", async ({ page }) => {
|
|
await page.goto("/settings");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
const html = page.locator("html");
|
|
const themeButton = page.getByRole("button", { name: /Dark Mode|Light Mode/i });
|
|
|
|
// Ensure light mode
|
|
const isDark = await html.evaluate((el) => el.classList.contains("dark"));
|
|
if (isDark) {
|
|
await themeButton.click();
|
|
await page.waitForTimeout(500);
|
|
}
|
|
|
|
// Navigate to dashboard
|
|
await page.goto("/");
|
|
await page.waitForLoadState("networkidle");
|
|
|
|
// Check that html doesn't have dark class
|
|
await expect(html).not.toHaveClass(/dark/);
|
|
});
|
|
});
|