Skip to content

Commit

Permalink
Test MessageBodyWriters resources works in native mode
Browse files Browse the repository at this point in the history
verifies: quarkusio/quarkus#25385

When we provide `MessageBodyWriter` that does not extend `ServerMessageBodyWriter`, related resource methods must be registered for the reflection. Previously, we had to do this manually, now it's done by the Quarkus for us.

Reproducer
1. go to Quarkus project
2. revert PR `git revert 224ec8c3e8e50930970e9c8b7316622ba898b0b9`
3. checkout this PR
4. `cd quarkus-test-suite/http/http-advanced-reactive`
5. run `mvn clean verify -Dit.test=HttpAdvancedReactiveIT -Dnative`
  • Loading branch information
michalvavrik committed Jul 25, 2022
1 parent 3a1275f commit 0a45b58
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,12 @@
import javax.ws.rs.Produces;

import io.quarkus.arc.properties.UnlessBuildProperty;
import io.quarkus.runtime.annotations.RegisterForReflection;
import io.smallrye.mutiny.Uni;

/**
* {@link Path#value()} set in this resource contains characters that were causing the build time
* validation failure. The issue was resolved in 2.8.3 with https://github.com/quarkusio/quarkus/issues/25258.
*/
@RegisterForReflection
@UnlessBuildProperty(name = QUARKUS_PLATFORM_VERSION_LESS_THAN_2_8_3, stringValue = QUARKUS_PLATFORM_VERSION_LESS_THAN_2_8_3_VAL, enableIfMissing = true)
@Path(NinetyNineBottlesOfBeerResource.PATH)
public class NinetyNineBottlesOfBeerResource {
Expand Down

0 comments on commit 0a45b58

Please sign in to comment.