Fix tests

Author: hsubox76

packages/storage/compat/reference.ts

@@ -47,7 +47,6 @@ import * as errorsExports from '../src/implementation/error';
 export class ReferenceCompat implements types.Reference {
   constructor(private readonly delegate: Reference) {}
 
-  root = new ReferenceCompat(this.delegate.root) as types.Reference;
   storage = new StorageServiceCompat(
     this.delegate.storage,
     ref => new ReferenceCompat(ref)
@@ -72,6 +71,10 @@ export class ReferenceCompat implements types.Reference {
     return new ReferenceCompat(reference);
   }
 
+  get root(): types.Reference {
+    return new ReferenceCompat(this.delegate.root) as types.Reference;
+  }
+
   /**
    * @return A reference to the parent of the
    *     current object, or null if the current object is the root.

packages/storage/compat/service.ts

@@ -60,6 +60,7 @@ export class StorageServiceCompat implements types.FirebaseStorage {
   app = this.delegate.app;
   maxOperationRetryTime = this.delegate.maxOperationRetryTime;
   maxUploadRetryTime = this.delegate.maxUploadRetryTime;
+  INTERNAL = new ServiceInternals(this);
 
   /**
    * Returns a firebaseStorage.Reference for the given path in the default
@@ -106,17 +107,17 @@ export class StorageServiceCompat implements types.FirebaseStorage {
  * @struct
  */
 export class ServiceInternals {
-  service_: StorageService;
+  service_: StorageServiceCompat;
 
-  constructor(service: StorageService) {
+  constructor(service: StorageServiceCompat) {
     this.service_ = service;
   }
 
   /**
    * Called when the associated app is deleted.
    */
   delete(): Promise<void> {
-    this.service_.deleteApp();
+    this.service_.delegate.deleteApp();
     return Promise.resolve();
   }
 }

packages/storage/exp/index.ts

@@ -39,8 +39,7 @@ export {
   list,
   listAll,
   getDownloadURL,
-  deleteObject,
-  getParent as parentReference
+  deleteObject
 } from '../src/reference';
 
 /**

packages/storage/src/reference.ts

@@ -107,6 +107,10 @@ export class Reference {
     return this.service;
   }
 
+  get parent(): Reference | null {
+    return getParent(this);
+  }
+
   throwIfRoot_(name: string): void {
     if (this.location.path === '') {
       throw errorsExports.invalidRootOperation(name);

packages/storage/test/integration/integration.test.ts

@@ -23,7 +23,7 @@ import '@firebase/auth';
 import * as storage from '@firebase/storage-types';
 
 import { expect } from 'chai';
-import '../../index';
+import '../../compat/index';
 
 // eslint-disable-next-line @typescript-eslint/no-require-imports
 const PROJECT_CONFIG = require('../../../../config/project.json');

packages/storage/test/unit/reference.test.ts renamed to packages/storage/test/unit/reference.compat.test.ts

@@ -19,30 +19,37 @@ import { FirebaseApp } from '@firebase/app-types';
 import { StringFormat } from '../../src/implementation/string';
 import { Headers } from '../../src/implementation/xhrio';
 import { Metadata } from '../../src/metadata';
-import { Reference } from '../../src/reference';
-import { StorageService } from '../../src/service';
+import { ReferenceCompat } from '../../compat/reference';
+import { StorageServiceCompat } from '../../compat/service';
 import * as testShared from './testshared';
 import { SendHook, TestingXhrIo } from './xhrio';
 import { DEFAULT_HOST } from '../../src/implementation/constants';
 import { FirebaseAuthInternalName } from '@firebase/auth-interop-types';
 import { Provider } from '@firebase/component';
+import { StorageService } from '../../src/service';
+import { Reference } from '../../src/reference';
 
 /* eslint-disable @typescript-eslint/no-floating-promises */
 function makeFakeService(
   app: FirebaseApp,
   authProvider: Provider<FirebaseAuthInternalName>,
   sendHook: SendHook
-): StorageService {
-  return new StorageService(app, authProvider, testShared.makePool(sendHook));
+): StorageServiceCompat {
+  return new StorageServiceCompat(
+    new StorageService(app, authProvider, testShared.makePool(sendHook)),
+    ref => new ReferenceCompat(ref)
+  );
 }
 
-function makeStorage(url: string): Reference {
+function makeStorage(url: string): ReferenceCompat {
+  console.log('makeStorage');
   const service = new StorageService(
     {} as FirebaseApp,
     testShared.emptyAuthProvider,
     testShared.makePool(null)
   );
-  return new Reference(service, url);
+  console.log('created service');
+  return new ReferenceCompat(new Reference(service, url));
 }
 
 describe('Firebase Storage > Reference', () => {