Skip to content

Commit ffedcb2

Browse files
committed
[AUTOMATED]: Prettier Code Styling
1 parent 82bb599 commit ffedcb2

File tree

7 files changed

+68
-52
lines changed

7 files changed

+68
-52
lines changed

packages-exp/auth-exp/src/api/authentication/token.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ describe('requestStsToken', () => {
3636
const mock = fetch.mock(endpoint, {
3737
'access_token': 'new-access-token',
3838
'expires_in': '3600',
39-
'refresh_token': 'new-refresh-token',
39+
'refresh_token': 'new-refresh-token'
4040
});
4141

4242
const response = await requestStsToken(mockAuth, 'old-refresh-token');
@@ -46,7 +46,7 @@ describe('requestStsToken', () => {
4646
const request = querystringDecode(`?${mock.calls[0].request}`);
4747
expect(request).to.eql({
4848
'grant_type': 'refresh_token',
49-
'refresh_token': 'old-refresh-token',
49+
'refresh_token': 'old-refresh-token'
5050
});
5151
expect(mock.calls[0].method).to.eq('POST');
5252
expect(mock.calls[0].headers).to.eql({
@@ -74,12 +74,12 @@ describe('requestStsToken', () => {
7474

7575
await expect(requestStsToken(mockAuth, 'old-token')).to.be.rejectedWith(
7676
FirebaseError,
77-
'Firebase: The user\'s credential is no longer valid. The user must sign in again. (auth/user-token-expired)'
77+
"Firebase: The user's credential is no longer valid. The user must sign in again. (auth/user-token-expired)"
7878
);
7979
const request = querystringDecode(`?${mock.calls[0].request}`);
8080
expect(request).to.eql({
8181
'grant_type': 'refresh_token',
82-
'refresh_token': 'old-token',
82+
'refresh_token': 'old-token'
8383
});
8484
});
8585
});

packages-exp/auth-exp/src/api/authentication/token.ts

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ const GRANT_TYPE = 'refresh_token';
2626
enum ServerField {
2727
ACCESS_TOKEN = 'access_token',
2828
EXPIRES_IN = 'expires_in',
29-
REFRESH_TOKEN = 'refresh_token',
29+
REFRESH_TOKEN = 'refresh_token'
3030
}
3131

3232
export interface RequestStsTokenResponse {
@@ -35,27 +35,32 @@ export interface RequestStsTokenResponse {
3535
refreshToken?: string;
3636
}
3737

38-
export async function requestStsToken(auth: Auth, refreshToken: string): Promise<RequestStsTokenResponse> {
39-
const response = await performFetchWithErrorHandling<{[key: string]: string}>(auth, {}, () => {
38+
export async function requestStsToken(
39+
auth: Auth,
40+
refreshToken: string
41+
): Promise<RequestStsTokenResponse> {
42+
const response = await performFetchWithErrorHandling<{
43+
[key: string]: string;
44+
}>(auth, {}, () => {
4045
const body = querystring({
4146
'grant_type': GRANT_TYPE,
42-
'refresh_token': refreshToken,
47+
'refresh_token': refreshToken
4348
}).slice(1);
4449

4550
return fetch(`${_ENDPOINT}?key=${auth.config.apiKey}`, {
4651
method: 'POST',
4752
headers: {
4853
'X-Client-Version': auth.config.sdkClientVersion,
49-
'Content-Type': 'application/x-www-form-urlencoded',
54+
'Content-Type': 'application/x-www-form-urlencoded'
5055
},
51-
body,
56+
body
5257
});
5358
});
5459

5560
// The response comes back in snake_case. Convert to camel:
5661
return {
5762
accessToken: response[ServerField.ACCESS_TOKEN],
5863
expiresIn: response[ServerField.EXPIRES_IN],
59-
refreshToken: response[ServerField.REFRESH_TOKEN],
64+
refreshToken: response[ServerField.REFRESH_TOKEN]
6065
};
61-
}
66+
}

packages-exp/auth-exp/src/api/index.ts

Lines changed: 23 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export async function _performApiRequest<T, V>(
5757
method: HttpMethod,
5858
path: Endpoint,
5959
request?: T,
60-
customErrorMap: Partial<ServerErrorMap<ServerError>> = {},
60+
customErrorMap: Partial<ServerErrorMap<ServerError>> = {}
6161
): Promise<V> {
6262
return performFetchWithErrorHandling(auth, customErrorMap, () => {
6363
let body = {};
@@ -77,31 +77,31 @@ export async function _performApiRequest<T, V>(
7777
...params
7878
}).slice(1);
7979

80-
return fetch(
81-
`${auth.config.apiScheme}://${auth.config.apiHost}${path}?${query}`,
82-
{
83-
method,
84-
headers: {
85-
'Content-Type': 'application/json',
86-
'X-Client-Version': auth.config.sdkClientVersion
87-
},
88-
referrerPolicy: 'no-referrer',
89-
...body
90-
}
91-
);
92-
});
80+
return fetch(
81+
`${auth.config.apiScheme}://${auth.config.apiHost}${path}?${query}`,
82+
{
83+
method,
84+
headers: {
85+
'Content-Type': 'application/json',
86+
'X-Client-Version': auth.config.sdkClientVersion
87+
},
88+
referrerPolicy: 'no-referrer',
89+
...body
90+
}
91+
);
92+
});
9393
}
9494

9595
export async function performFetchWithErrorHandling<V>(
9696
auth: Auth,
9797
customErrorMap: Partial<ServerErrorMap<ServerError>>,
98-
fetchFn: () => Promise<Response>,
98+
fetchFn: () => Promise<Response>
9999
): Promise<V> {
100100
const errorMap = { ...SERVER_ERROR_MAP, ...customErrorMap };
101101
try {
102102
const response: Response = await Promise.race<Promise<Response>>([
103103
fetchFn(),
104-
makeNetworkTimeout(auth.name),
104+
makeNetworkTimeout(auth.name)
105105
]);
106106
if (response.ok) {
107107
return response.json();
@@ -155,12 +155,11 @@ export async function _performSignInRequest<T, V extends IdTokenResponse>(
155155

156156
function makeNetworkTimeout<T>(appName: string): Promise<T> {
157157
return new Promise((_, reject) =>
158-
setTimeout(() => {
159-
return reject(
160-
AUTH_ERROR_FACTORY.create(AuthErrorCode.TIMEOUT, {
161-
appName,
162-
})
163-
);
164-
}, DEFAULT_API_TIMEOUT_MS.get())
165-
)
158+
setTimeout(() => {
159+
return reject(
160+
AUTH_ERROR_FACTORY.create(AuthErrorCode.TIMEOUT, {
161+
appName,
162+
})
163+
);
164+
}, DEFAULT_API_TIMEOUT_MS.get()));
166165
}

packages-exp/auth-exp/src/core/user/token_manager.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ describe('core/user/token_manager', () => {
8282
mock = fetch.mock(endpoint, {
8383
'access_token': 'new-access-token',
8484
'refresh_token': 'new-refresh-token',
85-
'expires_in': '3600',
85+
'expires_in': '3600'
8686
});
8787
});
8888

@@ -102,7 +102,7 @@ describe('core/user/token_manager', () => {
102102
expect(tokens).to.eql({
103103
accessToken: 'new-access-token',
104104
refreshToken: 'new-refresh-token',
105-
wasRefreshed: true,
105+
wasRefreshed: true
106106
});
107107
});
108108

@@ -122,7 +122,7 @@ describe('core/user/token_manager', () => {
122122
expect(tokens).to.eql({
123123
accessToken: 'new-access-token',
124124
refreshToken: 'new-refresh-token',
125-
wasRefreshed: true,
125+
wasRefreshed: true
126126
});
127127
});
128128
});
@@ -139,7 +139,7 @@ describe('core/user/token_manager', () => {
139139

140140
await expect(stsTokenManager.getToken(mockAuth)).to.be.rejectedWith(
141141
FirebaseError,
142-
'Firebase: The user\'s credential is no longer valid. The user must sign in again. (auth/user-token-expired)'
142+
"Firebase: The user's credential is no longer valid. The user must sign in again. (auth/user-token-expired)"
143143
);
144144
});
145145

@@ -154,7 +154,7 @@ describe('core/user/token_manager', () => {
154154
expect(tokens).to.eql({
155155
accessToken: 'token',
156156
refreshToken: 'refresh',
157-
wasRefreshed: false,
157+
wasRefreshed: false
158158
});
159159
});
160160
});

packages-exp/auth-exp/src/core/user/token_manager.ts

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -54,18 +54,18 @@ export class StsTokenManager {
5454
this.updateTokensAndExpiration(idToken, refreshToken, expiresInSec);
5555
}
5656

57-
async getToken(auth: Auth, forceRefresh = false): Promise<Tokens|null> {
57+
async getToken(auth: Auth, forceRefresh = false): Promise<Tokens | null> {
5858
if (!forceRefresh && this.accessToken && !this.isExpired) {
5959
return {
6060
accessToken: this.accessToken,
6161
refreshToken: this.refreshToken,
62-
wasRefreshed: false,
62+
wasRefreshed: false
6363
};
6464
}
6565

6666
if (this.accessToken && !this.refreshToken) {
6767
throw AUTH_ERROR_FACTORY.create(AuthErrorCode.TOKEN_EXPIRED, {
68-
appName: auth.name,
68+
appName: auth.name
6969
});
7070
}
7171

@@ -77,7 +77,7 @@ export class StsTokenManager {
7777
return {
7878
accessToken: this.accessToken!,
7979
refreshToken: this.refreshToken,
80-
wasRefreshed: true,
80+
wasRefreshed: true
8181
};
8282
}
8383

@@ -90,14 +90,23 @@ export class StsTokenManager {
9090
}
9191

9292
private async refresh(auth: Auth, oldToken: string): Promise<void> {
93-
const {accessToken, refreshToken, expiresIn} = await requestStsToken(auth, oldToken);
93+
const { accessToken, refreshToken, expiresIn } = await requestStsToken(
94+
auth,
95+
oldToken
96+
);
9497
this.updateTokensAndExpiration(accessToken, refreshToken, expiresIn);
9598
}
9699

97-
private updateTokensAndExpiration(accessToken: string|undefined, refreshToken: string|undefined, expiresInSec: string|undefined): void {
100+
private updateTokensAndExpiration(
101+
accessToken: string | undefined,
102+
refreshToken: string | undefined,
103+
expiresInSec: string | undefined
104+
): void {
98105
this.refreshToken = refreshToken || null;
99106
this.accessToken = accessToken || null;
100-
this.expirationTime = expiresInSec ? Date.now() + Number(expiresInSec) * 1000 : null;
107+
this.expirationTime = expiresInSec
108+
? Date.now() + Number(expiresInSec) * 1000
109+
: null;
101110
}
102111

103112
static fromPlainObject(

packages-exp/auth-exp/src/core/user/user_impl.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ export class UserImpl implements User {
7979
assert(tokens, this.auth.name);
8080

8181
// TODO: remove ! after #2934 is merged --
82-
const { refreshToken, accessToken, /* wasRefreshed */ } = tokens!;
82+
const { refreshToken, accessToken /* wasRefreshed */ } = tokens!;
8383
this.refreshToken = refreshToken || '';
8484

8585
// TODO: Uncomment after #2961 is merged

packages-exp/auth-exp/test/mock_fetch.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
import { SinonStub, stub } from 'sinon';
1919

2020
export interface Call {
21-
request?: object|string;
21+
request?: object | string;
2222
method?: string;
2323
headers?: HeadersInit;
2424
}
@@ -45,8 +45,11 @@ const fakeFetch: typeof fetch = (input: RequestInfo, request?: RequestInit) => {
4545
// Bang-assertion is fine since we check for routes.has() above
4646
const { response, status, calls } = routes.get(input)!;
4747

48-
const requestBody = request?.body && (request?.headers as any)?.['Content-Type'] === 'application/json' ?
49-
JSON.parse(request.body as string) : request?.body;
48+
const requestBody =
49+
request?.body &&
50+
(request?.headers as any)?.['Content-Type'] === 'application/json'
51+
? JSON.parse(request.body as string)
52+
: request?.body;
5053

5154
calls.push({
5255
request: requestBody,

0 commit comments

Comments
 (0)