fix ts errors

Author: jhuleatt

reactfire/components.test.tsx

@@ -171,7 +171,7 @@ describe('SuspenseWithPerf', () => {
       <SuspenseWithPerf
         traceId={'hello'}
         fallback={'loading'}
-        firePerf={mockPerf()}
+        firePerf={(mockPerf() as unknown) as performance.Performance}
       >
         {'children'}
       </SuspenseWithPerf>
@@ -204,16 +204,14 @@ describe('AuthCheck', () => {
       render(
         <React.Suspense fallback={'loading'}>
           <AuthCheck
-            fallback={'not signed in'}
+            fallback={<h1>not signed in</h1>}
             auth={(mockFirebase.auth() as unknown) as auth.Auth}
           >
             {'signed in'}
           </AuthCheck>
         </React.Suspense>
       )
     ).not.toThrow();
-
-    // if this doesn't throw, it's good
   });
 
   test.todo('renders the fallback if a user is not signed in');

reactfire/components.tsx

@@ -1,13 +1,13 @@
 import * as React from 'react';
-import { firebase, auth, performance, User } from 'firebase/app';
+import { auth, performance, User } from 'firebase/app';
 import { useUser, useFirebaseApp } from './index';
 const { Suspense, useState, useLayoutEffect } = React;
 
 export interface SuspensePerfProps {
   children: React.ReactNode;
   traceId: string;
   fallback: React.ReactNode;
-  firePerf?: firebase.performance.Performance; // TODO(jeff): Add firePerf here when it's available
+  firePerf?: performance.Performance; // TODO(jeff): Add firePerf here when it's available
 }
 
 function getPerfFromContext(): performance.Performance {
@@ -23,7 +23,7 @@ function getPerfFromContext(): performance.Performance {
 
   if (!perfFunc || !perfFunc()) {
     throw new Error(
-      "No auth object off of Firebase. Did you forget to import 'firebase/performance' in a component?"
+      "No perf object off of Firebase. Did you forget to import 'firebase/performance' in a component?"
     );
   }