More pruning.

Author: nedtwigg

browser-extension/tests/lib/enhancers/github.test.ts

@@ -7,7 +7,6 @@ import { EnhancerRegistry } from '../../../src/lib/registries'
 import { PAGES } from '../../har-index'
 
 vi.stubGlobal('defineContentScript', vi.fn())
-
 vi.mock('../../../src/overtype/overtype', () => {
   const mockConstructor = vi.fn().mockImplementation(() => [
     {
@@ -28,17 +27,12 @@ vi.mock('../../../src/overtype/overtype', () => {
 })
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-
 async function loadHtmlFromHar(key: keyof typeof PAGES): Promise<string> {
   const url = PAGES[key]
   const harPath = path.join(__dirname, '../../har', `${key}.har`)
   const harContent = await fs.readFile(harPath, 'utf-8')
   const harData = JSON.parse(harContent)
-
   const mainEntry = harData.log.entries.find((entry: any) => entry.request.url === url)
-  if (!mainEntry) {
-    throw new Error(`No HTML content found in HAR file: ${key}.har`)
-  }
   return mainEntry.response.content.text
 }
 

browser-extension/tests/setup.ts

@@ -28,25 +28,3 @@ Object.assign(globalThis, {
   Text: dom.Text,
   window: dom.window,
 })
-
-// Mock querySelector methods properly
-const originalQuerySelector = dom.document.querySelector.bind(dom.document)
-const originalQuerySelectorAll = dom.document.querySelectorAll.bind(dom.document)
-
-dom.document.querySelector = (selector: string) => {
-  try {
-    return originalQuerySelector(selector)
-  } catch (_e) {
-    return null
-  }
-}
-
-dom.document.querySelectorAll = ((selector: string) => {
-  try {
-    return originalQuerySelectorAll(selector)
-  } catch (_e) {
-    // Return an empty NodeList-like object instead of array
-    const emptyNodeList = document.createDocumentFragment().childNodes
-    return emptyNodeList as unknown as NodeListOf<Element>
-  }
-}) as typeof dom.document.querySelectorAll