Merge pull request #45 from mekras/fix-mkcol

Collections should be created with a slash at the end.
thephpleague · Dec 14, 2018 · 34783da · 34783da
2 parents b778244 + 793f9a3
commit 34783da
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -10,6 +10,10 @@
          stopOnFailure="true"
          syntaxCheck="true"
 >
+    <php>
+        <ini name="memory_limit" value="128M" />
+    </php>
+
     <testsuites>
         <testsuite name="flysystem/gridfs/tests">
             <directory suffix=".php">./tests/</directory>

diff --git a/src/WebDAVAdapter.php b/src/WebDAVAdapter.php
@@ -260,7 +260,7 @@ public function createDir($path, Config $config)
         }
 
         $location = $this->applyPathPrefix($encodedPath);
-        $response = $this->client->request('MKCOL', $location);
+        $response = $this->client->request('MKCOL', $location . $this->pathSeparator);
 
         if ($response['statusCode'] !== 201) {
             return false;

diff --git a/tests/WebDAVTests.php b/tests/WebDAVTests.php
@@ -288,7 +288,7 @@ public function testCreateDir()
 
         $mock->shouldReceive('request')
             ->once()
-            ->with('MKCOL', 'dirname')
+            ->with('MKCOL', 'dirname/')
             ->andReturn([
                 'statusCode' => 201,
             ]);
@@ -309,14 +309,14 @@ public function testCreateDirRecursive()
 
         $mock->shouldReceive('request')
             ->once()
-            ->with('MKCOL', 'dirname')
+            ->with('MKCOL', 'dirname/')
             ->andReturn([
                 'statusCode' => 201,
             ]);
 
         $mock->shouldReceive('request')
             ->once()
-            ->with('MKCOL', 'dirname/subdirname')
+            ->with('MKCOL', 'dirname/subdirname/')
             ->andReturn([
                 'statusCode' => 201,
             ]);
@@ -359,7 +359,7 @@ public function testCreateDirFail()
 
         $mock->shouldReceive('request')
             ->once()
-            ->with('MKCOL', 'dirname')
+            ->with('MKCOL', 'dirname/')
             ->andReturn([
                 'statusCode' => 500,
             ]);