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.

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

Need JAX-RS TCK test case for handling arrays as parameters (query, cookie, header, etc.) #413

Closed
andymc12 opened this issue Aug 6, 2020 · 4 comments
Assignees
Labels
future Target future Jakarta EE release

Comments

@andymc12
Copy link
Contributor

andymc12 commented Aug 6, 2020

In Jakarta RESTful Web Services issue jakartaee/rest#890 (targeted for 3.1 / post EE-9), implementations are required to handle array types as various types of parameters to resource methods - ex:

@GET
public List<Widget> getWidgetByTypes(@QueryParam("types") String[] typeNames) { //...

The TCK should already test collections in https://github.com/eclipse-ee4j/jakartaee-tck/blob/master/src/com/sun/ts/tests/jaxrs/ee/rs/queryparam/QueryParamTest.java - this class should probably add @QueryParam fields for array types.

@scottmarlow scottmarlow added the future Target future Jakarta EE release label Aug 7, 2020
@gurunrao gurunrao self-assigned this Dec 15, 2020
@scottmarlow
Copy link
Contributor

@andymc12 is this a TCK challenge or simply a TCK improvement for the future?

CC @spericas

@andymc12
Copy link
Contributor Author

@scottmarlow This is a request for a new/updated test (improvement). It is intended to test compatibility for a new function in RESTful WS 3.1 (EE 9.1?). Thanks

@alwin-joseph
Copy link
Contributor

alwin-joseph commented Dec 15, 2021

Duplicate issue raised jakartaee/rest#1019 .Now that the TCK is moved to new jaxrs-api, test should be added in jaxrs-api/jaxrs-tck

@gurunrao
Copy link
Contributor

closing duplicate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future Target future Jakarta EE release
Projects
None yet
Development

No branches or pull requests

4 participants