noImplicitAny TS fixes

Author: hsubox76

config/.eslintrc.json

@@ -20,7 +20,8 @@
                 "**/test/**/*.ts"
             ],
             "rules": {
-                "no-unused-expressions": "off"
+                "no-unused-expressions": "off",
+                "@typescript-eslint/no-explicit-any": "off"
             }
         }
     ],

packages/app/index.node.ts

@@ -18,8 +18,11 @@
 import { FirebaseNamespace } from '@firebase/app-types';
 import { _FirebaseNamespace } from '@firebase/app-types/private';
 import { createFirebaseNamespace } from './src/firebaseNamespace';
-import Storage from 'dom-storage';
-import { XMLHttpRequest } from 'xmlhttprequest';
+// Node specific packages.
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const Storage = require('dom-storage');
+// eslint-disable-next-line @typescript-eslint/no-require-imports
+const XMLHttpRequest = require('xmlhttprequest');
 
 const _firebase = createFirebaseNamespace() as _FirebaseNamespace;
 

packages/app/index.rn.ts

@@ -24,6 +24,7 @@ import { createFirebaseNamespace } from './src/firebaseNamespace';
  * some of our tests because of duplicate symbols, we are using require syntax
  * here
  */
+// eslint-disable-next-line
 const { AsyncStorage } = require('react-native');
 
 const _firebase = createFirebaseNamespace() as _FirebaseNamespace;

packages/app/index.ts

