fix: default to npm when testing

Author: danielroe

app/composables/useSettings.ts

@@ -41,7 +41,7 @@ const DEFAULT_SETTINGS: AppSettings = {
   hidePlatformPackages: true,
   selectedLocale: null,
   preferredBackgroundTheme: null,
-  searchProvider: 'algolia',
+  searchProvider: import.meta.test ? 'npm' : 'algolia',
   sidebar: {
     collapsed: [],
   },

test/nuxt/components/compare/PackageSelector.spec.ts

@@ -1,5 +1,5 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest'
-import { nextTick, ref } from 'vue'
+import { ref } from 'vue'
 import { mountSuspended } from '@nuxt/test-utils/runtime'
 import PackageSelector from '~/components/Compare/PackageSelector.vue'
 
@@ -141,7 +141,6 @@ describe('PackageSelector', () => {
 
       const input = component.find('input')
       await input.setValue('lodash')
-      await nextTick()
       await input.trigger('keydown', { key: 'Enter' })
 
       const emitted = component.emitted('update:modelValue')
@@ -158,7 +157,6 @@ describe('PackageSelector', () => {
 
       const input = component.find('input')
       await input.setValue('no dep')
-      await nextTick()
       await input.trigger('keydown', { key: 'Enter' })
 
       const emitted = component.emitted('update:modelValue')
@@ -175,7 +173,6 @@ describe('PackageSelector', () => {
 
       const input = component.find('input')
       await input.setValue('lodash')
-      await nextTick()
       await input.trigger('keydown', { key: 'Enter' })
 
       // Input should be cleared

test/nuxt/setup.ts

@@ -23,14 +23,4 @@ if (typeof window !== 'undefined') {
       removeColorScheme: () => {},
     }
   }
-
-  // force npm search provider in tests for compatibility with our mocks
-  const settingsKey = 'npmx-settings'
-  try {
-    const existing = JSON.parse(localStorage.getItem(settingsKey) || '{}')
-    existing.searchProvider = 'npm'
-    localStorage.setItem(settingsKey, JSON.stringify(existing))
-  } catch {
-    localStorage.setItem(settingsKey, JSON.stringify({ searchProvider: 'npm' }))
-  }
 }