Skip to content

Conversation

@oslivan
Copy link

@oslivan oslivan commented Sep 3, 2020

No description provided.

@coveralls
Copy link

coveralls commented Sep 3, 2020

Coverage Status

Coverage remained the same at 99.47% when pulling f788f43 on oslivan:master into 588579d on ruby-grape:master.

@oslivan oslivan closed this Sep 29, 2020
desc 'post in body with entity',
success: ::Entities::ResponseItem
params do
optional :names, type: Array[String], documentation: { desc: 'names', param_type: 'body' }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here you set the param as optional

specify do
expect(subject['paths']['/with_entities/in_body2']['post']['parameters']).to eql(
[
{ 'name' => 'WithEntitiesInBody2', 'in' => 'body', 'required' => true, 'schema' => { '$ref' => '#/definitions/postWithEntitiesInBody2' } }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but expecting required here and …

'properties' => {
'names' => { 'type' => 'array', 'description' => 'names', 'items' => { 'type' => 'string' } }
},
'required' => ['names'],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants