fix wrong resolving of responses component using $ref (#277)

Signed-off-by: Jihyeon Gim <[email protected]>

Author: potatogim

lib/JSON/Validator/Schema/OpenAPIv3.pm

@@ -2,7 +2,7 @@ package JSON::Validator::Schema::OpenAPIv3;
 use Mojo::Base 'JSON::Validator::Schema::Draft201909';
 
 use JSON::Validator::Util qw(E data_type negotiate_content_type schema_type);
-use Mojo::JSON qw(false true);
+use Mojo::JSON            qw(false true);
 use Mojo::Path;
 
 has moniker       => 'openapiv3';
@@ -91,8 +91,8 @@ sub parameters_for_response {
   my $cache_key = "parameters_for_response:$method:$path:$status";
   return $self->{cache}{$cache_key} if $self->{cache}{$cache_key};
 
-  my $responses = $self->get([paths => $path, $method, 'responses']);
-  my $response  = $responses->{$status} || $responses->{default};
+  my $response = $self->get([paths => $path, $method, 'responses', $status])
+    || $self->get([paths => $path, $method, 'responses', 'default']);
   return undef unless $response;
 
   my @parameters;

t/openapiv3-basic.t

@@ -127,6 +127,15 @@ subtest 'validate_response - accept' => sub {
     'negotiated content type';
 };
 
+subtest 'validate_response - resusable response' => sub {
+  $body   = {accept => 'application/*'};
+  @errors = $schema->validate_response([get => '/pets', 201], {body => \&body});
+  is "@errors", '', 'valid accept';
+  is_deeply $body,
+    {accept => 'application/*', content_type => 'application/json', in => 'body', name => 'body', valid => 1},
+    'negotiated content type';
+};
+
 subtest 'validate_response - content_type' => sub {
   $body   = {content_type => 'text/plain'};
   @errors = $schema->validate_response([get => '/pets'], {body => \&body});

t/spec/v3-petstore.json

@@ -57,6 +57,9 @@
               "application/xml": { "schema": { "$ref": "#/components/schemas/Pets" } }
             }
           },
+          "201": {
+            "$ref": "#/components/responses/Pets"
+          },
           "default": {
             "description": "unexpected error",
             "content": {
@@ -119,6 +122,15 @@
           "message": { "type": "string" }
         }
       }
+    },
+    "responses": {
+        "Pets": {
+            "description": "Reusable Pets response",
+            "content": {
+              "application/json": { "schema": { "$ref": "#/components/schemas/Pets" } },
+              "application/xml": { "schema": { "$ref": "#/components/schemas/Pets" } }
+            }
+        }
     }
   }
 }