Fix condition when stopping test servers

It was broken for solr; the same condition is used for Redis as well, for consistency.
SwissDataScienceCenter · Feb 7, 2024 · 32427e7 · 32427e7
1 parent 1373f5d
commit 32427e7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/project/RedisServer.scala b/project/RedisServer.scala
@@ -82,7 +82,8 @@ class RedisServer(module: String, port: Int) {
   }
 
   def stop(): Unit =
-    if (!skipServer && wasStartedHere.get()) {
+    if (skipServer || !wasStartedHere.get()) ()
+    else {
       println(s"Stopping Redis container for '$module'")
       stopCmd.!!
       ()

diff --git a/project/SolrServer.scala b/project/SolrServer.scala
@@ -100,7 +100,7 @@ class SolrServer(module: String, port: Int) {
   }
 
   def stop(): Unit =
-    if (!skipServer && !wasStartedHere.get()) ()
+    if (skipServer || !wasStartedHere.get()) ()
     else {
       println(s"Stopping Solr container for '$module'")
       stopCmd.!!