Skip to content

Commit 13c32bf

Browse files
committed
fix: acqusition test
1 parent 0c7363b commit 13c32bf

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

apps/server/src/routes/acquisition.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ router.openapi(routes.updateCheck, async (c) => {
229229
isAvailable: false,
230230
isMandatory: false,
231231
appVersion: receivedAppVersion,
232+
shouldRunBinaryVersion: true,
232233
},
233234
} satisfies UpdateCheckResponse);
234235
}
@@ -252,6 +253,7 @@ router.openapi(routes.updateCheck, async (c) => {
252253
isAvailable: false,
253254
isMandatory: false,
254255
appVersion: receivedAppVersion,
256+
shouldRunBinaryVersion: true,
255257
},
256258
};
257259

apps/server/test/routes/acquisition.test.ts

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -303,13 +303,17 @@ describe("Acquisition Routes", () => {
303303
});
304304

305305
// App version specific tests
306-
it.only("returns 200 and no update for greater app version", async () => {
306+
it("returns 200 and no update for greater app version", async () => {
307+
const packageHash = packages2
308+
.filter((p) => p.appVersion === "2.0.0")
309+
.sort((a, b) => b.label.localeCompare(a.label))[0].packageHash;
307310
const response = await SELF.fetch(
308-
`https://example.com/acquisition/updateCheck?deploymentKey=${deployment2.key}&appVersion=2.0.0&packageHash=${packages2[0].packageHash}`,
311+
`https://example.com/acquisition/updateCheck?deploymentKey=${deployment2.key}&appVersion=2.0.0&packageHash=${packageHash}`,
309312
);
310313
expect(response.status).toBe(200);
311314

312-
const data = UpdateCheckResponseSchema.parse(await response.json());
315+
const json = await response.json();
316+
const data = UpdateCheckResponseSchema.parse(json);
313317
expect(data.updateInfo.shouldRunBinaryVersion).toBe(true);
314318
});
315319

@@ -319,7 +323,8 @@ describe("Acquisition Routes", () => {
319323
);
320324
expect(response.status).toBe(200);
321325

322-
const data = UpdateCheckResponseSchema.parse(await response.json());
326+
const json = await response.json();
327+
const data = UpdateCheckResponseSchema.parse(json);
323328
expect(data.updateInfo.isAvailable).toBe(true);
324329
expect(data.updateInfo.label).toBe("v4");
325330
});

0 commit comments

Comments
 (0)