Skip to content

Commit 5f980bf

Browse files
committed
Formatting
1 parent 84a929e commit 5f980bf

File tree

4 files changed

+35
-13
lines changed

4 files changed

+35
-13
lines changed

packages-exp/auth-exp/src/core/auth/auth_event_manager.test.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,10 @@ import * as sinon from 'sinon';
2020
import * as sinonChai from 'sinon-chai';
2121

2222
import {
23-
AuthEvent, AuthEventConsumer, AuthEventError, AuthEventType
23+
AuthEvent,
24+
AuthEventConsumer,
25+
AuthEventError,
26+
AuthEventType
2427
} from '../../model/popup_redirect';
2528
import { AuthErrorCode } from '../errors';
2629
import { AuthEventManager } from './auth_event_manager';
@@ -115,7 +118,7 @@ describe('src/core/auth/auth_event_manager', () => {
115118
event.error = {
116119
code: `auth/${AuthErrorCode.INVALID_APP_CREDENTIAL}`,
117120
message: 'foo',
118-
name: 'name',
121+
name: 'name'
119122
};
120123

121124
manager.onEvent(event);
@@ -129,7 +132,7 @@ describe('src/core/auth/auth_event_manager', () => {
129132
const event = makeEvent(AuthEventType.REAUTH_VIA_POPUP);
130133
event.error = {
131134
message: 'foo',
132-
name: 'name',
135+
name: 'name'
133136
} as AuthEventError;
134137

135138
manager.onEvent(event);

packages-exp/auth-exp/src/core/auth/auth_event_manager.ts

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,11 @@
1515
* limitations under the License.
1616
*/
1717

18-
import { AuthEvent, AuthEventConsumer, EventManager } from '../../model/popup_redirect';
18+
import {
19+
AuthEvent,
20+
AuthEventConsumer,
21+
EventManager
22+
} from '../../model/popup_redirect';
1923
import { AUTH_ERROR_FACTORY, AuthErrorCode } from '../errors';
2024

2125
export class AuthEventManager implements EventManager {
@@ -39,10 +43,14 @@ export class AuthEventManager implements EventManager {
3943
) {
4044
if (event.error) {
4145
console.error('ERROR');
42-
const code = event.error.code?.split('auth/')[1] as AuthErrorCode || AuthErrorCode.INTERNAL_ERROR;
43-
consumer.onError(AUTH_ERROR_FACTORY.create(code, {
44-
appName: this.appName
45-
}));
46+
const code =
47+
(event.error.code?.split('auth/')[1] as AuthErrorCode) ||
48+
AuthErrorCode.INTERNAL_ERROR;
49+
consumer.onError(
50+
AUTH_ERROR_FACTORY.create(code, {
51+
appName: this.appName
52+
})
53+
);
4654
} else {
4755
consumer.onAuthEvent(event);
4856
}

packages-exp/auth-exp/src/platform_browser/popup_redirect.test.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,11 @@ import { TEST_AUTH_DOMAIN, TEST_KEY, testAuth } from '../../test/mock_auth';
2828
import { AuthEventManager } from '../core/auth/auth_event_manager';
2929
import { OAuthProvider } from '../core/providers/oauth';
3030
import { Auth } from '../model/auth';
31-
import { AuthEvent, AuthEventType, GapiAuthEvent } from '../model/popup_redirect';
31+
import {
32+
AuthEvent,
33+
AuthEventType,
34+
GapiAuthEvent
35+
} from '../model/popup_redirect';
3236
import * as gapiLoader from './iframe/gapi';
3337
import { BrowserPopupRedirectResolver } from './popup_redirect';
3438

packages-exp/auth-exp/src/platform_browser/popup_redirect.ts

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,11 @@ import { _getCurrentUrl } from '../core/util/location';
2828
import { _open, AuthPopup } from '../core/util/popup';
2929
import { ApiKey, AppName, Auth } from '../model/auth';
3030
import {
31-
AuthEventType, EventManager, GapiAuthEvent, GapiOutcome, PopupRedirectResolver
31+
AuthEventType,
32+
EventManager,
33+
GapiAuthEvent,
34+
GapiOutcome,
35+
PopupRedirectResolver
3236
} from '../model/popup_redirect';
3337
import { _openIframe } from './iframe/iframe';
3438

@@ -39,7 +43,7 @@ const WIDGET_URL = '__/auth/handler';
3943

4044
export class BrowserPopupRedirectResolver implements PopupRedirectResolver {
4145
private eventManager: EventManager | null = null;
42-
private initializationPromise: Promise<EventManager>|null = null;
46+
private initializationPromise: Promise<EventManager> | null = null;
4347

4448
// Wrapping in async even though we don't await anywhere in order
4549
// to make sure errors are raised as promise rejections
@@ -49,7 +53,10 @@ export class BrowserPopupRedirectResolver implements PopupRedirectResolver {
4953
authType: AuthEventType,
5054
eventId?: string
5155
): Promise<AuthPopup> {
52-
debugAssert(this.eventManager, '_initialize() not called before _openPopup()');
56+
debugAssert(
57+
this.eventManager,
58+
'_initialize() not called before _openPopup()'
59+
);
5360
const url = getRedirectUrl(auth, provider, authType, eventId);
5461
return _open(auth.name, url, _generateEventId());
5562
}
@@ -115,7 +122,7 @@ function getRedirectUrl(
115122
redirectUrl: _getCurrentUrl(),
116123
v: SDK_VERSION,
117124
eventId,
118-
tid: auth.tenantId || undefined,
125+
tid: auth.tenantId || undefined
119126
};
120127

121128
if (provider instanceof OAuthProvider) {

0 commit comments

Comments
 (0)