-
Notifications
You must be signed in to change notification settings - Fork 231
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
[enterprise-3.11] Fix bug where cannot create new role and new rolebinding in single operation using template #3116
[enterprise-3.11] Fix bug where cannot create new role and new rolebinding in single operation using template #3116
Conversation
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.
/lgtm
/test |
/retest |
/lgtm |
/retest |
1 similar comment
/retest |
/test |
…le operation using template
let routesPage = new RoutesPage(project); | ||
routesPage.visit(); | ||
expect(element(by.cssContainingText('td', 'nodejs-mongodb-example')).isPresent()).toBe(true); // TODO: use fixture | ||
return browser.sleep(3000).then(() => { |
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.
@spadgett, I took a guess this would resolve the failing test issue. Mind taking a look?
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.
This seems reasonable. Ideally we would wait for the page to update instead of an arbitrary sleep, but I think it's OK if it unblocks CI.
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.
/lgtm
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1673109
Big assist from @spadgett on this one. Thank you!