-
Notifications
You must be signed in to change notification settings - Fork 279
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introducing Swagger in Dredd #529
Conversation
|
||
#### Swagger | ||
|
||
1. [`response.schema`](http://swagger.io/specification/#responseSchema) - provided JSON Schema will be used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think response.examples is given preference.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when checking validity of response? I think it shouldn't
88bd38d
to
c40910b
Compare
@@ -20,6 +19,11 @@ Dredd reads your API description and step by step validates whether your API | |||
implementation replies with responses as they are described in the | |||
documentation. | |||
|
|||
## API Description Formats Support |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe "Supported API Description formats" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in 5e5c81b.
@@ -2,7 +2,7 @@ | |||
|
|||
This is an example how to create a simple [Express.js][] API backend application tested with Dredd | |||
|
|||
## Create API Description File | |||
## Create an API Description Document |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the article correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in 5e5c81b.
### Response Headers Expectations | ||
|
||
- All headers specified in the API description must be present in the response. | ||
- Names of headers are validated in case-insensitive way. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe "validated >the< case-insensitive way" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in 5e5c81b.
@netmilk All comments addressed. Good to merge? |
❤️ |
Also addressing: