Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 42d1a9b

Browse files
authored
fix client/web jest tests (#57887)
The jest-canvas-mock lib is no longer needed. It was erroring because it assumes that `global.jest` is defined, but we can just remove it.
1 parent c5f6e25 commit 42d1a9b

File tree

3 files changed

+1
-26
lines changed

3 files changed

+1
-26
lines changed

client/web/jest.config.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,7 @@ const exportedConfig = {
88
...config,
99
displayName: 'web',
1010
rootDir: __dirname,
11-
setupFiles: [
12-
...config.setupFiles,
13-
'jest-canvas-mock', // mocking canvas is required for Monaco editor to work in unit tests
14-
path.join(__dirname, 'dev/mocks/mockEventLogger.ts'),
15-
],
11+
setupFiles: [...config.setupFiles, path.join(__dirname, 'dev/mocks/mockEventLogger.ts')],
1612
}
1713

1814
module.exports = exportedConfig

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,6 @@
229229
"http-proxy-middleware": "^2.0.6",
230230
"identity-obj-proxy": "^3.0.0",
231231
"jest": "^29.7.0",
232-
"jest-canvas-mock": "^2.3.0",
233232
"jest-fetch-mock": "^3.0.3",
234233
"jest-junit": "^13.0.0",
235234
"jsdom": "^16.7.0",

pnpm-lock.yaml

Lines changed: 0 additions & 20 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)