mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
Expand test coverage across all layers via 14 parallel agents: Unit tests (3,378 total, +534): - First-ever AI sidecar tests (157 tests covering bridge lifecycle, all 12 tool modules) - API route infrastructure (auth, pipeline, batch, settings, teams, roles, audit, api-keys, files, docs) - Lib coverage improvements (audit 7%->95%, worker-pool 33%->100%) - Web store/lib gap fills (features-store, tool-registry) Integration tests (4,403 total, +903): - Expanded 19 tool test files with parameter variations, format edge cases, boundary values - Cross-format matrix: 290 tests covering 14 tools x 17 formats - Adversarial/edge cases: 63 tests for extreme inputs, concurrent requests, corrupted files E2E-Docker (125 new tests): - Expanded 8 spec files + 1 new file covering all 49 tools - Added HEIC/format handling, auth failures, download verification GUI E2E (expanded 28 spec files): - Navigation, responsive layout, keyboard shortcuts - All 51 tool UIs with settings, processing, display modes - Batch/pipeline workflows, settings/RBAC, visual regression - Resilience, accessibility (ARIA, contrast, focus), performance budgets
230 lines
9.7 KiB
TypeScript
230 lines
9.7 KiB
TypeScript
import { expect, openSettings, test } from "./helpers";
|
|
|
|
// ---------------------------------------------------------------------------
|
|
// Settings Dialog -- Security (change password) and API Keys tabs
|
|
// ---------------------------------------------------------------------------
|
|
|
|
test.describe("GUI Settings - Security Tab", () => {
|
|
test("shows Change Password form with required inputs", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await expect(page.locator("h3").filter({ hasText: "Security" })).toBeVisible();
|
|
await expect(page.getByText("Change Password").first()).toBeVisible();
|
|
|
|
// Three password fields
|
|
await expect(page.getByPlaceholder("Current Password")).toBeVisible();
|
|
await expect(page.getByPlaceholder("New Password").first()).toBeVisible();
|
|
await expect(page.getByPlaceholder("Confirm New Password")).toBeVisible();
|
|
|
|
// Submit button
|
|
await expect(page.getByRole("button", { name: /change password/i })).toBeVisible();
|
|
});
|
|
|
|
test("mismatched passwords show error message", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await page.getByPlaceholder("Current Password").fill("admin");
|
|
await page.getByPlaceholder("New Password").first().fill("NewPass123");
|
|
await page.getByPlaceholder("Confirm New Password").fill("DifferentPass456");
|
|
|
|
await page.getByRole("button", { name: /change password/i }).click();
|
|
|
|
await expect(page.getByText("Passwords do not match")).toBeVisible();
|
|
});
|
|
|
|
test("password visibility toggles work", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
const currentPwInput = page.getByPlaceholder("Current Password");
|
|
await expect(currentPwInput).toHaveAttribute("type", "password");
|
|
|
|
// Click the eye toggle button next to the current password field
|
|
const toggleButtons = page.locator("form button[type='button']");
|
|
await toggleButtons.first().click();
|
|
|
|
await expect(currentPwInput).toHaveAttribute("type", "text");
|
|
});
|
|
|
|
test("short password shows validation error", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await page.getByPlaceholder("Current Password").fill("admin");
|
|
await page.getByPlaceholder("New Password").first().fill("ab");
|
|
await page.getByPlaceholder("Confirm New Password").fill("ab");
|
|
|
|
await page.getByRole("button", { name: /change password/i }).click();
|
|
|
|
// Should show validation error about minimum length
|
|
await expect(page.getByText(/at least 4 characters/i)).toBeVisible({ timeout: 5_000 });
|
|
});
|
|
|
|
test("wrong current password shows error message", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await page.getByPlaceholder("Current Password").fill("wrongpassword");
|
|
await page.getByPlaceholder("New Password").first().fill("NewPass123");
|
|
await page.getByPlaceholder("Confirm New Password").fill("NewPass123");
|
|
|
|
await page.getByRole("button", { name: /change password/i }).click();
|
|
|
|
// The API returns 401 which maps to "Current password is incorrect"
|
|
await expect(page.getByText("Current password is incorrect")).toBeVisible({ timeout: 5_000 });
|
|
});
|
|
|
|
test("successful password change shows success message", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
// Change password from admin -> admin (same value, to avoid breaking other tests)
|
|
await page.getByPlaceholder("Current Password").fill("admin");
|
|
await page.getByPlaceholder("New Password").first().fill("admin");
|
|
await page.getByPlaceholder("Confirm New Password").fill("admin");
|
|
|
|
await page.getByRole("button", { name: /change password/i }).click();
|
|
|
|
await expect(page.getByText("Password changed successfully")).toBeVisible({ timeout: 5_000 });
|
|
});
|
|
|
|
test("form fields are cleared after successful password change", async ({
|
|
loggedInPage: page,
|
|
}) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await page.getByPlaceholder("Current Password").fill("admin");
|
|
await page.getByPlaceholder("New Password").first().fill("admin");
|
|
await page.getByPlaceholder("Confirm New Password").fill("admin");
|
|
|
|
await page.getByRole("button", { name: /change password/i }).click();
|
|
await expect(page.getByText("Password changed successfully")).toBeVisible({ timeout: 5_000 });
|
|
|
|
// All fields should be cleared after success
|
|
await expect(page.getByPlaceholder("Current Password")).toHaveValue("");
|
|
await expect(page.getByPlaceholder("New Password").first()).toHaveValue("");
|
|
await expect(page.getByPlaceholder("Confirm New Password")).toHaveValue("");
|
|
});
|
|
|
|
test("section heading and description are displayed", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await expect(page.locator("h3").filter({ hasText: "Security" })).toBeVisible();
|
|
await expect(page.getByText("Password and authentication settings.")).toBeVisible();
|
|
});
|
|
|
|
test("security section shows login attempt limit reference", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /security/i }).click();
|
|
|
|
await expect(page.getByText(/login attempt limits/i)).toBeVisible();
|
|
await expect(
|
|
page.getByText("Login attempt limits can be configured in System Settings."),
|
|
).toBeVisible();
|
|
});
|
|
});
|
|
|
|
test.describe("GUI Settings - API Keys Tab", () => {
|
|
test("shows Generate API Key button and name input", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
await expect(page.locator("h3").filter({ hasText: "API Keys" })).toBeVisible();
|
|
await expect(page.getByPlaceholder("Key name (optional)")).toBeVisible();
|
|
await expect(page.getByRole("button", { name: /generate api key/i })).toBeVisible();
|
|
});
|
|
|
|
test("generating an API key displays the key once", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
// Give a unique name so we can identify and clean it up
|
|
const keyName = `guiTest-${Date.now()}`;
|
|
await page.getByPlaceholder("Key name (optional)").fill(keyName);
|
|
await page.getByRole("button", { name: /generate api key/i }).click();
|
|
|
|
// The key should appear in a code element
|
|
const keyDisplay = page.locator("code.font-mono");
|
|
await expect(keyDisplay).toBeVisible({ timeout: 5_000 });
|
|
|
|
// The "Store this key" warning should be visible
|
|
await expect(page.getByText("Store this key securely")).toBeVisible();
|
|
|
|
// Clean up: delete the key we just created
|
|
const deleteBtn = page.locator("button[title='Delete key']").first();
|
|
if (await deleteBtn.isVisible()) {
|
|
page.on("dialog", (d) => d.accept());
|
|
await deleteBtn.click();
|
|
}
|
|
});
|
|
|
|
test("generated key appears in existing keys list", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
const keyName = `listTest-${Date.now()}`;
|
|
await page.getByPlaceholder("Key name (optional)").fill(keyName);
|
|
await page.getByRole("button", { name: /generate api key/i }).click();
|
|
|
|
// Wait for key to appear
|
|
await expect(page.locator("code.font-mono")).toBeVisible({ timeout: 5_000 });
|
|
|
|
// The key name should appear in the "Existing Keys" section
|
|
await expect(page.getByText("Existing Keys")).toBeVisible();
|
|
await expect(page.getByText(keyName)).toBeVisible();
|
|
|
|
// Clean up
|
|
page.on("dialog", (d) => d.accept());
|
|
await page.locator("button[title='Delete key']").first().click();
|
|
await page.waitForTimeout(500);
|
|
});
|
|
|
|
test("permission scoping toggle reveals checkboxes", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
const scopingToggle = page.getByText("Restrict permissions (optional)");
|
|
await expect(scopingToggle).toBeVisible();
|
|
|
|
await scopingToggle.click();
|
|
|
|
// After expanding, the collapse text and permission checkboxes appear
|
|
await expect(page.getByText("Remove permission scoping")).toBeVisible();
|
|
await expect(page.locator("input[type='checkbox']").first()).toBeVisible();
|
|
await expect(page.getByText("tools:use")).toBeVisible();
|
|
});
|
|
|
|
test("deleting an API key removes it from the list", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
// Create a key to delete
|
|
const keyName = `deleteTest-${Date.now()}`;
|
|
await page.getByPlaceholder("Key name (optional)").fill(keyName);
|
|
await page.getByRole("button", { name: /generate api key/i }).click();
|
|
await expect(page.locator("code.font-mono")).toBeVisible({ timeout: 5_000 });
|
|
|
|
// Confirm the key appears
|
|
await expect(page.getByText(keyName)).toBeVisible();
|
|
|
|
// Delete it
|
|
page.on("dialog", (d) => d.accept());
|
|
await page.locator("button[title='Delete key']").first().click();
|
|
|
|
// Key name should disappear
|
|
await expect(page.getByText(keyName)).not.toBeVisible({ timeout: 5_000 });
|
|
});
|
|
|
|
test("expiration date input is available", async ({ loggedInPage: page }) => {
|
|
await openSettings(page);
|
|
await page.getByRole("button", { name: /api keys/i }).click();
|
|
|
|
await expect(page.getByText("Expires:")).toBeVisible();
|
|
await expect(page.locator("input[type='datetime-local']")).toBeVisible();
|
|
});
|
|
});
|