Refactor jest configuration

Author: carbonrobot

package-scripts.js

@@ -12,9 +12,6 @@
  * `package.json:scripts`.
  */
 
-const path = require("path");
-const PKG_SRC = path.resolve("src");
-
 // For publishing, use the core package's version.
 const coreVersion = require("./packages/victory-core/package.json").version;
 if (!coreVersion) {
@@ -60,17 +57,6 @@ module.exports = {
     "lint:pkg": 'nps "lint:base src"',
     "lint:pkg:fix": 'nps "lint:base --fix src"',
 
-    // Tests
-    // - Jest
-    // TODO(2375): Can we cache / incremental?
-    // https://github.com/FormidableLabs/victory/issues/2375
-    "jest:native": `cross-env BABEL_ENV=commonjs jest --config=../../test/jest-native-config.js --passWithNoTests --testPathPattern=${PKG_SRC}`,
-    "jest:pkg": `cross-env BABEL_ENV=commonjs jest --config=../../test/jest-config.js --passWithNoTests --testPathPattern=${PKG_SRC}`,
-    // TODO(2348): Hook coverage up to CI
-    // https://github.com/FormidableLabs/victory/issues/2348
-    // TODO(2348): Add this to `check:ci`
-    "jest:cov": "echo TODO",
-
     // - TypeScript
     // TODO(2375): Can we cache / incremental?
     // https://github.com/FormidableLabs/victory/issues/2375

package.json

@@ -55,7 +55,7 @@
     "@testing-library/react-hooks": "^8.0.0",
     "@testing-library/react-native": "^11.0.0",
     "@types/fs-extra": "^11.0.3",
-    "@types/jest": "^28.1.6",
+    "@types/jest": "^29.5.12",
     "@types/lodash": "^4.14.149",
     "@types/node": "^18.6.1",
     "@types/prop-types": "^15.7.5",
@@ -64,7 +64,7 @@
     "@types/testing-library__jest-dom": "^5.14.5",
     "@typescript-eslint/eslint-plugin": "^5.32.0",
     "@typescript-eslint/parser": "^5.32.0",
-    "babel-jest": "^28.1.3",
+    "babel-jest": "^29.7.0",
     "babel-loader": "^8.2.5",
     "babel-plugin-lodash": "^3.1.2",
     "babel-plugin-module-resolver": "^4.1.0",
@@ -87,8 +87,8 @@
     "fs-extra": "^10.0.0",
     "glob": "8.0.3",
     "immutable": "^3.8.2",
-    "jest": "^28.1.3",
-    "jest-environment-jsdom": "^28.1.3",
+    "jest": "^29.7.0",
+    "jest-environment-jsdom": "^29.7.0",
     "lodash": "^4.17.19",
     "lodash-webpack-plugin": "^0.11.6",
     "mdast-util-to-string": "^1.0.6",
@@ -107,7 +107,7 @@
     "seedrandom": "^3.0.5",
     "storybook": "^7.6.7",
     "styled-components": "^5.3.5",
-    "ts-jest": "^28.0.7",
+    "ts-jest": "^29.1.2",
     "ts-loader": "^9.3.0",
     "ts-node": "^10.9.1",
     "typescript": "^4.7.3",

packages/victory-area/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};

packages/victory-area/package.json

@@ -28,8 +28,7 @@
     "react": ">=16.6.0"
   },
   "devDependencies": {
-    "victory-chart": "*",
-    "victory-area": "*"
+    "victory-chart": "*"
   },
   "publishConfig": {
     "provenance": true
@@ -275,7 +274,7 @@
       ]
     },
     "jest": {
-      "command": "nps jest:pkg",
+      "command": "jest --passWithNoTests",
       "files": [
         "src/**/*.test.*",
         "../../.babelrc.js",
@@ -284,7 +283,6 @@
       ],
       "output": [],
       "dependencies": [
-        "build:lib:cjs",
         "../victory-chart:build:lib:cjs",
         "../victory-vendor:build:lib:cjs",
         "../victory-voronoi:build:lib:cjs"

packages/victory-area/src/area.test.tsx

@@ -1,9 +1,10 @@
 import React from "react";
-import { Area } from "victory-area";
 import { render } from "@testing-library/react";
 import { VictoryContainer } from "victory-core";
 import * as d3Scale from "victory-vendor/d3-scale";
 
+import { Area } from "./area";
+
 describe("victory-primitives/area", () => {
   const baseProps = {
     data: [

packages/victory-area/src/helper-methods.test.tsx

@@ -1,6 +1,7 @@
-import { getDataWithBaseline } from "victory-area/lib/helper-methods";
 import * as d3Scale from "victory-vendor/d3-scale";
 
+import { getDataWithBaseline } from "./helper-methods";
+
 describe("victory-area/helper-methods", () => {
   describe("getDataWithBaseline", () => {
     const data = [

packages/victory-area/src/victory-area.test.tsx

@@ -1,10 +1,12 @@
 import { fireEvent, render, screen } from "@testing-library/react";
 import React from "react";
-import { Area, VictoryArea, VictoryAreaProps } from "victory-area";
 import { VictoryChart } from "victory-chart";
 import { Helpers } from "victory-core";
 import { curveCatmullRom } from "victory-vendor/d3-shape";
+
 import { calculateD3Path } from "../../../test/helpers/svg";
+import { Area } from './area';
+import { VictoryArea, VictoryAreaProps } from "./victory-area";
 
 describe("components/victory-area", () => {
   describe("default component rendering", () => {

packages/victory-area/tsconfig.build.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["**/*.test.*", "es", "lib"]
+  "exclude": ["**/*.test.*", "es", "lib", "jest.config.ts"]
 }

packages/victory-area/tsconfig.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["es", "lib"]
+  "exclude": ["es", "lib", "jest.config.ts"]
 }

packages/victory-axis/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};

packages/victory-axis/package.json

@@ -27,7 +27,6 @@
     "react": ">=16.6.0"
   },
   "devDependencies": {
-    "victory-axis": "*",
     "victory-chart": "*"
   },
   "publishConfig": {
@@ -267,7 +266,7 @@
       ]
     },
     "jest": {
-      "command": "nps jest:pkg",
+      "command": "jest --passWithNoTests",
       "files": [
         "src/**/*.test.*",
         "../../.babelrc.js",
@@ -276,7 +275,6 @@
       ],
       "output": [],
       "dependencies": [
-        "build:lib:cjs",
         "../victory-chart:build:lib:cjs",
         "../victory-vendor:build:lib:cjs",
         "../victory-voronoi:build:lib:cjs"

packages/victory-axis/src/victory-axis.test.tsx

@@ -1,7 +1,8 @@
 import React from "react";
-import { VictoryAxis } from "victory-axis";
-import { VictoryChart } from "victory-chart";
 import { render, screen } from "@testing-library/react";
+import { VictoryChart } from "victory-chart";
+
+import { VictoryAxis } from "./victory-axis";
 
 describe("components/victory-axis", () => {
   it("should render two axes by default", () => {

packages/victory-axis/tsconfig.build.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["**/*.test.*", "es", "lib"]
+  "exclude": ["**/*.test.*", "es", "lib", "jest.config.ts"]
 }

packages/victory-axis/tsconfig.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["es", "lib"]
+  "exclude": ["es", "lib", "jest.config.ts"]
 }

packages/victory-bar/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};

packages/victory-bar/package.json

@@ -28,8 +28,7 @@
     "react": ">=16.6.0"
   },
   "devDependencies": {
-    "victory-chart": "*",
-    "victory-bar": "*"
+    "victory-chart": "*"
   },
   "publishConfig": {
     "provenance": true
@@ -275,7 +274,7 @@
       ]
     },
     "jest": {
-      "command": "nps jest:pkg",
+      "command": "jest --passWithNoTests",
       "files": [
         "src/**/*.test.*",
         "../../.babelrc.js",
@@ -284,7 +283,6 @@
       ],
       "output": [],
       "dependencies": [
-        "build:lib:cjs",
         "../victory-chart:build:lib:cjs",
         "../victory-vendor:build:lib:cjs",
         "../victory-voronoi:build:lib:cjs"

packages/victory-bar/src/bar.test.tsx

@@ -2,8 +2,9 @@ import React from "react";
 import { render } from "@testing-library/react";
 import * as d3Scale from "victory-vendor/d3-scale";
 import { VictoryContainer } from "victory-core";
-import { Bar } from "victory-bar";
+
 import { getBarShape } from "../../../test/helpers";
+import { Bar } from "./bar";
 
 describe("victory-primitives/bar", () => {
   const baseProps = {

packages/victory-bar/src/geometry-helper-methods.test.ts

@@ -1,4 +1,4 @@
-import { circle, point } from "victory-bar/lib/geometry-helper-methods";
+import { circle, point } from './geometry-helper-methods';
 
 describe("point", () => {
   describe("calculates distances", () => {

packages/victory-bar/src/victory-bar.test.tsx

@@ -1,10 +1,11 @@
 import * as React from "react";
 import { render, fireEvent, screen } from "@testing-library/react";
 import { VictoryChart } from "victory-chart";
-import { Bar, VictoryBar } from "victory-bar";
 import { Helpers } from "victory-core";
 
 import { isBar, getBarHeight } from "../../../test/helpers";
+import { Bar } from './bar';
+import { VictoryBar } from './victory-bar';
 
 describe("components/victory-bar", () => {
   describe("default component rendering", () => {

packages/victory-bar/tsconfig.build.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["**/*.test.*", "es", "lib"]
+  "exclude": ["**/*.test.*", "es", "lib", "jest.config.ts"]
 }

packages/victory-bar/tsconfig.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["es", "lib"]
+  "exclude": ["es", "lib", "jest.config.ts"]
 }

packages/victory-box-plot/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};

packages/victory-box-plot/package.json

@@ -29,7 +29,7 @@
   },
   "devDependencies": {
     "victory-box-plot": "*",
-    "victory-chart": "^36.9.1"
+    "victory-chart": "*"
   },
   "publishConfig": {
     "provenance": true
@@ -275,7 +275,7 @@
       ]
     },
     "jest": {
-      "command": "nps jest:pkg",
+      "command": "jest --passWithNoTests",
       "files": [
         "src/**/*.test.*",
         "../../.babelrc.js",
@@ -284,7 +284,6 @@
       ],
       "output": [],
       "dependencies": [
-        "build:lib:cjs",
         "../victory-chart:build:lib:cjs",
         "../victory-vendor:build:lib:cjs",
         "../victory-voronoi:build:lib:cjs"

packages/victory-box-plot/src/victory-box-plot.test.tsx

@@ -1,9 +1,10 @@
 import { render, screen } from "@testing-library/react";
 import React from "react";
-import { VictoryBoxPlot } from "victory-box-plot";
 import { VictoryChart } from "victory-chart";
 import { Border, LineSegment, Whisker } from "victory-core";
 
+import { VictoryBoxPlot } from "./victory-box-plot";
+
 const TEST_GROUP_ID = "test-group-id";
 const dataset = [
   { x: 1, min: 1, max: 18, median: 8, q1: 5, q3: 15 },

packages/victory-box-plot/tsconfig.build.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["**/*.test.*", "es", "lib"]
+  "exclude": ["**/*.test.*", "es", "lib", "jest.config.ts"]
 }

packages/victory-box-plot/tsconfig.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["es", "lib"]
+  "exclude": ["es", "lib", "jest.config.ts"]
 }

packages/victory-brush-container/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};

packages/victory-brush-container/package.json

@@ -27,9 +27,6 @@
   "peerDependencies": {
     "react": ">=16.6.0"
   },
-  "devDependencies": {
-    "victory-brush-container": "*"
-  },
   "publishConfig": {
     "provenance": true
   },
@@ -264,7 +261,7 @@
       ]
     },
     "jest": {
-      "command": "nps jest:pkg",
+      "command": "jest --passWithNoTests",
       "files": [
         "src/**/*.test.*",
         "../../.babelrc.js",
@@ -273,7 +270,6 @@
       ],
       "output": [],
       "dependencies": [
-        "build:lib:cjs",
         "../victory-vendor:build:lib:cjs",
         "../victory-voronoi:build:lib:cjs"
       ],

packages/victory-brush-container/src/brush-helpers.test.tsx

@@ -1,4 +1,4 @@
-import { BrushHelpers } from "victory-brush-container";
+import { BrushHelpers } from "./brush-helpers";
 
 describe("containers/brush-helpers", () => {
   // eslint-disable-next-line @typescript-eslint/unbound-method

packages/victory-brush-container/tsconfig.build.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["**/*.test.*", "es", "lib"]
+  "exclude": ["**/*.test.*", "es", "lib", "jest.config.ts"]
 }

packages/victory-brush-container/tsconfig.json

@@ -1,4 +1,4 @@
 {
   "extends": "../../tsconfig.base.json",
-  "exclude": ["es", "lib"]
+  "exclude": ["es", "lib", "jest.config.ts"]
 }

packages/victory-brush-line/jest.config.ts

@@ -0,0 +1,5 @@
+import rootConfig from '../../test/jest.config';
+
+export default {
+  ...rootConfig,
+};