aboutsummaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* fix(tests): Fix most obvious errors in ObjectStore testsbackport/47986/stable29Côme Chilliet2024-10-313-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-10-303-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>
* test: add test for nested jail cross-storage movebackport/47847/stable29Robin Appelman2024-10-291-0/+23
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: crypto added manually as expectedyemkareems2024-10-281-1/+1
| | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* fix: crypto type made not nullable and tests run using ICryptoyemkareems2024-10-281-4/+3
| | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* fix: encrypt and store password, decrypt and retrieve the sameyemkareems2024-10-281-2/+19
| | | | Signed-off-by: yemkareems <yemkareems@gmail.com>
* Merge pull request #48764 from nextcloud/backport/48539/stable29Joas Schilling2024-10-171-5/+5
|\ | | | | [stable29] fix(message): Fix logic of Mail test
| * fix(message): Fix logic of Mail testbackport/48539/stable29Joas Schilling2024-10-171-5/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #48588 from nextcloud/backport/47896/stable29Ferdinand Thiessen2024-10-172-14/+70
|\ \ | |/ |/| [stable29] fix: Make user removal more resilient
| * fix(UserTrait): Fix backend initializationbackport/47896/stable29provokateurin2024-10-151-0/+3
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
| * fix: Make user removal more resilientFerdinand Thiessen2024-10-151-14/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently there is a problem if an exception is thrown in `User::delete`, because at that point the user is already removed from the backend, but not all data is deleted. There is no way to recover from this state, as the user is gone no information is available anymore. This means the data is still available on the server but can not removed by any API anymore. The solution here is to first set a flag and backup the user home, this can be used to recover failed user deletions in a way the delete can be re-tried. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(share): Return empty string if no label is setbackport/48673/stable29Ferdinand Thiessen2024-10-151-0/+34
|/ | | | | | | | | * Resolves: https://github.com/nextcloud/server/issues/48629 While the database supports NULL, the typing has always said it only returns *string*. So to not break any apps that might trust the typings we should return `''` if the database is set to `NULL`. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: gracefully parse non-standard trusted certificatesbackport/48307/stable29Richard Steinmetz2024-09-302-2/+32
| | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* fix(users): Don't crash if disabled user is missing in the databasebackport/48207/stable29Louis Chemineau2024-09-235-30/+48
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(tests): Fix Folder testsCôme Chilliet2024-09-191-3/+22
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* test: put parent items into cache in testsRobin Appelman2024-09-195-49/+82
| | | | 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 #48100 from nextcloud/backport/46140/stable29Robin Appelman2024-09-161-0/+7
|\ | | | | [stable29] fix(config): Add missing handling for `envCache` in `getKeys()`
| * fix(config): add envCache tests for getKeys()Josh Richards2024-09-161-0/+7
| | | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | fix(Entity): Fix magic setter call for custom strong typed settersbackport/48008/stable29provokateurin2024-09-161-2/+7
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #47941 from nextcloud/backport/47933/stable29Joas Schilling2024-09-161-0/+36
|\ | | | | [stable29] fix(config): Throw PreconditionException always when it didn't match
| * fix(config): Throw PreconditionException always when it didn't matchbackport/47933/stable29Joas 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>
* | fix(files): Check if the target path is a descendant of the shared folder pathbackport/47756/stable29Git'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(files): Create non-existent parents of mountpointsprovokateurin2024-09-041-0/+31
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(tests): Fix PHP 8.0 compatibilityCôme Chilliet2024-08-131-1/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(tests): Adapt tests to change of DefaultShareProvider constructorCôme Chilliet2024-08-131-6/+15
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #46889 from nextcloud/backport/46476/stable29Andy Scherzinger2024-08-081-0/+209
|\ | | | | [stable29] Migration Attributes
| * feat(upgrade): migration attributesMaxence Lange2024-08-061-0/+209
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | fix(session): Log when session_* calls are slowChristoph Wurst2024-08-073-17/+17
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* fix: `OCP\Files\Node\Folder::search` was not setting the ownerFerdinand Thiessen2024-07-301-3/+11
| | | | | | The owner was not set on the file info causing e.g. webdav searches to never return the known owner. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Ignore preview requests for invalid file idsJulius Härtl2024-07-291-0/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* feat: support excalidraw fileHoang Pham2024-07-252-2/+4
| | | | Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
* fix(mail): Fix big logos in mail templates for OutlookJoas Schilling2024-07-195-3/+8
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* feat: don't count failed CSRF as failed login attemptBenjamin Gaussorgues2024-07-111-1/+0
| | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* test: run tests in child processDaniel Kesselberg2024-07-101-0/+8
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix(tests): Strong type property $router in RouterTestCôme Chilliet2024-07-101-2/+1
| | | | Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
* fix(settings): define a 'heartbeat' route, so SecurityHeaders can handle ↵Steve Mokris2024-07-101-5/+15
| | | | | | redirected root Signed-off-by: Steve Mokris <steve@kosada.com>
* test: update DummyJobListRobin Appelman2024-07-101-4/+18
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(tests): Remove output when running testsJoas Schilling2024-07-031-0/+15
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(federation): Fix missing protocol on CloudID remoteJoas Schilling2024-06-272-16/+13
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* feat: add setup check for needed mimetype migrationsRobin Appelman2024-06-191-2/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #44977 from nextcloud/backport/44745/stable29Arthur Schiwon2024-06-121-3/+6
|\ | | | | [stable29] fix(auth): Keep redirect URL during 2FA setup and challenge
| * fix(auth): Keep redirect URL during 2FA setup and challengeChristoph Wurst2024-04-231-3/+6
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #45821 from nextcloud/backport/45811/stable29Daniel2024-06-121-0/+78
|\ \ | | | | | | [stable29] test: add tests for ProfilePageController
| * | test: add tests for ProfilePageControllerDaniel Kesselberg2024-06-121-0/+78
| | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #45809 from nextcloud/fix/noid/fix-new-token-scope-nameArthur Schiwon2024-06-121-1/+1
|\ \ \ | |/ / |/| | [stable29] fix(Token): make new scope future compatible
| * | fix(Token): make new scope future compatibleArthur Schiwon2024-06-121-1/+1
| | | | | | | | | | | | | | | | | | | | | - "password-unconfirmable" is the effective name for 30, but a draft name was backported. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | fix(files_sharing): dark avatar supportskjnldsv2024-06-121-2/+13
|/ / | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | fix(Session): avoid password confirmation on SSOArthur Schiwon2024-06-112-1/+63
| | | | | | | | | | | | | | | | | | | | | | SSO backends like SAML and OIDC tried a trick to suppress password confirmations as they are not possible by design. At least for SAML it was not reliable when existing user backends where used as user repositories. Now we are setting a special scope with the token, and also make sure that the scope is taken over when tokens are regenerated. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | fix(Mailer): Allow to enforce strict email formatFerdinand Thiessen2024-06-072-9/+33
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>