Fixed integration test failure of skipped tests (#1299)

* Fix integration test failure of skipped testss * Trigger integration tests
firebase · May 26, 2021 · f914f92 · f914f92
1 parent 03c66d8
commit f914f92
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/test/integration/auth.spec.ts b/test/integration/auth.spec.ts
@@ -1336,10 +1336,9 @@ describe('admin.auth', () => {
         clientId: 'CLIENT_ID1',
         responseType: {
           idToken: true,
-          code: false,
         },
       };
-      const modifiedConfigOptions = {
+      const deltaChanges = {
         displayName: 'OIDC_DISPLAY_NAME3',
         enabled: false,
         issuer: 'https://oidc.com/issuer3',
@@ -1350,6 +1349,17 @@ describe('admin.auth', () => {
           code: true,
         },
       };
+      const modifiedConfigOptions = {
+        providerId: authProviderConfig.providerId,
+        displayName: 'OIDC_DISPLAY_NAME3',
+        enabled: false,
+        issuer: 'https://oidc.com/issuer3',
+        clientId: 'CLIENT_ID3',
+        clientSecret: 'CLIENT_SECRET',
+        responseType: {
+          code: true,
+        },
+      };
 
       before(function() {
         if (!createdTenantId) {
@@ -1378,12 +1388,10 @@ describe('admin.auth', () => {
           .then((config) => {
             assertDeepEqualUnordered(authProviderConfig, config);
             return tenantAwareAuth.updateProviderConfig(
-              authProviderConfig.providerId, modifiedConfigOptions);
+              authProviderConfig.providerId, deltaChanges);
           })
           .then((config) => {
-            const modifiedConfig = deepExtend(
-              { providerId: authProviderConfig.providerId }, modifiedConfigOptions);
-            assertDeepEqualUnordered(modifiedConfig, config);
+            assertDeepEqualUnordered(modifiedConfigOptions, config);
             return tenantAwareAuth.deleteProviderConfig(authProviderConfig.providerId);
           })
           .then(() => {