Skip to content

Conversation

@SergioBertolinSG
Copy link
Contributor

Backport of #26370

@mention-bot
Copy link

@SergioBertolinSG, thanks for your PR! By analyzing the history of the files in this pull request, we identified @rullzer, @PVince81 and @LukasReschke to be potential reviewers.

@PVince81
Copy link
Contributor

👍

@PVince81 PVince81 added this to the 9.1.3 milestone Nov 23, 2016
@SergioBertolinSG
Copy link
Contributor Author

@PVince81 failing case:

[Thu Nov 24 09:18:46 2016] 127.0.0.1:45556 [404]: /remote.php/webdav/merge-test-outside-groups-renamebeforesecondshare-renamed
    Then as "user1" gets properties of folder "/merge-test-outside-groups-renamebeforesecondshare-renamed" with                                        # FeatureContext::asGetsPropertiesOfFolderWith()
      | {http://owncloud.org/ns}permissions |
      HTTP error: 404 (Sabre\DAV\Exception)

I guess you improved this test recently didn't you?

@PVince81
Copy link
Contributor

No, that's the test that usually randomly fails. Let's see if it passes during the next run...

@SergioBertolinSG
Copy link
Contributor Author

SergioBertolinSG commented Nov 24, 2016

What about removing it? 🔥 Those random failures are annoying.

@PVince81
Copy link
Contributor

This is an important test about merging shares, I don't want to remove it...

@PVince81
Copy link
Contributor

Green but result unpublished => merge

@PVince81 PVince81 merged commit 4a53f35 into stable9.1 Nov 24, 2016
@PVince81 PVince81 deleted the stable9.1-integration-tests-adding-etag-support branch November 24, 2016 10:53
@lock
Copy link

lock bot commented Aug 4, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants