summaryrefslogtreecommitdiffstats
path: root/build/integration
Commit message (Collapse)AuthorAgeFilesLines
* Update license headersChristoph Wurst2020-12-302-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Make integration tests work with both PHP 7.3 and 7.4Daniel Calviño Sánchez2020-12-304-3/+5
| | | | | | | | | | | | | | | | The "Trashbin" and "WebDav" traits were using each other in a circular dependency ("WebDav" -> "Sharing" -> "Provisioning" -> "BasicStructure" -> "Trashbin" -> "WebDav"). In PHP 7.3 this worked fine, but in PHP 7.4 the fatal error "Trait 'WebDav' not found in .../Trashbin.php" was thrown. To solve this now the "TrashBin" trait no longer explicitly uses "WebDav". However, due to this change, the class using "TrashBin" is now expected to also use "WebDav". As the "Trashbin" trait was not needed by most contexts using the "BasicStructure" trait "Trashbin" was removed from it and added only to those contexts that actually need it. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Update all license headers for Nextcloud 21Christoph Wurst2020-12-1612-9/+14
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #24486 from nextcloud/feature/noid/phone-number-validationJoas Schilling2020-12-082-4/+65
|\ | | | | Phone number validation and search
| * Unit tests for searching by phone numberJoas Schilling2020-12-071-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Add a config for default region of phone numbersJoas Schilling2020-12-071-2/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Add an integration test for the phone search APIJoas Schilling2020-12-072-4/+65
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Add integration tests for getting guest avatarsDaniel Calviño Sánchez2020-12-072-0/+30
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Add integration tests for resized user avatarsDaniel Calviño Sánchez2020-12-072-7/+88
| | | | | | | | | | | | | | | | Even on solid color images the resizing can cause some small artifacts that slightly modify the color of certain pixels. Due to this now the color comparison is no longer strict but fuzzy. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Add integration tests for user avatarsDaniel Calviño Sánchez2020-12-075-0/+344
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Make possible to set body in requesttoken requests in integration testsDaniel Calviño Sánchez2020-12-071-7/+17
| | | | | | | | | | | | | | | | | | | | "sendingAToWithRequesttoken" needs to be used to test some non OCS endpoints which require the request token to be sent in the request. Now it is possible to specify the body (or, rather, additional contents beside the cookies and the request token) for those requests, as it will be needed for example to upload an avatar. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Make possible to send requests as anonymous users in integration testsDaniel Calviño Sánchez2020-12-071-2/+2
|/ | | | | | | | | Until now requests always had "auth" headers either for an admin or a regular user, depending on the value of "currentUser". Now, if "currentUser" starts by "anonymous" no "auth" header is sent, which makes possible to also test requests with users not logged in. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Set frame-ancestors to none if none are filledRoeland Jago Douma2020-11-181-1/+1
| | | | | | | | frame-ancestors doesn't fall back to default-src. So when we apply a very restricted CSP we should make sure to set it to 'none' and not leave it empty. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Add integration tests for creating shares with default expiration datesDaniel Calviño Sánchez2020-11-162-0/+112
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* 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-112-0/+35
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Update behat/behat requirement in /build/integrationdependabot-preview[bot]2020-11-071-1/+1
| | | | | | | | Updates the requirements on [behat/behat](https://github.com/Behat/Behat) to permit the latest version. - [Release notes](https://github.com/Behat/Behat/releases) - [Changelog](https://github.com/Behat/Behat/blob/master/CHANGELOG.md) - [Commits](https://github.com/Behat/Behat/compare/v3.7.0...v3.8.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.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>
* Merge pull request #22761 from ↵Morris Jobke2020-09-141-1/+216
|\ | | | | | | | | nextcloud/extend-integration-tests-for-files-transfer-ownership-command Extend integration tests for "files:transfer-ownership" command
| * 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 behat/behat requirement in /build/integrationdependabot[bot]2020-09-081-1/+1
|/ | | | | | | | Updates the requirements on [behat/behat](https://github.com/Behat/Behat) to permit the latest version. - [Release notes](https://github.com/Behat/Behat/releases) - [Changelog](https://github.com/Behat/Behat/blob/master/CHANGELOG.md) - [Commits](https://github.com/Behat/Behat/compare/v3.6.1...v3.7.0) Signed-off-by: dependabot[bot] <support@github.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>
* Add more integration tests for resharing permissionsDaniel Calviño Sánchez2020-07-091-0/+244
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Merge pull request #21264 from stweil/masterMorris Jobke2020-07-054-50/+50
|\ | | | | build: Fix some typos (most of them found by codespell)
| * 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 another test for exact matching of a user in group restrictionJoas Schilling2020-06-171-0/+17
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add integration tests for video verificationDaniel Calviño Sánchez2020-05-293-0/+577
| | | | | | | | | | | | | | 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-295-0/+305
| | | | | | | | | | | | | | | | | | | | | 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-292-4/+18
| | | | | | | 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-292-10/+7
| | | | | | The step names were adjusted accordingly. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Update public link share permission codeRoeland Jago Douma2020-05-151-2/+2
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Update license headers for 19Christoph Wurst2020-04-2912-0/+12
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #19793 from nextcloud/bugfix/noid/link-to-federated-reshareMorris Jobke2020-04-271-3/+3
|\ | | | | Fix resharing of federated shares that were created out of links