Skip to content

Commit 7ae3d1d

Browse files
committed
Make fixes to ListenerType
1 parent 5e46878 commit 7ae3d1d

File tree

6 files changed

+19
-20
lines changed

6 files changed

+19
-20
lines changed

packages/app-check/src/api.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ export function onTokenChanged(
165165
addTokenListener(
166166
app,
167167
platformLoggerProvider,
168-
ListenerType['3P'],
168+
ListenerType.EXTERNAL,
169169
nextFn,
170170
errorFn
171171
);

packages/app-check/src/factory.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ export function internalFactory(
9595
addTokenListener(
9696
app,
9797
platformLoggerProvider,
98-
ListenerType['2P'],
98+
ListenerType.INTERNAL,
9999
listener
100100
),
101101
removeTokenListener: listener => removeTokenListener(app, listener)

packages/app-check/src/internal-api.test.ts

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -152,13 +152,13 @@ describe('internal api', () => {
152152
addTokenListener(
153153
app,
154154
fakePlatformLoggingProvider,
155-
ListenerType['2P'],
155+
ListenerType.INTERNAL,
156156
listener1
157157
);
158158
addTokenListener(
159159
app,
160160
fakePlatformLoggingProvider,
161-
ListenerType['2P'],
161+
ListenerType.INTERNAL,
162162
listener2
163163
);
164164

@@ -183,13 +183,13 @@ describe('internal api', () => {
183183
addTokenListener(
184184
app,
185185
fakePlatformLoggingProvider,
186-
ListenerType['2P'],
186+
ListenerType.INTERNAL,
187187
listener1
188188
);
189189
addTokenListener(
190190
app,
191191
fakePlatformLoggingProvider,
192-
ListenerType['2P'],
192+
ListenerType.INTERNAL,
193193
listener2
194194
);
195195

@@ -215,7 +215,7 @@ describe('internal api', () => {
215215
addTokenListener(
216216
app,
217217
fakePlatformLoggingProvider,
218-
ListenerType['3P'],
218+
ListenerType.EXTERNAL,
219219
listener1,
220220
errorFn1
221221
);
@@ -238,14 +238,14 @@ describe('internal api', () => {
238238
addTokenListener(
239239
app,
240240
fakePlatformLoggingProvider,
241-
ListenerType['2P'],
241+
ListenerType.INTERNAL,
242242
listener1,
243243
errorFn1
244244
);
245245
addTokenListener(
246246
app,
247247
fakePlatformLoggingProvider,
248-
ListenerType['2P'],
248+
ListenerType.INTERNAL,
249249
listener2
250250
);
251251

@@ -344,7 +344,7 @@ describe('internal api', () => {
344344
addTokenListener(
345345
app,
346346
fakePlatformLoggingProvider,
347-
ListenerType['2P'],
347+
ListenerType.INTERNAL,
348348
listener
349349
);
350350

@@ -361,7 +361,7 @@ describe('internal api', () => {
361361
addTokenListener(
362362
app,
363363
fakePlatformLoggingProvider,
364-
ListenerType['2P'],
364+
ListenerType.INTERNAL,
365365
listener
366366
);
367367

@@ -386,7 +386,7 @@ describe('internal api', () => {
386386
addTokenListener(
387387
app,
388388
fakePlatformLoggingProvider,
389-
ListenerType['2P'],
389+
ListenerType.INTERNAL,
390390
listener
391391
);
392392
await clock.runAllAsync();
@@ -416,7 +416,7 @@ describe('internal api', () => {
416416
addTokenListener(
417417
app,
418418
fakePlatformLoggingProvider,
419-
ListenerType['2P'],
419+
ListenerType.INTERNAL,
420420
fakeListener
421421
);
422422
});
@@ -435,7 +435,7 @@ describe('internal api', () => {
435435
addTokenListener(
436436
app,
437437
fakePlatformLoggingProvider,
438-
ListenerType['2P'],
438+
ListenerType.INTERNAL,
439439
listener
440440
);
441441
expect(getState(app).tokenObservers.length).to.equal(1);
@@ -452,7 +452,7 @@ describe('internal api', () => {
452452
addTokenListener(
453453
app,
454454
fakePlatformLoggingProvider,
455-
ListenerType['2P'],
455+
ListenerType.INTERNAL,
456456
listener
457457
);
458458
expect(getState(app).tokenObservers.length).to.equal(1);

packages/app-check/src/internal-api.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ function notifyTokenListeners(
307307

308308
for (const observer of observers) {
309309
try {
310-
if (observer.type === ListenerType['3P'] && token.error != null) {
310+
if (observer.type === ListenerType.EXTERNAL && token.error != null) {
311311
// If this listener was added by a 3P call, send any token error to
312312
// the supplied error handler. A 3P observer always has an error
313313
// handler.

packages/app-check/src/state.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ export interface AppCheckTokenObserver
3838
}
3939

4040
export const enum ListenerType {
41-
'2P' = '2P',
42-
'3P' = '3P'
41+
'INTERNAL' = 'INTERNAL',
42+
'EXTERNAL' = 'EXTERNAL'
4343
}
4444

4545
export interface AppCheckState {

packages/app-check/tsconfig.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
{
22
"extends": "../../config/tsconfig.base.json",
33
"compilerOptions": {
4-
"outDir": "dist",
5-
"preserveConstEnums": true
4+
"outDir": "dist"
65
},
76
"exclude": [
87
"dist/**/*"

0 commit comments

Comments
 (0)