add tests for #1633

test/bun.js/setImmediate.test.js

@@ -0,0 +1,47 @@
+import { it, expect } from "bun:test";
+
+it("setImmediate", async () => {
+  var lastID = -1;
+  const result = await new Promise((resolve, reject) => {
+    var numbers = [];
+
+    for (let i = 0; i < 10; i++) {
+      const id = setImmediate((...args) => {
+        numbers.push(i);
+        if (i === 9) {
+          resolve(numbers);
+        }
+        try {
+          expect(args.length).toBe(1);
+          expect(args[0]).toBe(i);
+        } catch (err) {
+          reject(err);
+        }
+      }, i);
+      expect(id > lastID).toBe(true);
+      lastID = id;
+    }
+  });
+
+  for (let j = 0; j < result.length; j++) {
+    expect(result[j]).toBe(j);
+  }
+  expect(result.length).toBe(10);
+});
+
+it("clearImmediate", async () => {
+  var called = false;
+  const id = setImmediate(() => {
+    called = true;
+    expect(false).toBe(true);
+  });
+  clearImmediate(id);
+
+  // assert it doesn't crash if you call clearImmediate twice
+  clearImmediate(id);
+
+  await new Promise((resolve, reject) => {
+    setImmediate(resolve);
+  });
+  expect(called).toBe(false);
+});

test/bun.js/setInterval.test.js

@@ -6,13 +6,20 @@ it("setInterval", async () => {
   const result = await new Promise((resolve, reject) => {
     start = performance.now();
 
-    var id = setInterval(() => {
+    var id = setInterval((...args) => {
       counter++;
       if (counter === 10) {
         resolve(counter);
         clearInterval(id);
       }
-    }, 1);
+      try {
+        expect(args.length).toBe(1);
+        expect(args[0]).toBe("foo");
+      } catch (err) {
+        reject(err);
+        clearInterval(id);
+      }
+    }, 1, "foo");
   });
 
   expect(result).toBe(10);

test/bun.js/setTimeout.test.js

@@ -5,22 +5,28 @@ it("setTimeout", async () => {
   const result = await new Promise((resolve, reject) => {
     var numbers = [];
 
-    for (let i = 1; i < 100; i++) {
-      const id = setTimeout(() => {
+    for (let i = 0; i < 10; i++) {
+      const id = setTimeout((...args) => {
         numbers.push(i);
-        if (i === 99) {
+        if (i === 9) {
           resolve(numbers);
         }
-      }, i);
+        try {
+          expect(args.length).toBe(1);
+          expect(args[0]).toBe("foo");
+        } catch (err) {
+          reject(err);
+        }
+      }, i, "foo");
       expect(id > lastID).toBe(true);
       lastID = id;
     }
   });
 
   for (let j = 0; j < result.length; j++) {
-    expect(result[j]).toBe(j + 1);
+    expect(result[j]).toBe(j);
   }
-  expect(result.length).toBe(99);
+  expect(result.length).toBe(10);
 });
 
 it("clearTimeout", async () => {
@@ -35,9 +41,7 @@ it("clearTimeout", async () => {
   clearTimeout(id);
 
   await new Promise((resolve, reject) => {
-    setTimeout(() => {
-      resolve();
-    }, 10);
+    setTimeout(resolve, 10);
   });
   expect(called).toBe(false);
 });