Skip to content

Add Test Cases for 'Multifactor Authentication Testing' #29#51

Open
kambojshalabh35 wants to merge 1 commit into
LoginRadius:masterfrom
kambojshalabh35:patch-1
Open

Add Test Cases for 'Multifactor Authentication Testing' #29#51
kambojshalabh35 wants to merge 1 commit into
LoginRadius:masterfrom
kambojshalabh35:patch-1

Conversation

@kambojshalabh35
Copy link
Copy Markdown

No description provided.

@Chhavig28
Copy link
Copy Markdown
Contributor

@kambojshalabh35 Could you please elaborate few of your cases like 14, 16, 17 and 18?

@kambojshalabh35
Copy link
Copy Markdown
Author

Yeah sure, so basically,
14, Testing the common validity range, refers to the time limit for getting the OTP or PIN during multifactor authentication.
16, Checking for Service Packs and Patches, - Checks whether the service packs and patches are intact in order to stand against hacking attacks.
17, Targeting WebAuthn Security Key Issues, - refers to testing for issues that might lead to the leakage of security keys while receiving a PIN or OTP
18, Ended the testing process, - if we have completed all the above steps, the testing process is concluded

@kambojshalabh35
Copy link
Copy Markdown
Author

In addition, if you want me to add comments to the test case, please merge the PR since I am getting errors while fetching the origin in my forked repo, therefore I need to open another PR if you do not merge. It would be easier for me to make changes if you could merge the PR.

@kambojshalabh35
Copy link
Copy Markdown
Author

@Chhavig28 kindly review the PR

@Chhavig28
Copy link
Copy Markdown
Contributor

@kambojshalabh35 Could you please update the cases in your PR as you mentioned above in the comment?

@kambojshalabh35
Copy link
Copy Markdown
Author

@kambojshalabh35 Could you please update the cases in your PR as you mentioned above in the comment?

@Chhavig28 Requested changes have been made in #69

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants