eclipse / microprofile-telemetry

microprofile telemetry
Apache License 2.0
16 stars 16 forks source link

Rewrite JaxRsServerAsyncTest to not depend directly on Jakarta EE Concurrency #140

Closed Azquelt closed 1 week ago

Azquelt commented 7 months ago

JaxRsServerAsyncTest currently depends on Jakarta EE Concurrency by injection of a ManagedExecutorService resource.

However, the only requirement for this test is that the CompletionStage is not completed until after the getCompletionStage resource method returns. While the concurrency spec is the way to do this in Jakarta EE, not all MP runtimes support that and there are other options.

We should rewrite this test so that it uses an Executor provided by the implementer running the TCK. That way the implementer can run the test with whatever async implementation is most appropriate in their runtime.

We should then make this test mandatory since it will no longer depend on Jakarta EE Concurrency.