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 initial UI test #151
Add initial UI test #151
Conversation
It seems I need to write some tests for camayoc, I am going to work on that later. Feel free to review as I should only add the tests later. |
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
Did leave comment about perhaps putting text into some constants and keeping them in a central location. I'll leave it up to you.
login.password.fill('pass') | ||
login.login.click() | ||
|
||
assert browser.selenium.title == 'Red Hat Entitlements Reporting' |
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.
Maybe make this a constant in constants.py
?
assert browser.selenium.title == 'Red Hat Entitlements Reporting' | ||
|
||
dashboard = DashboardView(browser) | ||
dashboard.user_dropdown.select_item('Logout') |
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.
See above comment.
I'm not certain this is better. Maybe even there should be a separate UI constants file, because there will probably be lots of text we will be working with.
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 think for now I will leave the string here. As UI test grows we decide if we would have a constants file for it or not.
Close #106
I will wait for travis and coveralls and if they are happy will merge the PR. I've added tests for the new util function to fetch the base URL. |
Close #106