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.

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

New core test scenario apiVersions/fileVersions.feature:438 fails #459

Closed
phil-davis opened this issue Jul 7, 2021 · 3 comments
Closed
Assignees
Labels

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Jul 7, 2021

https://drone.owncloud.com/owncloud/files_primary_s3/2356/98/17

  Scenario: download an old version of a restored file                                      # /var/www/owncloud/testrunner/tests/acceptance/features/apiVersions/fileVersions.feature:438
    Given user "Alice" has uploaded file with content "uploaded content" to "textfile0.txt" # FeatureContext::userHasUploadedAFileWithContentTo()
    And user "Alice" has uploaded file with content "version 1" to "textfile0.txt"          # FeatureContext::userHasUploadedAFileWithContentTo()
    And user "Alice" has uploaded file with content "version 2" to "textfile0.txt"          # FeatureContext::userHasUploadedAFileWithContentTo()
    And user "Alice" has restored version index "1" of file "textfile0.txt"                 # FilesVersionsContext::userRestoresVersionIndexOfFile()
    When user "Alice" downloads the version of file "textfile0.txt" with the index "1"      # FilesVersionsContext::downloadVersion()
    Then the HTTP status code should be "200"                                               # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And the following headers should be set                                                 # FeatureContext::theFollowingHeadersShouldBeSet()
      | header              | value                                                                |
      | Content-Disposition | attachment; filename*=UTF-8''textfile0.txt; filename="textfile0.txt" |
    And the downloaded content should be "version 2"                                        # FeatureContext::downloadedContentShouldBe()
    When user "Alice" downloads the version of file "textfile0.txt" with the index "2"      # FilesVersionsContext::downloadVersion()
    Then the HTTP status code should be "200"                                               # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And the following headers should be set                                                 # FeatureContext::theFollowingHeadersShouldBeSet()
      | header              | value                                                                |
      | Content-Disposition | attachment; filename*=UTF-8''textfile0.txt; filename="textfile0.txt" |
    And the downloaded content should be "uploaded content"                                 # FeatureContext::downloadedContentShouldBe()
      The downloaded content was expected to be 'uploaded content', but actually is 'version 1'. HTTP status was 200
      Failed asserting that two strings are equal.
      --- Expected
      +++ Actual
      @@ @@
      -'uploaded content'
      +'version 1'

This is a new scenario that was added in core PR owncloud/core#38921

Investigate why this fails in S3. (It passes in core, encryption, and user_ldap)

@phil-davis phil-davis self-assigned this Jul 7, 2021
@phil-davis
Copy link
Contributor Author

Fails on both Ceph and Scality storage.

@phil-davis
Copy link
Contributor Author

I removed my assignment. I left this in the QA project. Someone can try to look into more detail about how/why it fails, and report here. Then if it is a real issue when on S3 storage, hand over to devs.

@dpakach dpakach self-assigned this Jul 12, 2021
@dpakach
Copy link
Contributor

dpakach commented Jul 12, 2021

The reason for failure is #463

@dpakach dpakach closed this as completed Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants