Fixed EventConfigSourceTable test

Author: synqotik

ui/tests/components/EventConfiguration/EventConfigSourceTable.test.ts

@@ -28,7 +28,6 @@ describe('EventConfigSourceTable.vue', () => {
 
   beforeEach(async () => {
     vi.clearAllMocks()
-    // vi.useFakeTimers()
 
     const pinia = createTestingPinia({
       createSpy: vi.fn,
@@ -90,45 +89,19 @@ describe('EventConfigSourceTable.vue', () => {
   })
 
   afterEach(async () => {
-    const startTime = Date.now()
-    console.log('[afterEach] Starting cleanup...')
-    
-    // // Run any pending timers before cleanup
-    // if (vi.isFakeTimers()) {
-    //   console.log('[afterEach] Running pending timers...')
-    //   const timerStart = Date.now()
-    //   vi.runOnlyPendingTimers()
-    //   console.log(`[afterEach] Timers completed in ${Date.now() - timerStart}ms`)
-    // }
-    
     // Unmount wrapper to clean up watchers and event listeners
     if (wrapper) {
-      console.log('[afterEach] Unmounting wrapper...')
-      const unmountStart = Date.now()
       wrapper.unmount()
-      console.log(`[afterEach] Unmount completed in ${Date.now() - unmountStart}ms`)
     }
 
     // Flush any remaining promises
-    console.log('[afterEach] Flushing promises...')
-    const flushStart = Date.now()
     await flushPromises()
-    console.log(`[afterEach] Promises flushed in ${Date.now() - flushStart}ms`)
 
     // Additional nextTick to ensure all updates are processed
-    console.log('[afterEach] Waiting for nextTick...')
-    const nextTickStart = Date.now()
     await nextTick()
-    console.log(`[afterEach] nextTick completed in ${Date.now() - nextTickStart}ms`)
 
     // Restore mocks and timers
-    console.log('[afterEach] Restoring mocks and timers...')
-    const restoreStart = Date.now()
     vi.restoreAllMocks()
-    // vi.useRealTimers()
-    console.log(`[afterEach] Restore completed in ${Date.now() - restoreStart}ms`)
-    
-    console.log(`[afterEach] Total cleanup time: ${Date.now() - startTime}ms`)
   })
 
   it('renders correctly and calls fetchEventConfigs on mount', () => {
@@ -231,21 +204,24 @@ describe('EventConfigSourceTable.vue', () => {
     expect(rows[1].text()).toContain('Disabled')
   })
 
-  // it('handles search input changes with debouncing and calls onChangeSourcesSearchTerm', async () => {
-  //   vi.useFakeTimers()
+  // This test is skipped because it relies on debouncing which can be tricky to test reliably.
+  // Consider refactoring the search input handling to make it more testable or use a library like 
+  // @vue/test-utils' `setValue` with `flushPromises` to handle the debounce timing.
+  it.skip('handles search input changes with debouncing and calls onChangeSourcesSearchTerm', async () => {
+    vi.useFakeTimers()
 
-  //   store.sources = [mockSource]
-  //   await wrapper.vm.$nextTick()
+    store.sources = [mockSource]
+    await wrapper.vm.$nextTick()
 
-  //   const searchInput = wrapper.get('[data-test="search-input"] .feather-input')
-  //   await searchInput.setValue('test')
-  //   vi.advanceTimersByTime(500)
-  //   await wrapper.vm.$nextTick()
+    const searchInput = wrapper.get('[data-test="search-input"] .feather-input')
+    await searchInput.setValue('test')
+    vi.advanceTimersByTime(500)
+    await wrapper.vm.$nextTick()
 
-  //   expect(store.onChangeSourcesSearchTerm).toHaveBeenCalledWith('test')
+    expect(store.onChangeSourcesSearchTerm).toHaveBeenCalledWith('test')
 
-  //   vi.useRealTimers()
-  // })
+    vi.useRealTimers()
+  })
 
   it('clicks view details button in row and navigates correctly', async () => {
     store.sources = [mockSource]