Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable31] chore : Comprehensive e2e testing for public sharing #50625

Merged
merged 4 commits into from
Feb 3, 2025

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Feb 3, 2025

Backport of #50260

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

nfebe added 2 commits February 3, 2025 11:49
This update enhances the testing framework for public sharing by
improving the setup and management of share contexts.
It refactors the code to make the `createShare` function reusable externally,
ensuring better testability and flexibility.
Additionally, the update optimizes share permission adjustments and
improves state management, making public share setups more efficient and maintainable.

Signed-off-by: nfebe <[email protected]>
This commit introduces end-to-end tests for various public link sharing scenarios in the `files_sharing` app. It validates the behavior under multiple configurations:

- Password and expiration date enforced
- Password enforced with a default expiration date
- Expiration date enforced with optional password
- Default password and expiration date without enforcement
- Password enforced, expiration date set but not enforced
- Both password and expiration date not enforced, but defaults set
- Password not enforced, expiration date enforced
- Password not enforced, default expiration date set
- Password and expiration date not enforced with no defaults

The tests ensure proper validation and functionality of the configurations, leveraging the `setupData` and `createShare` utilities.

Signed-off-by: nfebe <[email protected]>
@backportbot backportbot bot requested review from artonge, szaimen and nfebe February 3, 2025 11:49
@backportbot backportbot bot added enhancement 3. to review Waiting for reviews labels Feb 3, 2025
@backportbot backportbot bot added this to the Nextcloud 31 milestone Feb 3, 2025
@nfebe nfebe force-pushed the backport/50260/stable31 branch from b422c89 to 6b6f55a Compare February 3, 2025 16:08
@nfebe
Copy link
Contributor

nfebe commented Feb 3, 2025

/compile

@nfebe nfebe marked this pull request as ready for review February 3, 2025 16:08
@nfebe nfebe enabled auto-merge February 3, 2025 16:09
@nfebe nfebe disabled auto-merge February 3, 2025 16:09
Signed-off-by: nextcloud-command <[email protected]>
@AndyScherzinger AndyScherzinger merged commit c4564ba into stable31 Feb 3, 2025
117 of 118 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/50260/stable31 branch February 3, 2025 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants