Merge pull request #1573 from okta/kg/OKTA-481893

fix: upgraded Jest to v29
okta · Sep 22, 2022 · 8f8f6b2 · 8f8f6b2
2 parents 3bb3a1f + 356afca
commit 8f8f6b2
Show file tree

Hide file tree

Showing 35 changed files with 3,284 additions and 2,912 deletions.
diff --git a/.nvmrc b/.nvmrc
@@ -1 +1 @@
-v16.15.0
+v16.17.0
diff --git a/.yarn/releases/yarn-3.2.0.cjs b/.yarn/releases/yarn-3.2.0.cjs
diff --git a/.yarn/releases/yarn-3.2.3.cjs b/.yarn/releases/yarn-3.2.3.cjs
diff --git a/.yarnrc.yml b/.yarnrc.yml
@@ -1,7 +1,11 @@
-yarnPath: .yarn/releases/yarn-3.2.0.cjs
-npmRegistryServer: "https://registry.yarnpkg.com"
 enableStrictSsl: false
-nodeLinker: "node-modules"
+
+nodeLinker: node-modules
+
+npmRegistryServer: "https://registry.yarnpkg.com"
+
 npmScopes:
   okta:
-    npmRegistryServer: ${OKTA_NPM_REGISTRY_SERVER-https://registry.yarnpkg.com}
+    npmRegistryServer: "${OKTA_NPM_REGISTRY_SERVER-https://registry.yarnpkg.com}"
+
+yarnPath: .yarn/releases/yarn-3.2.3.cjs
diff --git a/package.json b/package.json
@@ -7,7 +7,7 @@
     "packages/*"
   ],
   "resolutions": {
-    "@babel/plugin-transform-typescript": "^7.15.0"
+    "@babel/plugin-transform-typescript": "^7.19.1"
   },
   "engines": {
     "yarn": "^3.2.0",
@@ -45,5 +45,5 @@
     "stylelint": "^13.13.1"
   },
   "version": "0.5.0",
-  "packageManager": "yarn@3.2.0"
+  "packageManager": "yarn@3.2.3"
 }
diff --git a/packages/babel-plugin-fully-specified/jest.config.cjs b/packages/babel-plugin-fully-specified/jest.config.cjs
@@ -0,0 +1,15 @@
+/*!
+ * Copyright (c) 2021-present, Okta, Inc. and/or its affiliates. All rights reserved.
+ * The Okta software accompanied by this notice is provided pursuant to the Apache License, Version 2.0 (the "License.")
+ *
+ * You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0.
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *
+ * See the License for the specific language governing permissions and limitations under the License.
+ */
+
+module.exports = {
+  testEnvironment: "jsdom",
+};
diff --git a/packages/babel-plugin-fully-specified/package.json b/packages/babel-plugin-fully-specified/package.json
@@ -19,14 +19,14 @@
     "prepack": "yarn exec prepack"
   },
   "dependencies": {
-    "@babel/types": "^7.14.5"
+    "@babel/types": "^7.19.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.15.0",
+    "@babel/core": "^7.19.1",
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
-    "babel-jest": "^26.6.3",
-    "jest": "^26.6.3",
-    "typescript": "^4.4.4"
+    "jest": "^29.0.3",
+    "jest-environment-jsdom": "^29.0.3",
+    "typescript": "^4.8.3"
   }
 }
