fix: resolve TypeScript and Jest test issues in transform tests

package.json

@@ -11,18 +11,23 @@
     "jscodeshift": "jscodeshift"
   },
   "dependencies": {
+    "@codemod.com/codemod-utils": "^1.0.0",
+    "@types/jscodeshift": "^0.12.0",
     "chalk": "^2.4.2",
     "eslint": "^6.6.0",
     "execa": "^3.2.0",
     "globby": "^10.0.1",
     "inquirer": "^7.0.0",
     "is-git-clean": "^1.1.0",
-    "jscodeshift": "^0.11.0",
+    "jscodeshift": "^17.3.0",
     "meow": "^5.0.0"
   },
   "jest": {
     "globals": {
-      "baseDir": "../"
+      "baseDir": "../",
+      "ts-jest": {
+        "tsconfig": "./tsconfig.codemod.json"
+      }
     },
     "testEnvironment": "node",
     "roots": [
@@ -32,13 +37,21 @@
     "transform": {
       "^.+\\.jsx?$": "babel-jest",
       "^.+\\.tsx?$": "ts-jest"
+    },
+    "transformIgnorePatterns": [
+      "/node_modules/(?!@codemod\\.com/codemod-utils)/.+\\.js$"
+    ],
+    "moduleNameMapper": {
+      "^@codemod\\.com/codemod-utils$": "<rootDir>/node_modules/@codemod.com/codemod-utils/dist/index.cjs"
     }
   },
   "devDependencies": {
     "@babel/core": "^7.6.4",
     "@babel/plugin-proposal-object-rest-spread": "^7.6.2",
     "@babel/preset-env": "^7.6.3",
+    "@jest/globals": "^29.7.0",
     "@types/jest": "^24.9.0",
+    "@types/node": "^22.12.0",
     "@typescript-eslint/parser": "^7.8.0",
     "babel-eslint": "^10.0.3",
     "babel-jest": "^24.9.0",

transforms/__testfixtures__/create-element-to-jsx-preserve-comments.output.js

@@ -2,17 +2,17 @@ var React = require('react');
 
 const render = () => {
   return (
-    /*1*//*4*//*2*//*3*/<div
+    /*1*//*2*//*3*/<div
       /*8*/className/*9*/=/*10*/"foo"/*11*/
       /*12*/
       /*13*///17
       onClick/*14*/={/*15*/ this.handleClick}/*16*/>
-      {/*19*///25
-      <TodoList.Item />/*24*//*20*//*23*//*21*//*22*//*20*//*23*//*21*//*22*//*21*//*22*/}
-      <span {.../*26*/getProps()/*27*/} />
+      {/*18*//*19*/<TodoList.Item />/*24*//*20*//*23*//*21*//*22*//*20*//*23*//*21*//*22*//*21*//*22*/}
+      {//25
+      <span {.../*26*/getProps()/*27*/} />}
       {<input /> /*28*//*29*/}
-    </div>
-    /*5*//*6*//*7*//*18*/
+    </div>/*4*/
+    /*5*//*6*//*7*/
   );
 };
 

transforms/__testfixtures__/remove-context-provider/typescript/no-provider.input.js → transforms/__testfixtures__/remove-context-provider/typescript/no-provider.input.tsx

@@ -6,4 +6,4 @@ function App({ url }: { url: string }) {
       <Page />
     </Context>
   );
-}
+}

transforms/__testfixtures__/remove-context-provider/typescript/with-provider-2.input.js → transforms/__testfixtures__/remove-context-provider/typescript/with-provider-2.input.tsx

@@ -3,7 +3,6 @@ function App({ url }: { url: string }) {
 
   return (
     <Context.Provider value={theme}>
-      <Page />
     </Context.Provider>
   );
-}
+}

transforms/__testfixtures__/remove-context-provider/typescript/with-provider-2.output.js → transforms/__testfixtures__/remove-context-provider/typescript/with-provider-2.output.tsx

@@ -3,7 +3,6 @@ function App({ url }: { url: string }) {
 
   return (
     <Context value={theme}>
-      <Page />
     </Context>
   );
-}
+}

transforms/__testfixtures__/remove-context-provider/typescript/with-provider.input.js → transforms/__testfixtures__/remove-context-provider/typescript/with-provider.input.tsx

@@ -6,4 +6,4 @@ function App({ url }: { url: string }) {
       <Page />
     </ThemeContext.Provider>
   );
-}
+}

