jhthorsen / openapi-client

A client for talking to an Open API powered server
11 stars 17 forks source link

Duplicate keys not allowed #13

Closed eserte closed 6 years ago

eserte commented 6 years ago

The test suite fails on some of my smokers:

Duplicate keys not allowed, at character offset 595 (before "required": ["email",...") at /home/cpansand/.cpan/build/2018070703/Mojolicious-7.87-ThKlUI/blib/lib/Mojo/JSON.pm line 39.
t/body-validation.t ......... 
Dubious, test returned 255 (wstat 65280, 0xff00)
No subtests run 

The possibly problematic files in the test file:

              "required": true,                                                                                                                                                                                                                                               
              "required": ["email", "password"],                                                                                                                                                                                                                              
jberger commented 6 years ago

Interesting. @jhthorsen this is a result of the addition of Mojo::JSON's native Cpanel::JSON::XS handling. That module croaks on duplicate keys so this test is failing where C::JSON::XS is installed.