test: clear local storage before each test

Amp-Thread-ID: https://ampcode.com/threads/T-019feeff-305b-73a5-bb82-3ae3a74e816d

Author: ampagent

src/components/TopMenuSection.test.ts

@@ -196,7 +196,6 @@ function createComfyActionbarStub(actionbarTarget: HTMLElement) {
 
 describe('TopMenuSection', () => {
   beforeEach(() => {
-    localStorage.clear()
     mockData.isLoggedIn = false
     mockData.setShowConflictRedDot(false)
   })

src/components/actionbar/ComfyActionbar.test.ts

@@ -63,7 +63,6 @@ const renderActionbar = (showRunProgressBar: boolean) => {
 describe('ComfyActionbar', () => {
   beforeEach(() => {
     i18n.global.locale.value = 'en'
-    localStorage.clear()
   })
 
   it('teleports inline progress when run progress bar is enabled', async () => {

src/composables/maskeditor/useBrushPersistence.test.ts

@@ -28,7 +28,6 @@ const STORAGE_KEY = 'maskeditor_brush_settings'
 
 beforeEach(() => {
   setActivePinia(createTestingPinia({ stubActions: false }))
-  localStorage.clear()
 })
 
 describe('loadAndApply', () => {

src/composables/useFeatureFlags.test.ts

@@ -235,10 +235,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('dev override via localStorage', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('resolveFlag returns localStorage override over remoteConfig and server value', () => {
       vi.mocked(api.getServerFeature).mockReturnValue(false)
       localStorage.setItem('ff:model_upload_button_enabled', 'true')
@@ -301,7 +297,6 @@ describe('useFeatureFlags', () => {
       remoteConfig.value = {}
       cachedBillingControlEnabled.value = undefined
       cachedV1PaymentRecovery.value = undefined
-      localStorage.clear()
     })
 
     afterEach(() => {
@@ -310,7 +305,6 @@ describe('useFeatureFlags', () => {
       remoteConfig.value = {}
       cachedBillingControlEnabled.value = undefined
       cachedV1PaymentRecovery.value = undefined
-      localStorage.clear()
     })
 
     it('returns the cached session value during the auth window', () => {
@@ -359,10 +353,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('signupTurnstileMode', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('falls back to the server feature flag with default off', () => {
       vi.mocked(api.getServerFeature).mockImplementation(
         (path, defaultValue) => {
@@ -436,7 +426,6 @@ describe('useFeatureFlags', () => {
     afterEach(() => {
       vi.mocked(distributionTypes).isCloud = false
       remoteConfig.value = {}
-      localStorage.clear()
     })
 
     it('is disabled outside the cloud distribution', () => {
@@ -466,10 +455,6 @@ describe('useFeatureFlags', () => {
   })
 
   describe('unifiedCloudAuthEnabled', () => {
-    afterEach(() => {
-      localStorage.clear()
-    })
-
     it('reads the unified_cloud_auth server feature when set', () => {
       vi.mocked(api.getServerFeature).mockImplementation(
         (path, defaultValue) => {
@@ -497,7 +482,6 @@ describe('useFeatureFlags', () => {
       vi.mocked(distributionTypes).isCloud = false
       remoteConfigState.value = 'unloaded'
       cachedBillingControlEnabled.value = undefined
-      localStorage.clear()
       remoteConfig.value = {}
     })
 

src/composables/useRunButtonTelemetry.test.ts

@@ -45,7 +45,6 @@ import {
 
 describe('useRunButtonTelemetry', () => {
   beforeEach(() => {
-    localStorage.clear()
     state.telemetry.trackRunButton.mockClear()
     state.mode.value = 'graph'
     state.isAppMode.value = false

src/platform/cloud/subscription/components/CreditsTile.test.ts

@@ -209,7 +209,6 @@ describe('CreditsTile', () => {
     state.type = 'workspace'
     state.customerEventsError = null
     mockIsCloud.value = true
-    localStorage.clear()
   })
 
   it('renders the total balance (cents converted to credits) with the remaining suffix', () => {

src/platform/surveys/ErrorPanelSurveyCta.test.ts

@@ -1,6 +1,6 @@
 import { render, screen, within } from '@testing-library/vue'
 import userEvent from '@testing-library/user-event'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { nextTick, ref } from 'vue'
 
 const FEATURE_USAGE_KEY = 'Comfy.FeatureUsage'
@@ -82,7 +82,6 @@ describe('ErrorPanelSurveyCta', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
     vi.resetModules()
     mockOpen.mockReset()
 
@@ -97,10 +96,6 @@ describe('ErrorPanelSurveyCta', () => {
     }
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   async function renderComponent() {
     const { default: ErrorPanelSurveyCta } =
       await import('./ErrorPanelSurveyCta.vue')

src/platform/surveys/NightlySurveyPopover.test.ts

@@ -1,6 +1,6 @@
 import { render, screen } from '@testing-library/vue'
 import userEvent from '@testing-library/user-event'
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { nextTick } from 'vue'
 import { createI18n } from 'vue-i18n'
 
@@ -44,18 +44,13 @@ describe('NightlySurveyPopover', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
     vi.resetModules()
 
     mockIsNightly.value = true
     mockIsCloud.value = false
     mockIsDesktop.value = false
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   async function renderComponent(
     config = defaultConfig,
     eventHandlers: Record<string, ReturnType<typeof vi.fn>> = {}

src/platform/surveys/useFeatureUsageTracker.test.ts

@@ -1,18 +1,10 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 
 import { useFeatureUsageTracker } from './useFeatureUsageTracker'
 
 const STORAGE_KEY = 'Comfy.FeatureUsage'
 
 describe('useFeatureUsageTracker', () => {
-  beforeEach(() => {
-    localStorage.clear()
-  })
-
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   it('initializes with zero count for new feature', () => {
     const { useCount } = useFeatureUsageTracker('test-feature-1')
 

src/platform/surveys/useSurveyEligibility.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { useSurveyEligibility } from './useSurveyEligibility'
 
@@ -30,17 +30,11 @@ describe('useSurveyEligibility', () => {
   }
 
   beforeEach(() => {
-    localStorage.clear()
-
     mockDistribution.isNightly = true
     mockDistribution.isCloud = false
     mockDistribution.isDesktop = false
   })
 
-  afterEach(() => {
-    localStorage.clear()
-  })
-
   function setFeatureUsage(featureId: string, useCount: number) {
     const existing = JSON.parse(localStorage.getItem(FEATURE_USAGE_KEY) ?? '{}')
     existing[featureId] = {