Orphaned shares test and some requirements

[SergioBertolinSG]

@PVince81 it is failing getting a 500 instead of a 400 in the last step added.

[mention-bot]

@SergioBertolinSG, thanks for your PR! By analyzing the annotation information on this pull request, we identified @LukasReschke, @rullzer and @PVince81 to be potential reviewers

[PVince81]

Strange, am getting this:

--- FeatureContext has missing steps. Define them with these snippets:

    /**
     * @Given User :arg1 deletes file :arg2                                                                                                       
     */                                                                                                                                           
    public function userDeletesFile2($arg1, $arg2)                                                                                                
    {                                                                                                                                             
        throw new PendingException();                                                                                                             
    }     

[SergioBertolinSG]

Let me check.

[SergioBertolinSG]

Ok now it fails with the 500 (expecting 404):

https://jenkins.owncloud.org/job/owncloud-core/job/core/job/PR-25977/2/testReport/junit/(root)/sharing/orphaned_shares/

[PVince81]

Okay, got it reproduced now, thanks!

[PVince81]

Following the steps manually and at the end:

  % curl -X PROPFIND http://user1:test@localhost/owncloud/remote.php/webdav/sub
<?xml version="1.0" encoding="utf-8"?>
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">
  <s:exception>InvalidArgumentException</s:exception>
  <s:message>Jail rootPath is null</s:message>
</d:error>

[PVince81]

@SergioBertolinSG something to add to the test, maybe as separate case: can user1 create a folder called "sub" ? (the internal logic seems to forbid this, even though the share should not be mounted any more)

[PVince81]

Raised #26001 for the core issue. This will take a bit longer to fix...

[PVince81]

oops, pushed on the wrong branch

[PVince81]

restored to the previous state.

[PVince81]

The test seems to pass with #26004

[PVince81]

Rebased. The error should be gone now.

[PVince81]

Tests passed now, 👍

@SergioBertolinSG merge or did you want to add more stuff ?

[SergioBertolinSG]

Merge yes.

[lock]

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.