Author: pmouawad
Date: Sat Dec 31 14:47:47 2016
New Revision: 1776763
URL: http://svn.apache.org/viewvc?rev=1776763&view=rev
Log:
Sonar : fix squid:CommentedOutCodeLine "This block of commented-out lines of
code should be removed. "
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=1776763&r1=1776762&r2=1776763&view=diff
==============================================================================
---
jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
(original)
+++
jmeter/trunk/src/components/org/apache/jmeter/assertions/XMLSchemaAssertion.java
Sat Dec 31 14:47:47 2016
@@ -101,9 +101,6 @@ public class XMLSchemaAssertion extends
*/
private void setSchemaResult(AssertionResult result, String xmlStr, String
xsdFileName) {
try {
- // boolean toReturn = true;
-
- // Document doc = null;
DocumentBuilderFactory parserFactory =
DocumentBuilderFactory.newInstance();
parserFactory.setValidating(true);
parserFactory.setNamespaceAware(true);
@@ -113,8 +110,6 @@ public class XMLSchemaAssertion extends
// create a parser:
DocumentBuilder parser = parserFactory.newDocumentBuilder();
parser.setErrorHandler(new SAXErrorHandler(result));
-
- // doc =
parser.parse(new InputSource(new StringReader(xmlStr)));
// if everything went fine then xml schema validation is valid
} catch (SAXParseException e) {
@@ -189,7 +184,6 @@ public class XMLSchemaAssertion extends
*/
@Override
public void fatalError(SAXParseException exception) throws
SAXParseException {
-
String msg = "fatal: " + errorDetails(exception);
log.debug(msg);
result.setFailureMessage(msg);
@@ -202,13 +196,9 @@ public class XMLSchemaAssertion extends
*/
@Override
public void warning(SAXParseException exception) throws
SAXParseException {
-
String msg = "warning: " + errorDetails(exception);
log.debug(msg);
result.setFailureMessage(msg);
- // result.setError(true); // TODO is this the correct strategy?
- // throw exception; // allow assertion to pass
-
}
}
}