fix openapi version not validating

This is a change for JSON::Validator 4.00 (I think). Fixes #111. Thanks @eserte!
preaction · Jun 10, 2020 · decd083 · decd083
1 parent 8077199
commit decd083
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/dist.ini b/dist.ini
@@ -158,7 +158,7 @@ Mojolicious::Plugin::OpenAPI = 2.12 ; fix the GET /api breakage
 Mojolicious::Plugin::I18N = 1.6 ; Must be all-caps I18N!
 Role::Tiny = 2.000001 ; Required for Mojolicious roles
 Class::Method::Modifiers = 0 ; Required for around/before/after
-JSON::Validator = 3.08
+JSON::Validator = 4
 Sys::Hostname = 0
 File::Spec::Functions = 0
 FindBin = 0

diff --git a/lib/Mojolicious/Plugin/Yancy.pm b/lib/Mojolicious/Plugin/Yancy.pm
@@ -778,7 +778,7 @@ sub _build_validator {
         # This fixes HTML forms submitting the string "20" not being
         # detected as a number, or the number 1 not being detected as
         # a boolean
-        coerce => { booleans => 1, numbers => 1 },
+        coerce => { booleans => 1, numbers => 1, strings => 1 },
     );
     my $formats = $v->formats;
     $formats->{ password } = sub { undef };

diff --git a/lib/Yancy/Plugin/Editor.pm b/lib/Yancy/Plugin/Editor.pm
@@ -622,7 +622,7 @@ sub _openapi_spec_from_schema {
     }
 
     return {
-        info => $config->{info} || { title => $config->{title}, version => 1 },
+        info => $config->{info} || { title => $config->{title}, version => "1" },
         swagger => '2.0',
         host => $config->{host} // hostname(),
         basePath => '/api',

diff --git a/t/plugin/editor.t b/t/plugin/editor.t
@@ -16,6 +16,7 @@ use FindBin qw( $Bin );
 use Mojo::File qw( path );
 use lib "".path( $Bin, '..', 'lib' );
 use Local::Test qw( init_backend );
+use Mojo::JSON qw( true false );
 use Mojolicious;
 
 local $ENV{MOJO_HOME} = path( $Bin, '..', 'share' );
@@ -54,7 +55,7 @@ subtest 'non-default backend' => sub {
             properties => {
                 id => {
                     type => 'integer',
-                    readOnly => 1,
+                    readOnly => true,
                 },
                 name => {
                     type => 'string',