Skip to content

Commit 973c2e2

Browse files
committed
fix test issues
1 parent 2ce6a3f commit 973c2e2

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

packages/shared-tests/src/tests/adapter.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,11 @@ export function runTests<TDatabase = unknown>(ctx: SharedTestContext<TDatabase>)
111111

112112
await adapter.updateJobStatus(job.id, "processing")
113113

114-
const [updated]: [{ progressed_at: number; status: string }?] =
114+
const [updated]: [{ processed_at: Date; status: string }?] =
115115
await internalDbClient`SELECT processed_at, status FROM queue_jobs WHERE id=${job.id}`
116116

117117
expect(updated?.status).toBe("processing")
118-
expect(updated?.progressed_at).toBeTruthy()
118+
expect(updated?.processed_at).toBeTruthy()
119119
})
120120

121121
it("should get queue stats", async () => {
@@ -232,7 +232,7 @@ export function runTests<TDatabase = unknown>(ctx: SharedTestContext<TDatabase>)
232232
const result = { success: true, output: "processed" }
233233
await adapter.updateJobStatus(job.id, "completed", undefined, result)
234234

235-
const [updated]: [{ completed_at: number; status: string; result: unknown }?] =
235+
const [updated]: [{ completed_at: Date; status: string; result: unknown }?] =
236236
await internalDbClient`SELECT completed_at, status, result FROM queue_jobs WHERE id=${job.id}`
237237

238238
expect(updated?.status).toBe("completed")
@@ -301,7 +301,9 @@ export function runTests<TDatabase = unknown>(ctx: SharedTestContext<TDatabase>)
301301
await adapter.updateJobStatus(job.id, "completed")
302302

303303
const [updated]: [{ result: unknown; status: string }?] =
304-
await internalDbClient`SELECT result FROM queue_jobs WHERE id=${job.id}`
304+
await internalDbClient`SELECT result, status FROM queue_jobs WHERE id=${job.id}`
305+
306+
console.dir({ updated })
305307

306308
expect(updated?.result).toEqual(result)
307309
expect(updated?.status).toBe("completed")

packages/shared-tests/src/tests/timezone.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,21 +72,21 @@ export function runTests<TDatabase = unknown>(ctx: SharedTestContext<TDatabase>)
7272
})
7373

7474
// Retrieve the job and check the timestamp
75-
const [retrieved]: [{ process_at: number }?] =
75+
const [retrieved]: [{ process_at: Date }?] =
7676
await internalDbClient`SELECT process_at FROM queue_jobs WHERE id=${job.id}`
7777

7878
// The processAt should match our input UTC time
79-
expect(retrieved?.process_at).toBe(testDate.getTime())
79+
expect(retrieved?.process_at.getTime()).toBe(testDate.getTime())
8080

8181
// Update job status and check timestamp consistency
8282
await adapter.updateJobStatus(job.id, "processing")
8383

84-
const [updated]: [{ processed_at: number }?] =
84+
const [updated]: [{ processed_at: Date }?] =
8585
await internalDbClient`SELECT processed_at FROM queue_jobs WHERE id=${job.id}`
8686

8787
// processedAt should be a valid UTC timestamp
8888
expect(updated?.processed_at).toBeTruthy()
89-
expect(updated?.processed_at).toBeGreaterThan(testDate.getTime())
89+
expect(updated?.processed_at.getTime()).toBeGreaterThan(testDate.getTime())
9090
})
9191

9292
it("should handle dates with explicit timezone offsets", async () => {

0 commit comments

Comments
 (0)