Temporarily let junit tests runner allow no tests, like before

2 files changed, 2 insertions, 2 deletions

diff --git a/vespa-osgi-testrunner/src/main/java/com/yahoo/vespa/testrunner/TestReport.java b/vespa-osgi-testrunner/src/main/java/com/yahoo/vespa/testrunner/TestReport.java
index 4db5405029b..945b7603fe4 100644
--- a/vespa-osgi-testrunner/src/main/java/com/yahoo/vespa/testrunner/TestReport.java
+++ b/vespa-osgi-testrunner/src/main/java/com/yahoo/vespa/testrunner/TestReport.java
@@ -39,7 +39,7 @@ public class TestReport {
     }
 
     public TestRunner.Status status() {
-        return failedCount > 0 ? FAILURE : inconclusiveCount > 0 ? INCONCLUSIVE : (successCount + abortedCount + ignoredCount) > 0 ? SUCCESS : NO_TESTS;
+        return failedCount > 0 ? FAILURE : inconclusiveCount > 0 ? INCONCLUSIVE : (successCount + abortedCount + ignoredCount) > -1 ? SUCCESS : NO_TESTS; // TODO: require at least one test
     }
 
     public static Builder builder(){
diff --git a/vespa-osgi-testrunner/src/test/java/com/yahoo/vespa/testrunner/AggregateTestRunnerTest.java b/vespa-osgi-testrunner/src/test/java/com/yahoo/vespa/testrunner/AggregateTestRunnerTest.java
index 1af012f0fb2..2881da774f5 100644
--- a/vespa-osgi-testrunner/src/test/java/com/yahoo/vespa/testrunner/AggregateTestRunnerTest.java
+++ b/vespa-osgi-testrunner/src/test/java/com/yahoo/vespa/testrunner/AggregateTestRunnerTest.java
@@ -147,7 +147,7 @@ class AggregateTestRunnerTest {
 
     @Test
     void testReportStatus() {
-        assertEquals(NO_TESTS, TestReport.builder().build().status());
+        assertEquals(SUCCESS, TestReport.builder().build().status()); // TODO: require at least 1 test
         assertEquals(SUCCESS, TestReport.builder().withSuccessCount(1).build().status());
         assertEquals(INCONCLUSIVE, TestReport.builder().withSuccessCount(1).withInconclusiveCount(1).build().status());
         assertEquals(FAILURE, TestReport.builder().withSuccessCount(1).withFailedCount(1).build().status());