summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #23912 from nextcloud/objectstore-copyRoeland Jago Douma2020-11-253-0/+24
|\ | | | | use in objectstore copy
| * use in objectstore copyRobin Appelman2020-11-243-0/+24
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | set the display name of federated sharees from addressbookArthur Schiwon2020-11-243-9/+58
|/ | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* [encryption] Remove dependency fetching inside the constructor and move them ↵Morris Jobke2020-11-221-1/+3
| | | | | | to method call parameters Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #24007 from nextcloud/select-distinct-multipleMorris Jobke2020-11-191-0/+38
|\ | | | | allow selecting multiple columns with SELECT DISTINCT
| * allow selecting multiple columns with SELECT DISTINCTRobin Appelman2020-11-161-0/+38
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #24135 from ↵Roeland Jago Douma2020-11-193-6/+6
|\ \ | | | | | | | | | | | | medical-cloud/fix/23357-nextcloud-logo-in-email-notifications-is-misaligned-in-version-20 Fix nextcloud logo in email notifications misalignment
| * | Fix #23357medcloud2020-11-183-6/+6
| | | | | | | | | | | | Signed-off-by: medcloud <42641918+medcloud@users.noreply.github.com>
* | | Merge pull request #24189 from nextcloud/enh/csp/frame-ancestorsRoeland Jago Douma2020-11-184-60/+60
|\ \ \ | | | | | | | | Set frame-ancestors to none if none are filled
| * | | Set frame-ancestors to none if none are filledRoeland Jago Douma2020-11-184-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>
* / | Remove some IRouter methodsRoeland Jago Douma2020-11-171-3/+3
|/ / | | | | | | | | | | | | | | | | This is not the end. IRouter needs to burn. But it is a start. 🎵 we didn't start the fire 🎵 Signed-off-by: Roeland Jago Douma <roeland@famdouma.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-121-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-121-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-121-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-121-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-121-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-121-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-121-0/+21
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Remove wrong and misleading commentDaniel Calviño Sánchez2020-11-121-2/+0
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Remove extra spacesDaniel Calviño Sánchez2020-11-121-10/+6
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fix typo in variable nameDaniel Calviño Sánchez2020-11-121-129/+129
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fix typos in method namesDaniel Calviño Sánchez2020-11-121-9/+9
|/ | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Change to full class names for the encryption app DI in preparation of ↵Morris Jobke2020-11-111-2/+2
| | | | | | auto-wiring Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #23882 from nextcloud/tests/oracleJoas Schilling2020-11-114-25/+77
|\ | | | | Run unit tests against oracle
| * Fix unique key in test tableJoas Schilling2020-11-101-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-101-0/+20
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Easier debugging and spell fixJoas Schilling2020-11-101-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-102-12/+36
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Add github action for oci8Julius Härtl2020-11-102-3/+4
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | 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-098-24/+111
| | | | 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>
* Merge pull request #23934 from nextcloud/enh/avoid_scss_compileJohn Molakvoæ2020-11-071-0/+5
|\
| * Fix tests to return config default valuesMorris Jobke2020-11-071-0/+5
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #23851 from nextcloud/enh/read_env_only_onceMorris Jobke2020-11-061-33/+49
|\ \ | | | | | | Read the env variables only once
| * | Fix testsRoeland Jago Douma2020-11-061-33/+49
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Fix repair mimetype step to not leave stray cursorsJoas Schilling2020-11-061-1/+1
| |/ |/| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Add a function to get the unread count for multiple objects in one goJoas Schilling2020-11-041-0/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #21693 from ↵Morris Jobke2020-11-033-47/+24
|\ \ | | | | | | | | | | | | nextcloud/fix/noid/import-certificates-only-by-system Improve CertificateManager to not be user context dependent
| * | Improve CertificateManager to not be user context dependentMorris Jobke2020-11-033-47/+24
| |/ | | | | | | | | | | | | | | | | * removes the ability for users to import their own certificates (for external storage) * reliably returns the same certificate bundles system wide (and not depending on the user context and available sessions) The user specific certificates were broken in some cases anyways, as they are only loaded if the specific user is logged in and thus causing unexpected behavior for background jobs and other non-user triggered code paths. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* / Remove OCSResponse type hint - see #23827Morris Jobke2020-11-031-1/+0
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Remove deprecated OCSResponseRoeland Jago Douma2020-11-011-64/+0
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #23788 from ↵Morris Jobke2020-10-302-0/+33
|\ | | | | | | | | nextcloud/techdebt/noid/only-run-phpunit-when-php-changed Only run phpunit when "php" changed
| * 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>
* | use file name instead of path in 'not allowed to share' messageRobin Appelman2020-10-291-3/+6
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #23765 from ↵Morris Jobke2020-10-291-23/+31
|\ \ | | | | | | | | | | | | nextcloud/techdebt/noid/sudadmin-events-into-typed-events Add typed events for adding and removing a subadmin