remove old magic redirect codemod

Author: bjohansebas

commands/__test__/transform.spec.ts

@@ -22,15 +22,15 @@ describe('interactive mode', () => {
   it('runs without codemodName and source params provided', async () => {
     const spyOnConsole = jest.spyOn(console, 'log').mockImplementation()
 
-    prompts.inject(['magic-redirect'])
+    prompts.inject(['pluralized-methods'])
     prompts.inject(['./transforms/__testfixtures__'])
 
     await transform(undefined, undefined, defaultOptions)
 
     expect(spyOnConsole).not.toHaveBeenCalled()
     expect(run).toHaveBeenCalledTimes(1)
     expect(run).toHaveBeenCalledWith(
-      expect.stringContaining(getSystemPath('/transforms/magic-redirect.js')),
+      expect.stringContaining(getSystemPath('/transforms/pluralized-methods.js')),
       expect.arrayContaining([expect.stringContaining(getSystemPath('/transforms/__testfixtures__'))]),
       {
         babel: false,
@@ -46,14 +46,14 @@ describe('interactive mode', () => {
   it('runs properly on incorrect user input', async () => {
     const spyOnConsole = jest.spyOn(console, 'log').mockImplementation()
 
-    prompts.inject(['magic-redirect'])
+    prompts.inject(['pluralized-methods'])
 
     await transform('bad-codemod', './transforms/__testfixtures__', defaultOptions)
 
     expect(spyOnConsole).not.toHaveBeenCalled()
     expect(run).toHaveBeenCalledTimes(1)
     expect(run).toHaveBeenCalledWith(
-      expect.stringContaining(getSystemPath('/transforms/magic-redirect.js')),
+      expect.stringContaining(getSystemPath('/transforms/pluralized-methods.js')),
       expect.arrayContaining([expect.stringContaining(getSystemPath('/transforms/__testfixtures__'))]),
       {
         babel: false,
@@ -71,12 +71,12 @@ describe('interactive mode', () => {
 
     prompts.inject(['__testfixtures__'])
 
-    await transform('magic-redirect', undefined, defaultOptions)
+    await transform('pluralized-methods', undefined, defaultOptions)
 
     expect(spyOnConsole).not.toHaveBeenCalled()
     expect(run).toHaveBeenCalledTimes(1)
     expect(run).toHaveBeenCalledWith(
-      expect.stringContaining(getSystemPath('/transforms/magic-redirect.js')),
+      expect.stringContaining(getSystemPath('/transforms/pluralized-methods.js')),
       expect.arrayContaining([expect.stringContaining('__testfixtures__')]),
       {
         babel: false,
@@ -98,12 +98,12 @@ describe('Non-Interactive Mode', () => {
   it('Transforms code with codemodName and source params provided', async () => {
     const spyOnConsole = jest.spyOn(console, 'log').mockImplementation()
 
-    await transform('magic-redirect', '__testfixtures__', defaultOptions)
+    await transform('pluralized-methods', '__testfixtures__', defaultOptions)
 
     expect(spyOnConsole).not.toHaveBeenCalled()
     expect(run).toHaveBeenCalledTimes(1)
     expect(run).toHaveBeenCalledWith(
-      expect.stringContaining(getSystemPath('/transforms/magic-redirect.js')),
+      expect.stringContaining(getSystemPath('/transforms/pluralized-methods.js')),
       expect.arrayContaining([expect.stringContaining('__testfixtures__')]),
       {
         babel: false,

commands/__test__/upgrade.spec.ts

@@ -14,7 +14,7 @@ describe('interactive mode', () => {
   it('runs without source params provided and select is true', async () => {
     const spyOnConsole = jest.spyOn(console, 'log').mockImplementation()
 
-    prompts.inject([['magic-redirect', 'req-param']])
+    prompts.inject([['pluralized-methods', 'req-param']])
 
     await upgrade('__testfixtures__', { select: true })
 
@@ -26,7 +26,7 @@ describe('interactive mode', () => {
   it('runs with source params provided and select is true', async () => {
     const spyOnConsole = jest.spyOn(console, 'log').mockImplementation()
 
-    prompts.inject([['magic-redirect', 'req-param']])
+    prompts.inject([['pluralized-methods', 'req-param']])
 
     await upgrade('__testfixtures__', { select: true })
 
@@ -43,8 +43,8 @@ describe('interactive mode', () => {
     await upgrade(undefined)
 
     expect(spyOnConsole).toHaveBeenCalled()
-    expect(spyOnConsole).toHaveBeenCalledTimes(5)
-    expect(run).toHaveBeenCalledTimes(4)
+    expect(spyOnConsole).toHaveBeenCalledTimes(4)
+    expect(run).toHaveBeenCalledTimes(3)
   })
 })
 
@@ -58,12 +58,11 @@ describe('Non-Interactive Mode', () => {
 
     await upgrade('__testfixtures__')
 
-    expect(spyOnConsole).toHaveBeenCalledTimes(5)
-    expect(spyOnConsole).toHaveBeenCalledWith('> Applying codemod: magic-redirect')
+    expect(spyOnConsole).toHaveBeenCalledTimes(4)
     expect(spyOnConsole).toHaveBeenCalledWith('> Applying codemod: pluralized-methods')
     expect(spyOnConsole).toHaveBeenCalledWith('> Applying codemod: req-param')
     expect(spyOnConsole).toHaveBeenCalledWith('> Applying codemod: v4-deprecated-signatures')
     expect(spyOnConsole).toHaveBeenLastCalledWith('\n> All codemods have been applied successfully. \n')
-    expect(run).toHaveBeenCalledTimes(4)
+    expect(run).toHaveBeenCalledTimes(3)
   })
 })

config.ts

@@ -1,9 +1,4 @@
 export const TRANSFORM_OPTIONS = [
-  {
-    description: 'Transform the deprecated magic string "back"',
-    value: 'magic-redirect',
-    version: '5.0.0',
-  },
   {
     description: 'Transform the methods to their pluralized versions',
     value: 'pluralized-methods',