openclawd/SnapAPI
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: update test stack indices for BUG-021 validation middleware
All checks were successful
Build & Deploy to Staging / Build & Deploy to Staging (push) Successful in 9m48s
Details

Browse source
This commit is contained in:
Hoid 2026-03-06 18:15:40 +01:00
parent e11ae1e074
commit 126490feca
2 changed files with 17 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/routes/__tests__/pdf.test.ts
View file

@ -231,7 +231,7 @@ describe('PDF Output', () => {
const req = createMockRequest({ url: 'https://example.com', format: 'pdf' }) const req = createMockRequest({ url: 'https://example.com', format: 'pdf' })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
// Should NOT call addWatermark for PDF // Should NOT call addWatermark for PDF

32
src/routes/__tests__/playground.test.ts
View file

@ -61,7 +61,7 @@ describe('Playground Route', () => {
const res = createMockResponse() const res = createMockResponse()
// Get the handler from the router // Get the handler from the router
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[0].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(400) expect(res.status).toHaveBeenCalledWith(400)
@ -72,7 +72,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: 123 }) const req = createMockRequest({ url: 123 })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[0].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(400) expect(res.status).toHaveBeenCalledWith(400)
@ -83,7 +83,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "" }) const req = createMockRequest({ url: "" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[0].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(400) expect(res.status).toHaveBeenCalledWith(400)
@ -103,7 +103,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "https://example.com" }) const req = createMockRequest({ url: "https://example.com" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith({ expect(mockTakeScreenshot).toHaveBeenCalledWith({
@ -135,7 +135,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -153,7 +153,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -172,7 +172,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -191,7 +191,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -209,7 +209,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -223,7 +223,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "https://example.com" }) const req = createMockRequest({ url: "https://example.com" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(503) expect(res.status).toHaveBeenCalledWith(503)
@ -236,7 +236,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "https://example.com" }) const req = createMockRequest({ url: "https://example.com" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(504) expect(res.status).toHaveBeenCalledWith(504)
@ -249,7 +249,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "http://192.168.1.1" }) const req = createMockRequest({ url: "http://192.168.1.1" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(400) expect(res.status).toHaveBeenCalledWith(400)
@ -262,7 +262,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "https://nonexistent.invalid" }) const req = createMockRequest({ url: "https://nonexistent.invalid" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(400) expect(res.status).toHaveBeenCalledWith(400)
@ -275,7 +275,7 @@ describe('Playground Route', () => {
const req = createMockRequest({ url: "https://example.com" }) const req = createMockRequest({ url: "https://example.com" })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(res.status).toHaveBeenCalledWith(500) expect(res.status).toHaveBeenCalledWith(500)
@ -292,7 +292,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({
@ -310,7 +310,7 @@ describe('Playground Route', () => {
}) })
const res = createMockResponse() const res = createMockResponse()
const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[1].handle const handler = playgroundRouter.stack.find(layer => layer.route?.methods.post)?.route.stack[2].handle
await handler(req, res, vi.fn()) await handler(req, res, vi.fn())
expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({ expect(mockTakeScreenshot).toHaveBeenCalledWith(expect.objectContaining({