ocis: etags don't propagate correctly sometimes
The following feature failed in recent nightly https://drone.owncloud.com/owncloud/ocis/13031/46/6
Scenario Outline: copying a file inside a publicly shared folder by public changes etag for the sharer # /srv/app/testrunner/tests/acceptance/features/apiWebdavEtagPropagation2/copyFileFolder.feature:121
Given using <dav_version> DAV path # FeatureContext::usingOldOrNewDavPath()
And user "Alice" has created folder "/upload" # FeatureContext::userHasCreatedFolder()
And user "Alice" has uploaded file with content "uploaded content" to "/upload/file.txt" # FeatureContext::userHasUploadedAFileWithContentTo()
And user "Alice" has created a public link share with settings # FeatureContext::userHasCreatedAPublicLinkShareWithSettings()
| path | upload |
| permissions | change |
And user "Alice" has stored etag of element "/" # WebDavPropertiesContext::userHasStoredEtagOfElement()
And user "Alice" has stored etag of element "/upload" # WebDavPropertiesContext::userHasStoredEtagOfElement()
And user "Alice" has stored etag of element "/upload/file.txt" # WebDavPropertiesContext::userHasStoredEtagOfElement()
And user "Alice" has stored etag of element "/upload/file.txt" on path "/upload/renamedFile.txt" # WebDavPropertiesContext::userStoresEtagOfElementOnPath()
When the public copies file "file.txt" to "/renamedFile.txt" using the new public WebDAV API # PublicWebDavContext::thePublicCopiesFileUsingTheWebDAVApi()
Then the HTTP status code should be "201" # FeatureContext::thenTheHTTPStatusCodeShouldBe()
And these etags should have changed: # WebDavPropertiesContext::theseEtagsShouldHaveChanged()
| user | path |
| Alice | / |
| Alice | /upload |
| Alice | /upload/renamedFile.txt |
And these etags should not have changed: # WebDavPropertiesContext::theseEtagsShouldNotHaveChanged()
| user | path |
| Alice | /upload/file.txt |
Examples:
| dav_version |
| old |
| new |
| spaces |
WebDavPropertiesContext::theseEtagsShouldHaveChanged
The etag '"f1767bc7cff8f9573775e70ead429675"' of element '/upload' of user 'Alice' did not change.
Failed asserting that 1 matches expected 0.
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Comments: 17 (17 by maintainers)
@kiranparajuli589 please skip the failing tests on ocis and then we need to hand this over to the development team, maybe @butonic
The falkyness is seen again 😦 As discussed in this scrum handover, we should proceed with this. Assigning myself to it.
@kiranparajuli589 I think we need an other wait here, I don’t think we will see a 425 when checking the etag of a parent folder. I think we need an other waiting thing when comparing the etags