-
Notifications
You must be signed in to change notification settings - Fork 2.1k
UI test for PR #26979 #27316
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
UI test for PR #26979 #27316
Conversation
|
@individual-it, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @SergioBertolinSG and @PVince81 to be potential reviewers. |
|
👍 |
e8a1eed to
cee7b04
Compare
|
I've tested this locally and I am getting four failures: Any idea why those tests are failing? I thought it was because the size of the window, but maximizing it manually has not helped. Maybe it is not waiting for the elements? |
no - can you try to get an understanding of this @SergioBertolinSG - THX |
|
Ok, the problem was that my server was in spanish and this was looking for "Unlimited" instead of "Ilimitado". Works fine. |
cee7b04 to
81b736c
Compare
This sets up UI tests. It uses behat/mink/selenium to run the tests on saucelabs.com Included are some basic tests 1. Login as admin 2. Tests to cover the fix made in PR #26979
81b736c to
b90f10c
Compare
|
👍 |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
This is an UI test for the PR #26979 that fixes the issue #26975
This also includes the complete setup of the selenium infrastructure that can be used as the basis of further UI tests. After the discussion in #27057 behat+mink is used instead of the facebook webdriver.
See also discussion in #27191
For this to work travis starts behat/mink tests. And they connect to selenium running on https://saucelabs.com
Related Issue
PR #26979
issue #26975
List of PRs/issues that potentially would benefit of UI tests:
Can't set a user's quota back to Unlimited #26975
Fix long display names #27059
files_external: Keep checkbox and its label on one line #25120
Improve quota validation including a regexp which matches expected formats #25358
Rows are not updated after sharing via web ui. #27294 nextcloud/server#3688
Fixed disappearing of share info in file view #26534 nextcloud/server#2010
Motivation and Context
See issue #27055
Some bugs are hard to catch with karma & PHPUnit tests. Primarily this are front-end bugs. This PR implements end-to-end testing environment run on real browsers. Currently only Chrome browser is used, but it can be extended to multiple browsers/platforms including mobile browsers.
Currently my own sauce-labs account is used. Of course it would be good that a oC account would be used in future.
run this tests locally
java -jar selenium-server-standalone-3.0.1.jar -port 4445SRV_HOST_NAME, SRV_HOST_URL and SRV_HOST_PORTe.g.:bash tests/travis/start_php_dev_server.sh(leaveSRV_HOST_URLempty if you are using the php dev. serverexport SRV_HOST_URL="")it will bind to:
$SRV_HOST_NAME:$SRV_HOST_PORTbash tests/travis/start_behat_tests.shlimitations
Only PRs from local branches are tested. PRs that come from a fork cannot be tested as they do not have access to the saucelabs username+key.
Types of changes
Checklist:
How Has This Been Tested?
@individual-it joined travis with their own account on sauce labs and has run multiple tests. Created bugs and run tests again to see if they are failing.
Screenshots (if appropriate):
travis job:
https://travis-ci.org/individual-it/owncloud-core/jobs/203733741
saucelabs job:
https://saucelabs.com/beta/tests/840fed2401854c50ab20109ac2812fb3/watch#305