CCTC-team / redcap_cypress

REDCap Cypress Test Framework
MIT License
0 stars 0 forks source link

101. Two Factor Authentication #35

Open MintooXavier opened 6 months ago

MintooXavier commented 6 months ago

Brief description: This feature increases the security of the system by adding an additional security code on top of the password at user login.

User spec: https://github.com/CCTC-team/redcap_cypress/blob/redcap_val/user_requirement_specification/non_core/101_two_factor_authentication.spec

Feature link: https://github.com/CCTC-team/redcap_rsvc/blob/redcap_val/Feature%20Tests/D/Two%20Factor%20Authentication_101/D.101.100%20-%20Two%20Factor%20Authentication.feature https://github.com/CCTC-team/redcap_rsvc/blob/redcap_val/Feature%20Tests/D/Two%20Factor%20Authentication_101/D.101.200%20-%20REDUNDAN.feature https://github.com/CCTC-team/redcap_rsvc/blob/redcap_val/Feature%20Tests/D/Two%20Factor%20Authentication_101/D.101.300%20-%20REDUNDANT.feature

Pre review:

Post review:

MintooXavier commented 5 months ago

Add test to include the following:

  1. Enter wrong authentication code - message 'Sorry, but you did not enter a valid verification code.'
MintooXavier commented 1 month ago
  1. when you login as test-admin, then no need to give username and password. https://github.com/CCTC-team/redcap_cypress/blob/d81b8723fb22f4350d5defaadb9b8673bc7c0d23/manual_scripts/Two%20Factor%20Authentication_101/D.101.100%20-%20Two%20Factor%20Authentication.feature#L11
  2. Put the text what you see on the Hom page. https://github.com/CCTC-team/redcap_cypress/blob/d81b8723fb22f4350d5defaadb9b8673bc7c0d23/manual_scripts/Two%20Factor%20Authentication_101/D.101.100%20-%20Two%20Factor%20Authentication.feature#L14
  3. Verify 2Factor Auth is disabled. Then enable it and add the appropriate settings. https://github.com/CCTC-team/redcap_cypress/blob/d81b8723fb22f4350d5defaadb9b8673bc7c0d23/manual_scripts/Two%20Factor%20Authentication_101/D.101.100%20-%20Two%20Factor%20Authentication.feature#L21.
  4. When you add 'Given, When, Then, And' in front of Gherkin, make sure it makes sense while reading..

I have only reviewed the first part.

rukayat-yakub commented 1 month ago

@MintooXavier - Did the corrections. Please have a look when you can

MintooXavier commented 1 month ago

I have modified and completed the script. Code pushed to github

Script Pre-reviewed and Tests Pass

MintooXavier commented 14 hours ago

Scripts automated