You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2012/09/04 08:37:12 UTC
svn commit: r1380458 -
/jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
Author: pmouawad
Date: Tue Sep 4 06:37:11 2012
New Revision: 1380458
URL: http://svn.apache.org/viewvc?rev=1380458&view=rev
Log:
Rollback modification as per dev list discussion
Modified:
jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java?rev=1380458&r1=1380457&r2=1380458&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java Tue Sep 4 06:37:11 2012
@@ -175,9 +175,7 @@ public class XMLSchemaAssertion extends
public void error(SAXParseException exception) throws SAXParseException {
String msg = "error: " + errorDetails(exception);
- if(log.isDebugEnabled()) {
- log.debug(msg);
- }
+ log.debug(msg);
result.setFailureMessage(msg);
result.setError(true);
throw exception;
@@ -190,9 +188,7 @@ public class XMLSchemaAssertion extends
public void fatalError(SAXParseException exception) throws SAXParseException {
String msg = "fatal: " + errorDetails(exception);
- if(log.isDebugEnabled()) {
- log.debug(msg);
- }
+ log.debug(msg);
result.setFailureMessage(msg);
result.setError(true);
throw exception;
@@ -204,9 +200,7 @@ public class XMLSchemaAssertion extends
public void warning(SAXParseException exception) throws SAXParseException {
String msg = "warning: " + errorDetails(exception);
- if(log.isDebugEnabled()) {
- log.debug(msg);
- }
+ log.debug(msg);
result.setFailureMessage(msg);
// result.setError(true); // TODO is this the correct strategy?
// throw exception; // allow assertion to pass