svn commit: r1687546 - in /sling/trunk: bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java

[rombert]

Author: rombert
Date: Thu Jun 25 15:13:03 2015
New Revision: 1687546

URL: http://svn.apache.org/r1687546
Log:
Tolerate empty system options for coverage in Pax-Exam tests

Modified:
    
sling/trunk/bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java
    
sling/trunk/contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java

Modified: 
sling/trunk/bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java?rev=1687546&r1=1687545&r2=1687546&view=diff
==============================================================================
--- 
sling/trunk/bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java
 (original)
+++ 
sling/trunk/bundles/commons/log/src/test/java/org/apache/sling/commons/log/logback/integration/LogTestBase.java
 Thu Jun 25 15:13:03 2015
@@ -112,7 +112,7 @@ public abstract class LogTestBase {
 
     private static Option addCodeCoverageOption() {
         String coverageCommand = System.getProperty(COVERAGE_COMMAND);
-        if (coverageCommand != null) {
+        if (coverageCommand != null && !coverageCommand.isEmpty()) {
             return CoreOptions.vmOption(coverageCommand);
         }
         return null;

Modified: 
sling/trunk/contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java?rev=1687546&r1=1687545&r2=1687546&view=diff
==============================================================================
--- 
sling/trunk/contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java
 (original)
+++ 
sling/trunk/contrib/extensions/datasource/src/test/java/org/apache/sling/datasource/DataSourceTestBase.java
 Thu Jun 25 15:13:03 2015
@@ -96,7 +96,7 @@ public abstract class DataSourceTestBase
 
     private static Option addCodeCoverageOption() {
         String coverageCommand = System.getProperty(COVERAGE_COMMAND);
-        if (coverageCommand != null) {
+        if (coverageCommand != null && !coverageCommand.isEmpty()) {
             return CoreOptions.vmOption(coverageCommand);
         }
         return null;