fix test error

Author: khanniie

client/modules/IDE/components/Editor/codemirror.js

@@ -169,7 +169,6 @@ export default function useCodeMirror({
       [`${metaKey}-.`]: 'toggleComment' // Note: most adblockers use the shortcut ctrl+.
     });
 
-    console.log('setting up change handlers??', cmInstance.current);
     cmInstance.current.on('change', debouncedOnChange);
     cmInstance.current.on('keyup', onKeyUp);
     cmInstance.current.on('keydown', onKeyDown);

client/modules/IDE/components/Editor/index.jsx

@@ -115,13 +115,14 @@ function Editor({
   });
 
   // Lets the parent component access file content-specific functionality...
-  // TODO(connie) - Revisit the logic here, can we wrap this in useCallback or something?
-  provideController({
-    tidyCode: () => tidyCode(cmInstance.current),
-    showFind,
-    showReplace,
-    getContent
-  });
+  useEffect(() => {
+    provideController({
+      tidyCode: () => tidyCode(cmInstance.current),
+      showFind,
+      showReplace,
+      getContent
+    });
+  }, [showFind, showReplace, getContent]);
 
   // When the CM container div mounts, we set up CodeMirror.
   const onContainerMounted = useCallback(setupCodeMirrorOnContainerMounted, []);
@@ -149,11 +150,14 @@ function Editor({
     if (!autocompleteHinter) hideHinter(cmInstance.current);
   }, [autocompleteHinter]);
 
-  // TODO: Should this be watching more deps?
+  // TODO: test this
   useEffectWithComparison(
     (_, prevProps) => {
       if (runtimeErrorWarningVisible) {
-        if (consoleEvents.length !== prevProps.consoleEvents.length) {
+        if (
+          prevProps.consoleEvents &&
+          consoleEvents.length !== prevProps.consoleEvents.length
+        ) {
           consoleEvents.forEach((consoleEvent) => {
             if (consoleEvent.method === 'error') {
               // It doesn't work if you create a new Error, but this works