Skip to content

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Nov 15, 2019

Relates to: #5374

@geoand geoand requested a review from gsmet November 15, 2019 12:44
@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 15, 2019
@geoand geoand added the area/spring Issues relating to the Spring integration label Nov 15, 2019
@geoand geoand added this to the 1.1.0 milestone Nov 15, 2019
@machi1990 machi1990 merged commit a57de6d into quarkusio:master Nov 15, 2019
@machi1990
Copy link
Member

Merged. Thanks @geoand @gsmet

@geoand
Copy link
Contributor Author

geoand commented Nov 15, 2019

Thanks for merging!

@geoand geoand deleted the #5374-test branch November 15, 2019 19:13
RestAssured.when().get("/exception/re").then()
.contentType("application/json")
.body(containsString("hello from error"))
.statusCode(402);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, where does this status code come from? 402 seems pretty unusual, but maybe it's completely normal in that case.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just added a random one to make ensure the response was different

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It could have been anything, I just picked one in random 🤪

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation! :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NP!

@gsmet gsmet modified the milestones: 1.1.0, 1.0.1.Final Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/spring Issues relating to the Spring integration triage/waiting-for-ci Ready to merge when CI successfully finishes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants