fix: binding the methods to the context (#1466)

* fix: binding the methods to the context * fix(1465):changeset * fix: replace bind with arrow function * fix: types * loosen forwarding types --------- Co-authored-by: George Fu <[email protected]>
smithy-lang · Dec 6, 2024 · a257792 · a257792
1 parent b52b4e8
commit a257792
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.changeset/large-squids-rush.md b/.changeset/large-squids-rush.md
@@ -0,0 +1,5 @@
+---
+"@smithy/node-http-handler": patch
+---
+
+Added context binding to the setTimeout and clearTimeout functions
diff --git a/packages/node-http-handler/src/timing.ts b/packages/node-http-handler/src/timing.ts
@@ -3,6 +3,7 @@
  * For test spies.
  */
 export const timing = {
-  setTimeout: setTimeout,
-  clearTimeout: clearTimeout,
+  setTimeout: (cb: (...ignored: any[]) => void | unknown, ms?: number) => setTimeout(cb, ms),
+  clearTimeout: (timeoutId: string | number | undefined | unknown) =>
+    clearTimeout(timeoutId as Parameters<typeof clearTimeout>[0]),
 };