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

Flaky test on coreApiShareManagementToShares/moveReceivedShare.feature #9428

Closed
amrita-shrestha opened this issue Jun 21, 2024 · 7 comments · Fixed by #9454 or #9491
Closed

Flaky test on coreApiShareManagementToShares/moveReceivedShare.feature #9428

amrita-shrestha opened this issue Jun 21, 2024 · 7 comments · Fixed by #9454 or #9491
Assignees

Comments

@amrita-shrestha
Copy link
Contributor

Describe the bug

Flaky tests
https://drone.owncloud.com/owncloud/ocis/36078/42/6

  Scenario: keep group share when the one user renames the share and the user is deleted      # /drone/src/tests/acceptance/features/coreApiShareManagementToShares/moveReceivedShare.feature:167
    Given group "grp1" has been created                                                       # FeatureContext::groupHasBeenCreated()
    And user "Brian" has been added to group "grp1"                                           # FeatureContext::userHasBeenAddedToGroup()
    And user "Carol" has been added to group "grp1"                                           # FeatureContext::userHasBeenAddedToGroup()
    And user "Alice" has created folder "/TMP"                                                # FeatureContext::userHasCreatedFolder()
    And user "Alice" has sent the following resource share invitation:                        # SharingNgContext::userHasSentTheFollowingResourceShareInvitation()
      | resource        | TMP      |
      | space           | Personal |
      | sharee          | grp1     |
      | shareType       | group    |
      | permissionsRole | Viewer   |
    When user "Carol" moves folder "/Shares/TMP" to "/Shares/new" using the WebDAV API        # FeatureContext::userMovesFileOrFolderUsingTheWebDavAPI()
    And the administrator deletes user "Carol" using the provisioning API                     # FeatureContext::theAdminDeletesUserUsingTheProvisioningApi()
    Then the HTTP status code of responses on each endpoint should be "201, 204" respectively # FeatureContext::theHTTPStatusCodeOfResponsesOnEachEndpointShouldBe()
      Expected HTTP status codes: "201, 204". Found HTTP status codes: "404,204"
      Failed asserting that false is true.
    And as "Alice" file "Shares/TMP" should not exist    
@saw-jan saw-jan self-assigned this Jun 24, 2024
@saw-jan saw-jan added the QA:p1 label Jun 24, 2024
@saw-jan
Copy link
Member

saw-jan commented Jun 24, 2024

First impressions: shares are not immediately available to the share receivers

But it's strange to see only in that particular test scenario

@amrita-shrestha
Copy link
Contributor Author

@saw-jan PR related to this CI build has no changes related to this feature file but still fails. Maybe related to this issue
Build: https://drone.owncloud.com/owncloud/ocis/36208/43/5

Scenario: user included in multiple groups, shares a folder with a group                  # /drone/src/tests/acceptance/features/coreApiShareManagementBasicToShares/createShareToSharesFolder.feature:236
    Given these users have been created with default attributes and without skeleton files: # FeatureContext::theseUsersHaveBeenCreatedWithDefaultAttributesAndWithoutSkeletonFiles()
      | username |
      | Brian    |
    And group "grp1" has been created                                                       # FeatureContext::groupHasBeenCreated()
    And group "grp2" has been created                                                       # FeatureContext::groupHasBeenCreated()
    And user "Alice" has been added to group "grp1"                                         # FeatureContext::userHasBeenAddedToGroup()
    And user "Alice" has been added to group "grp2"                                         # FeatureContext::userHasBeenAddedToGroup()
    And user "Brian" has been added to group "grp1"                                         # FeatureContext::userHasBeenAddedToGroup()
    And user "Brian" has been added to group "grp2"                                         # FeatureContext::userHasBeenAddedToGroup()
    And user "Alice" has created folder "/PARENT"                                           # FeatureContext::userHasCreatedFolder()
    When user "Alice" shares folder "/PARENT" with group "grp1" using the sharing API       # FeatureContext::userSharesFileWithGroupUsingTheSharingApi()
    Then user "Brian" should see the following elements                                     # FeatureContext::userShouldSeeTheElements()
      | /Shares/PARENT/ |
      /remote.php/webdav/Shares/PARENT/ is not in propfind answer but should be

@saw-jan
Copy link
Member

saw-jan commented Jun 24, 2024

Reproducible on a low-spec machine

@saw-jan
Copy link
Member

saw-jan commented Jun 24, 2024

@saw-jan PR related to this CI build has no changes related to this feature file but still fails. Maybe related to this issue
Build: https://drone.owncloud.com/owncloud/ocis/36208/43/5

yes, the test failure is related to this and I have already added in the list above #9428 (comment)

@saw-jan
Copy link
Member

saw-jan commented Jun 24, 2024

First impressions: shares are not immediately available to the share receivers

But it's strange to see only in that particular test scenario

Can confirm that the shares weren't there while doing MOVE request.

@saw-jan
Copy link
Member

saw-jan commented Jun 24, 2024

Scenarios that can be flaky due to the same reason:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants