CompletionStageResponseTest intermittent failures #2390
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
asoldano
changed the title
asoldano
force-pushed
the
CompletionStageResponseTest-intermittent-failure
branch
from
8d45718 to
2494255
Compare
asoldano
added a commit
to asoldano/Resteasy
that referenced
this pull request
May 12, 2020
…steasy#2390) * Prevent memory visibility issues * Refactor AsyncResponseCallback to isolate data from different tests * Same sleep time for similar methods
ronsigal
added a commit
that referenced
this pull request
Jun 16, 2020
* 4.5.0-SNAPSHOT -> 4.5.0.Final (#2307) * 4.5.0.Final -> 4.6.0-SNAPSHOT * [RESTEASY-2496]:Add test case * [RESTEASY-2510] Split RESTUtils to links provider and injector * RESTEASY-2445 * [RESTEASY-2517] Async io fixes (#2315) * Apply same fix for SSE streams into raw streams To avoid onNext finishing after onComplete * Test for raw stream and async io * Fixed typo in onComplete/onWriteComplete * Fix potential bug spotted by Stuart * [RESTEASY-2520] CDI BeanManager fix for Quarkus (#2318) * Fix for Quarkus (quarkusio/quarkus#7639) Signed-off-by:Phillip Kruger <[email protected]> * Added check for CDI in construct. Signed-off-by:Phillip Kruger <[email protected]> * Checkstyle fixes Signed-off-by:Phillip Kruger <[email protected]> * [RESTEASY-2518] added value check and test for query param replacement * [RESTEASY-2503] Throw ViolationException for EJBs with just parameter violations. * RESTEASY-2519 (#2320) * RESTEASY-2519 * update * [RESTEASY-2535] Upgrade Jackson2 to 2.10.3 * [RESTEASY-2541] Fix RestClientListeners * [RESTEASY-2542] Excluding tests * RESTEASY-2554 Update section "Upgrading RESTEasy within WildFly" (#2349) * update * RESTEASY-2554 * remove module * update doc * minor fix * minor addition * minor fix * minor addition * fix version * minor fix * [RESTEASY-2559] Improper validation of response header in MediaTypeHeaderDelegate.java class * [RESTEASY-2551] Upgraded io.netty:netty-all to version 4.1.48.Final * [RESTEASY-2543] add regex change * [RESTEASY-2226]:NullPointerException in PatchMethodFilter * [RESTEASY-2385] Added support for embedded MultipartOutput types. Added testcase. * [RESTEASY-2480] re-enable code handling base64 * [RESTEASY-2532] replace reference to jakarta-json * [RESTEASY-2565] Mp client async interceptors (#2359) * Tie the AsyncInvocationInterceptor to each invocation * Spotted a bug, I think * [RESTEASY-2566] Set response code to 500 if writing exception fails (#2358) * [RESTEASY-2562] Incorrect status code when WebApplicationException is… (#2365) * [RESTEASY-2562] Incorrect status code when WebApplicationException is thrown while parsing @*Param * [RESTEASY-2562] Checkstyle - make arg final * [RESTEASY-2495] HeaderParam could have a ParamConverter related (#2292) AbstractInvocationCollectionProcessor is processing Array and Collection of elements being ParamConverter aware. But with single values is delegating in final class to process the value. HeaderParamProcessor is not being aware of such transformation. * [RESTEASY-2495] made HeaderParamProcessor aware of ParamConverter and provided testcase * [RESTEASY-2526] Fix by using asyncutil lib for async loops (#2368) * [RESTEASY-2466] removed profile arquillian.remote * [RESTEASY-2531] expanded and refactored chapter * [RESTEASY-2544] pom updates to arquillian-util and integration-tests * [RESTEASY-2542] ignore any IOException in SSEOutputStream.close() * Test against WildFly 19.0.0.Final instead of Beta2 * [RESTEASY-2526] Turn the com.ibm.async.asyncutil module into private * [RESTEASY-2550] Skip null parameters being processed by ParamConverter's. Also throw an NPE if a parameter annotated with @PathParam it not allowed. https://issues.redhat.com/browse/RESTEASY-2550 * [RESTEASY-2542] Restore tests * [RESTEASY-2239] updated mime4j to version 0.8.2 * Add GitHub Action for running CI builds * [RESTEASY-2561] Allow tests to run in environments other than a Linux local environment. * [RESTEASY-2564] Use a single thread executor instead of the default executor for a more graceful shutdown. * Remove TravisCI runs duplicated by Github Actions * [RESTEASY-2542] Exclude a test again, still failing * [RESTEASY-2542] Exlcude the whole SseTest, not just testReconnect() * Only upload archives for failed runs. Also fix the name to not end with ".zip" * [RESTEASY-2548] add new provider with higher priority than JaxrsFormProvider adding test * [RESTEASY-2560] set scope for must junit dependencies to test * [RESTEASY-2391] changed code to use getMethod() * [RESTEASY-2472]:@suspended AsyncResponse timeout still runs, even if … (#2372) * [RESTEASY-2472]:@suspended AsyncResponse timeout still runs, even if the response was resumed before * Update AsyncTimeoutResource.java * Update AsyncTimeoutResource.java Bogus change to rerun tests. Co-authored-by: Ron Sigal <[email protected]> * [RESTEASY-2563: SseBroadcasterImpl.close() puts notifyOnCloseListeners(eventSink) in finally block. (#2396) * [RESTEASY-2576] added client modules * [RESTEASY-2582] CompletionStageResponseTest intermittent failures (#2390) * Prevent memory visibility issues * Refactor AsyncResponseCallback to isolate data from different tests * Same sleep time for similar methods * [RESTEASY-2571] Avoid a NullPointerException of the MediaType associated with request headers is null. https://issues.redhat.com/browse/RESTEASY-2571 * [RESTEASY-2569] RestClient does not propogate incoming headers with inherited Interface * [RESTEASY-2582] Add logs to the test * [RESTEASY-2582] Fix race condition in test * [RESTEASY-1203] Fix validation for JNDI resources. (#2406) * [RESTEASY-2542][RESTEASY-2585] Workaround SseTest intermittent failures (#2404) * Enable SseTest * [RESTEASY-2542]:Fix Sse reconnect test * [RESTEASY-2542]:More Sse test fix * Fix typo and add comment * [RESTEASY-2017]:Fix SSE doesn't work with inherited annotations * Updated links to the new project site * [RESTEASY-2588] Upgrade jackson to 2.10.4 * [RESTEASY-2587] Update classmate to 1.5.1 * Fix RESTEASY-2573 by using ResteashContext.addCloseable variants * Test RESTEASY-2573 for SSE/raw/collect async streaming * [RESTEASY-2542] wait for proxy to be started * [RESTEASY-2576] Added modules to resteasy-bom/pom.xml (#2417) * Added note to testsuite/README.MD on how to run single microprofile-tck tests (#2426) * SseEventOutputImpl: avoid deadlock by avoiding getting into critical sections early RESTEASY-2585 * HttpServletResponseWrapper: try working around UNDERTOW-1713: RESTEASY-2585 * RESTEASY-2524 (#2332) * RESTEASY-2524 * trailing spaces removed * [RESTEASY-2585]:Remove Thread.sleep() workaround in SseTest * Updating target test containers * [RESTEASY-2598] Upgrade Infinispan to 10.1.8 * [RESTEASY-2598] Exclude ServerCacheInterceptorTest in WFLY19 * Fix modules test references to properly run with WFLY20 too * Exclude test failing on GithubActions CI due to microprofile/microprofile-rest-client#265 * [RESTESY-2601] Exclude test * [RESTEASY-2605] Remove the fallback to URLConnection when SSL is disabled for REST Client (#2431) The fallback was originally added to get around limitation in Apache HTTP Client when SSL is completely disabled in native binaries. However we can get around these limitations in a more elegant manner in Quarkus (see quarkusio/quarkus#9773). By defaulting to Apache HTTP Client, things like HTTP Patch can be supported OOTB (see quarkusio/quarkus#9342). Co-authored-by: Ron Sigal <[email protected]> Co-authored-by: Jim Ma <[email protected]> Co-authored-by: Gytis Trikleris <[email protected]> Co-authored-by: 阿男 <[email protected]> Co-authored-by: Stéphane Épardaud <[email protected]> Co-authored-by: Phillip Krüger <[email protected]> Co-authored-by: R Searls <[email protected]> Co-authored-by: Bartosz Spyrko-Smietanko <[email protected]> Co-authored-by: Stuart Douglas <[email protected]> Co-authored-by: Anil Gursel <[email protected]> Co-authored-by: David Santos <[email protected]> Co-authored-by: James Perkins <[email protected]> Co-authored-by: Duncan <[email protected]> Co-authored-by: bobbyphilip <[email protected]> Co-authored-by: Tommaso Borgato <[email protected]> Co-authored-by: Georgios Andrianakis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A bunch of changes in test classes to prevent concurrency related issues that might be causing the intermittent failure of org.jboss.resteasy.test.response.CompletionStageResponseTest and similar tests.