Skip to content
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

WINDUP-2260 Changes to have the tests to work with a remote RHAMT instance #4

Merged
merged 3 commits into from Jan 30, 2019

Conversation

mrizzi
Copy link
Member

@mrizzi mrizzi commented Jan 11, 2019

  • strengthened tests behaviour to work when the UI is remote so there's more delay in responding due to network latency and remote server performances
  • added optional login action using -DwithLogin parameter
  • added optional URL to call a remote RHAMT instance using -DbaseUrl=http://rhamt.example.com/ parameter
  • added optional parameter -Dheadless=false to disable the browser headless mode
  • refactored some method and moved them in CommonProject class (login, waitForProjectList, waitForNoProjectWelcomePage, waitForProjectLoad, navigateProject, closeDriver)
  • added SeleniumSuiteTest class to execute all of the tests in the right order with just one parameter -Dtest=SeleniumSuiteTest
  • updated version in pom.xml to 4.2.1-SNAPSHOT
  • commented a check about sorting for having it fixed later

@PhilipCattanach
Copy link
Contributor

I executed the tests several time using the remote upshift 4.2.1 snapshot build of the RHAMT Web UI via the following command.
mvn -Dtest=Selenium01Test#test05RunAnalysis -DwithLogin -DbaseUrl=http://rhamt-web-console-rhamt.cloud.paas.upshift.redhat.com/ clean test
I also ran the same test running the WebUI on localhost. There were no failures due to issues with the Package selection controls.

@mrizzi
Copy link
Member Author

mrizzi commented Jan 25, 2019

@m-brophy i changed the pattern to wait for the package tree to appear in the page: now it waits for an li element with class jstree-node that i really hope would be able to exclude the Loading... label to be retrieved.
Please, make me know if it works for you.

@PhilipCattanach PhilipCattanach merged commit 9b52e13 into windup:master Jan 30, 2019
@mrizzi mrizzi deleted the WINDUP-2260 branch January 30, 2019 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants