Fix ErrorParams

Author: hsubox76

common/api-review/app-exp.api.md

@@ -5,9 +5,9 @@
 ```ts
 
 import { Component } from '@firebase/component';
-import { FirebaseApp } from '@firebase/app-types';
-import { FirebaseAppConfig } from '@firebase/app-types';
-import { FirebaseOptions } from '@firebase/app-types';
+import { FirebaseApp } from '@firebase/app-types-exp';
+import { FirebaseAppConfig } from '@firebase/app-types-exp';
+import { FirebaseOptions } from '@firebase/app-types-exp';
 import { LogCallback } from '@firebase/logger';
 import { LogLevelString } from '@firebase/logger';
 import { LogOptions } from '@firebase/logger';

packages-exp/app-exp/src/api.ts

@@ -284,10 +284,7 @@ export function onLog(
   options?: LogOptions
 ): void {
   if (logCallback !== null && typeof logCallback !== 'function') {
-    throw ERROR_FACTORY.create(AppError.INVALID_LOG_ARGUMENT, {
-      // appName isn't used in INVALID_LOG_ARGUMENT error but ErrorFactory requires it
-      appName: ''
-    });
+    throw ERROR_FACTORY.create(AppError.INVALID_LOG_ARGUMENT);
   }
   setUserLogHandler(logCallback, options);
 }

packages-exp/app-exp/src/errors.ts

@@ -40,7 +40,13 @@ const ERRORS: ErrorMap<AppError> = {
     'First argument to `onLog` must be null or a function.'
 };
 
-type ErrorParams = { [key in AppError]: { appName: string } };
+interface ErrorParams {
+  [AppError.NO_APP]: { appName: string };
+  [AppError.BAD_APP_NAME]: { appName: string };
+  [AppError.DUPLICATE_APP]: { appName: string };
+  [AppError.APP_DELETED]: { appName: string };
+  [AppError.INVALID_APP_ARGUMENT]: { appName: string };
+}
 
 export const ERROR_FACTORY = new ErrorFactory<AppError, ErrorParams>(
   'app',

packages/app/src/errors.ts

@@ -40,7 +40,13 @@ const ERRORS: ErrorMap<AppError> = {
     'First argument to `onLog` must be null or a function.'
 };
 
-type ErrorParams = { [key in AppError]: { appName: string } };
+interface ErrorParams {
+  [AppError.NO_APP]: { appName: string };
+  [AppError.BAD_APP_NAME]: { appName: string };
+  [AppError.DUPLICATE_APP]: { appName: string };
+  [AppError.APP_DELETED]: { appName: string };
+  [AppError.INVALID_APP_ARGUMENT]: { appName: string };
+}
 
 export const ERROR_FACTORY = new ErrorFactory<AppError, ErrorParams>(
   'app',

packages/app/src/firebaseNamespaceCore.ts

@@ -287,10 +287,7 @@ export function createFirebaseNamespaceCore(
 
   function onLog(logCallback: LogCallback | null, options?: LogOptions): void {
     if (logCallback !== null && typeof logCallback !== 'function') {
-      throw ERROR_FACTORY.create(AppError.INVALID_LOG_ARGUMENT, {
-        // appName isn't used in INVALID_LOG_ARGUMENT error but ErrorFactory requires it
-        appName: ''
-      });
+      throw ERROR_FACTORY.create(AppError.INVALID_LOG_ARGUMENT);
     }
     setUserLogHandler(logCallback, options);
   }