diff --git a/tests/acceptance/features/apiSpacesDavOperation/copyByFileId.feature b/tests/acceptance/features/apiSpacesDavOperation/copyByFileId.feature new file mode 100644 index 00000000000..6a6f9c90684 --- /dev/null +++ b/tests/acceptance/features/apiSpacesDavOperation/copyByFileId.feature @@ -0,0 +1,70 @@ +Feature: copying file using file id + As a user + I want to copy the file using file id + So that I can manage my resource + + Background: + Given using spaces DAV path + And user "Alice" has been created with default attributes and without skeleton files + + + Scenario Outline: copy a file into a folder in personal space + Given user "Alice" has created folder "/folder" + And user "Alice" has uploaded file with content "some data" to "/textfile.txt" + And we save it into "FILEID" + When user "Alice" copies a file "/textfile.txt" into "/folder" inside space "Personal" using file-id path "" + Then the HTTP status code should be "201" + And for user "Alice" the space "Personal" should contain these entries: + | folder/textfile.txt | + | textfile.txt | + Examples: + | dav-path | + | /remote.php/dav/spaces/<> | + | /dav/spaces/<> | + + + Scenario Outline: copy a file into a sub-folder in personal space + Given user "Alice" has created folder "/folder" + And user "Alice" has created folder "folder/sub-folder" + And user "Alice" has uploaded file with content "some data" to "/textfile.txt" + And we save it into "FILEID" + When user "Alice" copies a file "/textfile.txt" into "/folder/sub-folder" inside space "Personal" using file-id path "" + Then the HTTP status code should be "201" + And for user "Alice" the space "Personal" should contain these entries: + | folder/sub-folder/textfile.txt | + | textfile.txt | + Examples: + | dav-path | + | /remote.php/dav/spaces/<> | + | /dav/spaces/<> | + + + Scenario Outline: copy a file from a folder into root of personal space + Given user "Alice" has created folder "/folder" + And user "Alice" has uploaded file with content "some data" to "folder/textfile.txt" + And we save it into "FILEID" + When user "Alice" copies a file "folder/textfile.txt" into "/" inside space "Personal" using file-id path "" + Then the HTTP status code should be "201" + And for user "Alice" the space "Personal" should contain these entries: + | textfile.txt | + | folder/textfile.txt | + Examples: + | dav-path | + | /remote.php/dav/spaces/<> | + | /dav/spaces/<> | + + + Scenario Outline: copy a file from sub-folder into root of personal space + Given user "Alice" has created folder "/folder" + And user "Alice" has created folder "folder/sub-folder" + And user "Alice" has uploaded file with content "some data" to "folder/sub-folder/textfile.txt" + And we save it into "FILEID" + When user "Alice" copies a file "folder/sub-folder/textfile.txt" into "/" inside space "Personal" using file-id path "" + Then the HTTP status code should be "201" + And for user "Alice" the space "Personal" should contain these entries: + | textfile.txt | + | folder/sub-folder/textfile.txt | + Examples: + | dav-path | + | /remote.php/dav/spaces/<> | + | /dav/spaces/<> | diff --git a/tests/acceptance/features/bootstrap/SpacesContext.php b/tests/acceptance/features/bootstrap/SpacesContext.php index 87f21cf4599..4c9fb4a8113 100644 --- a/tests/acceptance/features/bootstrap/SpacesContext.php +++ b/tests/acceptance/features/bootstrap/SpacesContext.php @@ -1753,15 +1753,19 @@ public function userMovesFileFromAndToSpaceBetweenSpaces( * @param string $user * @param string $fileDestination * @param string $spaceName + * @param string|null $endPath * * @return string * @throws GuzzleException */ - public function destinationHeaderValueWithSpaceName(string $user, string $fileDestination, string $spaceName):string { + public function destinationHeaderValueWithSpaceName(string $user, string $fileDestination, string $spaceName, string $endPath = null):string { $space = $this->getSpaceByName($user, $spaceName); - $fileDestination = $this->escapePath(\ltrim($fileDestination, "/")); - + if ($endPath && str_contains($endPath, 'remote.php')) { + // this is check for when we want to test with the endpoint having `remote.php` in space webdav + // by default spaces webdav is '/dav/spaces' + return $this->featureContext->getBaseUrl() . '/remote.php/dav/spaces/' . $space['id'] . '/' . $fileDestination; + } return $space["root"]["webDavUrl"] . '/' . $fileDestination; } @@ -1786,6 +1790,27 @@ public function copyFilesAndFoldersRequest(string $user, string $fullUrl, array ); } + /** + * @When /^user "([^"]*)" copies a file "([^"]*)" into "([^"]*)" inside space "([^"]*)" using file-id path "([^"]*)"$/ + * + * @param string $user + * @param string $sourceFile + * @param string $destinationFile + * @param string $toSpaceName + * @param string $url + * + * @throws GuzzleException + * @return void + */ + public function userCopiesFileWithFileIdFromAndToSpaceBetweenSpaces(string $user, string $sourceFile, string $destinationFile, string $toSpaceName, string $url): void { + // split the source when there are sub-folders + $sourceFile = explode("/", $sourceFile); + $fileDestination = $this->escapePath(\ltrim($destinationFile, "/")) . '/' . $this->escapePath(\ltrim(end($sourceFile), "/")); + $headers['Destination'] = $this->destinationHeaderValueWithSpaceName($user, $fileDestination, $toSpaceName, $url); + $fullUrl = $this->featureContext->getBaseUrl() . $url; + $this->featureContext->setResponse($this->copyFilesAndFoldersRequest($user, $fullUrl, $headers)); + } + /** * @Given /^user "([^"]*)" has uploaded a file inside space "([^"]*)" with content "([^"]*)" to "([^"]*)"$/ *