You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What steps will reproduce the problem? 1. Use rest-assured@2.4.0
2. Call RestAssured.enableLoggingOfRequestAndResponseIfValidationFails()
3. Invoke HTTP resource that return an unknown code, like 601 What is the expected output? What do you see instead? java.lang.IllegalArgumentException: Status code must be greater than 100 and less than 600, was 601.
at com.jayway.restassured.builder.ResponseBuilder.build(ResponseBuilder.java:175)
Without logging enabled things works well.
Enabling logging should not fail tests that pass without it.
From my understanding, HTTP RFCs do not forbid usage of status codes >599, they are simply unspecified.
I created a pull request 1773 where I removed the check from the Builder class. We really need this fix to check our application because we're using "911" as application specific error code.
From eskato...@gmail.com on January 10, 2015 12:07:23
What steps will reproduce the problem? 1. Use rest-assured@2.4.0
2. Call RestAssured.enableLoggingOfRequestAndResponseIfValidationFails()
3. Invoke HTTP resource that return an unknown code, like 601 What is the expected output? What do you see instead? java.lang.IllegalArgumentException: Status code must be greater than 100 and less than 600, was 601.
at com.jayway.restassured.builder.ResponseBuilder.build(ResponseBuilder.java:175)
Without logging enabled things works well.
Enabling logging should not fail tests that pass without it.
From my understanding, HTTP RFCs do not forbid usage of status codes >599, they are simply unspecified.
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=382
The text was updated successfully, but these errors were encountered: