Skip to content

Commit 9b95f30

Browse files
authored
Revert "Remove request body for deleteTenant (#1452)"
This reverts commit 475927c.
1 parent 475927c commit 9b95f30

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

src/auth/auth-api-request.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1827,14 +1827,12 @@ export abstract class AbstractAuthRequestHandler {
18271827
*/
18281828
protected invokeRequestHandler(
18291829
urlBuilder: AuthResourceUrlBuilder, apiSettings: ApiSettings,
1830-
requestData: object | undefined, additionalResourceParams?: object): Promise<object> {
1830+
requestData: object, additionalResourceParams?: object): Promise<object> {
18311831
return urlBuilder.getUrl(apiSettings.getEndpoint(), additionalResourceParams)
18321832
.then((url) => {
18331833
// Validate request.
1834-
if (requestData != null) {
1835-
const requestValidator = apiSettings.getRequestValidator();
1836-
requestValidator(requestData);
1837-
}
1834+
const requestValidator = apiSettings.getRequestValidator();
1835+
requestValidator(requestData);
18381836
// Process request.
18391837
const req: HttpRequestConfig = {
18401838
method: apiSettings.getHttpMethod(),
@@ -2062,7 +2060,7 @@ export class AuthRequestHandler extends AbstractAuthRequestHandler {
20622060
if (!validator.isNonEmptyString(tenantId)) {
20632061
return Promise.reject(new FirebaseAuthError(AuthClientErrorCode.INVALID_TENANT_ID));
20642062
}
2065-
return this.invokeRequestHandler(this.tenantMgmtResourceBuilder, DELETE_TENANT, undefined, { tenantId })
2063+
return this.invokeRequestHandler(this.tenantMgmtResourceBuilder, DELETE_TENANT, {}, { tenantId })
20662064
.then(() => {
20672065
// Return nothing.
20682066
});

test/unit/auth/auth-api-request.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4554,7 +4554,7 @@ AUTH_REQUEST_HANDLER_TESTS.forEach((handler) => {
45544554
return requestHandler.deleteTenant(tenantId)
45554555
.then((result) => {
45564556
expect(result).to.be.undefined;
4557-
expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, undefined));
4557+
expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, {}));
45584558
});
45594559
});
45604560

@@ -4589,7 +4589,7 @@ AUTH_REQUEST_HANDLER_TESTS.forEach((handler) => {
45894589
throw new Error('Unexpected success');
45904590
}, (error) => {
45914591
expect(error).to.deep.include(expectedError);
4592-
expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, undefined));
4592+
expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, {}));
45934593
});
45944594
});
45954595
});

0 commit comments

Comments
 (0)