Adding tests for password optional and identifier first login #179
Conversation
ivanezeigbo-okta
requested review from
bretterer,
bryanapellanes-okta and
laura-rodriguez
bryanapellanes-okta
left a comment
bryanapellanes-okta
left a comment
There was a problem hiding this comment.
Is this work still in progress?
| { | ||
| var options = new ChromeOptions(); | ||
| options.AddArgument("--headless"); | ||
| //options.AddArgument("--headless"); |
There was a problem hiding this comment.
uncomment this; the builds should run headless.
There was a problem hiding this comment.
Ah yes, missed that! :) Thanks
| And a user named "Mary" | ||
| And Mary does not have an account in the org | ||
|
|
||
| @ignore |
There was a problem hiding this comment.
Is there a reason that this is ignored?
There was a problem hiding this comment.
yes, ignored those because email passcode comes in twice so sometimes the test fails if it picks the first code and not the second. I believe we prob had this conversation before about that limitation.
| And the cell for the value of name is shown and contains her first name and last name | ||
| And the preferred_username claim is shown and matches Mary's email | ||
|
|
||
| @ignore |
There was a problem hiding this comment.
Same question as previous, is this ignored for a reason?
There was a problem hiding this comment.
yes, see above comment. If you run it locally however and put a wait before entering the passcode, it should pass fine.
There was a problem hiding this comment.
@ivanezeigbo-okta , can you please include a quick recording showing how the e2e works? We usually do that to help reviewers to understand the PR better. Thank you!
Summary
Adding cucumber feature tests for password optional and login identifier first among other .NET cucumber scenarios to be covered. This is the first batch of tests from password optional.
Type of PR
Test Information
.NET Version:
Os Version:
Signoff