aboutsummaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * | fix(tests): Fix most obvious errors in ObjectStore testsbackport/30/fix_move_on_same_bucketCôme Chilliet2024-09-193-56/+60
| | | | | | | | | | | | | | | | | | Some are still failing Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | perf(ObjectStoreStorage): Improve (slow) move on same object bucketChristoph Fiehe2024-09-193-0/+178
| | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes the issue #47856. When you upload a file into a group folder and when you use a single S3 bucket as primary storage, the final move operation hangs for a long time. In the background, Nextcloud initiates a copy-delete sequence from the bucket into the bucket, with causes a lot unnecessary overhead. Nextcloud thinks that the file must be imported to another storage and does not recognize that everything is done on the same object bucket. In that case, the import step can be completely skipped, which saves time, network bandwidth and reduces the load on the object storage. The behavior improves a lot with https://github.com/nextcloud/server/pull/46013. However, there are still some put messages that are being sent to the object storage when you use an object storage as primary storage and upload files into a group folder. Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Christoph Fiehe <c.fiehe@eurodata.de>
* | | fix(tests): Fix Folder testsCôme Chilliet2024-09-191-3/+22
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | fix(tests): Remove deprecated tests for ServerTestCôme Chilliet2024-09-191-113/+7
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | test: put parent items into cache in testsRobin Appelman2024-09-195-50/+83
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | test: fix incorrect ltrim usage in testRobin Appelman2024-09-191-2/+2
|/ / | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #48088 from nextcloud/backport/48008/stable30Kate2024-09-171-2/+7
|\ \
| * | fix(Entity): Fix magic setter call for custom strong typed settersbackport/48008/stable30provokateurin2024-09-161-2/+7
| |/ | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* / fix(config): add envCache tests for getKeys()Josh Richards2024-09-161-0/+7
|/ | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* Merge pull request #47940 from nextcloud/backport/47933/stable30Andy Scherzinger2024-09-151-0/+36
|\ | | | | [stable30] fix(config): Throw PreconditionException always when it didn't match
| * fix(config): Throw PreconditionException always when it didn't matchbackport/47933/stable30Joas Schilling2024-09-131-0/+36
| | | | | | | | | | | | | | | | Previously even when the precondition did not match, the call "passed" when the after value was the expected one. This however can lead to race conditions, duplicate code excutions and other things. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #47796 from nextcloud/backport/47756/stable30Andy Scherzinger2024-09-151-1/+9
|\ \ | |/ |/| [stable30] fix(files): Check if target path is a descendant of the shared folder
| * fix(files): Check if the target path is a descendant of the shared folder pathbackport/47756/stable30Git'Fellow2024-09-061-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: fix tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: add tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests
* | fix: Add whiteboard to education and public sector bundleJulius Knorr2024-09-121-0/+1
|/ | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* fix(files): Create non-existent parents of mountpointsprovokateurin2024-09-041-0/+31
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Adjust filename validation messagesFerdinand Thiessen2024-08-301-1/+1
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(TaskProcessing): Fix namespace of TaskProcessingTest.phpMarcel Klehr2024-08-301-1/+1
| | | | | Signed-off-by: Marcel Klehr <mklehr@gmx.net> Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* fix(taskprocessing): fix testsJulien Veyssier2024-08-302-13/+5
| | | | Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
* test(phpunit): skip sharding tests on 32bitArthur Schiwon2024-08-292-0/+6
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix: Renaming does not need update but delete permissionsFerdinand Thiessen2024-08-281-3/+3
| | | | | | | Renaming is basically copy + delete (a move), so no need to update permissions. Especially if the node is in a invalid directory the node should be moveable but not editable. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Backport to 30Louis Chemineau2024-08-281-2/+2
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* chore: Apply php:cs recommendationsLouis Chemineau2024-08-285-18/+18
| | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me> [skip ci]
* fix: implement sharding compatible cleanup for various bitsRobin Appelman2024-08-282-9/+8
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* test: fix share provider tests for shardingRobin Appelman2024-08-282-6/+6
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* test: run sharding tests in ciRobin Appelman2024-08-281-0/+21
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* test: mark share test cleanup as running across all shardsRobin Appelman2024-08-281-0/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat: implement distributing partitioned queries over multiple shardsRobin Appelman2024-08-285-2/+154
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat: add option to automatically partition queries by specific tablesRobin Appelman2024-08-282-0/+290
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: delay calculating global cache prefix untill a cache is createdRobin Appelman2024-08-281-3/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat: add negative compare-and-delete to imemcacheRobin Appelman2024-08-231-0/+23
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: `FilenameValidator::isForbidden` should only check forbidden filesFerdinand Thiessen2024-08-221-17/+7
| | | | | | And not forbidden basenames as this is used for different purposes. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* ci: Skip flaky test on PHP 8.3Joas Schilling2024-08-211-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #43573 from nextcloud/HolgerHees-fix-csp-nonce-handlingFerdinand Thiessen2024-08-134-262/+267
|\ | | | | Fix for ignored `CSP_NONCE` in ContentSecurity Header
| * feat: Provide CSP nonce as `<meta>` elementFerdinand Thiessen2024-08-131-1/+0
| | | | | | | | | | | | | | | | This way we use the CSP nonce for dynamically loaded scripts. Important to notice: The CSP nonce must NOT be injected in `content` as this can lead to value exfiltration using e.g. side-channel attacts (CSS selectors). Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * test: Adjust tests for CSP nonceFerdinand Thiessen2024-08-134-261/+267
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | feat(auth): Clean-up unused auth tokens and wipe tokensChristoph Wurst2024-08-131-11/+10
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | fix(tests): Adapt tests to change of DefaultShareProvider constructorCôme Chilliet2024-08-121-6/+15
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(SetupCheck): Properly check public access to data directoryFerdinand Thiessen2024-08-082-10/+10
| | | | | | | | | | | | | When checking for public (web) access to the data directory the status is not enough as you might have a webserver that forwards to e.g. a login page. So instead check that the content of the file matches. For this the `.ncdata` file (renamed from `.ocdata`¹) has minimal text content to allow checking. ¹The file was renamed from the legacy `.ocdata`, there is a repair step to remove the old one. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(session): Log when session_* calls are slowChristoph Wurst2024-08-073-17/+17
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* fix: Fix failing test for group listingJulius Härtl2024-08-061-0/+6
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #46967 from nextcloud/fix/share-not-foundJohn Molakvoæ2024-08-061-2/+4
|\
| * fix(files_sharing): show proper share not found error messageskjnldsv2024-08-061-2/+4
| | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | Merge pull request #46985 from nextcloud/feat/email-share-formatJohn Molakvoæ2024-08-063-20/+20
|\ \
| * | fix: emails tests after new design fixesskjnldsv2024-08-063-20/+20
| | | | | | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | | chore(tests): Adapt tests to the group:list changeCôme Chilliet2024-08-061-28/+32
| |/ |/| | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix: unify bundle namingJérôme Herbinet2024-08-061-1/+1
| | | | | | | | | | Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com> Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* | chore: delete repair step for 8y old oc_mounts issueRobin Appelman2024-08-021-114/+0
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* chore: Remove deprecated legacy search backendJulius Härtl2024-08-011-2/+0
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #46476 from nextcloud/enh/noid/migration-attributesStephan Orbaugh2024-07-302-4/+386
|\ | | | | Migration Attributes
| * feat(migration-attributes): testsMaxence Lange2024-07-292-4/+386
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>