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
Selenium testing #587
Selenium testing #587
Conversation
Can one of the admins verify this patch? |
retest |
@elisedd please 1. push a commit which removes .recommenders and then push another commit on top of that that adds .recommenders to the gitignore file. |
/* | ||
* step 02 | ||
*/ | ||
// have to click cancel twice to get out of the name input |
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.
actually this should not be the case, right? Is that an error in the UI? then it should fail instead and we should log a ticket?
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.
Yes of course - https://issues.jboss.org/browse/WINDUP-2154
Sorry about this Janine, with Selenium there is no way to actually test
that the name input box is automatically selected when moving to the page.
It isn't really a bug with the code just a limitation of Selenium.
(Apologies if I am unnecessarily explaining things)
Selenium has 3 methods to check the status of a web element,
".isSelected()", ".isEnabled()", ".isDisplayed()". The ".isDisplayed()"
method of course works, but I was hoping for the "isSelected()" method to
return true.
At the moment I just have it looking for the actual element.
I wrote a method to test this, then just left it for if selenium is updated
so that we all know that this should ultimately be tested, I should have
written more comments explaining this.
…On Tue, Aug 7, 2018 at 12:13 PM, Janine Eichler ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In ui/src/test/org/jboss/windup/web/selenium/Selenium01Test.java
<#587 (comment)>:
> + */
+ assertEquals("http://127.0.0.1:8080/rhamt-web/project-list", selenium.checkURL());
+
+ selenium.clickNewProjButton();
+ assertEquals("http://127.0.0.1:8080/rhamt-web/wizard/create-project", selenium.checkURL());
+
+ //checks that the project name field has focus, then the cancel/next buttons are enabled/disabled
+ //.nameInputSelected does not work
+ assertTrue(selenium.nameInputSelected());
+ assertTrue(selenium.cancelEnabled());
+ assertFalse(selenium.nextEnabled());
+
+ /*
+ * step 02
+ */
+ // have to click cancel twice to get out of the name input
actually this should not be the case, right? Is that an error in the UI?
then it should fail instead and we should log a ticket?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#587 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Ali5QLVP8mzACKQ4V6vGRxLuf4KCcqIPks5uObyPgaJpZM4ViCkT>
.
|
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.
No description provided.