transforms/__testfixtures__/remove-context-provider/typescript/with-provider.output.js → transforms/__testfixtures__/remove-context-provider/typescript/with-provider.output.tsx

@@ -6,4 +6,4 @@ function App({ url }: { url: string }) {
       <Page />
     </ThemeContext>
   );
-}
+}

transforms/__testfixtures__/remove-context-provider/with-provider-2.input.js

@@ -3,7 +3,6 @@ function App() {
 
   return (
     <Context.Provider value={theme}>
-    <Page />
     </Context.Provider>
   );
 }

transforms/__testfixtures__/remove-context-provider/with-provider-2.output.js

@@ -3,7 +3,6 @@ function App() {
 
   return (
     <Context value={theme}>
-    <Page />
     </Context>
   );
 }

transforms/__testfixtures__/remove-forward-ref/typescript/props-type-literal.output.js → transforms/__testfixtures__/remove-forward-ref/typescript/props-type-literal.output.tsx

@@ -3,7 +3,7 @@ const MyComponent = function Component(
         ref: myRef,
         ...myProps
     }: { a: 1 } & {
-        ref: React.RefObject<unknown>
+        ref: React.RefObject<unknown>;
     }
 ) {
     return null;

transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments-custom-names.output.js → transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments-custom-names.output.tsx

@@ -3,7 +3,7 @@ const MyComponent = function Component(
     ref: myRef,
     ...myProps
   }: Props & {
-    ref: React.RefObject<HTMLButtonElement>
+    ref: React.RefObject<HTMLButtonElement>;
   }
 ) {
   return null;

transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments-type-literals.output.js → transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments-type-literals.output.tsx

@@ -3,7 +3,7 @@ const MyInput = (
     ref,
     ...props
   }: { a: string } & {
-    ref: React.RefObject<RefValueType>
+    ref: React.RefObject<RefValueType>;
   }
 ) => {
   return null;

transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments.output.js → transforms/__testfixtures__/remove-forward-ref/typescript/type-arguments.output.tsx

@@ -5,7 +5,7 @@ const MyInput = (
     ref,
     ...props
   }: Props & {
-    ref: React.RefObject<HTMLInputElement>
+    ref: React.RefObject<HTMLInputElement>;
   }
 ) => {
   return null;

transforms/__tests__/React-PropTypes-to-prop-types-test.js

@@ -9,7 +9,7 @@
 'use strict';
 
 jest.mock('../React-PropTypes-to-prop-types', () => {
-  return Object.assign(require.requireActual('../React-PropTypes-to-prop-types'), {
+  return Object.assign(jest.requireActual('../React-PropTypes-to-prop-types'), {
     parser: 'flow'
   });
 });

transforms/__tests__/ReactNative-View-propTypes-test.js

@@ -9,7 +9,7 @@
 'use strict';
 
 jest.mock('../ReactNative-View-propTypes', () => {
-  return Object.assign(require.requireActual('../ReactNative-View-propTypes'), {
+  return Object.assign(jest.requireActual('../ReactNative-View-propTypes'), {
     parser: 'flow'
   });
 });

transforms/__tests__/create-element-to-jsx-test.js

@@ -9,7 +9,7 @@
 'use strict';
 
 jest.mock('../create-element-to-jsx', () => {
-  return Object.assign(require.requireActual('../create-element-to-jsx'), {
+  return Object.assign(jest.requireActual('../create-element-to-jsx'), {
     parser: 'flow'
   });
 });

transforms/__tests__/manual-bind-to-arrow-test.js

@@ -9,7 +9,7 @@
 'use strict';
 
 jest.mock('../manual-bind-to-arrow', () => {
-  return Object.assign(require.requireActual('../manual-bind-to-arrow'), {
+  return Object.assign(jest.requireActual('../manual-bind-to-arrow'), {
     parser: 'flow'
   });
 });

transforms/__tests__/pure-component-test.js

@@ -9,7 +9,7 @@
 'use strict';
 
 jest.mock('../pure-component', () => {
-  return Object.assign(require.requireActual('../pure-component'), {
+  return Object.assign(jest.requireActual('../pure-component'), {
     parser: 'flow'
   });
 });

transforms/__tests__/remove-context-provider.test.ts

@@ -1,51 +1,39 @@
+"use strict";
 
-'use strict';
+const tests = ["with-provider", "with-provider-2", "no-provider"];
 
-const tests = [
-  'with-provider',
-  'with-provider-2',
-  'no-provider',
-];
+const defineTest = require("jscodeshift/dist/testUtils").defineTest;
 
-const defineTest = require('jscodeshift/dist/testUtils').defineTest;
-
-describe('remove-context-provider', () => {
-
-  tests.forEach(test =>
+describe("remove-context-provider", () => {
+  tests.forEach((test) =>
     defineTest(
       __dirname,
-      'remove-context-provider',
+      "remove-context-provider",
       null,
-      `remove-context-provider/${ test }`
+      `remove-context-provider/${test}`
     )
   );
 
-
-  describe('typescript', () => {
-
+  describe("typescript", () => {
     beforeEach(() => {
-      jest.mock('../remove-context-provider', () => {
-        return Object.assign(
-          require.requireActual('../remove-context-provider'),
-          {
-            parser: 'tsx'
-          }
-        );
+      jest.mock("../remove-context-provider", () => {
+        return Object.assign(jest.requireActual("../remove-context-provider"), {
+          parser: "tsx",
+        });
       });
     });
 
     afterEach(() => {
       jest.resetModules();
     });
 
-    tests.forEach(test => {
+    tests.forEach((test) => {
       defineTest(
         __dirname,
-        'remove-context-provider',
+        "remove-context-provider",
         null,
-        `remove-context-provider/typescript/${ test }`
+        `remove-context-provider/typescript/${test}`
       );
     });
-
   });
 });

transforms/__tests__/remove-forward-ref.test.ts

@@ -1,62 +1,54 @@
-
-'use strict';
+"use strict";
 
 const jsTests = [
-  'function-expression',
-  'arrow-function-expression',
-  'forward-ref-import',
-  'forward-ref-import-2',
-  'props-identifier', 
-  'props-object-pattern', 
-  'callee-is-member-expression'
+  "function-expression",
+  "arrow-function-expression",
+  "forward-ref-import",
+  "forward-ref-import-2",
+  "props-identifier",
+  "props-object-pattern",
+  "callee-is-member-expression",
 ];
 
 const tsTests = [
-  'type-arguments', 
-  'type-arguments-custom-names', 
-  'type-arguments-type-literals', 
-  'props-type-literal'
+  "type-arguments",
+  "type-arguments-custom-names",
+  "type-arguments-type-literals",
+  "props-type-literal",
 ];
 
-const defineTest = require('jscodeshift/dist/testUtils').defineTest;
-
-describe('remove-forward-ref', () => {
+const defineTest = require("jscodeshift/dist/testUtils").defineTest;
 
-  jsTests.forEach(test =>
+describe("remove-forward-ref", () => {
+  jsTests.forEach((test) =>
     defineTest(
       __dirname,
-      'remove-forward-ref',
+      "remove-forward-ref",
       null,
-      `remove-forward-ref/${ test }`
+      `remove-forward-ref/${test}`
     )
   );
 
-
-  describe('typescript', () => {
-
+  describe("typescript", () => {
     beforeEach(() => {
-      jest.mock('../remove-forward-ref', () => {
-        return Object.assign(
-          require.requireActual('../remove-forward-ref'),
-          {
-            parser: 'tsx'
-          }
-        );
+      jest.mock("../remove-forward-ref", () => {
+        return Object.assign(jest.requireActual("../remove-forward-ref"), {
+          parser: "tsx",
+        });
       });
     });
 
     afterEach(() => {
       jest.resetModules();
     });
 
-    tsTests.forEach(test => {
+    tsTests.forEach((test) => {
       defineTest(
         __dirname,
-        'remove-forward-ref',
+        "remove-forward-ref",
         null,
-        `remove-forward-ref/typescript/${ test }`
+        `remove-forward-ref/typescript/${test}`
       );
     });
-
   });
 });

transforms/__tests__/rename-unsafe-lifecycles-test.js

@@ -26,7 +26,7 @@ describe('rename-unsafe-lifecycles', () => {
     beforeEach(() => {
       jest.mock('../rename-unsafe-lifecycles', () => {
         return Object.assign(
-          require.requireActual('../rename-unsafe-lifecycles'),
+          jest.requireActual('../rename-unsafe-lifecycles'),
           {
             parser: 'flow'
           }
@@ -52,7 +52,7 @@ describe('rename-unsafe-lifecycles', () => {
     beforeEach(() => {
       jest.mock('../rename-unsafe-lifecycles', () => {
         return Object.assign(
-          require.requireActual('../rename-unsafe-lifecycles'),
+          jest.requireActual('../rename-unsafe-lifecycles'),
           {
             parser: 'tsx'
           }