AsyncMethodTest#testAsyncInvocationInterceptorProvider is racey #203
Comments
stuartwdouglas
added a commit
to stuartwdouglas/quarkus
that referenced
this issue
Jun 13, 2019
This is due to eclipse/microprofile-rest-client#203
patriot1burke
added a commit
to patriot1burke/quarkus
that referenced
this issue
Jun 14, 2019
root path docs fix imports fix poms and typos doc suggestions Add mp rest-client tck configuration Introduce handling for mp/uri in accordance to spec Fixes: quarkusio#2795 Remove misleading sentence from Kubernetes / ap4k guide The guide will updated when dekorateio/dekorate#227 is implemented and the new version included into Quarkus Implement mock mailer by default on dev/test fix(quarkusio#2789): fix Jackson version issue The oder of import in the bom is important, we need to import Jackson first before RESTEasy. After this, exluding and importing Jackson in the quarkus-smallrye-openapi extention is useless. Arc - correct circular dependency check to detect self-injection as well, added test. feat(openapi): Allows OpenAPI path to contain segments e.g /api/openapi/ Fixes quarkusio#2756 Arc - Implement inheritance of scopes for beans. register reflective classes from bean archive index by providing the index as part of the reflective class build item Document InjectionPointTransformer. Arc - register certain CDI components under BuildContext so extensions can use that. Modify SpringDIProcessor to make use of this. Subclasses/client proxies - skip delegation if not constructed yet - resolves quarkusio#2774 Docs - CDI guide - remove unnecessary dummy constructor feat(agroal): Adds a configuration option to enable datasource metrics Fixes quarkusio#2820 Quarkus bypasses Nexus and downloads artefacts directly from maven central Fix title levels in Undertow reference guide Remove two ArC injection warnings Update to smallrye-jwt-1.1 Moved common stuff to panache-common-deployment, ported `@GenerateBridge` to non-rx panache Added two finals Skip racey TCK test This is due to eclipse/microprofile-rest-client#203 Undertow 2.0.21.Final Better error reporting if test resource fails to start Remove license headers
|
Hey @stuartwdouglas, we've been seeing the same issue with CXF's MP Client implementation when running the TCK. I am wondering what MP Client implementation you are running TCK against? Thank you. |
|
Quarkus, so a modified version of the Smallrye implementation. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
org.eclipse.microprofile.rest.client.tck.providers.TLAsyncInvocationInterceptor#removeContext happens after the CompletionStage in the test is complete. This means there is no guarentee the test will actually see this method as being run, as it may not have happened yet.
The text was updated successfully, but these errors were encountered: