aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/tests
Commit message (Collapse)AuthorAgeFilesLines
* test(files_sharing): split `testCanAccessShare` into separate tests and fix ↵test/files-sharing-phpunitFerdinand Thiessen2025-06-021-36/+81
| | | | | | | | | | for PHPUnit 10 - split the test into individual test cases - fix invalid call to `onConsecutiveCalls` (it was called more than defined values and is deprecated in v10 anyways). Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* test(files_sharing): adjust mocks of `Talk` classestest/fix-files-sharingFerdinand Thiessen2025-05-311-7/+19
| | | | | | | Classes need to exist to be mocked (reflection), thus unknown classes only can be mocked as `stdClass`. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* test(files_sharing): adjust tests for PHPUnit v10Ferdinand Thiessen2025-05-315-131/+79
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): show note, label and list of uploaded files on file dropFerdinand Thiessen2025-05-261-2/+6
| | | | | | | | | This was missing from the Vue migration of the public share view: - Show the note as the description of the file drop - Show the label as the heading of the file drop if available - Show list of uploaded files for verification Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #52810 from ↵Louis2025-05-221-4/+12
|\ | | | | | | nextcloud/artonge/feat/do_not_require_samesite_strict_cookie_on_public.php
| * fix: Replace the deprecated direct download link with the public DAV endpointartonge/feat/do_not_require_samesite_strict_cookie_on_public.phpLouis Chemineau2025-05-211-4/+12
| | | | | | | | | | | | Follow-up of #48098 Signed-off-by: Louis Chemineau <louis@chmn.me>
* | fix(tests): Register mount in loginHelper as wellfix/noid/encrypted-propagation-testMarcel Müller2025-05-191-2/+7
| | | | | | | | Signed-off-by: Marcel Müller <marcel-mueller@gmx.de>
* | refactor: move tag handling from files handler to files_sharingchore/remove-dead-code-filesFerdinand Thiessen2025-05-162-6/+47
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #48560 from nextcloud/fix/migrate-encryption-away-from-hooksCôme Chilliet2025-05-142-3/+12
|\ | | | | feat(encryption): Migrate from hooks to events
| * fix(tests): Set encryption configuration even earlier so that all users are ↵fix/migrate-encryption-away-from-hooksCôme Chilliet2025-05-131-1/+5
| | | | | | | | | | | | created with private key Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix(tests): Avoid user login before a private key is setupCôme Chilliet2025-05-132-3/+8
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix(files_sharing): add mime fallback to public preview toofix/mime-fallback-publicJohn Molakvoæ (skjnldsv)2025-05-131-1/+3
|/ | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* fix(files_sharing): bring back owner and ownerDisplayName initial statefix/public-displayname-ownerskjnldsv2025-04-291-0/+4
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix: Replace all usages of OC_User backend method calls by IUserManagerfix/cleanup-user-backendsCôme Chilliet2025-04-101-4/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: more optimized caching for share target verificationshared-target-verify-cacheRobin Appelman2025-04-081-1/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(phpunit10): Migrate away from PHPUnit at() callsJoas Schilling2025-03-271-20/+18
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(lookup-server): disable lookup server for non-global scale setupsFerdinand Thiessen2025-03-111-6/+7
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(lookup-server): do not query data by defaultFerdinand Thiessen2025-03-111-1/+1
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Replace OC_App calls by IAppManagerCôme Chilliet2025-02-241-3/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(files_sharing): block downloading if neededfix/download-permsFerdinand Thiessen2025-02-181-0/+29
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore: Apply new rector configuration to apps folderCôme Chilliet2025-02-1320-119/+166
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: re-apply current rector configuration to apps folderCôme Chilliet2025-02-131-58/+59
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* test: add test for shared storage root being changed after watchershared-cache-watcher-updateRobin Appelman2025-02-061-0/+35
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(sharing): Ensure download restrictions are not droppedfix/proper-download-checkFerdinand Thiessen2025-02-041-69/+79
| | | | | | | | When a user receives a share with share-permissions but also with download restrictions (hide download or the modern download permission attribute), then re-shares of that share must always also include those restrictions. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): Respect permissions passed when creating link sharesFerdinand Thiessen2025-01-281-146/+155
| | | | | | | | | | | | | | | | | Given: User creates a link or email share with permissions=4 (create only = file drop). Problem: Currently the permissions are automatically extended to permissions = 5 (READ + CREATE). Work around was to create the share and directly update it. Solution: Respect what the user is requesting, create a file drop share. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* test: Update testsChristopher Ng2025-01-151-0/+2
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* feat(federatedfilesharing): auto-accept shares from trusted serversskjnldsv2025-01-091-6/+6
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* test: add test for permissions of copied shareRobin Appelman2024-12-041-0/+26
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* chore: remove legacy action to test remote endpointBenjamin Gaussorgues2024-11-291-89/+0
| | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* Merge pull request #43025 from ↵Côme Chilliet2024-11-251-0/+116
|\ | | | | | | | | nextcloud/bugfix/error-on-reshare-after-transfer-ownership fix: update re-share if shared-by user has been revoked
| * fix(sharing): add command to fix broken shares after ownership transferringbugfix/error-on-reshare-after-transfer-ownershipLuka Trovic2024-11-181-0/+116
| | | | | | | | Signed-off-by: Luka Trovic <luka@nextcloud.com>
* | feat(files_sharing): Allow users with share permission to manage shares on ↵provokateurin2024-11-251-0/+124
|/ | | | | | IShareOwnerlessMount Signed-off-by: provokateurin <kate@provokateurin.de>
* feat(files_sharing): use share node or label as page titlefeat/sharing-titleskjnldsv2024-11-081-0/+3
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix: Adjust preview for view-only sharesfix/view-only-previewFerdinand Thiessen2024-10-281-10/+111
| | | | | | | | | | | | | | | | | Previously there was a different behavior for public shares (link-shares) and internal shares, if the user disabled the view permission. The legacy UI for public shares simply "disabled" the context menu and hided all download actions. With Nextcloud 31 all share types use the consistent permissions attributes, which simplifies code, but caused a regression: Images can no longer been viewed. Because on 30 and before the attribute was not set, previews for view-only files were still allowed. Now with 31 we need a new way to allow "viewing" shares. So this is allowing previews for those files, but only for internal usage. This is done by settin a special header, which only works with custom requests, and not by opening the URL directly. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): Cleanup error messagesfix/files_sharing/cleanup-error-messagesprovokateurin2024-10-282-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(apps): Use constructor property promotion when possibleprovokateurin2024-10-2116-29/+44
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(deps): Bump nextcloud/coding-standard in /vendor-bin/cs-fixerdependabot/composer/vendor-bin/cs-fixer/nextcloud/coding-standard-1.3.2dependabot[bot]2024-10-191-1/+1
| | | | | | | | | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 1.3.1 to 1.3.2. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v1.3.1...v1.3.2) --- updated-dependencies: - dependency-name: nextcloud/coding-standard dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-1527-505/+562
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor(files_sharing): Add Storage parameter strong typesprovokateurin2024-10-071-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* test: fix share storage move test with object storeRobin Appelman2024-09-271-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(files_sharing): Fix all IStorage return typesprovokateurin2024-09-261-3/+4
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: Apply rector configuration to apps folderCôme Chilliet2024-09-202-5/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-196-24/+24
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48200 from nextcloud/share-reminder-shardingRobin Appelman2024-09-191-1/+3
|\ | | | | fix: add sharding compatible version of share reminder job
| * fix: add sharding compatible version of share reminder jobshare-reminder-shardingRobin Appelman2024-09-191-1/+3
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #48162 from nextcloud/bugfix/noid/array-keysJoas Schilling2024-09-191-5/+5
|\ \ | |/ |/| fix(autocomplete): Move known array keys to psalm docs
| * fix(autocomplete): Move known array keys to psalm docsbugfix/noid/array-keysJoas Schilling2024-09-181-5/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | test: fix ShareControllerTest.phpobject-store-move-fixesRobin Appelman2024-09-181-0/+1
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | test: fix ShareAPIControllerTest.phpRobin Appelman2024-09-181-1/+41
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | test: put parent items into cache in testsRobin Appelman2024-09-181-0/+2
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
generated by cgit v1.2.3 (git 2.39.1) at 2025-06-29 22:02:08 +0000