Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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

wants to merge 1 commit into from

Conversation

kambojshalabh35
Copy link

No description provided.

@Chhavig28
Copy link
Contributor

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

@kambojshalabh35
Copy link
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
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
Author

@Chhavig28 kindly review the PR

@Chhavig28
Copy link
Contributor

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

@kambojshalabh35
Copy link
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