summaryrefslogtreecommitdiffstats
path: root/tests/acceptance/features/bootstrap
Commit message (Collapse)AuthorAgeFilesLines
* Handle single action unshareJulius Härtl2021-06-291-4/+16
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Change label also in the acceptance testsJoas Schilling2021-03-111-4/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Find directly the label instead of falling back to itDaniel Calviño Sánchez2021-03-081-8/+1
| | | | | | | | The input element is always hidden, so the check always ended falling back to the label. Moreover, the label is the element that the user interacts with, so it must be the one used. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Assert also element visibility instead of just finding itDaniel Calviño Sánchez2021-03-081-4/+12
| | | | | | | | Although if the element could not be found an exception would be thrown and the test aborted if an element is in the DOM but hidden it would be found and the test would pass. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add explicit locator for "Enable all" bundle buttonDaniel Calviño Sánchez2021-03-081-5/+7
| | | | | | | | | | | Instead of looking for the bundle button and then checking its value now the expected value is included in the locator and the button is checked similarly to other elements. No "Disable all" locator was added as it was not currently needed anywhere. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Find elements through the actor rather than the driverDaniel Calviño Sánchez2021-03-081-4/+30
| | | | | | | | | | "Actor::find" is a more robust way to look for elements, as it handles some exceptions that may be thrown. Therefore, even if the elements are not actually used and it is only checked whether they exist or not using the actor is the preferred way when possible (and it also makes it consistent with the rest of the acceptance tests). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add locator for apps listDaniel Calviño Sánchez2021-03-081-4/+16
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix identationDaniel Calviño Sánchez2021-03-081-14/+14
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix delete user acceptance testDaniel Calviño Sánchez2021-03-081-1/+1
| | | | | | | | | | | | | Adding some missing asserts showed that the "delete user" acceptance test was silently failing, as the deletion was not being confirmed in the dialog and thus the user was not being deleted. The dialog button contains a single quote ("user0's"), so the XPath expression had to be adjusted (it seems that it is not possible to escape a single quote in a string enclosed in single quotes in XPath 1.0). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Verify WaitFor::element... results with an assertionDaniel Calviño Sánchez2021-03-085-11/+74
| | | | | | | | WaitFor::element... calls only perform the waiting and return whether the condition succeeded or not, but that result needs to be explicitly checked to prevent further steps from being executed if the wait failed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add missing waits when finding elements in the acceptance testsDaniel Calviño Sánchez2021-03-083-7/+7
| | | | | | | | As no timeout was specified the elements were tried to be found just once. This caused the steps to fail if the elements did not appear yet in the page when they were tried to be found. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add acceptance tests for reshares by link when resharing is disabledDaniel Calviño Sánchez2020-11-111-0/+31
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add acceptance tests for reshares when resharing is disabledDaniel Calviño Sánchez2020-11-112-0/+90
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust acceptance tests to changes in @nextcloud/vue 2.6.3Daniel Calviño Sánchez2020-09-031-75/+136
| | | | | | | | | | | | | | | | The menu button and the menu of a share are no longer direct childs of the actions of the share row. The menu button is now a child of a ".trigger" element, while the menu is a direct child of the body and has an id defined in the "aria-describedby" attribute of the ".trigger" element. In XPath 1.0 it does not seem possible to "backreference" a value or create variables, so when the share menu or one of its item is needed now the ".trigger" element is first found and then its XPath expression is used to compose its "aria-describedby" attribute in the XPath expression for the menu. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix clicks on actions menu of non opaque file rows in acceptance testsDaniel Calviño Sánchez2020-08-291-7/+37
| | | | | | | | | | | | | | | | When a row is added to the file list the opacity of the file row is animated from transparent to fully opaque. As the file actions menu is a descendant of the row but overflows it when the row is not fully opaque clicks on the menu entries "fall-through" and are received instead by the rows behind. The opacity animation is a CSS animation, and it is not possible to know if a row is appearing or not except from its opacity (the row will have the "appear" CSS class even after the animation ended). Therefore it should be waited until the row of the file is fully opaque before using the menu. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Hide share link menu before copying the share link in acceptance testsDaniel Calviño Sánchez2020-07-221-0/+11
| | | | | | | | | | | The (old) Firefox version used in the acceptance tests does not properly render the share link menu. As the menu is taller than it should sometimes it covers the copy link button, so it is not possible to click it without hiding the share link menu. Moreover, in those cases the share menu button is also covered by the share menu, so the menu needs to be closed by pressing the "Esc" key. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix breaking changes of nc/vue updateGeorg Ehrke2020-07-225-10/+10
| | | | Signed-off-by: Georg Ehrke <developer@georgehrke.com>
* Add acceptance test for sharing a folder without create permissionDaniel Calviño Sánchez2020-06-111-0/+55
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add acceptance test for sharing a file without edit permissionDaniel Calviño Sánchez2020-06-111-0/+55
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Generalize functions to get locators for share permission checkboxesDaniel Calviño Sánchez2020-06-111-9/+23
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Merge pull request #20419 from nextcloud/dependabot/npm_and_yarn/variousJohn Molakvoæ2020-04-111-4/+4
|\ | | | | Bump p-limit from 2.2.2 to 2.3.0, @nextcloud/auth from 1.2.2 to 1.2.3, @babel/preset-env from 7.8.7 to 7.9.5, @nextcloud/event-bus from 1.1.3 to 1.1.4, @nextcloud/password-confirmation from 1.0.0 to 1.0.1, @nextcloud/l10n from 1.2.2 to 1.2.3, @nextcloud/initial-state from 1.1.1 to 1.1.2, core-js from 3.6.4 to 3.6.5 …
| * Bump acceptance testsJohn Molakvoæ (skjnldsv)2020-04-101-4/+4
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Format control structures, classes, methods and functionChristoph Wurst2020-04-1021-44/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use elseif instead of else ifChristoph Wurst2020-04-101-1/+1
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove trailing whitespaces from commentsChristoph Wurst2020-04-094-6/+6
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove trailing and in between spacesChristoph Wurst2020-04-091-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-093-7/+7
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix (array) indent style to always use one tabChristoph Wurst2020-04-092-13/+13
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* tests: fix failing header acceptance testAzul2020-04-071-1/+1
| | | | | | The username is listed in a div with class `fullname` now. Signed-off-by: Azul <azul@riseup.net>
* Merge pull request #20139 from ↵Roeland Jago Douma2020-04-041-0/+44
|\ | | | | | | | | nextcloud/unify-default-value-for-restricting-user-enumeration-with-settings Unify default value for restricting user enumeration with settings
| * Add acceptance test for restricting user enumeration to groupsDaniel Calviño Sánchez2020-03-241-0/+44
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Merge pull request #20170 from nextcloud/techdebt/remove-unused-importsChristoph Wurst2020-03-271-1/+0
|\ \ | | | | | | Remove unused imports
| * | Remove unused importsChristoph Wurst2020-03-251-1/+0
| |/ | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* / Use the short array syntax, everywhereChristoph Wurst2020-03-261-1/+1
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add acceptance test for accepting an incoming shareDaniel Calviño Sánchez2020-03-201-0/+44
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix timeout cappingDaniel Calviño Sánchez2020-03-201-1/+1
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* fix the folder shareGretaD2019-12-191-4/+4
| | | | | Signed-off-by: GretaD <gretadoci@gmail.com> Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
* Move users management to multi lineGreta Doci2019-12-121-30/+61
| | | | Signed-off-by: Greta Doci <gretadoci@gmail.com>
* Accept incoming shares in acceptance testsDaniel Calviño Sánchez2019-11-201-0/+98
| | | | | | | | | | | | | | | | | Incoming shares are no longer automatically added to the file list of the sharee. Instead, the user now needs to explictly accept the share. Currently shares can be accepted only from the Notifications app, so it must be explicitly cloned before installing Nextcloud if it is not found in the "apps" directory. Note that the development branches are already built, so there is no need to explicitly build the app. With the new sharing behaviour the "share a skeleton file with another user before first login" scenario is no longer valid (as the user will need to log in to accept the share, so at that point the skeleton is already created), so it was removed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Rename "Notification" to "Toast"Daniel Calviño Sánchez2019-11-201-9/+9
| | | | | | | | Since Nextcloud 17 the proper name for the old built-in notifications is "Toast". Moreover, this will reduce ambiguity when using the "notification" term to refer to elements in the Notifications app. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust acceptance tests to new OCA.SidebarJohn Molakvoæ (skjnldsv)2019-10-292-35/+52
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix tag acceptance tests after #16682John Molakvoæ (skjnldsv)2019-10-041-0/+16
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Remove input id on login formJohn Molakvoæ (skjnldsv)2019-09-281-1/+1
| | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com> Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
* Adjust acceptance tests to changes in link share menu behaviourGreta Doci2019-08-291-8/+0
| | | | | | | | | | | | | | | | Now the link share menu is not automatically opened after a link share is created, so waiting until it was opened failed in iShareTheLinkFor. Note that the steps that interact with the link share menu take care themselves of showing the menu if needed, so there is no need to explicitly show it despite the change. Also, the waiting in iShareTheLinkFor was introduced when the link share menu was changed to automatically open after creating a link share, as that caused some issues with the steps that opened the menu by themselves (fec8d12fc5). Due to all this, now that the link share menu is again not automatically opened the wait can be simply removed. Signed-off-by: Greta Doci <gretadoci@gmail.com>
* Adjust acceptance tests to notifications based on toastify (#15938)John Molakvoæ2019-06-131-2/+2
|\ | | | | Adjust acceptance tests to notifications based on toastify
| * Adjust acceptance tests to notifications based on toastifyDaniel Calviño Sánchez2019-06-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | The old notifications were added as ".row" elements to the "#notification-container" element; the new notifications based on toastify are added as ".toastify .on .toast..." elements to the "#content" element. Besides that, they also include a span element with an X to close the notification, so now only the first child text node should be compared to the expected message. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Add messages to assertFalse/True to clarify the failed assertionDaniel Calviño Sánchez2019-06-121-3/+6
|/ | | | | | | | Otherwise the output would just read "Failed asserting that true is false." or "Failed asserting that false is true.", which is not very informative when there are several assertFalse/True in a row. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust acceptance tests to changes of file names in file pickerDaniel Calviño Sánchez2019-04-031-1/+3
| | | | | | | | | File names are no longer shown directly in the ".filename" element, but split in two "span" elements inside a ".filename-parts" element, so now the texts in those span elements need to be concatenated to get the file name. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Adjust acceptance testsMorris Jobke2019-03-181-0/+18
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix acceptance tests for wrong password message due to changed messageMorris Jobke2019-03-111-1/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-09 02:37:27 +0000