Skip to content
Prev Previous commit
Next Next commit
fix rebasing caused tests failure
  • Loading branch information
Xiaoshouzi-gh committed Apr 12, 2023
commit 6868fd20c718a8be5dcb331c2f34b3da3aac3f78
83 changes: 23 additions & 60 deletions test/integration/auth.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1274,6 +1274,10 @@ describe('admin.auth', () => {
}
},
multiFactorConfig: mfaConfig,
recaptchaConfig: {
emailPasswordEnforcementState: 'OFF',
managedRules: [{ endScore: 0.1, action: 'BLOCK' }],
},
};
const expectedProjectConfigSmsEnabledTotpDisabled: any = {
smsRegionConfig: expectedProjectConfig2.smsRegionConfig,
Expand All @@ -1287,64 +1291,6 @@ describe('admin.auth', () => {
}
],
},
};

it('updateProjectConfig() should resolve with the updated project config', () => {
return getAuth().projectConfigManager().updateProjectConfig(projectConfigOption1)
.then((actualProjectConfig) => {
expect(actualProjectConfig.toJSON()).to.deep.equal(expectedProjectConfig1);
return getAuth().projectConfigManager().updateProjectConfig(projectConfigOption2);
})
.then((actualProjectConfig) => {
expect(actualProjectConfig.toJSON()).to.deep.equal(expectedProjectConfig2);
return getAuth().projectConfigManager().updateProjectConfig(projectConfigOptionSmsEnabledTotpDisabled);
}).then((actualProjectConfig) => {
expect(actualProjectConfig.toJSON()).to.deep.equal(expectedProjectConfigSmsEnabledTotpDisabled);
});
});

it('getProjectConfig() should resolve with expected project config', () => {
return getAuth().projectConfigManager().getProjectConfig()
.then((actualConfig) => {
const actualConfigObj = actualConfig.toJSON();
expect(actualConfigObj).to.deep.equal(expectedProjectConfigSmsEnabledTotpDisabled);
});
});
});

describe('Project config management operations', () => {
before(function() {
if (authEmulatorHost) {
this.skip(); // getConfig is not supported in Auth Emulator
}
});
const projectConfigOption1: UpdateProjectConfigRequest = {
recaptchaConfig: {
emailPasswordEnforcementState: 'AUDIT',
managedRules: [{ endScore: 0.1, action: 'BLOCK' }],
useAccountDefender: true,
},
};
const projectConfigOption2: UpdateProjectConfigRequest = {
recaptchaConfig: {
emailPasswordEnforcementState: 'OFF',
useAccountDefender: false,
},
};
const projectConfigOption3: UpdateProjectConfigRequest = {
recaptchaConfig: {
emailPasswordEnforcementState: 'OFF',
useAccountDefender: true,
},
};
const expectedProjectConfig1: any = {
recaptchaConfig: {
emailPasswordEnforcementState: 'AUDIT',
managedRules: [{ endScore: 0.1, action: 'BLOCK' }],
useAccountDefender: true,
},
};
const expectedProjectConfig2: any = {
recaptchaConfig: {
emailPasswordEnforcementState: 'OFF',
managedRules: [{ endScore: 0.1, action: 'BLOCK' }],
Expand All @@ -1361,14 +1307,18 @@ describe('admin.auth', () => {
})
.then((actualProjectConfig) => {
expect(actualProjectConfig.toJSON()).to.deep.equal(expectedProjectConfig2);
return getAuth().projectConfigManager().updateProjectConfig(projectConfigOptionSmsEnabledTotpDisabled);
})
.then((actualProjectConfig) => {
expect(actualProjectConfig.toJSON()).to.deep.equal(expectedProjectConfigSmsEnabledTotpDisabled);
});
});

it('getProjectConfig() should resolve with expected project config', () => {
return getAuth().projectConfigManager().getProjectConfig()
.then((actualConfig) => {
const actualConfigObj = actualConfig.toJSON();
expect(actualConfigObj).to.deep.equal(expectedProjectConfig2);
expect(actualConfigObj).to.deep.equal(expectedProjectConfigSmsEnabledTotpDisabled);
});
});
});
Expand Down Expand Up @@ -1486,6 +1436,16 @@ describe('admin.auth', () => {
disallowedRegions: ['AC', 'AD'],
}
},
recaptchaConfig: {
emailPasswordEnforcementState: 'OFF',
managedRules: [
{
endScore: 0.3,
action: 'BLOCK',
},
],
useAccountDefender: false,
},
};
const expectedUpdatedTenantSmsEnabledTotpDisabled: any = {
displayName: 'testTenantUpdated',
Expand Down Expand Up @@ -2070,7 +2030,10 @@ describe('admin.auth', () => {
}
return getAuth().tenantManager().updateTenant(createdTenantId, updateRequestSMSEnabledTOTPDisabled)
.then((actualTenant) => {
expect(actualTenant.toJSON()).to.deep.equal(expectedUpdatedTenantSmsEnabledTotpDisabled);
// response from backend ignores account defender status is recaptcha status is OFF.
const expectedUpdatedTenantCopy = deepCopy(expectedUpdatedTenantSmsEnabledTotpDisabled);
delete expectedUpdatedTenantCopy.recaptchaConfig.useAccountDefender;
expect(actualTenant.toJSON()).to.deep.equal(expectedUpdatedTenantCopy);
});
});

Expand Down
8 changes: 8 additions & 0 deletions test/unit/auth/tenant.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,14 @@ describe('Tenant', () => {
mfaConfig: {
state: 'ENABLED',
enabledProviders: ['PHONE_SMS'],
providerConfigs: [
{
state: 'ENABLED',
totpProviderConfig: {
adjacentIntervals: 5,
},
},
],
},
testPhoneNumbers: {
'+16505551234': '019287',
Expand Down