fix: test issues

Author: ryanseddon

src/Frame.tsx

@@ -58,7 +58,6 @@ function Frame({
   const [iframeLoaded, setIframeLoaded] = useState(false);
   const internalRef = useRef<HTMLIFrameElement | null>(null);
   const nodeRef = externalRef || internalRef;
-  const isMounted = useRef(false);
   const loadCheckInterval = useRef<ReturnType<typeof setInterval> | null>(null);
 
   const getDoc = useCallback((): Document | null => {
@@ -85,8 +84,6 @@ function Frame({
   }, [iframeLoaded]);
 
   useEffect(() => {
-    isMounted.current = true;
-
     const doc = getDoc();
     const frame = nodeRef.current;
     const interval = loadCheckInterval.current;
@@ -96,8 +93,6 @@ function Frame({
     }
 
     return () => {
-      isMounted.current = false;
-
       if (frame?.contentWindow) {
         frame.contentWindow.removeEventListener('DOMContentLoaded', handleLoad);
       }
@@ -106,15 +101,11 @@ function Frame({
         clearInterval(interval);
       }
     };
-    // nodeRef is stable (either internalRef from useRef or external ref from props)
+    // nodeRef is a ref and should not be in dependencies per React best practices
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [getDoc, handleLoad]);
 
   const renderFrameContents = (): ReactNode => {
-    if (!isMounted.current) {
-      return null;
-    }
-
     const doc = getDoc();
 
     if (!doc) {

test/Context.spec.jsx

@@ -1,11 +1,8 @@
 import React from 'react';
 import { render } from '@testing-library/react';
 import { expect, describe, it } from 'vitest';
-import {
-  FrameContextProvider,
-  FrameContextConsumer,
-  useFrame
-} from '../src/Context';
+import { FrameContextProvider, FrameContextConsumer } from '../src/Context';
+import { useFrame } from '../src/useFrame';
 
 describe('The DocumentContext Component', () => {
   it('will establish context variables', async () => {

test/Frame.spec.jsx

@@ -2,7 +2,7 @@ import React from 'react';
 import { render, waitFor } from '@testing-library/react';
 import { expect, vi, describe, it, afterEach, beforeEach } from 'vitest';
 import ForwardedRefFrame, { Frame } from '../src/Frame';
-import { FrameContext } from '../src/Context';
+import { FrameContext } from '../src/FrameContext';
 
 describe('The Frame Component', () => {
   let div;