Fix mocking the locaiton.

Author: nedtwigg

browser-extension/src/entrypoints/content.ts

@@ -10,10 +10,15 @@ const enhancedTextareas = new TextareaRegistry()
 ;(window as any).gitcassoTextareaRegistry = enhancedTextareas
 
 function detectLocation(): StrippedLocation {
-  return {
+  if ((window as any).gitcassoMockLocation) {
+    return (window as any).gitcassoMockLocation
+  }
+  const result = {
     host: window.location.host,
     pathname: window.location.pathname,
   }
+  logger.debug('[gitcasso] detectLocation called, returning:', result)
+  return result
 }
 
 function sendEventToBackground(type: 'ENHANCED' | 'DESTROYED', spot: CommentSpot): void {
@@ -33,6 +38,7 @@ enhancedTextareas.setEventHandlers(
 
 export default defineContentScript({
   main() {
+    logger.debug('Main was called')
     const textAreasOnPageLoad = document.querySelectorAll<HTMLTextAreaElement>(`textarea`)
     for (const textarea of textAreasOnPageLoad) {
       enhanceMaybe(textarea)
@@ -42,7 +48,7 @@ export default defineContentScript({
       childList: true,
       subtree: true,
     })
-    logger.debug('Extension loaded with', enhancers.getEnhancerCount, 'handlers')
+    logger.debug('Extension loaded with', enhancers.getEnhancerCount(), 'handlers')
   },
   matches: ['<all_urls>'],
   runAt: 'document_end',
@@ -87,6 +93,7 @@ function handleMutations(mutations: MutationRecord[]): void {
 }
 
 function enhanceMaybe(textarea: HTMLTextAreaElement) {
+  logger.debug('[gitcasso] enhanceMaybe called for textarea:', textarea.id, textarea.className)
   if (enhancedTextareas.get(textarea)) {
     logger.debug('textarea already registered {}', textarea)
     return
@@ -96,7 +103,9 @@ function enhanceMaybe(textarea: HTMLTextAreaElement) {
   injectStyles()
 
   try {
-    const enhancedTextarea = enhancers.tryToEnhance(textarea, detectLocation())
+    const location = detectLocation()
+    logger.debug('[gitcasso] Calling tryToEnhance with location:', location)
+    const enhancedTextarea = enhancers.tryToEnhance(textarea, location)
     if (enhancedTextarea) {
       logger.debug(
         'Identified textarea:',

browser-extension/tests/har-view.ts

@@ -328,17 +328,10 @@ function injectGitcassoScript(key: keyof typeof PAGES, html: string) {
                 'throw new Error("This script should only be loaded in a browser extension.")',
                 'console.warn("Webextension-polyfill check bypassed for HAR testing")'
               );
-
-              // Override detectLocation function with correct domain and pathname
-              patchedCode = patchedCode.replace(
-                /function detectLocation(): StrippedLocation {[sS]*?}/,
-                'function detectLocation() {\\n' +
-                '  return {\\n' +
-                '    host: \\'${urlParts.host}\\',\\n' +
-                '    pathname: \\'${urlParts.pathname}\\'\\n' +
-                '  };\\n' +
-                '}'
-              );
+              window.gitcassoMockLocation = {
+                host: '${urlParts.host}',
+                pathname: '${urlParts.pathname}'
+              };
 
               // Execute the patched script with browser API mocks prepended
               const browserMocks = 'window.chrome=window.chrome||{runtime:{getURL:path=>"chrome-extension://gitcasso-test/"+path,onMessage:{addListener:()=>{}},sendMessage:()=>Promise.resolve(),id:"gitcasso-test"}};window.browser=window.chrome;';