summaryrefslogtreecommitdiffstats
path: root/tests/acceptance/features
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #10322 from ↵Roeland Jago Douma2018-07-242-0/+22
|\ | | | | | | | | weeman1337/feature-9978-improve-disabled-user-login-message Login: Implements the "user disabled" message like the "wrong password" message
| * Adds disabled user acceptance testsMichael Weimann2018-07-242-0/+22
| | | | | | | | Signed-off-by: Michael Weimann <mail@michael-weimann.eu>
* | Acceptance and mobile navigation fixesJohn Molakvoæ (skjnldsv)2018-07-241-2/+12
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Adjust acceptance tests to the new share link UIDaniel Calviño Sánchez2018-07-211-24/+66
|/ | | | | | | | | | | | | | | The share link UI no longer uses its own layout below the other shares; now it is shown as a share row with a menu for the actions (except enabling it, which is shown in the row itself), just like the other shares. The share link is no longer shown, either; now the link is got by clicking on a "Copy URL" menu item, which copies the link to the clipboard. As the clipboard is not accessible from the acceptance tests the URL is now extracted from the attributes of that menu item (although the menu item is clicked anyway to mimic the user behaviour). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust acceptance tests to sidebar changes in Files appJohn Molakvoæ (skjnldsv)2018-07-203-89/+73
| | | | | | | | Before, each section of the Files app ("All files", "Favorites"...) had its own sidebar element. Now there is a single sidebar element for all the sections in the Files app. Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix accceptance testsJulius Härtl2018-07-201-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Adds tests for testing removing an empty groupMichael Weimann2018-07-203-1/+31
| | | | Signed-off-by: Michael Weimann <mail@michael-weimann.eu>
* Merge pull request #9518 from ↵Roeland Jago Douma2018-06-211-2/+2
|\ | | | | | | | | nextcloud/feature/5986/public_share_controller_middleware Public share middleware & controller
| * Fix acceptance testRoeland Jago Douma2018-06-201-2/+2
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Adjust theming acceptance tests to new header colour transitionDaniel Calviño Sánchez2018-06-202-10/+20
| | | | | | | | | | | | | | | | | | | | | | Before, the acceptance tests checked the header colour just once, as the header colour was immediately changed once the new theming colour was saved. This is no longer the case, as currently a transition is used to change between the original colour and the new one, so now the acceptance tests check repeteadly for the expected header colour until it matches or the timeout expires. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fixed tests and improved app-navigation-captionJohn Molakvoæ (skjnldsv)2018-06-201-1/+1
|/ | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Properly check for the sidebarJulius Härtl2018-06-092-3/+26
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Add basic acceptance tests for apps managementJulius Härtl2018-06-093-0/+315
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Allow 0 quota by provisioning apiJohn Molakvoæ (skjnldsv)2018-06-071-1/+10
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fixed root url detection, new tests, default new user group to currentJohn Molakvoæ (skjnldsv)2018-05-162-33/+107
| | | | | | opened group and user removal update fix Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Bump script and fixed copyright templateJohn Molakvoæ (skjnldsv)2018-05-164-118/+122
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Tabs fixJohn Molakvoæ (skjnldsv)2018-05-162-72/+76
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Users list acceptance testsJohn Molakvoæ (skjnldsv)2018-05-166-44/+392
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* newt acceptance testsJohn Molakvoæ (skjnldsv)2018-05-162-0/+67
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fixed routerJohn Molakvoæ (skjnldsv)2018-05-161-1/+0
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fixed new user form detection in acceptanceJohn Molakvoæ (skjnldsv)2018-05-161-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix row detection acceptance testJohn Molakvoæ (skjnldsv)2018-05-161-2/+2
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Rename helper for PHP built-in web serverDaniel Calviño Sánchez2018-04-092-14/+14
| | | | | | | For consistency with the helper for the Apache web server the helper for the PHP built-in web server was renamed too. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add helper to run acceptance tests on an Apache serverDaniel Calviño Sánchez2018-04-091-0/+130
| | | | | | | | | | | | | | | | | | | | | | The default and only helper to run acceptance tests run them on the PHP built-in web server. This commit introduces a new helper that can be used to run them on an Apache web server instead. This helper is meant to be used by the acceptance tests of apps that require a multi-threaded web server to run (like Talk, due to its use of long polling). To use the helper it is only needed to set it in the Behat configuration for the acceptance tests of the app, as explained in the "NextcloudTestServerContext" documentation. It is assumed that the acceptance tests are run using the default setup, and therefore inside a Docker container based on the image for acceptance tests from Nextcloud. Due to that the helper is expected to have root permissions, and thus it starts and stops the Apache web server directly using "service start/stop apache2". In the same way it also restores the owner and group for "apps", "config" and "data" to "www-data", as it is the user that Apache sub-processes are run as. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix files acceptance testJulius Härtl2018-04-051-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Move styling and menu handling to publicpage.jsJulius Härtl2018-04-051-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Header acceptance featuresJohn Molakvoæ (skjnldsv)2018-03-095-10/+238
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Add acceptance tests for creation of subfolders in public shared foldersDaniel Calviño Sánchez2018-03-093-0/+121
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Extract common "wait for" functions to a helper classDaniel Calviño Sánchez2018-03-093-46/+86
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Generalize file list steps so a specific ancestor can be usedDaniel Calviño Sánchez2018-03-094-3/+115
| | | | | | | | | | | | | | | | | | | | | | | | | The "FileListContext" provides steps to interact with and check the behaviour of a file list. However, the "FileListContext" does not know the right file list ancestor that has to be used by the file list steps, so until now the file list steps were explicitly wired to the Files app and they could be used only in that case. Instead of duplicating the steps with a slightly different name (for example, "I create a new folder named :folderName in the public shared folder" instead of "I create a new folder named :folderName") the steps were generalized; now contexts that "know" that certain file list ancestor has to be used by the FileListContext steps performed by certain actor from that point on (until changed again) set it explicitly. For example, when the current page is the Files app then the ancestor of the file list is the main view of the current section of the Files app, but when the current page is a shared link then the ancestor is set to null (because there will be just one file list, and thus its ancestor is not relevant to differentiate between instances) A helper trait, "FileListAncestorSetter", was introduced to reduce the boilerplate needed to set the file list ancestor from other contexts. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Generalize file list locators so a specific ancestor can be usedDaniel Calviño Sánchez2018-03-092-54/+66
| | | | | | | | | | | | | | | | | | | | The file list is used in other places besides the Files app (for example, the File sharing app); in those cases the locators for the file list elements are the same, but not for the ancestor of the file list. To make possible to reuse the file list locators in those cases too now they receive the ancestor to use. Note that the locators for the file actions menu were not using an ancestor locator because it is expected that there is only one file actions menu at a time in the whole page; that may change in the future, but for the time being it is a valid assumption and thus the ancestor was not added to those locators in this commit. Although the locators were generalized the steps themselves still use the "FilesAppContext::currentSectionMainView" locator as ancestor; the steps will be generalized in a following commit. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Store the name of the actor in the Actor objectDaniel Calviño Sánchez2018-03-092-3/+19
| | | | | | This is needed to be able to easily use the actor as a key in an array. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Extract file list locators and steps to its own classDaniel Calviño Sánchez2018-03-092-250/+287
| | | | | | | | | | Besides the extraction some minor adjustments (sorting locators for file action menu entries to reflect the order of the menu entries in the UI, moving parametrized locators like "createMenuItemFor" above the locators that use them and placing "descendantOf" calls always in a new line) were made too. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add automatic handling of NoSuchElement exceptionsDaniel Calviño Sánchez2018-03-081-6/+16
| | | | | | | | | | | | NoSuchElement exceptions are sometimes thrown instead of StaleElementReference exceptions. This can happen when the Selenium2 driver for Mink performs an action on an element through the WebDriver session instead of directly through the WebDriver element. In that case, if the element with the given ID does not exist, a NoSuchElement exception would be thrown instead of a StaleElementReference exception, so those cases are handled like StaleElementReference exceptions. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add automatic handling of MoveTargetOutOfBounds exceptionsDaniel Calviño Sánchez2018-03-081-3/+17
| | | | | | | | | | | | | | | | | | MoveTargetOutOfBounds exceptions are sometimes thrown instead of ElementNotVisible exceptions. This can happen when the Selenium2 driver for Mink moves the cursor on an element using the "moveto" method of the Webdriver session, for example, before clicking on an element. In that case, if the element is not visible, "moveto" would throw a MoveTargetOutOfBounds exception instead of an ElementNotVisible exception, so those cases are handled like ElementNotVisible exceptions. Note that MoveTargetOutOfBounds exceptions could be thrown too if the element was visible but "out of reach"; there is no problem in handling those cases as if the element was not visible, as the exception will be thrown again anyway once it is verified that the element is indeed visible. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* fixup! Acceptance fixJohn Molakvoæ (skjnldsv)2018-03-011-1/+2
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Acceptance fixJohn Molakvoæ (skjnldsv)2018-03-012-1/+37
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Merge pull request #7600 from nextcloud/new-user-button-to-sidebarRoeland Jago Douma2018-02-282-1/+34
|\ | | | | New user button to sidebar
| * Fix acceptance testsJohn Molakvoæ (skjnldsv)2018-02-282-1/+34
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Fix acceptance test for new menu structureJulius Härtl2018-02-271-1/+1
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Take into account the comment message when looking for itDaniel Calviño Sánchez2018-02-152-24/+17
| | | | | | | | Instead of checking that the list contains one comment it is now checked that a comment with certain message is visible. This makes the step (and the locator) more reusable in future tests and also simplifies the code. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust timeouts in the step to create a new commentDaniel Calviño Sánchez2018-02-151-2/+2
| | | | | | | | | | | | | Depending on the previous steps the new comment field may be already shown or not when the step to create a new comment is executed. Therefore, the timeout was increased from 2 to the "standard" 10 seconds used in other tests. If the new comment field was found there is no need to use a timeout when looking for the new comment button; it is either there or not, it will not appear after some time. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Move locators above step definitionsDaniel Calviño Sánchez2018-02-151-21/+26
| | | | | | | | | The locators are moved above the step definitions for consistency with other context files; besides that I made some minor adjustments for consistency too in the locator descriptions and identation, and moved the locators for ".newCommentRow" descendants together. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Merge pull request #7650 from rpirritano/masterMorris Jobke2018-01-111-1/+1
|\ | | | | made small changes to login screen
| * made small changes to login screenRobert Pirritano2018-01-111-1/+1
| | | | | | | | | | | | added height back 70px back to footer and removed .htaccess file Signed-off-by: Robert R Pirritano <rpirritano@gmail.com>
* | Add acceptance test for renaming a file with the details view openDaniel Calviño Sánchez2018-01-112-0/+57
|/ | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Wait for the shared link to be set in the acceptance testsDaniel Calviño Sánchez2017-12-221-1/+24
| | | | | | | | | | | | | | | | | When clicking on "Share link" in the "Sharing" tab of the Files app an input field with the link appears. That input field already exists in the DOM, although empty, before clicking on "Share link", and when that is done the proper value is set and then the input field is shown. In the acceptance tests "getValue()" can return the value of hidden elements too, so as long as an element exists its value is returned without waiting for the field to be visible. Due to this if the test code runs too fast the "I write down the shared link" step could be executed before the proper value was set, so the shared link got in that case would be an empty value, and this would lead to failures when the following steps were executed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add missing timeout multiplierDaniel Calviño Sánchez2017-12-221-2/+6
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix typo in callback nameDaniel Calviño Sánchez2017-12-221-2/+2
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add acceptance test for opening the menu in a public shared linkDaniel Calviño Sánchez2017-12-192-0/+95
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>