Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tests-only][full-ci] add test to download files and folders as archive by ocm user #10740

Merged
merged 1 commit into from
Dec 23, 2024
Merged

[tests-only][full-ci] add test to download files and folders as archive by ocm user #10740

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 22 additions & 9 deletions tests/acceptance/bootstrap/ArchiverContext.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ class ArchiverContext implements Context {
* @var FeatureContext
*/
private FeatureContext $featureContext;
private SpacesContext $spacesContext;

/**
* @BeforeScenario
Expand All @@ -58,6 +59,7 @@ public function before(BeforeScenarioScope $scope): void {
$environment = $scope->getEnvironment();
// Get all the contexts you need in this context
$this->featureContext = BehatHelper::getContext($scope, $environment, 'FeatureContext');
$this->spacesContext = BehatHelper::getContext($scope, $environment, 'SpacesContext');
}

/**
Expand Down Expand Up @@ -120,13 +122,15 @@ private function getArchiverQueryString(
case 'id':
case 'ids':
return 'id=' . $this->featureContext->getFileIdForPath($user, $resource);
case 'remoteItemIds':
return 'id=' . $this->spacesContext->getSharesRemoteItemId($user, $resource);
case 'path':
case 'paths':
return 'path=' . $resource;
default:
throw new Exception(
'"' . $addressType .
'" is not a legal value for $addressType, must be id|ids|path|paths'
'" is not a legal value for $addressType, must be id|ids|remoteItemIds|path|paths'
);
}
}
Expand Down Expand Up @@ -276,7 +280,7 @@ public function theDownloadedArchiveShouldContainTheseFiles(string $type, TableN
$tar = $this->getArchiveClass($type);
$tar->open($tempFile);
$archiveData = $tar->contents();

// extract the archive
$tar->open($tempFile);
$tar->extract($tempExtractFolder);
Expand All @@ -291,13 +295,22 @@ public function theDownloadedArchiveShouldContainTheseFiles(string $type, TableN

if ($expectedPath === $actualPath) {
if (!$info->getIsdir()) {
$fileContent = \file_get_contents("$tempExtractFolder/$actualPath");
Assert::assertEquals(
$expectedItem['content'],
$fileContent,
__METHOD__ .
" content of '" . $expectedPath . "' not as expected"
);
$fileFullPath = "$tempExtractFolder/$actualPath";
$fileMimeType = \mime_content_type($fileFullPath);

if ($fileMimeType === "text/plain") {
$fileContent = \file_get_contents($fileFullPath);
Assert::assertEquals(
$expectedItem['content'],
$fileContent,
__METHOD__ . " content of '" . $expectedPath . "' not as expected"
);
} else {
Assert::assertFileExists(
$fileFullPath,
__METHOD__ . " File '" . $expectedPath . "' is not in the downloaded archive."
);
}
}
$found = true;
break;
Expand Down
33 changes: 33 additions & 0 deletions tests/acceptance/features/apiOcm/share.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1297,3 +1297,36 @@ Feature: an user shares resources using ScienceMesh application
| Viewer |
| Uploader |
| Editor |

@issue-10272
Scenario: federated user downloads shared resources as an archive
Given using spaces DAV path
And using server "REMOTE"
And "Brian" has created the federation share invitation
And using server "LOCAL"
And "Alice" has accepted invitation
And user "Alice" has uploaded file with content "some data" to "textfile.txt"
And user "Alice" has created folder "imageFolder"
And user "Alice" has uploaded file "filesForUpload/testavatar.png" to "imageFolder/testavatar.png"
And user "Alice" has sent the following resource share invitation to federated user:
| resource | textfile.txt |
| space | Personal |
| sharee | Brian |
| shareType | user |
| permissionsRole | Viewer |
And user "Alice" has sent the following resource share invitation to federated user:
| resource | imageFolder |
| space | Personal |
| sharee | Brian |
| shareType | user |
| permissionsRole | Viewer |
And using server "REMOTE"
When user "Brian" downloads the archive of these items using the resource remoteItemIds
| textfile.txt |
| imageFolder |
Then the HTTP status code should be "200"
And the downloaded zip archive should contain these files:
| name | content |
| textfile.txt | some data |
| imageFolder | |
| imageFolder/testavatar.png | |