move safe migration tests to is-safe-migration tests

Author: RobinMalfait

packages/@tailwindcss-upgrade/src/codemods/template/is-safe-migration.test.ts

@@ -29,4 +29,26 @@ test('does not replace classes in invalid positions', async () => {
   await shouldNotReplace(`<div v-else-if="!border"></div>\n`)
   await shouldNotReplace(`<div v-show="!border"></div>\n`)
   await shouldNotReplace(`<div x-if="!border"></div>\n`)
+
+  await shouldNotReplace(`let notShadow = shadow    \n`, 'shadow')
+  await shouldNotReplace(`{ "foo": shadow.something + ""}\n`, 'shadow')
+  await shouldNotReplace(`<div v-if="something && shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-else-if="something && shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-show="something && shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-if="shadow || shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-else-if="shadow || shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-show="shadow || shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-if="shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-else-if="shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div v-show="shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(`<div x-if="shadow"></div>\n`, 'shadow')
+  await shouldNotReplace(
+    `<div style={{filter: 'drop-shadow(30px 10px 4px #4444dd)'}}/>\n`,
+    'shadow',
+  )
+
+  // Next.js Image placeholder cases
+  await shouldNotReplace(`<Image placeholder="blur" src="/image.jpg" />`, 'blur')
+  await shouldNotReplace(`<Image placeholder={'blur'} src="/image.jpg" />`, 'blur')
+  await shouldNotReplace(`<Image placeholder={blur} src="/image.jpg" />`, 'blur')
 })

packages/@tailwindcss-upgrade/src/codemods/template/migrate-legacy-classes.test.ts

@@ -1,7 +1,6 @@
 import { __unstable__loadDesignSystem } from '@tailwindcss/node'
 import { expect, test, vi } from 'vitest'
 import * as versions from '../../utils/version'
-import { isSafeMigration } from './is-safe-migration'
 import { migrateLegacyClasses } from './migrate-legacy-classes'
 vi.spyOn(versions, 'isMajor').mockReturnValue(true)
 
@@ -44,43 +43,3 @@ test.each([
 
   expect(await migrateLegacyClasses(designSystem, {}, candidate)).toEqual(result)
 })
-
-test('does not replace classes in invalid positions', async () => {
-  let designSystem = await __unstable__loadDesignSystem('@import "tailwindcss";', {
-    base: __dirname,
-  })
-
-  async function shouldNotReplace(example: string, candidate = 'shadow') {
-    let location = {
-      contents: example,
-      start: example.indexOf(candidate),
-      end: example.indexOf(candidate) + candidate.length,
-    }
-
-    // Skip this migration if we think that the migration is unsafe
-    if (location && !isSafeMigration(candidate, location, designSystem)) {
-      return candidate
-    }
-
-    expect(await migrateLegacyClasses(designSystem, {}, candidate)).toEqual(candidate)
-  }
-
-  await shouldNotReplace(`let notShadow = shadow    \n`)
-  await shouldNotReplace(`{ "foo": shadow.something + ""}\n`)
-  await shouldNotReplace(`<div v-if="something && shadow"></div>\n`)
-  await shouldNotReplace(`<div v-else-if="something && shadow"></div>\n`)
-  await shouldNotReplace(`<div v-show="something && shadow"></div>\n`)
-  await shouldNotReplace(`<div v-if="shadow || shadow"></div>\n`)
-  await shouldNotReplace(`<div v-else-if="shadow || shadow"></div>\n`)
-  await shouldNotReplace(`<div v-show="shadow || shadow"></div>\n`)
-  await shouldNotReplace(`<div v-if="shadow"></div>\n`)
-  await shouldNotReplace(`<div v-else-if="shadow"></div>\n`)
-  await shouldNotReplace(`<div v-show="shadow"></div>\n`)
-  await shouldNotReplace(`<div x-if="shadow"></div>\n`)
-  await shouldNotReplace(`<div style={{filter: 'drop-shadow(30px 10px 4px #4444dd)'}}/>\n`)
-
-  // Next.js Image placeholder cases
-  await shouldNotReplace(`<Image placeholder="blur" src="/image.jpg" />`, 'blur')
-  await shouldNotReplace(`<Image placeholder={'blur'} src="/image.jpg" />`, 'blur')
-  await shouldNotReplace(`<Image placeholder={blur} src="/image.jpg" />`, 'blur')
-})