Skip to content

Commit 7c52be8

Browse files
committed
Update tests
1 parent bcba81b commit 7c52be8

2 files changed

Lines changed: 11 additions & 11 deletions

File tree

src/integration-tests/ServiceRegistryClient.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { describeForOrkesV5 } from "./utils/customJestDescribe";
1010
// when testing against a remote cluster (e.g. a public Swagger URL it can reach).
1111
const TEST_SERVICE_URI =
1212
process.env.CONDUCTOR_TEST_SERVICE_URI ??
13-
"http://httpbin-server:8081/api-docs";
13+
"https://orkes-api-tester.orkesconductor.com/api";
1414

1515
describeForOrkesV5("ServiceRegistryClient", () => {
1616
const clientPromise = orkesConductorClient();

src/integration-tests/WorkerRegistration.test.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ describe("SDK Worker Registration", () => {
9393
expect(handler.running).toBe(false);
9494

9595
// Start workers BEFORE registering workflow
96-
handler.startWorkers();
96+
await handler.startWorkers();
9797

9898
// Wait a bit for workers to start polling
9999
await new Promise(resolve => setTimeout(resolve, 100));
@@ -189,7 +189,7 @@ describe("SDK Worker Registration", () => {
189189
});
190190

191191
// Start workers BEFORE registering workflow
192-
handler.startWorkers();
192+
await handler.startWorkers();
193193

194194
// Wait a bit for workers to start polling
195195
await new Promise(resolve => setTimeout(resolve, 100));
@@ -264,7 +264,7 @@ describe("SDK Worker Registration", () => {
264264
expect(registeredWorkers[0]?.domain).toBe(domain);
265265

266266
// Start workers and verify they start properly
267-
handler.startWorkers();
267+
await handler.startWorkers();
268268
expect(handler.running).toBe(true);
269269

270270
// Wait a bit for workers to initialize
@@ -300,7 +300,7 @@ describe("SDK Worker Registration", () => {
300300
});
301301

302302
// Start workers BEFORE registering workflow (important!)
303-
handler.startWorkers();
303+
await handler.startWorkers();
304304

305305
// Wait a bit for workers to start polling
306306
await new Promise(resolve => setTimeout(resolve, 100));
@@ -392,7 +392,7 @@ describe("SDK Worker Registration", () => {
392392
});
393393

394394
// Start workers BEFORE registering workflow
395-
handler.startWorkers();
395+
await handler.startWorkers();
396396

397397
// Wait a bit for workers to start polling
398398
await new Promise(resolve => setTimeout(resolve, 100));
@@ -473,7 +473,7 @@ describe("SDK Worker Registration", () => {
473473
expect(handler.workerCount).toBe(2);
474474

475475
// Start workers BEFORE registering workflow
476-
handler.startWorkers();
476+
await handler.startWorkers();
477477

478478
// Wait a bit for workers to start polling
479479
await new Promise(resolve => setTimeout(resolve, 100));
@@ -547,12 +547,12 @@ describe("SDK Worker Registration", () => {
547547
expect(handler.runningWorkerCount).toBe(0);
548548

549549
// Start workers
550-
handler.startWorkers();
550+
await handler.startWorkers();
551551
expect(handler.running).toBe(true);
552552
expect(handler.runningWorkerCount).toBe(1);
553553

554554
// Starting again should be idempotent
555-
handler.startWorkers();
555+
await handler.startWorkers();
556556
expect(handler.running).toBe(true);
557557
expect(handler.runningWorkerCount).toBe(1);
558558

@@ -567,7 +567,7 @@ describe("SDK Worker Registration", () => {
567567
expect(handler.runningWorkerCount).toBe(0);
568568

569569
// Can restart after stopping
570-
handler.startWorkers();
570+
await handler.startWorkers();
571571
expect(handler.running).toBe(true);
572572
expect(handler.runningWorkerCount).toBe(1);
573573

@@ -607,7 +607,7 @@ describe("SDK Worker Registration", () => {
607607

608608
expect(handler.workerCount).toBe(2);
609609

610-
handler.startWorkers();
610+
await handler.startWorkers();
611611
expect(handler.runningWorkerCount).toBe(2);
612612

613613
await handler.stopWorkers();

0 commit comments

Comments
 (0)