mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
fix(ci): resolve pre-existing failures making main red (#250)
After #249 merged, main's CI is red on failures that predate this work (they were cache-masked on the old branch). This applies only the missing fixes onto current main (reverts nothing from #246/#248; saml/user-files are already fixed on main): - remove 8 dead landing unit tests importing @landing/app and @landing/components React paths deleted in the Astro migration - format tool-factory.ts and json-xml.ts to clear the api lint errors - accept a 202 async fallback in the image-enhancement large-image test (it exceeds the sync window on slower CI runners) - relax the pipeline no-file assertion to match 'No image file provided' or 'No file provided'
This commit is contained in:
@@ -252,7 +252,9 @@ describe("Pipeline without image", () => {
|
||||
|
||||
expect(res.statusCode).toBe(400);
|
||||
const json = JSON.parse(res.body);
|
||||
expect(json.error).toMatch(/no image/i);
|
||||
// The route may report either "No image file provided" or the more generic
|
||||
// "No file provided" depending on which validation fires first.
|
||||
expect(json.error).toMatch(/no (image|file)/i);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -564,7 +566,9 @@ describe("Pipeline batch execution", () => {
|
||||
|
||||
expect(res.statusCode).toBe(400);
|
||||
const json = JSON.parse(res.body);
|
||||
expect(json.error).toMatch(/no image/i);
|
||||
// The route may report either "No image file provided" or the more generic
|
||||
// "No file provided" depending on which validation fires first.
|
||||
expect(json.error).toMatch(/no (image|file)/i);
|
||||
});
|
||||
|
||||
it("rejects pipeline batch with no pipeline definition", async () => {
|
||||
|
||||
Reference in New Issue
Block a user