Merge pull request #1527 from Automattic/remove/babel-plugins

chore(dev-deps): remove @babel/plugin-transform-modules-commonjs and babel-plugin-module-resolver
Automattic · Oct 17, 2023 · 67c0c7b · 67c0c7b
2 parents fc99e03 + cb6543e
commit 67c0c7b
Show file tree

Hide file tree

Showing 11 changed files with 8 additions and 246 deletions.
diff --git a/__tests__/bin/vip-config-envvar-delete.js b/__tests__/bin/vip-config-envvar-delete.js
@@ -19,7 +19,7 @@ function mockExit() {
 jest.spyOn( console, 'log' ).mockImplementation( () => {} );
 jest.spyOn( process, 'exit' ).mockImplementation( mockExit );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-config-envvar-get-all.js b/__tests__/bin/vip-config-envvar-get-all.js
@@ -18,7 +18,7 @@ function mockExit() {
 jest.spyOn( console, 'log' ).mockImplementation( () => {} );
 jest.spyOn( process, 'exit' ).mockImplementation( mockExit );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-config-envvar-get.js b/__tests__/bin/vip-config-envvar-get.js
@@ -17,7 +17,7 @@ function mockExit() {
 jest.spyOn( console, 'log' ).mockImplementation( () => {} );
 jest.spyOn( process, 'exit' ).mockImplementation( mockExit );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-config-envvar-list.js b/__tests__/bin/vip-config-envvar-list.js
@@ -18,7 +18,7 @@ function mockExit() {
 jest.spyOn( console, 'log' ).mockImplementation( () => {} );
 jest.spyOn( process, 'exit' ).mockImplementation( mockExit );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-config-envvar-set.js b/__tests__/bin/vip-config-envvar-set.js
@@ -20,7 +20,7 @@ function mockExit() {
 jest.spyOn( console, 'log' ).mockImplementation( () => {} );
 jest.spyOn( process, 'exit' ).mockImplementation( mockExit );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-logs.js b/__tests__/bin/vip-logs.js
@@ -13,7 +13,7 @@ jest.spyOn( exit, 'withError' ).mockImplementation( () => {
 	throw 'EXIT WITH ERROR'; // throws to break the flow (the real implementation does a process.exit)
 } );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/bin/vip-slowlogs.js b/__tests__/bin/vip-slowlogs.js
@@ -13,7 +13,7 @@ jest.spyOn( exit, 'withError' ).mockImplementation( () => {
 	throw 'EXIT WITH ERROR'; // throws to break the flow (the real implementation does a process.exit)
 } );
 
-jest.mock( 'lib/cli/command', () => {
+jest.mock( '../../src/lib/cli/command', () => {
 	const commandMock = {
 		argv: () => commandMock,
 		examples: () => commandMock,

diff --git a/__tests__/lib/cli/exit.js b/__tests__/lib/cli/exit.js
@@ -19,7 +19,7 @@ jest.spyOn( global.console, 'log' );
 const mockExit = jest.spyOn( process, 'exit' ).mockImplementation( () => {} );
 const ERROR_CODE = 1;
 
-describe( 'lib/cli/exit', () => {
+describe( '../../src/lib/cli/exit', () => {
 	beforeAll( async () => {
 		output = '';
 		mockExit.mockClear();

diff --git a/babel.config.js b/babel.config.js
@@ -15,17 +15,5 @@ module.exports = {
 			},
 		],
 	],
-	plugins: [
-		'@babel/plugin-transform-modules-commonjs',
-		[
-			'module-resolver',
-			{
-				root: [ './src' ],
-				alias: {
-					root: './',
-				},
-			},
-		],
-	],
 	ignore: [ '**/*.d.ts' ],
 };
diff --git a/npm-shrinkwrap.json b/npm-shrinkwrap.json
diff --git a/package.json b/package.json
@@ -105,7 +105,6 @@
     "@automattic/eslint-plugin-wpvip": "0.6.0",
     "@babel/cli": "7.22.15",
     "@babel/core": "7.22.17",
-    "@babel/plugin-transform-modules-commonjs": "7.22.15",
     "@babel/preset-env": "7.22.15",
     "@babel/preset-typescript": "7.22.15",
     "@jest/globals": "^29.5.0",
@@ -127,7 +126,6 @@
     "@types/uuid": "^9.0.4",
     "@types/xml2js": "^0.4.12",
     "babel-jest": "^29.5.0",
-    "babel-plugin-module-resolver": "5.0.0",
     "dockerode": "^3.3.4",
     "eslint": "^8.35.0",
     "jest": "^29.5.0",