Making getShares more reusable

tests/integration/features/bootstrap/Sharing.php

@@ -512,14 +512,8 @@ trait Sharing {
 		}
 	}
 
-	/**
-	 * @When /^user "([^"]*)" gets shares of (file|folder) "([^"]*)"$/
-	 * @param string $user
-	 * @param string $type
-	 * @param string $path
-	 * @param \Behat\Gherkin\Node\TableNode|null $body
-	 */
-	public function getShares($user, $type, $path, $TableNode){
+	/* Returns shares of a file or folders as an array of elements */
+	public function getShares($user, $path) {
 		$fullUrl = $this->baseUrl . "v{$this->apiVersion}.php/apps/files_sharing/api/v{$this->sharingApiVersion}/shares";
 		$fullUrl = $fullUrl . '?path=' . $path;
 
@@ -533,7 +527,18 @@ trait Sharing {
 		}
 
 		$this->response = $client->send($client->createRequest("GET", $fullUrl, $options));
-		$dataResponded = $this->response->xml()->data->element;
+		return $this->response->xml()->data->element;
+	}
+
+	/**
+	 * @When /^user "([^"]*)" checks public shares of (file|folder) "([^"]*)"$/
+	 * @param string $user
+	 * @param string $type
+	 * @param string $path
+	 * @param \Behat\Gherkin\Node\TableNode|null $body
+	 */
+	public function checkPublicShares($user, $type, $path, $TableNode){
+		$dataResponded = $this->getShares($user, $path);
 
 		if ($TableNode instanceof \Behat\Gherkin\Node\TableNode) {
 			$elementRows = $TableNode->getRows();

tests/integration/features/sharing-v1.feature

@@ -1198,7 +1198,7 @@ Feature: sharing
       | permissions | 1 |
     Then the OCS status code should be "100"
     And the HTTP status code should be "200"
-    And user "user0" gets shares of folder "/FOLDER"
+    And user "user0" checks public shares of folder "/FOLDER"
       | /FOLDER | 15 | sharedlink2 |
       | /FOLDER | 15 | sharedlink1 |
       | /FOLDER | 1 | sharedlink3 |