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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+57 −2
Open

[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
8 changes: 6 additions & 2 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 Down
51 changes: 51 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 @@ -1229,3 +1229,54 @@ Feature: an user shares resources using ScienceMesh application
}
}
"""

@issue-10272
Scenario: federated user downloads multiple files and folders as archive
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Scenario: federated user downloads multiple files and folders as archive
Scenario: federated user downloads shared files and folders as an archive

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Scenario: federated user downloads multiple files and folders as archive
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 "textfile0.txt"
And user "Alice" has uploaded file with content "other data" to "textfile1.txt"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we upload images here?

And user "Alice" has created folder "my_data"
And user "Alice" has uploaded file with content "some data" to "/my_data/textfile2.txt"
And user "Alice" has created folder "more_data"
And user "Alice" has uploaded file with content "more data" to "/more_data/an_other_file.txt"
And user "Alice" has sent the following resource share invitation to federated user:
| resource | textfile0.txt |
| space | Personal |
| sharee | Brian |
| shareType | user |
| permissionsRole | Viewer |
And user "Alice" has sent the following resource share invitation to federated user:
| resource | textfile1.txt |
| space | Personal |
| sharee | Brian |
| shareType | user |
| permissionsRole | Viewer |
And user "Alice" has sent the following resource share invitation to federated user:
| resource | my_data |
| space | Personal |
| sharee | Brian |
| shareType | user |
| permissionsRole | Viewer |
And user "Alice" has sent the following resource share invitation to federated user:
| resource | more_data |
| 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
| textfile0.txt |
| textfile1.txt |
| my_data |
| more_data |
Then the HTTP status code should be "200"
And the downloaded zip archive should contain these files:
| name | content |
| textfile0.txt | some data |
| textfile1.txt | other data |
| my_data/textfile2.txt | some data |
| more_data/an_other_file.txt | more data |