GoTimeUtilTest : Use assertThat(actual).contains(expected) instead of…

… fetching and comparing optional#1378

jkube-kit/enricher/api/src/test/java/org/eclipse/jkube/kit/enricher/api/util/GoTimeUtilTest.java

@@ -13,15 +13,12 @@
  */
 package org.eclipse.jkube.kit.enricher.api.util;
 
-import org.assertj.core.api.Assertions;
 import org.junit.Test;
 
 import java.util.Arrays;
 import java.util.List;
-import java.util.Optional;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
 
 public class GoTimeUtilTest {
 
@@ -33,28 +30,24 @@ public void testConversion() {
         List<Integer> expectations = Arrays.asList(23, 0, 0, 0, 1, 123, 3663, 1810, -15, 30);
 
         for (int i = 0; i < inputs.size(); i++) {
-            // When
-            Optional<Integer> result = GoTimeUtil.durationSeconds(inputs.get(i));
-
-            // Then
-            assertTrue(result.isPresent());
-            Assertions.assertThat(result.get()).isEqualTo(expectations.get(i).intValue());
+            assertThat(GoTimeUtil.durationSeconds(inputs.get(i)))
+                    .contains(expectations.get(i));
         }
     }
 
     @Test
     public void testNull() {
-        assertEquals(Optional.empty(), GoTimeUtil.durationSeconds(null));
+        assertThat(GoTimeUtil.durationSeconds(null)).isEmpty();
     }
 
     @Test
     public void testEmpty() {
-        assertEquals(Optional.empty(), GoTimeUtil.durationSeconds(""));
+        assertThat(GoTimeUtil.durationSeconds("")).isEmpty();
     }
 
     @Test
     public void testBlankSpace() {
-        assertEquals(Optional.empty(), GoTimeUtil.durationSeconds(" "));
+        assertThat(GoTimeUtil.durationSeconds(" ")).isEmpty();
     }
 
     @Test(expected = IllegalArgumentException.class)
@@ -76,7 +69,4 @@ public void testErrorUnknownUnit() {
     public void testErrorUnparsable() {
         GoTimeUtil.durationSeconds("ms");
     }
-
-
-
 }