Skip to content

[tests-only][full-ci] test: verify managers can remove resource shares created by others - #13030

Draft
Asmitapaudel wants to merge 1 commit into
owncloud:masterfrom
Asmitapaudel:test/remove-share-different-manager
Draft

Asmitapaudel wants to merge 1 commit into
owncloud:masterfrom
Asmitapaudel:test/remove-share-different-manager

Conversation

@Asmitapaudel

Copy link
Copy Markdown
Contributor

Description

Adds an acceptance test covering the removal of resource shares by users with different roles in a project space.

The test verifies that:

  • A resource share is created by one user (Alice) for another user (Carol).
  • A different user (Brian) is granted access to the same project space with different roles.
  • Users with Space Viewer and Space Editor roles cannot remove Carol's resource share and receive 403.
  • A user with the Manager role can remove a resource share created by another user and receives 204.
  • After successful removal, the shared resource is no longer visible in Carol's Shares space.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
    • Locally
    • CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@kw-security

kw-security commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants