summaryrefslogtreecommitdiffstats
path: root/build/integration/features
Commit message (Collapse)AuthorAgeFilesLines
* Reset app configs by deleting the values instead of setting the defaultsDaniel Calviño Sánchez2020-11-165-18/+31
| | | | | | | | This avoids the need to keep the default values in the integration tests in sync with the code, and also makes possible to reset values with "dynamic" defaults (defaults that depend on other values). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration tests for default share permissionsDaniel Calviño Sánchez2020-11-111-0/+1
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix the expected outputJoas Schilling2020-10-271-6/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Simplify the function looking for outputJoas Schilling2020-10-271-27/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* This is not javascriptJoas Schilling2020-10-261-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix undefined variableJoas Schilling2020-10-261-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Format code to a single space around binary operatorsChristoph Wurst2020-10-053-6/+6
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add integration tests to check that only the given path is transferredDaniel Calviño Sánchez2020-09-181-0/+51
| | | | | | | | | | | | Until recently (it was fixed in ac2999a26a) when a path was transferred other shares with the target user were removed, so a test was added to ensure that it does not happen again. Besides that a test to ensure that other files with the target user are not transferred was added too (it did not fail before, but seemed convenient to have that covered too :-) ). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration tests for transferring files of a user with a risky nameDaniel Calviño Sánchez2020-09-183-1/+41
| | | | | | | | | | | The files:transfer-ownership performs a sanitization of users with "risky" display names (including characters like "\" or "/"). In order to allow (escaped) double quotes in the display name the regular expression used in the "user XXX with displayname YYY exists" step had to be adjusted. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration test for transferring the path of a single fileDaniel Calviño Sánchez2020-09-181-0/+14
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration tests for transferring ownership of resharesDaniel Calviño Sánchez2020-09-101-0/+100
| | | | | | | | | | | | | | | | | | Currently only transferring ownership of a reshare with a group to a user in the group is possible. When transferring ownership of a reshare with another user or with a group to a user not in the group restoring the share fails (but the command succeeds, it only fails for the specific files that are reshares). When transferring ownership of a path that is a reshare the command fails (as when a specific path is provided the path tries to move the file, it does not take into account reshares). The added integration tests reflect the above behaviours. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Check share ownership after transferring file ownershipDaniel Calviño Sánchez2020-09-101-0/+62
| | | | | | | | The integration tests did not verify that the shares were actually transferred between the users (or that they were removed due to being transferred to the sharee). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Check whether file exists or not after transferring ownershipDaniel Calviño Sánchez2020-09-101-1/+54
| | | | | | | The integration tests did not verify that the files were actually transferred between the users, only that the files were downloadable. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Update license headers for Nextcloud 20 (again)Christoph Wurst2020-09-071-0/+1
| | | | | | | There are still lots of outdated headers, so time for another round of updates. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Change Content-Disposition HeaderJacob Neplokh2020-09-051-1/+1
| | | | | | | | | | - Make ALLOWED_CONTENT_TYPES public in order to use - Add $fileName variable which uses "$node->getName()" to get the proper file name and "$file->getMimeType()" along with the ALLOWED_CONTENT_TYPES array in PhotoCache.php to get the proper file extension - Make "$fileName" the name of the file in the Content-Disposition header when downloading a Contact's photo - Add filename to the CardDAV integration image export test header - Change headers in ImageExportPluginTest to reflect changes Signed-off-by: Jacob Neplokh <me@jacobneplokh.com>
* Update the license headers for Nextcloud 20Christoph Wurst2020-08-243-1/+23
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* new weather status used in dashboardJulien Veyssier2020-08-201-0/+1
| | | | Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
* Merge pull request #21138 from ↵Morris Jobke2020-08-052-1/+61
|\ | | | | | | | | nextcloud/fix/noid/search-in-group-displayname-email Search also the email and displayname in user mangement for groups
| * Add integration tests for search in groupsMorris Jobke2020-08-052-1/+61
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Fix integration test failJoas Schilling2020-08-031-0/+1
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix integration test due to new dashboard appMorris Jobke2020-07-151-0/+1
| | | | | | See #21346 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* build: Fix two typos in function namesStefan Weil2020-06-061-2/+2
| | | | Signed-off-by: Stefan Weil <sw@weilnetz.de>
* build: Fix some typos (most of them found by codespell)Stefan Weil2020-06-064-48/+48
| | | | Signed-off-by: Stefan Weil <sw@weilnetz.de>
* Add integration tests for video verificationDaniel Calviño Sánchez2020-05-291-0/+72
| | | | | | | | | | | | | | Enabling the "send password by Talk" property of shares require that Talk is installed and enabled, so the Drone step that runs them has to first clone the Talk repository. When the integration tests are run on a local development instance, however, it is not guaranteed that Talk is installed. Due to this the "@Talk" tag was added, which ensures that any feature or scenario marked with it will first check if Talk is installed and, if not, skip the scenario (instead of failing). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration tests for creating and updating a mail shareDaniel Calviño Sánchez2020-05-294-0/+219
| | | | | | | | | | | | | | | | | | | | | In most cases, when a mail share is created or updated an e-mail is sent to the sharee, which is done by connecting to the SMTP server set in the configuration. If the server can not be contacted then the creation or update of the mail share fails. To make possible to test mail shares without using a real SMTP server a fake one has been added. The original script, which is MIT licensed, was based on inetd, so it was slightly modified to run on its own. In order to use it from the integration tests the "Given dummy mail server is listening" step has to be called in the scenarios in which the mail server is needed. For now that is the only available step; things like checking the sent mails, while possible (as the script can log the mails to certain file), have not been added yet. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Generalize integration test steps to download last shareDaniel Calviño Sánchez2020-05-291-2/+16
| | | | | | | Note that the "last link share can be downloaded" step was kept as it tests the "url" property specific of link shares. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Remove unused variablesDaniel Calviño Sánchez2020-05-291-7/+4
| | | | | | The step names were adjusted accordingly. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Update license headers for 19Christoph Wurst2020-04-2912-0/+12
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix app not listed in provisioning integration testDaniel Calviño Sánchez2020-04-131-0/+1
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-1015-122/+90
| | | | | | | | | | | | | | | 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-104-5/+5
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Make sure every file ends with an empty newlineChristoph Wurst2020-04-092-2/+0
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Remove trailing and in between spacesChristoph Wurst2020-04-091-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Unify function spacing to PSR2 recommendationChristoph Wurst2020-04-099-29/+29
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update the license headers for Nextcloud 19Christoph Wurst2020-03-315-2/+4
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #20170 from nextcloud/techdebt/remove-unused-importsChristoph Wurst2020-03-276-13/+0
|\ | | | | Remove unused imports
| * Remove unused importsChristoph Wurst2020-03-256-13/+0
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use the short array syntax, everywhereChristoph Wurst2020-03-263-9/+9
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use lowercase true, false and null constantsChristoph Wurst2020-03-251-12/+12
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix testsJohn Molakvoæ (skjnldsv)2019-12-101-0/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Update license headersChristoph Wurst2019-12-0525-33/+75
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Do not call count on nullJulius Härtl2019-11-291-2/+6
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Accept incoming shares in integration testsDaniel Calviño Sánchez2019-11-209-0/+90
| | | | | | | | | Now all incoming shares need to be explicitly accepted before being able to use the shared file or get information about a reshare (although getting the information of the incoming share is possible before accepting it). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Replace direct calls to share API with specific share stepsDaniel Calviño Sánchez2019-11-203-30/+30
| | | | | | | This makes possible to use steps that reference the last share, which will be needed to accept pending shares. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add integration tests for getting shares including subfilesDaniel Calviño Sánchez2019-10-251-0/+108
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Replace fully qualified name with aliasDaniel Calviño Sánchez2019-10-251-10/+11
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Do not expect a 200 HTTP status code when updating a shareDaniel Calviño Sánchez2019-10-041-2/+0
| | | | | | | | | | | | | This will be needed to test scenarios in which updating a share return a different HTTP status code, like 401. The assertion for the 200 HTTP status code was added in those scenarios that tested updating a share (that is, those that were also checking the OCS status code), but not in those in which updating a share was just a preparatory step for the actual test (in the same way that the HTTP status code is not checked in those tests when creating a share). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Move sharing integration tests to their own directoryJohn Molakvoæ (skjnldsv)2019-10-044-1274/+40
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Move settings to an appChristoph Wurst2019-09-281-0/+1
| | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at> Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
* Better check reshare permissions when creating a shareJoas Schilling2019-07-031-0/+60
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>