diff --git a/packages/babel-plugin-fully-specified/src/__tests__/__snapshots__/plugin.test.js.snap b/packages/babel-plugin-fully-specified/src/__tests__/__snapshots__/plugin.test.js.snap
@@ -1,45 +1,45 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`plugin should fully qualify the module exports specifiers 1`] = `
-"export { level } from \\"./index.js\\";
-export { levelIndex } from \\"./index.js\\";
-export { oneBackLevel } from \\"../index.js\\";
-export { oneBackLevelIndex } from \\"../index.js\\";
-export { twoBackLevel } from \\"../../index.js\\";
-export { twoBackLevelIndex } from \\"../../index.js\\";
-export { somethingBack } from \\"../lib/something.js\\";
-export { somethingBackTest } from \\"../lib/something.test.js\\";
-export { something } from \\"./lib/something.js\\";
-export { somethingTest } from \\"./lib/something.test.js\\";
-export { something as another } from \\"./lib/something.js\\";
-export { somethingTest as anotherTest } from \\"./lib/something.test.js\\";
-export * as anotherModule from \\"./lib/something.js\\";
-export * as anotherModuleTest from \\"./lib/something.test.js\\";
-export * from \\"./lib/something.js\\";
-export * from \\"./lib/something.test.js\\";"
+"export { level } from "./index.js";
+export { levelIndex } from "./index.js";
+export { oneBackLevel } from "../index.js";
+export { oneBackLevelIndex } from "../index.js";
+export { twoBackLevel } from "../../index.js";
+export { twoBackLevelIndex } from "../../index.js";
+export { somethingBack } from "../lib/something.js";
+export { somethingBackTest } from "../lib/something.test.js";
+export { something } from "./lib/something.js";
+export { somethingTest } from "./lib/something.test.js";
+export { something as another } from "./lib/something.js";
+export { somethingTest as anotherTest } from "./lib/something.test.js";
+export * as anotherModule from "./lib/something.js";
+export * as anotherModuleTest from "./lib/something.test.js";
+export * from "./lib/something.js";
+export * from "./lib/something.test.js";"
 `;
 
 exports[`plugin should fully qualify the module imports specifiers 1`] = `
-"import { level } from \\"./index.js\\";
-import { levelIndex } from \\"./index.js\\";
-import { oneBackLevel } from \\"../index.js\\";
-import { oneBackLevelIndex } from \\"../index.js\\";
-import { twoBackLevel } from \\"../../index.js\\";
-import { twoBackLevelIndex } from \\"../../index.js\\";
-import { somethingBack } from \\"../lib/something.js\\";
-import { somethingBackTest } from \\"../lib/something.test.js\\";
-import { export1, export2 as alias2 } from \\"./lib/something.js\\";
-import { export1Test, export2 as alias2Test } from \\"./lib/something.test.js\\";
-import { something } from \\"./lib/something.js\\";
-import { somethingTest } from \\"./lib/something.test.js\\";
-import { something as other } from \\"./lib/something.js\\";
-import { something as otherTest } from \\"./lib/something.test.js\\";
-import anotherImport from \\"./lib/something.js\\";
-import anotherImportTest from \\"./lib/something.test.js\\";
-import another, { otherImport } from \\"./lib/something.js\\";
-import anotherTest, { otherImportTest } from \\"./lib/something.test.js\\";
-import * as Something from \\"./lib/something.js\\";
-import * as SomethingTest from \\"./lib/something.test.js\\";"
+"import { level } from "./index.js";
+import { levelIndex } from "./index.js";
+import { oneBackLevel } from "../index.js";
+import { oneBackLevelIndex } from "../index.js";
+import { twoBackLevel } from "../../index.js";
+import { twoBackLevelIndex } from "../../index.js";
+import { somethingBack } from "../lib/something.js";
+import { somethingBackTest } from "../lib/something.test.js";
+import { export1, export2 as alias2 } from "./lib/something.js";
+import { export1Test, export2 as alias2Test } from "./lib/something.test.js";
+import { something } from "./lib/something.js";
+import { somethingTest } from "./lib/something.test.js";
+import { something as other } from "./lib/something.js";
+import { something as otherTest } from "./lib/something.test.js";
+import anotherImport from "./lib/something.js";
+import anotherImportTest from "./lib/something.test.js";
+import another, { otherImport } from "./lib/something.js";
+import anotherTest, { otherImportTest } from "./lib/something.test.js";
+import * as Something from "./lib/something.js";
+import * as SomethingTest from "./lib/something.test.js";"
 `;
 
 exports[`plugin should skip bare module specifiers 1`] = `

diff --git a/packages/browserslist-config-odyssey/package.json b/packages/browserslist-config-odyssey/package.json
@@ -19,6 +19,6 @@
   "devDependencies": {
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   }
 }
diff --git a/packages/eslint-plugin-odyssey/package.json b/packages/eslint-plugin-odyssey/package.json
@@ -14,6 +14,6 @@
   },
   "devDependencies": {
     "eslint": "^7.27.0",
-    "jest": "^26.6.3"
+    "jest": "^29.0.3"
   }
 }
diff --git a/packages/odyssey-babel-plugin/package.json b/packages/odyssey-babel-plugin/package.json
@@ -18,19 +18,18 @@
     "prepack": "yarn exec prepack"
   },
   "dependencies": {
-    "@babel/template": "^7.12.13",
-    "@babel/types": "^7.14.5",
+    "@babel/template": "^7.18.10",
+    "@babel/types": "^7.19.0",
     "postcss-load-config": "^3.1.0"
   },
   "devDependencies": {
-    "@babel/core": "^7.15.0",
+    "@babel/core": "^7.19.1",
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-postcss-preset": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
-    "babel-jest": "^26.6.3",
-    "jest": "^26.6.3",
+    "jest": "^29.0.3",
     "postcss": "^8.3.6",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   },
   "peerDependencies": {
     "@okta/odyssey-postcss-preset": ">= 0.8.2",

diff --git a/packages/odyssey-babel-plugin/src/__tests__/__snapshots__/importVisitor.test.js.snap b/packages/odyssey-babel-plugin/src/__tests__/__snapshots__/importVisitor.test.js.snap
@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`transformStyles import visitor transforms styles as expected 1`] = `
-Object {
+{
   "__digest": "0776a6",
   "__template": [Function],
   "autoprefixed": "ods-58h7mt",

diff --git a/packages/odyssey-babel-plugin/src/__tests__/__snapshots__/requireVisitor.test.js.snap b/packages/odyssey-babel-plugin/src/__tests__/__snapshots__/requireVisitor.test.js.snap
@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`transformStyles require visitor transforms styles as expected 1`] = `
-Object {
+{
   "__digest": "0776a6",
   "__template": [Function],
   "autoprefixed": "ods-58h7mt",

diff --git a/packages/odyssey-babel-plugin/src/compileFactory.ts b/packages/odyssey-babel-plugin/src/compileFactory.ts
@@ -47,10 +47,10 @@ export default function compileFactory(): Compile {
       delete result.message.warning;
     }
 
+    worker.unref();
+
     return result.message;
   };
 
-  worker.unref();
-
   return compile;
 }
diff --git a/packages/odyssey-babel-preset/package.json b/packages/odyssey-babel-preset/package.json
@@ -17,16 +17,16 @@
     "prepack": "yarn exec prepack"
   },
   "dependencies": {
-    "@babel/core": "^7.15.0",
-    "@babel/preset-env": "^7.15.0",
-    "@babel/preset-react": "^7.14.5",
-    "@babel/preset-typescript": "^7.15.0",
+    "@babel/core": "^7.19.1",
+    "@babel/preset-env": "^7.19.1",
+    "@babel/preset-react": "^7.18.6",
+    "@babel/preset-typescript": "^7.18.6",
     "@okta/odyssey-babel-plugin": "^0.15.0"
   },
   "devDependencies": {
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
     "@types/babel__core": "^7.1.15",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   }
 }
diff --git a/packages/odyssey-design-tokens/package.json b/packages/odyssey-design-tokens/package.json
@@ -9,7 +9,6 @@
   "module": "dist/index.js",
   "types": "dist/index.d.ts",
   "type": "module",
-  "sideEffects": false,
   "exports": {
     ".": {
       "default": "./dist/index.js"

diff --git a/packages/odyssey-ie-11-sandbox/package.json b/packages/odyssey-ie-11-sandbox/package.json
@@ -25,10 +25,10 @@
     "react-dom": "^17.0.2"
   },
   "devDependencies": {
-    "@babel/core": "7.15.0",
-    "@babel/preset-env": "7.15.0",
-    "@babel/preset-react": "7.14.5",
-    "@babel/preset-typescript": "7.15.0",
+    "@babel/core": "7.19.1",
+    "@babel/preset-env": "7.19.1",
+    "@babel/preset-react": "7.18.6",
+    "@babel/preset-typescript": "7.18.6",
     "@okta/browserslist-config-odyssey": "^0.15.0",
     "@okta/odyssey-babel-preset": "^0.15.0",
     "babel-loader": "8.2.2",

diff --git a/packages/odyssey-postcss-preset/package.json b/packages/odyssey-postcss-preset/package.json
@@ -29,7 +29,7 @@
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
     "@types/cssnano": "^4.0.1",
-    "jest": "^26.6.3",
+    "jest": "^29.0.3",
     "typescript": "^4.4.4"
   },
   "peerDependencies": {

diff --git a/packages/odyssey-postcss-scss/package.json b/packages/odyssey-postcss-scss/package.json
@@ -24,9 +24,9 @@
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
     "@types/sass": "^1.16.1",
-    "jest": "^26.6.3",
+    "jest": "^29.0.3",
     "postcss": "^8.3.6",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   },
   "peerDependencies": {
     "postcss": "^8.3.6"

diff --git a/packages/odyssey-postcss-theme/package.json b/packages/odyssey-postcss-theme/package.json
@@ -20,8 +20,8 @@
   "devDependencies": {
     "@okta/odyssey-lifecycle": "^0.15.0",
     "@okta/odyssey-typescript": "^0.15.0",
-    "jest": "^26.6.3",
-    "typescript": "^4.4.4"
+    "jest": "^29.0.3",
+    "typescript": "^4.8.3"
   },
   "peerDependencies": {
     "postcss": "^8.3.6"

diff --git a/packages/odyssey-react-mui/jest.config.cjs b/packages/odyssey-react-mui/jest.config.cjs
@@ -12,4 +12,5 @@
 
 module.exports = {
   setupFilesAfterEnv: ["./jest.setup.js"],
+  testEnvironment: "jsdom",
 };
diff --git a/packages/odyssey-react-mui/package.json b/packages/odyssey-react-mui/package.json
@@ -22,8 +22,8 @@
     "@okta/odyssey-design-tokens": "^0.15.0"
   },
   "devDependencies": {
-    "@babel/cli": "^7.14.8",
-    "@babel/core": "^7.15.0",
+    "@babel/cli": "^7.18.10",
+    "@babel/core": "^7.19.1",
     "@emotion/react": "^11.10.4",
     "@emotion/styled": "^11.10.4",
     "@mui/icons-material": "^5.10.3",
@@ -46,14 +46,15 @@
     "babel-jest": "^26.6.3",
     "babel-plugin-import": "^1.13.5",
     "eslint": "^7.27.0",
-    "jest": "^26.6.3",
+    "jest": "^29.0.3",
     "jest-axe": "^5.0.1",
+    "jest-environment-jsdom": "^29.0.3",
     "npm-run-all": "^4.1.5",
     "react": "^17.0.2",
     "react-dom": "^17.0.2",
     "regenerator-runtime": "^0.13.7",
     "stylelint": "^13.13.1",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   },
   "peerDependencies": {
     "@emotion/react": "^11",

diff --git a/packages/odyssey-react-theme/jest.config.cjs b/packages/odyssey-react-theme/jest.config.cjs
@@ -12,6 +12,7 @@
 
 module.exports = {
   setupFilesAfterEnv: ["./jest.setup.js"],
+  testEnvironment: "jsdom",
   transformIgnorePatterns: [
     "<rootDir>/node_modules/(?!@okta/odyssey-design-tokens/)",
   ],

diff --git a/packages/odyssey-react-theme/package.json b/packages/odyssey-react-theme/package.json
@@ -9,7 +9,6 @@
   "module": "dist/index.js",
   "types": "dist/index.d.ts",
   "type": "module",
-  "sideEffects": false,
   "exports": {
     ".": {
       "default": "./dist/index.js"
@@ -25,8 +24,8 @@
     "hoist-non-react-statics": "^3.3.2"
   },
   "devDependencies": {
-    "@babel/cli": "^7.14.8",
-    "@babel/core": "^7.15.0",
+    "@babel/cli": "^7.18.10",
+    "@babel/core": "^7.19.1",
     "@okta/browserslist-config-odyssey": "^0.15.0",
     "@okta/odyssey-babel-preset": "^0.15.0",
     "@okta/odyssey-lifecycle": "^0.15.0",
@@ -38,11 +37,12 @@
     "@types/react-dom": "^17.0.5",
     "babel-jest": "^26.6.3",
     "eslint": "^7.27.0",
-    "jest": "^26.6.3",
+    "jest": "^29.0.3",
+    "jest-environment-jsdom": "^29.0.3",
     "react": "^17.0.2",
     "react-dom": "^17.0.2",
     "regenerator-runtime": "^0.13.7",
-    "typescript": "^4.4.4"
+    "typescript": "^4.8.3"
   },
   "peerDependencies": {
     "react": ">=16 <18",

diff --git a/packages/odyssey-react/jest.config.cjs b/packages/odyssey-react/jest.config.cjs
@@ -12,4 +12,5 @@
 
 module.exports = {
   setupFilesAfterEnv: ["./jest.setup.js"],
+  testEnvironment: "jsdom",
 };