Skip to content

Commit 305895f

Browse files
committed
Remove leftover testing console logs
- Remove 🧪 TESTING console logs from checkpoint handlers - Clean up debug output that was still printing during tests - Tests now run cleanly without verbose checkpoint logging
1 parent df3fd4c commit 305895f

File tree

3 files changed

+27
-8
lines changed

3 files changed

+27
-8
lines changed

packages/aws-durable-execution-sdk-js-examples/src/examples/parallel/wait/parallel-wait.history.json

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,17 @@
205205
"Result": {}
206206
}
207207
},
208+
{
209+
"EventType": "InvocationCompleted",
210+
"EventId": 19,
211+
"EventTimestamp": "2025-12-05T00:16:52.929Z",
212+
"InvocationCompletedDetails": {
213+
"StartTimestamp": "2025-12-05T00:16:52.873Z",
214+
"EndTimestamp": "2025-12-05T00:16:52.929Z",
215+
"Error": {},
216+
"RequestId": "5907d455-ef8a-4021-b822-bcffa5dd2d07"
217+
}
218+
},
208219
{
209220
"EventType": "WaitSucceeded",
210221
"SubType": "Wait",
@@ -242,6 +253,17 @@
242253
}
243254
}
244255
},
256+
{
257+
"EventType": "InvocationCompleted",
258+
"EventId": 23,
259+
"EventTimestamp": "2025-12-05T00:16:55.881Z",
260+
"InvocationCompletedDetails": {
261+
"StartTimestamp": "2025-12-05T00:16:55.874Z",
262+
"EndTimestamp": "2025-12-05T00:16:55.881Z",
263+
"Error": {},
264+
"RequestId": "6fa7d6b2-831c-4e48-a0c6-9cb81bdfe3c6"
265+
}
266+
},
245267
{
246268
"EventType": "ExecutionSucceeded",
247269
"EventId": 24,

packages/aws-durable-execution-sdk-js-examples/src/examples/parallel/wait/parallel-wait.test.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,11 @@ createTests({
2626
expect(wait2SecondsOp.getWaitDetails()!.waitSeconds!).toBe(2);
2727
expect(wait5SecondsOp.getWaitDetails()!.waitSeconds!).toBe(5);
2828

29-
// TODO: Still investigating why local produces 2 events vs CI 4 events
29+
// Not compatible with latest changes applied.
30+
// There is a good change that this issue is related to
31+
// testing library handling PENDING items in a different way than
32+
// backend. Backend only cound them after LAn SDK received the changes
33+
// in checkpoint response.
3034
// assertEventSignatures(execution);
3135
}, 10000);
3236
},

packages/aws-durable-execution-sdk-js-testing/src/checkpoint-server/handlers/checkpoint-handlers.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -107,13 +107,6 @@ export function processCheckpointDurableExecution(
107107

108108
validateCheckpointUpdates(updates, storage.operationDataMap);
109109

110-
console.log(`🧪 TESTING: Registering ${updates.length} checkpoint updates:`);
111-
updates.forEach((update, index) => {
112-
console.log(
113-
`🧪 TESTING: Update ${index}: Id=${update.Id}, Action=${update.Action}, Name=${update.Name}`,
114-
);
115-
});
116-
117110
storage.registerUpdates(updates);
118111

119112
const output: CheckpointDurableExecutionResponse = {

0 commit comments

Comments
 (0)