summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix app fetcher php comparison to allow wider rangesChristoph Wurst2021-01-271-0/+8
| | | | | | | | | | | When app app specifies php 7.4 as upper limit we have to allow the installation on php>7.4.0. The previous version check didn't do that. This adjusts the regexes to discard any irrelevant suffix after the three version numbers so that we can use more fine granular checks than php's version_compare can do out of the box, like for php 7.4 we only compare the major and minor version numbers and ignore the patch level. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Add "composer.lock" for acceptance tests to gitDaniel Calviño Sánchez2021-01-182-3/+2774
| | | | | | | | | | As "composer.lock" was not versioned the dependencies had to be resolved everytime that the acceptance tests run, which took some precious time. Besides that the dependency versions were also tightened for better control. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* DB: warn on parameter number constraintsArthur Schiwon2021-01-141-6/+82
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #24592 from nextcloud/backport/23912/stable20Morris Jobke2021-01-073-0/+24
|\ | | | | [stable20] use in objectstore copy
| * use in objectstore copyRobin Appelman2020-12-073-0/+24
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Properly handle result when copying with failure and throw in testsJulius Härtl2021-01-051-1/+2
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Fix writeStream mockingJulius Härtl2021-01-051-1/+1
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Show unique displayname context in the sharing inputJulius Härtl2021-01-052-23/+23
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #24653 from nextcloud/backport/23044/stable20Roeland Jago Douma2020-12-144-0/+696
|\ \ | | | | | | [stable20] Handle owncloud migration to latest release
| * | Readd repair steps that are relevant when migrating from ownCloudJulius Härtl2020-12-144-0/+696
| |/ | | | | | | | | | | This reverts commit d9b1492e03ab9fe58bb87baaeba745790ca15c53. Signed-off-by: Julius Härtl <jus@bitgrid.net>
* / Add temporary test for migrating int to stringJulius Härtl2020-12-141-0/+49
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Revert "Do not read certificate bundle from data dir by default"Morris Jobke2020-12-041-2/+3
| | | | This reverts commit 18b0d753f2d76da10fefbf9a34e22dfdcbdc93b0.
* Fix vsprint parameterRoeland Jago Douma2020-12-031-1/+1
| | | | | | | %2\$; is not valid. On php7 this is just ignored but on php8 it gives an error. %2\$s; works. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Add notification for user limitMorris Jobke2020-12-021-1/+31
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Allow subscription to indicate that a userlimit is reachedMorris Jobke2020-12-022-3/+89
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Set frame-ancestors to none if none are filledRoeland Jago Douma2020-12-014-60/+60
| | | | | | | | 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>
* set the display name of federated sharees from addressbookArthur Schiwon2020-11-243-9/+58
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #24230 from nextcloud/backport/24007/stable20Morris Jobke2020-11-201-0/+38
|\ | | | | [stable20] allow selecting multiple columns with SELECT DISTINCT
| * allow selecting multiple columns with SELECT DISTINCTRobin Appelman2020-11-191-0/+38
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #24228 from nextcloud/backport/24135/stable20Morris Jobke2020-11-203-6/+6
|\ \ | | | | | | [stable20] Fix nextcloud logo in email notifications misalignment
| * | Fix #23357medcloud2020-11-193-6/+6
| |/ | | | | | | Signed-off-by: medcloud <42641918+medcloud@users.noreply.github.com>
* / use file name instead of path in 'not allowed to share' messageRobin Appelman2020-11-191-3/+6
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Add unit tests for "validateExpirationDateInternal"Daniel Calviño Sánchez2020-11-161-0/+284
| | | | | | | They were copied and adjusted as needed from the tests for "validateExpirationDate". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add explicit time to test date for clarityDaniel Calviño Sánchez2020-11-161-3/+5
| | | | | | | This shows in a clearer way that the expected expiration date is the original one, but without time. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add unit test for expiration date with date and with defaultDaniel Calviño Sánchez2020-11-161-0/+28
| | | | | | | This adds back what was being actually tested in the unit test fixed in the previous commit. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix unit test for expiration date without date but with defaultDaniel Calviño Sánchez2020-11-161-7/+5
| | | | | | | | | The expiration date was explicitly given, so it was not testing what it should. Moreover, "link_defaultExpDays" needs to be mapped when testing the default value (even if it will just fallback to the value of "shareapi_expire_after_n_days"). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix unit test method nameDaniel Calviño Sánchez2020-11-161-1/+1
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix unit test for verifying enforced valid expiration date.Daniel Calviño Sánchez2020-11-161-0/+1
| | | | | | | The enforced expiration date was not actually enabled, as for that "shareapi_default_expire_date" needs to be explicitly set to "yes". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix unit test for verifying enforced expire date too far in the futureDaniel Calviño Sánchez2020-11-161-8/+5
| | | | | | | | | | | | The test should fail if no exception was thrown, and no exception was thrown due to a missing configuration parameter. However, the use of "addToAssertionCount(1)" made the test pass even if no exception was thrown and, therefore, to silently fail. Now the missing configuration parameter was added and the test will fail if the expected exception is not thrown. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add unit tests to enforce a relaxed default share expiration dateDaniel Calviño Sánchez2020-11-161-0/+21
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Remove wrong and misleading commentDaniel Calviño Sánchez2020-11-161-2/+0
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Remove extra spacesDaniel Calviño Sánchez2020-11-161-10/+6
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix typo in variable nameDaniel Calviño Sánchez2020-11-161-129/+129
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix typos in method namesDaniel Calviño Sánchez2020-11-161-9/+9
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix unique key in test tableJoas Schilling2020-11-121-4/+8
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Skip the insertIfNotExists() tests on Oracle because it doesn't work with clobJoas Schilling2020-11-121-0/+20
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Easier debugging and spell fixJoas Schilling2020-11-121-6/+9
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Use a different column for the primary key as we can not insert it on oracleJoas Schilling2020-11-122-12/+36
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add github action for oci8Julius Härtl2020-11-122-3/+4
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #24044 from nextcloud/backport/24042/stable20Morris Jobke2020-11-114-0/+305
|\ | | | | [stable20] Fix sharing tab state not matching resharing admin settings
| * Add acceptance tests for reshares by link when resharing is disabledDaniel Calviño Sánchez2020-11-112-0/+111
| | | | | | | | 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-113-0/+194
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Don't leave cursors open when tests failJoas Schilling2020-11-108-24/+111
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix repair mimetype step to not leave stray cursorsJoas Schilling2020-11-101-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Use query builder instead of OC_DB in OC\Files\*Joas Schilling2020-11-091-11/+15
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also run for .sh and .json (when not package or package-lock)Joas Schilling2020-10-302-0/+8
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also run all tests when drone config is modifiedJoas Schilling2020-10-302-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Only run phpunit when php, xml or test files are modifiedJoas Schilling2020-10-302-0/+21
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Only retry fetching app store data once every 5 minutes in case it failsJulius Härtl2020-10-221-75/+36
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #23621 from nextcloud/backport/23613/stable20Morris Jobke2020-10-221-6/+7
|\ | | | | [stable20] Disable unreliable app-files.feature:108