Unit test fix

Author: schmidt-sebastian

packages/firestore/src/core/target.ts

@@ -38,7 +38,8 @@ export interface Target {
   readonly endAt: Bound | null;
 }
 
-class TargetImpl implements Target {
+// Visible for testing
+export class TargetImpl implements Target {
   memoizedCanonicalId: string | null = null;
   constructor(
     readonly path: ResourcePath,

packages/firestore/test/unit/local/target_cache.test.ts

@@ -34,7 +34,12 @@ import {
 import { Timestamp } from '../../../src/api/timestamp';
 import * as persistenceHelpers from './persistence_test_helpers';
 import { TestTargetCache } from './test_target_cache';
-import { canonifyTarget, Target, targetEquals } from '../../../src/core/target';
+import {
+  canonifyTarget,
+  Target,
+  targetEquals,
+  TargetImpl
+} from '../../../src/core/target';
 
 describe('MemoryTargetCache', () => {
   genericTargetCacheTests(persistenceHelpers.testMemoryEagerPersistence);
@@ -106,7 +111,7 @@ describe('IndexedDbTargetCache', () => {
 function genericTargetCacheTests(
   persistencePromise: () => Promise<Persistence>
 ): void {
-  addEqualityMatcher({ equalsFn: targetEquals, forType: Target });
+  addEqualityMatcher({ equalsFn: targetEquals, forType: TargetImpl });
   let cache: TestTargetCache;
 
   const QUERY_ROOMS = Query.atPath(path('rooms')).toTarget();

packages/firestore/test/unit/specs/describe_spec.ts

@@ -26,7 +26,7 @@ import { SpecBuilder } from './spec_builder';
 import { SpecStep } from './spec_test_runner';
 
 import * as stringify from 'json-stable-stringify';
-import { Target, targetEquals } from '../../../src/core/target';
+import { Target, targetEquals, TargetImpl } from '../../../src/core/target';
 
 // Disables all other tests; useful for debugging. Multiple tests can have
 // this tag and they'll all be run (but all others won't).
@@ -240,7 +240,7 @@ export function describeSpec(
 
   if (!writeJSONFile) {
     describe(name, () => {
-      addEqualityMatcher({ equalsFn: targetEquals, forType: Target });
+      addEqualityMatcher({ equalsFn: targetEquals, forType: TargetImpl });
       return builder();
     });
   } else {