Skip to content

Conversation

@adambarreiro
Copy link
Contributor

@adambarreiro adambarreiro commented Feb 18, 2025

This PR adds a test for Tenant Content Libraries.

Tenant Content libraries need the org_id argument, when this is set, the Content Library will belong to an Organization.
The test creates a few libraries in a created Organization, and also logins as a tenant user to create a Content Library in this Organization.

The added test caught a bug in CL deletion (the error was being ignored). This PR also fixes that and updates documentation on this regard.

Signed-off-by: abarreiro <[email protected]>
abarreiro added 3 commits February 18, 2025 16:06
#
Signed-off-by: abarreiro <[email protected]>
#
Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro marked this pull request as draft February 18, 2025 16:11
abarreiro added 4 commits February 18, 2025 18:12
Fix
Signed-off-by: abarreiro <[email protected]>
Fix
Signed-off-by: abarreiro <[email protected]>
#
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro marked this pull request as ready for review February 19, 2025 11:43
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
abarreiro added 3 commits February 19, 2025 12:49
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Copy link
Contributor

@Didainius Didainius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM tests passed.

abarreiro added 2 commits February 20, 2025 12:23
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro merged commit 1bd47b1 into vmware:main Feb 20, 2025
3 checks passed
@adambarreiro adambarreiro deleted the test-tenant-cl branch February 20, 2025 11:30
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