commit eee38b8d7ba709e4cfe8cd46cea9b313b16e925a
author David Pursehouse <dpursehouse@collab.net> Fri May 24 13:04:49 2019 +0900
committer David Pursehouse <dpursehouse@collab.net> Fri May 24 13:04:49 2019 +0900
tree 511e90f8063b0d8e627a1867dd63ccaa57c0e9bb
parent 373bb75b9b323324d9382fe9686c7c127e71cc74

ExecutorModuleTest: Use Truth to assert about Optional.isPresent

Change-Id: I81998aca3eb89e56a78ff6b5586b32335b1ffb9c

src/test/java/com/ericsson/gerrit/plugins/gcconductor/executor/ExecutorModuleTest.java

diff --git a/src/test/java/com/ericsson/gerrit/plugins/gcconductor/executor/ExecutorModuleTest.java b/src/test/java/com/ericsson/gerrit/plugins/gcconductor/executor/ExecutorModuleTest.java
index f4d3cef..3e88f2b 100644
--- a/src/test/java/com/ericsson/gerrit/plugins/gcconductor/executor/ExecutorModuleTest.java
+++ b/src/test/java/com/ericsson/gerrit/plugins/gcconductor/executor/ExecutorModuleTest.java
@@ -15,7 +15,7 @@
 package com.ericsson.gerrit.plugins.gcconductor.executor;
 
 import static com.google.common.truth.Truth.assertThat;
-import static org.junit.Assert.fail;
+import static com.google.common.truth.Truth8.assertThat;
 import static org.mockito.Mockito.when;
 
 import com.google.inject.Binder;
@@ -45,9 +45,7 @@
     String host = "hostname";
     when(execCfg.isPickOwnHostOnly()).thenReturn(true);
     Optional<String> wasQueuedFrom = executorModule.wasQueuedFrom(execCfg, host);
-    if (!wasQueuedFrom.isPresent()) {
-      fail("wasQueued from is empty");
-    }
+    assertThat(wasQueuedFrom).isPresent();
     assertThat(wasQueuedFrom.get()).isEqualTo(host);
 
     when(execCfg.isPickOwnHostOnly()).thenReturn(false);