test: debounce

Author: gabrieljablonski

src/test/utils.spec.js

@@ -62,22 +62,48 @@ describe('compute positions', () => {
 describe('debounce', () => {
   jest.useFakeTimers()
 
-  let func
-  let debouncedFunc
-
-  beforeEach((timeout = 1000) => {
-    func = jest.fn()
-    debouncedFunc = debounce(func, timeout)
-  })
+  const func = jest.fn()
 
   test('execute just once', () => {
+    const debouncedFunc = debounce(func, 1000)
     for (let i = 0; i < 100; i += 1) {
       debouncedFunc()
     }
 
-    // Fast-forward time
+    expect(func).not.toHaveBeenCalled()
+
     jest.runAllTimers()
 
     expect(func).toBeCalledTimes(1)
   })
+
+  test('execute immediately just once', () => {
+    const debouncedFunc = debounce(func, 1000, true)
+
+    debouncedFunc()
+    expect(func).toBeCalledTimes(1)
+
+    for (let i = 0; i < 100; i += 1) {
+      debouncedFunc()
+    }
+
+    jest.runAllTimers()
+
+    expect(func).toHaveBeenCalledTimes(1)
+  })
+
+  test('does not execute after cancel', () => {
+    const debouncedFunc = debounce(func, 1000)
+
+    debouncedFunc()
+
+    expect(func).not.toHaveBeenCalled()
+
+    debouncedFunc.cancel()
+
+    jest.runAllTimers()
+
+    expect(func).not.toHaveBeenCalled()
+  })
+})
 })

src/utils/debounce.ts

@@ -22,7 +22,7 @@ const debounce = <T, A extends any[]>(
 
     if (immediate && !timeout) {
       /**
-       * there's not need to clear the timeout
+       * there's no need to clear the timeout
        * since we expect it to resolve and set `timeout = null`
        */
       func.apply(this, args)
@@ -38,9 +38,11 @@ const debounce = <T, A extends any[]>(
   }
 
   debounced.cancel = () => {
+    /* c8 ignore start */
     if (!timeout) {
       return
     }
+    /* c8 ignore end */
     clearTimeout(timeout)
     timeout = null
   }