-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat(core): add support for OAS3 Server object #428
Conversation
Closes APIARY-5855
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 two of the properties proposes can be removed as the API Elements design won't allow for those to exist.
@kyle I am trying to add the support for the server object to the Fury adapter, but I am missing something, can you please have a check? |
ea4487b
to
2ff7549
Compare
2ff7549
to
60a1f98
Compare
packages/fury-adapter-oas3-parser/lib/parser/oas/parseServerObject.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/lib/parser/oas/parseServerObject.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/lib/parser/oas/parseOpenAPIObject.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/lib/parser/oas/parseServerObject.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/test/unit/parser/oas/parseOpenAPIObject-test.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/test/unit/parser/oas/parseServerObject-test.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/test/unit/parser/oas/parseServersArray-test.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/test/unit/parser/oas/parseServersArray-test.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/test/unit/parser/oas/parseServersArray-test.js
Outdated
Show resolved
Hide resolved
packages/fury-adapter-oas3-parser/lib/parser/oas/parseHostsObject.js
Outdated
Show resolved
Hide resolved
|
||
const { minim: namespace } = new Fury(); | ||
|
||
describe('#parseHostsObject', () => { |
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.
describe('#parseHostsObject', () => { | |
describe('#parseServersArray', () => { |
To match the function we are testing.
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.
changed
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 this is coming a long and looks good now. I'll approve it, but let's not merge this until we have the API Element's specification PR completed.
@kylef As apiaryio/api-elements#62 has been merged I am merging also this PR |
Closes APIARY-5855, APIARY-5856