WIP: new composition prepend / append

Author: n.gnato

README.md

@@ -19,10 +19,10 @@ public function test()
         $responseFactory, 
         new \FreeElephants\StaticHttpClient\PathResolver\PathBuilderBasedResolver(
             new \FreeElephants\StaticHttpClient\PathBuilder\Composite(
-                new \FreeElephants\StaticHttpClient\PathBuilder\BasePath(__DIR__),
-                new \FreeElephants\StaticHttpClient\PathBuilder\HostnameAsDirectory(),
+                new \FreeElephants\StaticHttpClient\PathBuilder\PrependBasePath(__DIR__),
+                new \FreeElephants\StaticHttpClient\PathBuilder\PrependHostnameAsDirectory(),
                 new \FreeElephants\StaticHttpClient\PathBuilder\AppendRequestPath(),
-                new \FreeElephants\StaticHttpClient\PathBuilder\DefaultFileExtension('.json'),
+                new \FreeElephants\StaticHttpClient\PathBuilder\AppendDefaultFileExtension('.json'),
             ) 
         )
     );

src/FreeElephants/StaticHttpClient/PathBuilder/DefaultFileExtension.php renamed to src/FreeElephants/StaticHttpClient/PathBuilder/AppendDefaultFileExtension.php

@@ -4,7 +4,7 @@
 
 use Psr\Http\Message\RequestInterface;
 
-class DefaultFileExtension implements PathBuilderInterface
+class AppendDefaultFileExtension implements PathBuilderInterface
 {
 
     private string $fileExtension;

src/FreeElephants/StaticHttpClient/PathBuilder/AppendHostnameAsDirectory.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace FreeElephants\StaticHttpClient\PathBuilder;
+
+use Psr\Http\Message\RequestInterface;
+
+class AppendHostnameAsDirectory implements PathBuilderInterface
+{
+    public function build(RequestInterface $request, string $path = ''): string
+    {
+        return $path . DIRECTORY_SEPARATOR . $request->getUri()->getHost();
+    }
+
+}

src/FreeElephants/StaticHttpClient/PathBuilder/BasePath.php renamed to src/FreeElephants/StaticHttpClient/PathBuilder/PrependBasePath.php

@@ -4,13 +4,12 @@
 
 use Psr\Http\Message\RequestInterface;
 
-class BasePath implements PathBuilderInterface
+class PrependBasePath implements PathBuilderInterface
 {
     private string $basePath;
 
     public function __construct(string $basePath)
     {
-
         $this->basePath = $basePath;
     }
 

src/FreeElephants/StaticHttpClient/PathBuilder/HostnameAsDirectory.php renamed to src/FreeElephants/StaticHttpClient/PathBuilder/PrependHostnameAsDirectory.php

@@ -4,7 +4,7 @@
 
 use Psr\Http\Message\RequestInterface;
 
-class HostnameAsDirectory implements PathBuilderInterface
+class PrependHostnameAsDirectory implements PathBuilderInterface
 {
     public function build(RequestInterface $request, string $path = ''): string
     {

src/FreeElephants/StaticHttpClient/PathResolver/PathBuilderBasedResolver.php

@@ -19,6 +19,7 @@ public function resolve(RequestInterface $request): string
     {
         $filename = $this->pathBuilder->build($request);
 
+        var_dump($filename);
         if (file_exists($filename)) {
             return $filename;
         }

tests/FreeElephants/StaticHttpClient/PathBuilder/CompositeTest.php

@@ -11,13 +11,14 @@ class CompositeTest extends AbstractTestCase
     public function testBuild(): void
     {
         $composite = new Composite(
+            new PrependBasePath('/root'),
+            new AppendHostnameAsDirectory(),
             new AppendRequestPath(),
-            new DefaultFileExtension(),
-            new HostnameAsDirectory(),
+            new AppendDefaultFileExtension(),
         );
 
         $path = $composite->build(new Request('GET', 'http://example.com/foo'));
 
-        $this->assertSame('example.com/foo.json', $path, 'Composite should build path from all builders');
+        $this->assertSame('/root/example.com/foo.json', $path, 'Composite should build path from all builders');
     }
 }

tests/FreeElephants/StaticHttpClient/PathResolver/PathBuilderBasedResolverTest.php

@@ -4,10 +4,11 @@
 
 use FreeElephants\StaticHttpClient\AbstractTestCase;
 use FreeElephants\StaticHttpClient\PathBuilder\AppendRequestPath;
-use FreeElephants\StaticHttpClient\PathBuilder\BasePath;
+use FreeElephants\StaticHttpClient\PathBuilder\PrependBasePath;
 use FreeElephants\StaticHttpClient\PathBuilder\Composite;
-use FreeElephants\StaticHttpClient\PathBuilder\DefaultFileExtension;
+use FreeElephants\StaticHttpClient\PathBuilder\AppendDefaultFileExtension;
 use FreeElephants\StaticHttpClient\PathBuilder\PathBuilderInterface;
+use FreeElephants\StaticHttpClient\PathBuilder\PrependHostnameAsDirectory;
 use FreeElephants\StaticHttpClient\PathResolver\Exception\UnresolvablePathException;
 use Nyholm\Psr7\Request;
 use PHPUnit\Framework\TestCase;
@@ -18,21 +19,22 @@ public function testResolve(): void
     {
         $pathBuilderBasedResolver = new PathBuilderBasedResolver(
             new Composite(
-                new BasePath(self::FIXTURE_PATH),
+                new PrependBasePath(self::FIXTURE_PATH),
+                new PrependHostnameAsDirectory(),
                 new AppendRequestPath(),
-                new DefaultFileExtension()
+                new AppendDefaultFileExtension('.html')
             )
         );
 
-        $actual = $pathBuilderBasedResolver->resolve(new Request('GET', 'http://example.com/foo'));
-        $this->assertSame(self::FIXTURE_PATH . DIRECTORY_SEPARATOR . 'foo.json', $actual);
+        $actual = $pathBuilderBasedResolver->resolve(new Request('GET', 'http://example.com/bar'));
+        $this->assertSame(self::FIXTURE_PATH . '/example.com/bar.html', $actual);
     }
 
     public function testUnresolvedException(): void
     {
         $pathBuilderBasedResolver = new PathBuilderBasedResolver($this->createMock(PathBuilderInterface::class));;
 
         $this->expectException(UnresolvablePathException::class);
-        $pathBuilderBasedResolver->resolve(new Request('GET', 'http://example.com/foo'));
+        $pathBuilderBasedResolver->resolve(new Request('GET', 'http://example.com/bar'));
     }
 }

tests/_fixtures/example.com/bar.html

@@ -0,0 +1,4 @@
+<html>
+    <head><title>bar</title></head>
+    <body></body>
+</html>