Generalize integration test steps to download last share

Note that the "last link share can be downloaded" step was kept as it
tests the "url" property specific of link shares.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

diff --git a/build/integration/features/bootstrap/Sharing.php b/build/integration/features/bootstrap/Sharing.php
index 0958ab5b64b1b6df6487b8bd9e7eeb5d3bd979b6..4ce2e12e29d8f20a885a85f710755cbe385cb8c4 100644 (file)
--- a/build/integration/features/bootstrap/Sharing.php
+++ b/build/integration/features/bootstrap/Sharing.php
@@ -164,9 +164,23 @@ trait Sharing {
        }
 
        /**
-        * @Then /^last link share with password "([^"]*)" can be downloaded$/
+        * @Then /^last share can be downloaded$/
         */
-       public function lastLinkShareWithPasswordCanBeDownloaded($password) {
+       public function lastShareCanBeDownloaded() {
+               if (count($this->lastShareData->data->element) > 0) {
+                       $token = $this->lastShareData->data[0]->token;
+               } else {
+                       $token = $this->lastShareData->data->token;
+               }
+
+               $fullUrl = substr($this->baseUrl, 0, -4) . "index.php/s/" . $token . "/download";
+               $this->checkDownload($fullUrl, null, 'text/plain');
+       }
+
+       /**
+        * @Then /^last share with password "([^"]*)" can be downloaded$/
+        */
+       public function lastShareWithPasswordCanBeDownloaded($password) {
                if (count($this->lastShareData->data->element) > 0) {
                        $token = $this->lastShareData->data[0]->token;
                } else {

diff --git a/build/integration/sharing_features/sharing-v1.feature b/build/integration/sharing_features/sharing-v1.feature
index 8440a2d874e928a5afec0f0ffda2336486276600..498fe1456f68d75ffbbd853dd6a51b8cc2dddaf3 100644 (file)
--- a/build/integration/sharing_features/sharing-v1.feature
+++ b/build/integration/sharing_features/sharing-v1.feature
@@ -75,7 +75,7 @@ Feature: sharing
       | password | publicpw |
     Then the OCS status code should be "100"
     And the HTTP status code should be "200"
-    And last link share with password "publicpw" can be downloaded
+    And last share with password "publicpw" can be downloaded
 
   Scenario: Creating a new public share of a folder
    Given user "user0" exists
@@ -108,7 +108,7 @@ Feature: sharing
       | expireDate | +3 days |
     Then the OCS status code should be "100"
     And the HTTP status code should be "200"
-    And last link share with password "publicpw" can be downloaded
+    And last share with password "publicpw" can be downloaded
 
   Scenario: Creating a new public share, updating its expiration date and getting its info
     Given user "user0" exists