Fix tests

Author: tonybaroneee

packages/telemetry/src/api.test.ts

@@ -42,7 +42,6 @@ import { _FirebaseInstallationsInternal } from '@firebase/installations';
 const PROJECT_ID = 'my-project';
 const APP_ID = 'my-appid';
 const API_KEY = 'my-api-key';
-const FID = 'fid-1234';
 
 const emittedLogs: LogRecord[] = [];
 
@@ -70,8 +69,7 @@ const fakeTelemetry: Telemetry = {
       appId: APP_ID
     }
   },
-  loggerProvider: fakeLoggerProvider,
-  fid: FID
+  loggerProvider: fakeLoggerProvider
 };
 
 describe('Top level API', () => {
@@ -127,7 +125,6 @@ describe('Top level API', () => {
       expect(log.severityNumber).to.equal(SeverityNumber.ERROR);
       expect(log.body).to.equal('This is a test error');
       expect(log.attributes).to.deep.equal({
-        'user.id': FID,
         'error.type': 'TestError',
         'error.stack': '...stack trace...'
       });
@@ -144,7 +141,6 @@ describe('Top level API', () => {
       expect(log.severityNumber).to.equal(SeverityNumber.ERROR);
       expect(log.body).to.equal('error with no stack');
       expect(log.attributes).to.deep.equal({
-        'user.id': FID,
         'error.type': 'Error',
         'error.stack': 'No stack trace available'
       });
@@ -157,9 +153,7 @@ describe('Top level API', () => {
       const log = emittedLogs[0];
       expect(log.severityNumber).to.equal(SeverityNumber.ERROR);
       expect(log.body).to.equal('a string error');
-      expect(log.attributes).to.deep.equal({
-        'user.id': 'fid-1234'
-      });
+      expect(log.attributes).to.deep.equal({});
     });
 
     it('should capture an unknown error type correctly', () => {
@@ -169,9 +163,7 @@ describe('Top level API', () => {
       const log = emittedLogs[0];
       expect(log.severityNumber).to.equal(SeverityNumber.ERROR);
       expect(log.body).to.equal('Unknown error type: number');
-      expect(log.attributes).to.deep.equal({
-        'user.id': 'fid-1234'
-      });
+      expect(log.attributes).to.deep.equal({});
     });
 
     it('should propagate trace context', async () => {
@@ -195,7 +187,6 @@ describe('Top level API', () => {
       await provider.shutdown();
 
       expect(emittedLogs[0].attributes).to.deep.equal({
-        'user.id': FID,
         'error.type': 'TestError',
         'error.stack': '...stack trace...',
         'logging.googleapis.com/trace': `projects/${PROJECT_ID}/traces/my-trace`,
@@ -220,7 +211,6 @@ describe('Top level API', () => {
       expect(emittedLogs.length).to.equal(1);
       const log = emittedLogs[0];
       expect(log.attributes).to.deep.equal({
-        'user.id': FID,
         'error.type': 'TestError',
         'error.stack': '...stack trace...',
         strAttr: 'string attribute',
@@ -254,7 +244,7 @@ function getFakeApp(): FirebaseApp {
       'installations-internal',
       () =>
         ({
-          getId: async () => 'FID',
+          getId: async () => 'iid',
           getToken: async () => 'authToken'
         } as _FirebaseInstallationsInternal),
       ComponentType.PUBLIC

packages/telemetry/src/logging/logger-provider.ts

@@ -33,7 +33,6 @@ import { FetchTransport } from './fetch-transport';
 import { DynamicHeaderProvider, DynamicLogAttributeProvider } from '../types';
 import { FirebaseApp } from '@firebase/app';
 import { ExportResult } from '@opentelemetry/core';
-import { _FirebaseInstallationsInternal } from '@firebase/installations';
 
 /**
  * Create a logger provider for the current execution environment.