Skip to content

Commit

Permalink
fix(js): Remove focused test, fix tests (#1138)
Browse files Browse the repository at this point in the history
  • Loading branch information
dqbd authored Oct 28, 2024
2 parents 4945163 + 609ac8c commit ba14f75
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions js/src/tests/batch_client.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ describe.each(ENDPOINT_TYPES)(
await new Promise((resolve) => setTimeout(resolve, 300));
});

it.only("Create + update batching should merge into a single call", async () => {
it("Create + update batching should merge into a single call", async () => {
const client = new Client({
apiKey: "test-api-key",
autoBatchTracing: true,
Expand Down Expand Up @@ -298,7 +298,7 @@ describe.each(ENDPOINT_TYPES)(
end_time: endTime,
});

await new Promise((resolve) => setTimeout(resolve, 100));
await client.awaitPendingTraceBatches();

expect(serverInfoFailedOnce).toBe(true);

Expand Down Expand Up @@ -857,7 +857,7 @@ describe.each(ENDPOINT_TYPES)(
end_time: endTime,
});

await new Promise((resolve) => setTimeout(resolve, 100));
await client.awaitPendingTraceBatches();

const calledRequestParam: any = callSpy.mock.calls[0][2];
expect(await parseMockRequestBody(calledRequestParam?.body)).toEqual({
Expand Down

0 comments on commit ba14f75

Please sign in to comment.