fix(parser) Allow URI template to be / only.

Hywan · Dec 11, 2017 · 896a32c · 896a32c
1 parent 2cbde0c
commit 896a32c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Parser.php b/src/Parser.php
@@ -489,12 +489,12 @@ public function getHeaderType(string $headerContent, &$matches = []): int
         }
 
         // Resource section.
-        if (0 !== preg_match('/^(?<name>[^\[]+)\[(?<uriTemplate>\/[^\]]+)\]/', $headerContent, $matches)) {
+        if (0 !== preg_match('/^(?<name>[^\[]+)\[(?<uriTemplate>\/[^\]]*)\]/', $headerContent, $matches)) {
             return self::HEADER_RESOURCE;
         }
 
         // Action section.
-        if (0 !== preg_match('/(?:^(?<requestMethod1>[A-Z]+)$)|(?:^(?<name>[^\[]+)\[(?<requestMethod2>[A-Z]+)(?:\h+(?<uriTemplate>\/[^\]]+))?\])/', $headerContent, $matches)) {
+        if (0 !== preg_match('/(?:^(?<requestMethod1>[A-Z]+)$)|(?:^(?<name>[^\[]+)\[(?<requestMethod2>[A-Z]+)(?:\h+(?<uriTemplate>\/[^\]]*))?\])/', $headerContent, $matches)) {
             if (empty($matches['requestMethod1'])) {
                 $matches['requestMethod1'] = null;
             }

diff --git a/test/integration/Parser.php b/test/integration/Parser.php
@@ -115,14 +115,14 @@ public function test_one_empty_resource()
         $this
             ->given(
                 $parser = new SUT(),
-                $datum  = '# Foo Bar [/foo/bar]'
+                $datum  = '# Foo Bar [/]'
             )
             ->when($result = $parser->parse($datum))
             ->then
                 ->let(
                     $resource              = new IR\Resource(),
                     $resource->name        = 'Foo Bar',
-                    $resource->uriTemplate = '/foo/bar',
+                    $resource->uriTemplate = '/',
 
                     $document              = new IR\Document(),
                     $document->resources[] = $resource