-
Notifications
You must be signed in to change notification settings - Fork 187
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.
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 upload file to a shared folder via TUS by federated user #10415
Conversation
35308d7
to
acf049e
Compare
5c929d2
to
67a6e20
Compare
ce3a92e
to
3ec2a4e
Compare
3ec2a4e
to
0580219
Compare
8466d85
to
0ecd3f0
Compare
fef6e9d
to
d93d28d
Compare
please revert all changes in |
d93d28d
to
d922695
Compare
@issue-10285 @issue-10536 | ||
Scenario: local user upload file to a federated shared folder via TUS | ||
Given using spaces DAV path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this matter about spaces dav path in this scenario?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it matters, as we need to hit endpoint with spaceId.
:
if ($davPathVersion === WebDavHelper::DAV_VERSION_SPACES) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we must specify space dav path for federation shares (doesn't work with other dav paths, see #10536). by default new dav path is used in the tests
9b8b92c
to
5516b7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's 👍 for me.
Signed-off-by: prashant-gurung899 <[email protected]>
5516b7d
to
e40015c
Compare
Quality Gate passedIssues Measures |
[tests-only][full-ci] add test to upload file to a shared folder via TUS by federated user
Description
This PR adds test to upload a file to a shared folder via TUS by a federated user.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: