Api tests for deleting the expiration date of a space

owncloud · Feb 21, 2023 · 3dc0cdc · 3dc0cdc
1 parent 22f3577
commit 3dc0cdc
Show file tree

Hide file tree

Showing 2 changed files with 43 additions and 4 deletions.
diff --git a/tests/acceptance/features/apiSpacesShares/shareSpaces.feature b/tests/acceptance/features/apiSpacesShares/shareSpaces.feature
@@ -347,4 +347,44 @@ Feature: Share spaces
       | role    |
       | manager |
       | editor  |
-      | viewer  |
+      | viewer  |
+
+
+  Scenario Outline: delete the expiration date of a space in user share
+    Given user "Alice" has shared a space "share space" with settings:
+      | shareWith  | Brian                    |
+      | role       | <role>                   |
+      | expireDate | 2042-03-25T23:59:59+0100 |
+    When user "Alice" updates the space "share space" with settings:
+      | shareWith  | Brian  |
+      | expireDate |        |
+      | role       | <role> |
+    Then the HTTP status code should be "200"
+    And the user "Brian" should have a space called "share space" granted to user "Brian" with role "<role>" and expiration date ""
+    Examples:
+      | role    |
+      | manager |
+      | editor  |
+      | viewer  |
+
+
+  Scenario Outline: update the expiration date of a space in group share
+    Given group "sales" has been created
+    And the administrator has added a user "Brian" to the group "sales" using GraphApi
+    And user "Alice" has shared a space "share space" with settings:
+      | shareWith  | sales                    |
+      | shareType  | 8                        |
+      | role       | <role>                   |
+      | expireDate | 2042-03-25T23:59:59+0100 |
+    When user "Alice" updates the space "share space" with settings:
+      | shareWith  | sales  |
+      | shareType  | 8      |
+      | expireDate |        |
+      | role       | <role> |
+    Then the HTTP status code should be "200"
+    And the user "Brian" should have a space called "share space" granted to group "sales" with role "<role>" and expiration date ""
+    Examples:
+      | role    |
+      | manager |
+      | editor  |
+      | viewer  |
diff --git a/tests/acceptance/features/bootstrap/SpacesContext.php b/tests/acceptance/features/bootstrap/SpacesContext.php
@@ -69,7 +69,7 @@ class SpacesContext implements Context {
 	 * @var ChecksumContext
 	 */
 	private ChecksumContext $checksumContext;
-	
+
 	/**
 	 * @var FilesVersionsContext
 	 */
@@ -3228,8 +3228,7 @@ public function theUserShouldHaveSpaceWithRecipient(
 		foreach ($spaceAsArray['root']['permissions'] as $permission) {
 			if (isset($permission['grantedToIdentities'][0][$recipientType]) && $permission['roles'][0] === $role && $permission['grantedToIdentities'][0][$recipientType]['id'] === $recipientId) {
 				$foundRoleInResponse = true;
-				if ($expirationDate !== null) {
-					Assert::assertArrayHasKey('expirationDateTime', $permission, 'expirationDateTime key not found in response');
+				if ($expirationDate !== null && isset($permission['expirationDateTime'])) {
 					Assert::assertEquals($expirationDate, (preg_split("/[\sT]+/", $permission['expirationDateTime']))[0], "$expirationDate is different in the response");
 				}
 				break;