-
Notifications
You must be signed in to change notification settings - Fork 189
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
[full-ci] bump reva to test the spaceid change #7975
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
cannot build it
|
00dd58d
to
f668a72
Compare
hm, this smells like a new flakyness: Scenario Outline: send last chunk twice # /drone/src/tests/acceptance/features/coreApiWebdavUploadTUS/lowLevelUpload.feature:53
Given using <dav-path-version> DAV path # FeatureContext::usingOldOrNewDavPath()
And user "Alice" has created a new TUS resource on the WebDAV API with these headers: # TUSContext::userHasCreatedNewTUSResourceWithHeaders()
| Upload-Length | 10 |
| Upload-Metadata | filename ZmlsZS50eHQ= |
When user "Alice" sends a chunk to the last created TUS Location with offset "0" and data "123" using the WebDAV API # TUSContext::userSendsAChunkToTUSLocationWithOffsetAndData()
And user "Alice" sends a chunk to the last created TUS Location with offset "3" and data "4567890" using the WebDAV API # TUSContext::userSendsAChunkToTUSLocationWithOffsetAndData()
And user "Alice" sends a chunk to the last created TUS Location with offset "3" and data "0000000" using the WebDAV API # TUSContext::userSendsAChunkToTUSLocationWithOffsetAndData()
Then the HTTP status code should be "404" # FeatureContext::thenTheHTTPStatusCodeShouldBe()
And the content of file "/file.txt" for user "Alice" should be "1234567890" # FeatureContext::contentOfFileForUserShouldBe()
Examples:
| dav-path-version |
| old |
| new |
Failed step: Then the HTTP status code should be "404"
HTTP status code 500 is not the expected value 404
Failed asserting that 500 matches expected '404'.
| spaces | |
f668a72
to
e378e77
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.
👍 works for me as expected
e378e77
to
0c68af5
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.
works, used in owncloud/ocis-php-sdk#130
please rebase |
cs3org/reva#4407 was merged as part of #7989 |
testing cs3org/reva#4407
cc @kobergj @individual-it