-
Notifications
You must be signed in to change notification settings - Fork 494
[TASK] RT-2525 e2e tests for account rename #1748
Conversation
LGTM |
function pressRenameSubmit() { | ||
$("form[name='renameForm'] button[type='submit']").click().then(function() { | ||
|
||
browser.wait(function () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can make this more concise with browser.wait($("div.showPublic div.alert-success").isDisplayed)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, please make the change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll wait until you merge #1766 - I've added useful utility function there
Merged #1766 |
50fcb41
to
6efdbe9
Compare
use utility functions to make code shorter
6efdbe9
to
156bcca
Compare
|
||
|
||
it('should rename user', function(done) { | ||
protractor.getInstance().ignoreSynchronization = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you need this here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because, without that, protractor will wait for angular to finish all of it internal jobs.
but, when you submit password, ripple client start timer for 5 minutes to expire this password.
so protractor will wait for that to finish and will run out of time for test.
it took long time to identify this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@darkdarkdragon ok, makes sense.
LGTM |
[TASK] RT-2525 e2e tests for account rename
No description provided.