External/SnapOtter
1
0
Fork 0
mirror of https://github.com/snapotter-hq/SnapOtter.git synced 2026-08-03 07:46:42 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: sync API docs, register content-aware-resize, normalize tool counts

Browse Source 
- Fix 23 OpenAPI schema discrepancies across 16+ tools (wrong ranges,
  missing fields, incorrect schemas for gif-tools/collage/ocr)
- Add content-aware-resize to canonical TOOLS array and landing BentoGrid
- Normalize tool count to 47 across README, docs, landing, i18n, OpenAPI
- Remove dead "automation" ToolCategory variant
- Add BMP and JPEG XL format decoding via ImageMagick
- Add libopenexr-dev to Docker runtime image
- Update e2e test selectors for current pipeline builder UI
This commit is contained in:
SnapOtter
2026-04-24 23:27:08 +08:00
parent 7f62bc32db
commit 8633dba431
18 changed files with 211 additions and 84 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/e2e/automate.spec.ts
+17 -14
View File
@@ -10,16 +10,16 @@ test.describe("Automate Page", () => {
*/
async function gotoAutomate(page: import("@playwright/test").Page) {
const heading = page.getByRole("heading", {
name: /automate/i,
name: /pipeline builder|automate/i,
});
for (let attempt = 0; attempt < 3; attempt++) {
if (attempt === 0) {
await page.goto("/automate", { waitUntil: "networkidle" });
await page.goto("/automate", { waitUntil: "load" });
} else {
// On retry, wait then reload
await page.waitForTimeout(500);
await page.goto("/automate", { waitUntil: "networkidle" });
await page.goto("/automate", { waitUntil: "load" });
}
try {
@@ -31,7 +31,7 @@ test.describe("Automate Page", () => {
}
// Final attempt - let it throw if it fails
await page.goto("/automate", { waitUntil: "networkidle" });
await page.goto("/automate", { waitUntil: "load" });
await expect(heading).toBeVisible({ timeout: 10_000 });
}
@@ -48,8 +48,6 @@ test.describe("Automate Page", () => {
name: string,
expectedCount: number,
) {
await page.getByRole("button", { name: /add step/i }).click();
await expect(page.getByText("Add a step")).toBeVisible();
await page.getByPlaceholder("Search tools...").fill(name);
await page
.getByRole("button", { name: new RegExp(name, "i") })
@@ -74,12 +72,16 @@ test.describe("Automate Page", () => {
test("automate page renders pipeline builder", async ({ loggedInPage: page }) => {
await gotoAutomate(page);
await expect(page.getByText(/chain tools into a pipeline/i).first()).toBeVisible();
await expect(
page.getByText(/add tools from the palette|chain tools into a pipeline/i).first(),
).toBeVisible();
});
test("shows empty state message when no steps", async ({ loggedInPage: page }) => {
await gotoAutomate(page);
await expect(page.getByText(/add steps to build your pipeline/i)).toBeVisible();
await expect(
page.getByText(/add tools from the palette|add steps to build your pipeline/i),
).toBeVisible();
});
test("shows dropzone when no file uploaded", async ({ loggedInPage: page }) => {
@@ -89,9 +91,9 @@ test.describe("Automate Page", () => {
await expect(page.getByRole("button", { name: /upload from computer/i })).toBeVisible();
});
test("has Add Step button", async ({ loggedInPage: page }) => {
test("has tool palette search", async ({ loggedInPage: page }) => {
await gotoAutomate(page);
await expect(page.getByRole("button", { name: /add step/i })).toBeVisible();
await expect(page.getByPlaceholder("Search tools...")).toBeVisible();
});
test("has Process button (disabled when no steps or file)", async ({ loggedInPage: page }) => {
@@ -116,17 +118,18 @@ test.describe("Automate Page", () => {
// --- Add Step ---
test("clicking Add Step opens tool picker", async ({ loggedInPage: page }) => {
test("tool palette is visible with search", async ({ loggedInPage: page }) => {
await gotoAutomate(page);
await page.getByRole("button", { name: /add step/i }).click();
await expect(page.getByText("Add a step")).toBeVisible();
await expect(page.getByPlaceholder("Search tools...")).toBeVisible();
});
test("selecting a tool from picker adds a step", async ({ loggedInPage: page }) => {
await gotoAutomate(page);
await addToolStep(page, "Resize", 1);
// Verify empty state is gone
await expect(page.getByText(/add steps to build your pipeline/i)).not.toBeVisible();
await expect(
page.getByText(/add tools from the palette|add steps to build your pipeline/i),
).not.toBeVisible();
});
test("can add multiple steps", async ({ loggedInPage: page }) => {