update tests

Author: gameroman

test/nuxt/composables/use-compare-replacements.spec.ts

@@ -1,3 +1,4 @@
+import { ref, watchEffect, h, defineComponent } from 'vue'
 import { afterEach, describe, expect, it, vi } from 'vitest'
 import { mountSuspended } from '@nuxt/test-utils/runtime'
 import type { ModuleReplacement } from 'module-replacements'
@@ -50,12 +51,13 @@ describe('useCompareReplacements', () => {
         vi.fn().mockImplementation((url: string) => {
           if (url.includes('/api/replacements/array-includes')) {
             return Promise.resolve({
+              id: 'array-includes',
               type: 'native',
-              moduleName: 'array-includes',
-              nodeVersion: '6.0.0',
-              replacement: 'Array.prototype.includes',
-              mdnPath: 'Global_Objects/Array/includes',
-              category: 'native',
+              url: { type: 'mdn', id: 'Global_Objects/Array/includes' },
+              engines: [{ engine: 'nodejs', minVersion: '6.0.0' }],
+              nodeFeatureId: {
+                moduleName: 'Array.prototype.includes',
+              },
             } satisfies ModuleReplacement)
           }
           return Promise.resolve(null)
@@ -70,6 +72,7 @@ describe('useCompareReplacements', () => {
         expect(noDepSuggestions.value).toHaveLength(1)
       })
 
+      // Note: forPackage likely maps to the 'id' in the new schema or the original query
       expect(noDepSuggestions.value[0]?.forPackage).toBe('array-includes')
       expect(noDepSuggestions.value[0]?.replacement.type).toBe('native')
       expect(infoSuggestions.value).toHaveLength(0)
@@ -81,10 +84,9 @@ describe('useCompareReplacements', () => {
         vi.fn().mockImplementation((url: string) => {
           if (url.includes('/api/replacements/is-even')) {
             return Promise.resolve({
+              id: 'is-even',
               type: 'simple',
-              moduleName: 'is-even',
-              replacement: 'Use (n % 2) === 0',
-              category: 'micro-utilities',
+              description: 'Use (n % 2) === 0',
             } satisfies ModuleReplacement)
           }
           return Promise.resolve(null)
@@ -110,10 +112,11 @@ describe('useCompareReplacements', () => {
         vi.fn().mockImplementation((url: string) => {
           if (url.includes('/api/replacements/moment')) {
             return Promise.resolve({
+              id: 'moment',
               type: 'documented',
-              moduleName: 'moment',
-              docPath: 'moment',
-              category: 'preferred',
+              url: 'https://momentjs.com/docs/#/use-it/built-in-alternatives/',
+              replacementModule: 'date-fns',
+              preferred: true,
             } satisfies ModuleReplacement)
           }
           return Promise.resolve(null)
@@ -139,28 +142,25 @@ describe('useCompareReplacements', () => {
         vi.fn().mockImplementation((url: string) => {
           if (url.includes('/api/replacements/is-odd')) {
             return Promise.resolve({
+              id: 'is-odd',
               type: 'simple',
-              moduleName: 'is-odd',
-              replacement: 'Use (n % 2) !== 0',
-              category: 'micro-utilities',
+              description: 'Use (n % 2) !== 0',
             } satisfies ModuleReplacement)
           }
           if (url.includes('/api/replacements/lodash')) {
             return Promise.resolve({
+              id: 'lodash',
               type: 'documented',
-              moduleName: 'lodash',
-              docPath: 'lodash-underscore',
-              category: 'preferred',
+              url: 'https://github.com/you-dont-need/You-Dont-Need-Lodash-Underscore',
+              replacementModule: 'native',
             } satisfies ModuleReplacement)
           }
           if (url.includes('/api/replacements/array-map')) {
             return Promise.resolve({
+              id: 'array-map',
               type: 'native',
-              moduleName: 'array-map',
-              nodeVersion: '0.10.0',
-              replacement: 'Array.prototype.map',
-              mdnPath: 'Global_Objects/Array/map',
-              category: 'native',
+              url: { type: 'mdn', id: 'Global_Objects/Array/map' },
+              engines: [{ engine: 'nodejs', minVersion: '0.10.0' }],
             } satisfies ModuleReplacement)
           }
           return Promise.resolve(null)
@@ -178,12 +178,9 @@ describe('useCompareReplacements', () => {
         expect(infoSuggestions.value).toHaveLength(1)
       })
 
-      // no dep should have simple and native
       const noDepTypes = noDepSuggestions.value.map(s => s.replacement.type)
       expect(noDepTypes).toContain('simple')
       expect(noDepTypes).toContain('native')
-
-      // Info should have documented
       expect(infoSuggestions.value[0]?.replacement.type).toBe('documented')
     })
   })
@@ -192,12 +189,7 @@ describe('useCompareReplacements', () => {
     it('does not include packages with no replacement data', async () => {
       vi.stubGlobal(
         '$fetch',
-        vi.fn().mockImplementation((url: string) => {
-          if (url.includes('/api/replacements/react')) {
-            return Promise.resolve(null) // No replacement for react
-          }
-          return Promise.resolve(null)
-        }),
+        vi.fn().mockImplementation(() => Promise.resolve(null)),
       )
 
       const { noDepSuggestions, infoSuggestions, replacements } =
@@ -214,21 +206,16 @@ describe('useCompareReplacements', () => {
     it('handles fetch errors gracefully', async () => {
       vi.stubGlobal(
         '$fetch',
-        vi.fn().mockImplementation(() => {
-          return Promise.reject(new Error('Network error'))
-        }),
+        vi.fn().mockImplementation(() => Promise.reject(new Error('Network error'))),
       )
 
-      const { noDepSuggestions, infoSuggestions, replacements } =
-        await useCompareReplacementsInComponent(['some-package'])
+      const { replacements } = await useCompareReplacementsInComponent(['some-package'])
 
       await vi.waitFor(() => {
         expect(replacements.value.has('some-package')).toBe(true)
       })
 
       expect(replacements.value.get('some-package')).toBeNull()
-      expect(noDepSuggestions.value).toHaveLength(0)
-      expect(infoSuggestions.value).toHaveLength(0)
     })
   })
 
@@ -237,10 +224,9 @@ describe('useCompareReplacements', () => {
       const fetchMock = vi.fn().mockImplementation((url: string) => {
         if (url.includes('/api/replacements/is-even')) {
           return Promise.resolve({
+            id: 'is-even',
             type: 'simple',
-            moduleName: 'is-even',
-            replacement: 'Use (n % 2) === 0',
-            category: 'micro-utilities',
+            description: 'Use (n % 2) === 0',
           } satisfies ModuleReplacement)
         }
         return Promise.resolve(null)

test/nuxt/composables/use-replacement-dependencies.spec.ts

@@ -114,7 +114,7 @@ describe('useReplacementDependencies', () => {
 
     expect(simple?.type).toBe('simple')
     expect(native?.type).toBe('native')
-    expect(native?.url?.type).toBe('mdn')
+    expect(native?.url).toHaveProperty('type', 'mdn')
     expect(replacements.value['picoquery']).toBeUndefined()
   })