Implement all the things

Author: maxdavidson

src/main/java/se/cygni/palmithor/tdd/Calculator.java

@@ -10,11 +10,8 @@ public class Calculator {
     public Optional<Integer> sumAll(final Integer... valuesParam) {
         if (valuesParam == null) {
             return Optional.empty();
-        } else if (valuesParam.length < 1) {
-            return Optional.of(0);
-        } else {
-            return Optional.of(Arrays.stream(valuesParam)
-                    .reduce(0, (a, b) -> a + b));
         }
+
+        return Optional.of(Arrays.stream(valuesParam).reduce(0, Integer::sum));
     }
 }

src/main/java/se/cygni/palmithor/tdd/StringUtils.java

@@ -4,7 +4,7 @@ public class StringUtils {
 
 
     public boolean isPalindrome(final String str) {
-        throw new RuntimeException("Not yet implemented");
+        return str != null && str.equals(new StringBuilder(str).reverse().toString());
     }
 
 
@@ -15,6 +15,6 @@ public boolean isPalindrome(final String str) {
      * @return true if str is null, empty or whitespace only, otherwise false
      */
     public boolean isBlank(final String str) {
-        throw new RuntimeException("Not yet implemented");
+        return str == null || str.trim().isEmpty();
     }
 }

src/test/java/se/cygni/palmithor/tdd/CalculatorTest.java

@@ -13,7 +13,8 @@ public class CalculatorTest {
 
     @Test
     public void test() {
-        assertThat(calculator.sumAll().get()).isEqualTo(1); // TODO failing on purpose please fix
+        assertThat(calculator.sumAll()).isEqualTo(Optional.of(0));
+        assertThat(calculator.sumAll(null)).isEqualTo(Optional.empty());
     }
 
 

src/test/java/se/cygni/palmithor/tdd/StringUtilsTest.java

@@ -24,6 +24,11 @@ public void isPalindrome() {
      */
     @Test
     public void isBlank() {
-        // stringUtils.isBlank()
+
+        assertThat(stringUtils.isBlank("")).isTrue();
+        assertThat(stringUtils.isBlank("   ")).isTrue();
+        assertThat(stringUtils.isBlank(null)).isTrue();
+
+        assertThat(stringUtils.isBlank("hej")).isFalse();
     }
 }