Skip to content
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.

Sign up for GitHub

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

Jump to bottom

TCK tests for openapi endpoint #69

Closed
arthurdm opened this issue Nov 27, 2017 · 18 comments
Closed

TCK tests for openapi endpoint #69

arthurdm opened this issue Nov 27, 2017 · 18 comments
Labels
tck

Comments

@arthurdm
Copy link
Member

We will be using the /openapi endpoint for most of our TCK tests already, but this issue will track the inclusion of specific endpoint tests that will use different Accept headers for json and yaml.
@navidsh
Copy link
Contributor

navidsh commented Dec 4, 2017 •
edited
Loading

@navidsh navidsh mentioned this issue Dec 5, 2017
Merged
@mrglavas
Copy link
Contributor

mrglavas commented Dec 5, 2017 •
edited by arthurdm
Loading

We'll want to write assertions which cover each of the annotations. I've copied the checklist below.

Annotation list:

  • Callback
  • Callbacks
  • CallbackOperation
  • Components
  • Explode
  • ParameterIn
  • ParameterStyle
  • SecuritySchemeIn
  • SecuritySchemeType
  • Extension
  • Extensions
  • ExternalDocumentation
  • Header
  • Contact
  • Info
  • License
  • Link
  • LinkParameter
  • Content
  • DiscriminatorMapping
  • Encoding
  • ExampleObject
  • Schema
  • OpenAPIDefinition
  • Operation
  • Parameter
  • Parameters
  • RequestBody
  • APIResponse
  • APIResponses
  • OAuthFlow
  • OAuthFlows
  • OAuthScope
  • SecurityRequirement
  • SecurityRequirements
  • SecurityScheme
  • SecuritySchemes
  • Server
  • Servers
  • ServerVariable
  • Tag
  • Tags

@mrglavas
Copy link
Contributor

mrglavas commented Dec 5, 2017

Going to start with the Info, Contact and License annotations.

@leochr
Copy link
Contributor

leochr commented Dec 5, 2017

I am covering Component, Tag, Tags and Header.

@turkeylurkey
Copy link
Contributor

turkeylurkey commented Dec 5, 2017 •
edited
Loading

Starting with Schema

@janamanoharan
Copy link

janamanoharan commented Dec 5, 2017 •
edited
Loading

doing

  • RequestBody
  • Parameter
  • Explode
  • Callback
  • Callbacks
  • CallbackOperation

@navidsh
Copy link
Contributor

navidsh commented Dec 5, 2017 •
edited
Loading

I'm covering the following annotations:

  • ExternalDocumentation
  • Server
  • Servers
  • ServerVariable
  • OpenAPIDefinition

@a-saf
Copy link

a-saf commented Dec 5, 2017

I'll do SecurityScheme and SecurityRequirement

@arthurdm
Copy link
Member Author

arthurdm commented Dec 5, 2017

thanks everyone. I updated the checkboxes based on the comments.

@navidsh navidsh mentioned this issue Dec 6, 2017
Merged
@mrglavas
Copy link
Contributor

mrglavas commented Dec 6, 2017 •
edited
Loading

I'm now working on Operation, APIResponse and APIResponses.

@a-saf
Copy link

a-saf commented Dec 6, 2017

I'm working on Link, Encoding

@navidsh navidsh mentioned this issue Dec 7, 2017
Merged
@arthurdm
Copy link
Member Author

arthurdm commented Dec 7, 2017

updated checkboxes

@turkeylurkey
Copy link
Contributor

I'll handle Extension, Extensions, Example Object

@arthurdm
Copy link
Member Author

arthurdm commented Dec 8, 2017

updated

@a-saf
Copy link

a-saf commented Dec 8, 2017 •
edited
Loading

Also doing OAuthFlow, OAuthFlows, OAuthScope, as these relate to SecurityScheme (confirmed this with Jana @janamanoharan).
@arthurdm Also completed LinkParameter, and SecuritySchemeIn/SecuritySchemeType (these are enums for SecurityScheme annotation)

Note: SecurityRequirements is covered in petStore app, so I will write test for it there.

@navidsh navidsh mentioned this issue Dec 8, 2017
Merged
@leochr
Copy link
Contributor

leochr commented Dec 8, 2017

I am working on Content

@turkeylurkey
Copy link
Contributor

I am working on DiscriminatorMapping

@navidsh navidsh mentioned this issue Dec 11, 2017
Merged
@arthurdm
Copy link
Member Author

All done - thanks everyone!

Azquelt pushed a commit to Azquelt/microprofile-open-api that referenced this issue Mar 17, 2022
@EricWittmann
Upgraded to version 1.1.1 of the spec (eclipse#69)
f871b22 
* Added a unit test for simple jax-rs processing.  Specifically added a test for eclipse#28

* Upgraded to MP+OAI 1.1.  Fixed some deprecation warnings.

* Updates to smallrye due to change in MP+OAI v1.1

* Upgraded to version 1.1.1 of the MP+OAI spec.

Signed-off-by: Eric Wittmann <eric.wittmann@gmail.com>

* Replaced tabs with spaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tck
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@mrglavas @navidsh @arthurdm @janamanoharan @a-saf @leochr @turkeylurkey