changed wording of mount unmount to enable and disable sync

owncloud · Apr 5, 2024 · 963af78 · 963af78
1 parent d7ad04c
commit 963af78
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 31 deletions.
diff --git a/tests/TestHelpers/GraphHelper.php b/tests/TestHelpers/GraphHelper.php
@@ -1905,7 +1905,7 @@ public static function getSharesSharedByMe(
 	 * @return ResponseInterface
 	 * @throws GuzzleException
 	 */
-	public static function unmountShare(
+	public static function disableSyncForShare(
 		string $baseUrl,
 		string $xRequestId,
 		string $user,
@@ -1934,7 +1934,7 @@ public static function unmountShare(
 	 * @return ResponseInterface
 	 * @throws GuzzleException
 	 */
-	public static function mountShare(
+	public static function enableSyncForShare(
 		string $baseUrl,
 		string $xRequestId,
 		string $user,

diff --git a/.../apiSharingNg/mountOrUnmountShare.feature → ...ingNg/enableOrDisableSyncForShare.feature b/.../apiSharingNg/mountOrUnmountShare.feature → ...ingNg/enableOrDisableSyncForShare.feature
@@ -1,4 +1,4 @@
-Feature:  mount or unmount incoming share
+Feature:  enable or disable sync for incoming share
   As a user
   I want to have control over the share received
   So that I can filter out the files and folder shared with Me
@@ -11,7 +11,7 @@ Feature:  mount or unmount incoming share
     And using spaces DAV path
 
 
-  Scenario Outline: unmount shared resource
+  Scenario Outline: disable sync for shared resource
     And user "Alice" has created folder "FolderToShare"
     And user "Alice" has uploaded file with content "hello world" to "/textfile0.txt"
     And user "Alice" has sent the following share invitation:
@@ -20,7 +20,7 @@ Feature:  mount or unmount incoming share
       | sharee          | Brian      |
       | shareType       | user       |
       | permissionsRole | Viewer     |
-    When user "Brian" unmounts share "<resource>" using the Graph API
+    When user "Brian" disables sync for share "<resource>" using the Graph API
     And user "Brian" lists the shares shared with him using the Graph API
     Then the HTTP status code of responses on all endpoints should be "200"
     And the JSON data of the response should match
@@ -56,7 +56,7 @@ Feature:  mount or unmount incoming share
       | FolderToShare |
 
 
-  Scenario Outline: mount shared resource when auto-sync is disabled
+  Scenario Outline: enable sync for shared resource when auto-sync is disabled
     Given user "Brian" has disabled the auto-sync share
     And user "Alice" has uploaded file with content "hello world" to "/textfile0.txt"
     And user "Alice" has created folder "folder"
@@ -66,7 +66,7 @@ Feature:  mount or unmount incoming share
       | sharee          | Brian      |
       | shareType       | user       |
       | permissionsRole | Viewer     |
-    When user "Brian" mounts share "<resource>" offered by "Alice" from "Personal" space using the Graph API
+    When user "Brian" enables sync for share "<resource>" offered by "Alice" from "Personal" space using the Graph API
     Then the HTTP status code should be "201"
     And the JSON data of the response should match
         """
@@ -103,7 +103,7 @@ Feature:  mount or unmount incoming share
       | sharee          | grp1       |
       | shareType       | group      |
       | permissionsRole | Viewer     |
-    When user "Alice" mounts share "<resource>" offered by "Carol" from "Personal" space using the Graph API
+    When user "Alice" enables sync for share "<resource>" offered by "Carol" from "Personal" space using the Graph API
     Then the HTTP status code should be "201"
     And the JSON data of the response should match
       """
@@ -140,7 +140,7 @@ Feature:  mount or unmount incoming share
       | sharee          | grp1       |
       | shareType       | group      |
       | permissionsRole | Viewer     |
-    When user "Alice" unmounts share "<resource>" using the Graph API
+    When user "Alice" disables sync for share "<resource>" using the Graph API
     Then the HTTP status code should be "200"
     And user "Alice" should have sync disabled for share "<resource>"
     And user "Brian" should have sync enabled for share "<resource>"

diff --git a/tests/acceptance/features/bootstrap/SharingNgContext.php b/tests/acceptance/features/bootstrap/SharingNgContext.php
@@ -561,42 +561,31 @@ public function forUserTheSpaceSharesShouldContainTheseEntries(string $user, str
 	}
 
 	/**
+	 * @When user :user disables sync for share :share using the Graph API
 	 *
 	 * @param string $user
-	 * @param string $itemId
-	 * @param string $shareSpaceId
 	 *
-	 * @return ResponseInterface
+	 * @return void
+	 * @throws Exception
 	 */
-	public function unmountShare(string $user, string $itemId, string $shareSpaceId): ResponseInterface {
-		return GraphHelper::unmountShare(
+	public function userUnmountsShareUsingTheGraphApi(string $user):void {
+		$shareItemId = $this->featureContext->shareNgGetLastCreatedUserGroupShareID();
+		$shareSpaceId = FeatureContext::SHARES_SPACE_ID;
+		$itemId = $shareSpaceId . '!' . $shareItemId;
+		$response = GraphHelper::disableSyncForShare(
 			$this->featureContext->getBaseUrl(),
 			$this->featureContext->getStepLineRef(),
 			$this->featureContext->getActualUsername($user),
 			$this->featureContext->getPasswordForUser($user),
 			$itemId,
 			$shareSpaceId,
 		);
-	}
-
-	/**
-	 * @When user :user unmounts share :share using the Graph API
-	 *
-	 * @param string $user
-	 *
-	 * @return void
-	 * @throws Exception
-	 */
-	public function userUnmountsShareUsingTheGraphApi(string $user):void {
-		$shareItemId = $this->featureContext->shareNgGetLastCreatedUserGroupShareID();
-		$shareSpaceId = FeatureContext::SHARES_SPACE_ID;
-		$itemId = $shareSpaceId . '!' . $shareItemId;
-		$this->featureContext->setResponse($this->unmountShare($user, $itemId, $shareSpaceId));
+		$this->featureContext->setResponse($response);
 		$this->featureContext->pushToLastStatusCodesArrays();
 	}
 
 	/**
-	 * @When user :user mounts share :share offered by :offeredBy from :space space using the Graph API
+	 * @When user :user enables sync for share :share offered by :offeredBy from :space space using the Graph API
 	 *
 	 * @param string $user
 	 * @param string $share
@@ -610,7 +599,7 @@ public function userMountsShareOfferedByFromSpaceUsingTheGraphApi(string $user,
 		$share = ltrim($share, '/');
 		$itemId = $this->spacesContext->getResourceId($offeredBy, $space, $share);
 		$shareSpaceId = FeatureContext::SHARES_SPACE_ID;
-		$response =  GraphHelper::mountShare(
+		$response =  GraphHelper::enableSyncForShare(
 			$this->featureContext->getBaseUrl(),
 			$this->featureContext->getStepLineRef(),
 			$this->featureContext->getActualUsername($user),