@@ -29,6 +29,7 @@ if (isBrowser() && 'firebase' in self) {
     Firebase library is only loaded once.
   `);
 
+  // eslint-disable-next-line
   const sdkVersion = ((self as any).firebase as FirebaseNamespace).SDK_VERSION;
   if (sdkVersion && sdkVersion.indexOf('LITE') >= 0) {
     logger.warn(`

packages/app/package.json

@@ -18,7 +18,7 @@
     "lint:fix": "eslint --fix -c .eslintrc.json '**/*.ts'",
     "build": "rollup -c",
     "dev": "rollup -c -w",
-    "test": "run-p test:browser test:node",
+    "test": "run-p lint test:browser test:node",
     "test:browser": "karma start --single-run",
     "test:browser:debug": "karma start --browsers Chrome --auto-watch",
     "test:node": "TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --opts ../../config/mocha.node.opts",

packages/app/src/firebaseApp.ts

@@ -38,7 +38,7 @@ interface ServicesCache {
 
 // An array to capture listeners before the true auth functions
 // exist
-let tokenListeners: any[] = [];
+let tokenListeners: Array<(token: string | null) => void> = [];
 
 /**
  * Global context object for a collection of services using
@@ -175,6 +175,7 @@ export class FirebaseAppImpl implements FirebaseApp {
    * Callback function used to extend an App instance at the time
    * of service instance creation.
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   private extendApp(props: { [name: string]: any }): void {
     // Copy the object onto the FirebaseAppImpl prototype
     deepExtend(this, props);

packages/app/src/firebaseNamespace.ts

@@ -32,11 +32,11 @@ export function createFirebaseNamespace(): FirebaseNamespace {
   const namespace = createFirebaseNamespaceCore(FirebaseAppImpl);
   (namespace as _FirebaseNamespace).INTERNAL = {
     ...(namespace as _FirebaseNamespace).INTERNAL,
-    createFirebaseNamespace: createFirebaseNamespace,
-    extendNamespace: extendNamespace,
-    createSubscribe: createSubscribe,
-    ErrorFactory: ErrorFactory,
-    deepExtend: deepExtend
+    createFirebaseNamespace,
+    extendNamespace,
+    createSubscribe,
+    ErrorFactory,
+    deepExtend
   };
 
   /**

packages/app/src/firebaseNamespaceCore.ts

@@ -29,17 +29,13 @@ import {
   FirebaseServiceNamespace,
   AppHook
 } from '@firebase/app-types/private';
-import { deepExtend, patchProperty } from '@firebase/util';
+import { deepExtend, contains } from '@firebase/util';
 import { FirebaseAppImpl } from './firebaseApp';
 import { ERROR_FACTORY, AppError } from './errors';
 import { FirebaseAppLiteImpl } from './lite/firebaseAppLite';
 import { DEFAULT_ENTRY_NAME } from './constants';
 import { version } from '../../firebase/package.json';
 
-function contains(obj: object, key: string) {
-  return Object.prototype.hasOwnProperty.call(obj, key);
-}
-
 /**
  * Because auth can't share code with other components, we attach the utility functions
  * in an internal namespace to share code.
@@ -60,9 +56,11 @@ export function createFirebaseNamespaceCore(
     // as the firebase namespace.
     // @ts-ignore
     __esModule: true,
-    initializeApp: initializeApp,
-    app: app as any,
-    apps: null as any,
+    initializeApp,
+    // @ts-ignore
+    app,
+    // @ts-ignore
+    apps: null,
     SDK_VERSION: version,
     INTERNAL: {
       registerService,
@@ -82,7 +80,8 @@ export function createFirebaseNamespaceCore(
   //
   //   import * as firebase from 'firebase';
   //   which becomes: var firebase = require('firebase');
-  patchProperty(namespace, 'default', namespace);
+  // @ts-ignore
+  namespace['default'] = namespace;
 
   // firebase.apps is a read-only getter.
   Object.defineProperty(namespace, 'apps', {
@@ -105,12 +104,13 @@ export function createFirebaseNamespaceCore(
   function app(name?: string): FirebaseApp {
     name = name || DEFAULT_ENTRY_NAME;
     if (!contains(apps, name)) {
-      throw ERROR_FACTORY.create(AppError.NO_APP, { name: name });
+      throw ERROR_FACTORY.create(AppError.NO_APP, { name });
     }
     return apps[name];
   }
 
-  patchProperty(app, 'App', firebaseAppImpl);
+  // @ts-ignore
+  app['App'] = firebaseAppImpl;
   /**
    * Create a new App instance (name must be unique).
    */
@@ -119,7 +119,10 @@ export function createFirebaseNamespaceCore(
     config?: FirebaseAppConfig
   ): FirebaseApp;
   function initializeApp(options: FirebaseOptions, name?: string): FirebaseApp;
-  function initializeApp(options: FirebaseOptions, rawConfig = {}) {
+  function initializeApp(
+    options: FirebaseOptions,
+    rawConfig = {}
+  ): FirebaseApp {
     if (typeof rawConfig !== 'object' || rawConfig === null) {
       const name = rawConfig;
       rawConfig = { name };
@@ -138,7 +141,7 @@ export function createFirebaseNamespaceCore(
     }
 
     if (contains(apps, name)) {
-      throw ERROR_FACTORY.create(AppError.DUPLICATE_APP, { name: name });
+      throw ERROR_FACTORY.create(AppError.DUPLICATE_APP, { name });
     }
 
     const app = new firebaseAppImpl(
@@ -177,7 +180,7 @@ export function createFirebaseNamespaceCore(
   ): FirebaseServiceNamespace<FirebaseService> {
     // Cannot re-register a service that already exists
     if (factories[name]) {
-      throw ERROR_FACTORY.create(AppError.DUPLICATE_SERVICE, { name: name });
+      throw ERROR_FACTORY.create(AppError.DUPLICATE_SERVICE, { name });
     }
 
     // Capture the service factory for later service instantiation
@@ -194,16 +197,18 @@ export function createFirebaseNamespaceCore(
     }
 
     // The Service namespace is an accessor function ...
-    function serviceNamespace(appArg: FirebaseApp = app()) {
+    function serviceNamespace(appArg: FirebaseApp = app()): FirebaseService {
+      // @ts-ignore
       if (typeof appArg[name] !== 'function') {
         // Invalid argument.
         // This happens in the following case: firebase.storage('gs:/')
         throw ERROR_FACTORY.create(AppError.INVALID_APP_ARGUMENT, {
-          name: name
+          name
         });
       }
 
       // Forward service instance lookup to the FirebaseApp.
+      // @ts-ignore
       return appArg[name]();
     }
 
@@ -213,9 +218,11 @@ export function createFirebaseNamespaceCore(
     }
 
     // Monkey-patch the serviceNamespace onto the firebase namespace
+    // @ts-ignore
     namespace[name] = serviceNamespace;
 
     // Patch the FirebaseAppImpl prototype
+    // @ts-ignore
     firebaseAppImpl.prototype[name] = function(...args) {
       const serviceFxn = this._getService.bind(this, name);
       return serviceFxn.apply(this, allowMultipleInstances ? args : []);
@@ -224,7 +231,7 @@ export function createFirebaseNamespaceCore(
     return serviceNamespace;
   }
 
-  function callAppHooks(app: FirebaseApp, eventName: string) {
+  function callAppHooks(app: FirebaseApp, eventName: string): void {
     for (const serviceName of Object.keys(factories)) {
       // Ignore virtual services
       const factoryName = useAsService(app, serviceName);

packages/app/src/lite/firebaseAppLite.ts

@@ -157,7 +157,7 @@ export class FirebaseAppLiteImpl implements FirebaseApp {
    * Callback function used to extend an App instance at the time
    * of service instance creation.
    */
-  private extendApp(props: { [name: string]: any }): void {
+  private extendApp(props: { [name: string]: unknown }): void {
     // Copy the object onto the FirebaseAppImpl prototype
     deepExtend(this, props);
   }

packages/app/src/lite/firebaseNamespaceLite.ts

@@ -20,7 +20,9 @@ import {
   _FirebaseApp,
   _FirebaseNamespace,
   FirebaseServiceFactory,
-  AppHook
+  AppHook,
+  FirebaseServiceNamespace,
+  FirebaseService
 } from '@firebase/app-types/private';
 import { FirebaseAppLiteImpl } from './firebaseAppLite';
 import { createFirebaseNamespaceCore } from '../firebaseNamespaceCore';
@@ -41,10 +43,10 @@ export function createFirebaseNamespaceLite(): FirebaseNamespace {
   function registerServiceForLite(
     name: string,
     createService: FirebaseServiceFactory,
-    serviceProperties?: { [prop: string]: any },
+    serviceProperties?: { [prop: string]: unknown },
     appHook?: AppHook,
     allowMultipleInstances?: boolean
-  ) {
+  ): FirebaseServiceNamespace<FirebaseService> {
     // only allow performance to register with firebase lite
     if (name !== 'performance' && name !== 'installations') {
       throw Error(`${name} cannot register with the standalone perf instance`);

packages/app/test/firebaseApp.test.ts

@@ -28,7 +28,7 @@ import { assert } from 'chai';
 executeFirebaseTests();
 executeFirebaseLiteTests();
 
-function executeFirebaseTests() {
+function executeFirebaseTests(): void {
   firebaseAppTests('Firebase App Tests', createFirebaseNamespace);
 
   describe('Firebase Service Registration', () => {
@@ -40,24 +40,24 @@ function executeFirebaseTests() {
     it('Register App Hook', done => {
       const events = ['create', 'delete'];
       let hookEvents = 0;
-      let app: FirebaseApp;
       (firebase as _FirebaseNamespace).INTERNAL.registerService(
         'test',
         (app: FirebaseApp) => {
           return new TestService(app);
         },
         undefined,
-        (event: string, app: FirebaseApp) => {
+        (event: string, _app: FirebaseApp) => {
           assert.equal(event, events[hookEvents]);
           hookEvents += 1;
           if (hookEvents === events.length) {
             done();
           }
         }
       );
-      app = firebase.initializeApp({});
+      const app = firebase.initializeApp({});
       // Ensure the hook is called synchronously
       assert.equal(hookEvents, 1);
+      // tslint:disable-next-line:no-floating-promises
       app.delete();
     });
 
@@ -140,7 +140,7 @@ function executeFirebaseTests() {
           return new TestService(app);
         },
         undefined,
-        (event: string, app: FirebaseApp) => {
+        (event: string, _app: FirebaseApp) => {
           assert.equal(event, events[hookEvents]);
           hookEvents += 1;
           if (hookEvents === events.length) {
@@ -150,6 +150,7 @@ function executeFirebaseTests() {
       );
       // Ensure the hook is called synchronously
       assert.equal(hookEvents, 1);
+      // tslint:disable-next-line:no-floating-promises
       app.delete();
     });
 
@@ -221,6 +222,7 @@ function executeFirebaseTests() {
       );
       assert.strictEqual(service, service2);
     });
+
     it(`Should pass null to the factory method if using default instance`, () => {
       // Register Multi Instance Service
       (firebase as _FirebaseNamespace).INTERNAL.registerService(
@@ -235,11 +237,8 @@ function executeFirebaseTests() {
         }
       );
       firebase.initializeApp({});
-
-      // Capture a given service ref
-      const serviceIdentifier = 'custom instance identifier';
-      const service = (firebase.app() as any).testService();
     });
+
     it(`Should extend INTERNAL per app instance`, () => {
       let counter: number = 0;
       (firebase as _FirebaseNamespace).INTERNAL.registerService(
@@ -280,7 +279,7 @@ function executeFirebaseTests() {
   });
 }
 
-function executeFirebaseLiteTests() {
+function executeFirebaseLiteTests(): void {
   firebaseAppTests('Firebase App Lite Tests', createFirebaseNamespaceLite);
 
   describe('Firebase Lite Service Registration', () => {
@@ -291,14 +290,13 @@ function executeFirebaseLiteTests() {
     });
 
     it('should allow Performance service to register', () => {
-      let app: FirebaseApp;
       (firebase as _FirebaseNamespace).INTERNAL.registerService(
         'performance',
         (app: FirebaseApp) => {
           return new TestService(app);
         }
       );
-      app = firebase.initializeApp({});
+      const app = firebase.initializeApp({});
       const perf = (app as any).performance();
       assert.isTrue(perf instanceof TestService);
     });
@@ -316,7 +314,7 @@ function executeFirebaseLiteTests() {
   });
 }
 
-function firebaseAppTests(testName, firebaseNamespaceFactory) {
+function firebaseAppTests(testName: string, firebaseNamespaceFactory: () => FirebaseNamespace): void {
   describe(testName, () => {
     let firebase: FirebaseNamespace;
 

packages/database/index.node.ts

@@ -40,7 +40,7 @@ import * as types from '@firebase/database-types';
 
 const ServerValue = Database.ServerValue;
 
-export function initStandalone(app, url, version?: string) {
+export function initStandalone(app: FirebaseApp, url: string, version?: string) {
   /**
    * This should allow the firebase-admin package to provide a custom version
    * to the backend

packages/database/src/core/PersistentConnection.ts

@@ -948,7 +948,7 @@ export class PersistentConnection extends ServerActions {
 
     // Puts depend on having received the corresponding data update from the server before they complete, so we must
     // make sure to send listens before puts.
-    forEach(this.listens_, (pathString: string, queries: Object) => {
+    forEach(this.listens_, (pathString: string, queries: { [key: string]: ListenSpec }) => {
       forEach(queries, (key: string, listenSpec: ListenSpec) => {
         this.sendListen_(listenSpec);
       });

packages/database/src/core/snap/ChildrenNode.ts

@@ -201,7 +201,7 @@ export class ChildrenNode implements Node {
   val(exportFormat?: boolean): object {
     if (this.isEmpty()) return null;
 
-    const obj: { [k: string]: Object } = {};
+    const obj: { [k: string]: unknown } = {};
     let numKeys = 0,
       maxKey = 0,
       allIntegerKeys = true;
@@ -218,7 +218,7 @@ export class ChildrenNode implements Node {
 
     if (!exportFormat && allIntegerKeys && maxKey < 2 * numKeys) {
       // convert to array.
-      const array: Object[] = [];
+      const array: unknown[] = [];
       for (let key in obj) array[(key as any) as number] = obj[key];
 
       return array;