test: reinstate test for optional dependency canvas

foxxyz · Jun 9, 2024 · 23519a3 · 23519a3
1 parent 2e3699e
commit 23519a3
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 24 deletions.
diff --git a/test/device-ct.test.js b/test/device-ct.test.js
@@ -166,10 +166,6 @@ describe('Drawing (Callback API)', () => {
         await delay(10)
         expect(sender).toHaveBeenCalledTimes(1)
     })
-    // TODO: mock canvas
-    // it('informs the user if the canvas library is not installed', () => {
-    //     expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
-    // })
 })
 describe('Drawing (Buffer API)', () => {
     beforeEach(() => {

diff --git a/test/device-live-s.test.js b/test/device-live-s.test.js
@@ -124,11 +124,6 @@ describe('Drawing (Callback API)', () => {
         await delay(10)
         expect(sender).toHaveBeenCalledTimes(1)
     })
-    // TODO: mock canvas lib
-    // it('informs the user if the canvas library is not installed', () => {
-    //     jest.mock('canvas', () => {})
-    //     expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
-    // })
 })
 describe('Drawing (Buffer API)', () => {
     beforeEach(() => {

diff --git a/test/device-live.test.js b/test/device-live.test.js
@@ -150,11 +150,6 @@ describe('Drawing (Callback API)', () => {
         await delay(10)
         expect(sender).toHaveBeenCalledTimes(1)
     })
-    // TODO: Mock canvas lib
-    // it('informs the user if the canvas library is not installed', () => {
-    //     jest.mock('canvas', () => {})
-    //     expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
-    // })
 })
 describe('Drawing (Buffer API)', () => {
     beforeEach(() => {

diff --git a/test/device-razer-x.test.js b/test/device-razer-x.test.js
@@ -112,11 +112,6 @@ describe('Drawing (Callback API)', () => {
         await delay(10)
         expect(sender).toHaveBeenCalledTimes(1)
     })
-    // TODO: Mock canvas lib
-    // it('informs the user if the canvas library is not installed', () => {
-    //     jest.mock('canvas', () => {})
-    //     expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
-    // })
 })
 describe('Drawing (Buffer API)', () => {
     beforeEach(() => {

diff --git a/test/device-razer.test.js b/test/device-razer.test.js
@@ -144,11 +144,6 @@ describe('Drawing (Callback API)', () => {
         await delay(10)
         expect(sender).toHaveBeenCalledTimes(1)
     })
-    // TODO: Mock canvas lib
-    // it('informs the user if the canvas library is not installed', () => {
-    //     jest.mock('canvas', () => {})
-    //     expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
-    // })
 })
 describe('Drawing (Buffer API)', () => {
     beforeEach(() => {

diff --git a/test/optional-deps.test.js b/test/optional-deps.test.js
@@ -0,0 +1,14 @@
+import { jest } from '@jest/globals'
+jest.unstable_mockModule('canvas', () => ({}))
+const { LoupedeckLiveS } = await import('../index.js')
+
+let device
+describe('Optional Dependencies', () => {
+    beforeEach(() => {
+        device = new LoupedeckLiveS({ autoConnect: false })
+        device.connection = { send: () => {}, isReady: () => true }
+    })
+    it('informs the user if the canvas library is not installed', () => {
+        expect(() => device.drawKey(6, () => {})).toThrow(/using callbacks requires the `canvas` library/i)
+    })
+})