Revert "Remove request body for deleteTenant (#1452)" (#1456)

This reverts commit 475927c.

Author: hiranya911

src/auth/auth-api-request.ts

@@ -1827,14 +1827,12 @@ export abstract class AbstractAuthRequestHandler {
    */
   protected invokeRequestHandler(
     urlBuilder: AuthResourceUrlBuilder, apiSettings: ApiSettings,
-    requestData: object | undefined, additionalResourceParams?: object): Promise<object> {
+    requestData: object, additionalResourceParams?: object): Promise<object> {
     return urlBuilder.getUrl(apiSettings.getEndpoint(), additionalResourceParams)
       .then((url) => {
         // Validate request.
-        if (requestData != null) {
-          const requestValidator = apiSettings.getRequestValidator();
-          requestValidator(requestData);
-        }
+        const requestValidator = apiSettings.getRequestValidator();
+        requestValidator(requestData);
         // Process request.
         const req: HttpRequestConfig = {
           method: apiSettings.getHttpMethod(),
@@ -2062,7 +2060,7 @@ export class AuthRequestHandler extends AbstractAuthRequestHandler {
     if (!validator.isNonEmptyString(tenantId)) {
       return Promise.reject(new FirebaseAuthError(AuthClientErrorCode.INVALID_TENANT_ID));
     }
-    return this.invokeRequestHandler(this.tenantMgmtResourceBuilder, DELETE_TENANT, undefined, { tenantId })
+    return this.invokeRequestHandler(this.tenantMgmtResourceBuilder, DELETE_TENANT, {}, { tenantId })
       .then(() => {
         // Return nothing.
       });

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

@@ -4554,7 +4554,7 @@ AUTH_REQUEST_HANDLER_TESTS.forEach((handler) => {
           return requestHandler.deleteTenant(tenantId)
             .then((result) => {
               expect(result).to.be.undefined;
-              expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, undefined));
+              expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, {}));
             });
         });
 
@@ -4589,7 +4589,7 @@ AUTH_REQUEST_HANDLER_TESTS.forEach((handler) => {
               throw new Error('Unexpected success');
             }, (error) => {
               expect(error).to.deep.include(expectedError);
-              expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, undefined));
+              expect(stub).to.have.been.calledOnce.and.calledWith(callParams(path, method, {}));
             });
         });
       });