You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2004/01/07 00:51:18 UTC
cvs commit: jakarta-jmeter/src/components/org/apache/jmeter/assertions ResponseAssertion.java
sebb 2004/01/06 15:51:18
Modified: src/components/org/apache/jmeter/assertions
ResponseAssertion.java
Log:
Make Response Assertion behave like the other Assertions
Revision Changes Path
1.14 +18 -14 jakarta-jmeter/src/components/org/apache/jmeter/assertions/ResponseAssertion.java
Index: ResponseAssertion.java
===================================================================
RCS file: /home/cvs/jakarta-jmeter/src/components/org/apache/jmeter/assertions/ResponseAssertion.java,v
retrieving revision 1.13
retrieving revision 1.14
diff -u -r1.13 -r1.14
--- ResponseAssertion.java 15 Oct 2003 23:05:33 -0000 1.13
+++ ResponseAssertion.java 6 Jan 2004 23:51:18 -0000 1.14
@@ -193,16 +193,20 @@
public AssertionResult getResult(SampleResult response)
{
AssertionResult result;
- if (!response.isSuccessful())
- {
- result = new AssertionResult();
- result.setError(true);
- byte [] ba = response.getResponseData();
- result.setFailureMessage(
- ba == null ? "Unknown Error (responseData is empty)" : new String(ba)
- );
- return result;
- }
+
+// None of the other Assertions check the response status, so remove this check
+// for the time being, at least...
+// if (!response.isSuccessful())
+// {
+// result = new AssertionResult();
+// result.setError(true);
+// byte [] ba = response.getResponseData();
+// result.setFailureMessage(
+// ba == null ? "Unknown Error (responseData is empty)" : new String(ba)
+// );
+// return result;
+// }
+
result = evaluateResponse(response);
return result;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org