Skip to content

Commit 56c3437

Browse files
committed
fix(sdk): resolve ESLint warnings in build process
- Remove unused MAX_POLL_DURATION_MS import from checkpoint test - Add ESLint disable comments for intentionally skipped tests - Tests are skipped pending future SDK concurrency features
1 parent b401803 commit 56c3437

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

packages/aws-durable-execution-sdk-js-testing/src/test-runner/local/__tests__/integration/invoke.integration.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ describe("LocalDurableTestRunner Invoke operations integration", () => {
141141
});
142142

143143
// TODO: handling errors for callback and checkpoint updates
144+
// eslint-disable-next-line jest/no-disabled-tests
144145
it.skip("should fail execution if invoking a function that does not exist", async () => {
145146
const handler = withDurableExecution(async (_, ctx) => {
146147
await ctx.invoke("durableOperation", "nonExistentFunction", {

packages/aws-durable-execution-sdk-js-testing/src/test-runner/local/__tests__/integration/local-durable-test-runner.integration.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -311,6 +311,7 @@ describe("LocalDurableTestRunner Integration", () => {
311311
});
312312

313313
// enable when language SDK supports concurrent waits
314+
// eslint-disable-next-line jest/no-disabled-tests
314315
it.skip("should prevent scheduled function interference in parallel wait scenario", async () => {
315316
// This test creates a scenario where multiple wait operations could create
316317
// scheduled functions that fire while invocations are still active.

packages/aws-durable-execution-sdk-js-testing/src/test-runner/local/__tests__/integration/wait-for-callback-operations.integration.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ describe("WaitForCallback Operations Integration", () => {
2121
// Concurrent and Complex Workflow Tests Category
2222
describe("Concurrent and Complex Workflow Tests", () => {
2323
// todo: add test when language SDK adds concurrency support
24+
// eslint-disable-next-line jest/no-disabled-tests
2425
it.skip("should handle multiple concurrent waitForCallback operations with different submitters", async () => {
2526
let callback1Id: string | undefined;
2627
let callback2Id: string | undefined;

packages/aws-durable-execution-sdk-js/src/utils/checkpoint/checkpoint-central-termination.test.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,7 @@ import { OperationLifecycleState, OperationSubType } from "../../types";
55
import { OperationType } from "@aws-sdk/client-lambda";
66
import { EventEmitter } from "events";
77
import { hashId } from "../step-id-utils/step-id-utils";
8-
import {
9-
CHECKPOINT_TERMINATION_COOLDOWN_MS,
10-
MAX_POLL_DURATION_MS,
11-
} from "../constants/constants";
8+
import { CHECKPOINT_TERMINATION_COOLDOWN_MS } from "../constants/constants";
129

1310
jest.mock("../logger/logger");
1411

0 commit comments

Comments
 (0)