Исправил ошибки в тестах. Убрал задачу - калькулятор.

TimurSokolov · TimurSokolov · commit 7fe924ca38b3 · 2019-11-14T06:59:19.000+04:00
diff --git a/README.md b/README.md
@@ -1,19 +1,4 @@
 # Шаблон для практической работы по теме Java Basics
-
-### Задание №1
 Реализуйте методы в классе SimpleMathService. Следите за правильностью написания кода - метод должен
 делать только то, что от него требуется. Также уделите внимание оптимизации, некоторые методы можно выполнить несколькими
-способами, от быстрого и красивого до нечитаемого и медленного.
-
-### Задание №2
-Реализовать класс Expression, который создается конструктором с единственным аргументом (стартовое значение) и к 
-экземпляру возможно применить следующие методы: `plus`, `minus`, `divide`, `multiply`, `sqrt`, `pow`, `mod`, `factorial`,
-и `calculate`.
-Пример:\
-`new Expression(2)`\
-     `.plus(6)`\
-     `.divide(2)`\
-     `.sqrt()`\
-     `.calculate()` - возвращает ответ (в конкретном случае 2)
-При возникновении исключительных ситуаций (например, деление на 0) должно выводиться сообщение об ошибке.
-В методе Main необходимо написать демо, демострирующее работу данного класса.
+способами, от быстрого и красивого до нечитаемого и медленного.
diff --git a/src/main/java/com/epam/izh/rd/online/calculator/Expression.java b/src/main/java/com/epam/izh/rd/online/calculator/Expression.java
diff --git a/src/main/java/com/epam/izh/rd/online/service/MathService.java b/src/main/java/com/epam/izh/rd/online/service/MathService.java
@@ -2,6 +2,8 @@
 
 public interface MathService {
 
+    int compare(int value1, int value2);
+
     int maxFrom(int value1, int value2);
 
     int maxFrom(int[] values);
@@ -14,9 +16,9 @@ public interface MathService {
 
     long calcFibonacci(int number);
 
-    int[] sort(int[] arr);
+    int[] sort(int[] values);
 
     boolean isPrimary(int number);
 
-    int[] reverseArray(int[] arr);
+    int[] reverseArray(int[] values);
 }
diff --git a/src/main/java/com/epam/izh/rd/online/service/SimpleMathService.java b/src/main/java/com/epam/izh/rd/online/service/SimpleMathService.java
@@ -2,22 +2,36 @@
 
 public class SimpleMathService implements MathService {
 
+    /**
+     * Метод возвращает 0, если value1 = value2.
+     * Метод возвращает -1, если value1 < value2.
+     * Метод возвращает 1, если value1 > value2.
+     *
+     * Например для (-1, -1) метод должен вернуть 0;
+     * Например для (-3, -1) метод должен вернуть -1;
+     * Например для (3, 1) метод должен вернуть 1;
+     */
+    @Override
+    public int compare(int value1, int value2) {
+        return -2;
+    }
+
     /**
      * Метод возвращает максимальное число из пары.
      * Например для списка (-1, 2) метод должен вернуть 2
      */
     @Override
     public int maxFrom(int value1, int value2) {
-        return -1; //TODO
+        return -1;
     }
 
     /**
-     * Метод возвращает максимальное число из массива.
+     * Метод возвращает максимальное число из переданного массива.
      * Например для списка {-1, -3, 4, 8, 5, 22, -5} метод должен вернуть 22
      */
     @Override
     public int maxFrom(int[] values) {
-        return -1; //TODO
+        return -1;
     }
 
     /**
@@ -26,7 +40,7 @@ public int maxFrom(int[] values) {
      */
     @Override
     public int sum(int[] values) {
-        return -1; //TODO
+        return -1;
     }
 
     /**
@@ -35,50 +49,52 @@ public int sum(int[] values) {
      */
     @Override
     public int[] getEvenDigits(int[] values) {
-        return null; //TODO
+        return null;
     }
 
     /**
      * Метод считает факториал из заданного числа.
-     * Например для числа 5 метод должен вернуть 120
+     * Например для числа 5 метод должен вернуть 120.
+     * Факториал 0 должен быть равен 1.
      */
     @Override
     public long calcFactorial(int initialVal) {
-        return -1L; //TODO
+        return -1L;
     }
 
     /**
-     * Метод возвращает число, которе находится на заданной позиции в ряду фибоначчи.
+     * Метод возвращает число, которе находится на заданной позиции (счет начинается с нуля) в ряду фибоначчи.
      *
      * Ряд фибоначчи - ряд, следующие элементы которого состоят из суммы двух предыдущих.
-     * Ряд начинается с 0 и 1.
+     * Ряд начинается 0 и 1.
+     * Пример 0, 1, 1, 2, 3, 5, 8, 13, 21, 34, 55 ...
      *
-     * Например для числа 9 метод должен вернуть 34
+     * Для числа 9 метод должен вернуть 34
+     * Для числа 0 метод должен вернуть 0
      */
     @Override
     public long calcFibonacci(int number) {
-        return -1L; //TODO
+        return -1L;
     }
 
     /**
      * Метод возвращает отсортированный по возрастанию массив.
      * Например для массива {-1, -3, 4, 8, 5, 22, -5} метод должен вернуть {-5, -3, -1, 4, 5, 8, 22}
      */
     @Override
-    public int[] sort(int[] arr) {
-        return null; //TODO
+    public int[] sort(int[] values) {
+        return null;
     }
 
     /**
      * Метод определяет, является ли заданное число простым.
-     *
      * Простое число - число, которое делится только на 1 и на само себя.
      *
      * Например для числа 22 вернется false, а для числа 23 true.
      */
     @Override
     public boolean isPrimary(int number) {
-        return false; //TODO
+        return false;
     }
 
     /**
@@ -87,7 +103,7 @@ public boolean isPrimary(int number) {
      * Например для массива {-1, -3, 4, 8, 5, 22, -5} метод вернет {-5, 22, 5, 8, 4, -3, -1}
      */
     @Override
-    public int[] reverseArray(int[] arr) {
+    public int[] reverseArray(int[] values) {
         return null;
     }
 }
diff --git a/src/test/java/com.epam.izh.rd.online/MathServiceTest.java b/src/test/java/com.epam.izh.rd.online/MathServiceTest.java
@@ -8,7 +8,7 @@
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.MethodSource;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.*;
 
 public class MathServiceTest {
 
@@ -19,6 +19,13 @@ static void setup() {
         mathService = new SimpleMathService();
     }
 
+    @ParameterizedTest
+    @MethodSource("com.epam.izh.rd.online.Providers#testCompare")
+    @DisplayName("Тест метода MathService.compare(int value1, int value2)")
+    void testCompare(int value1, int value2, int expected) {
+        assertEquals(expected, mathService.compare(value1, value2));
+    }
+
     @Test
     @DisplayName("Тест метода MathService.maxFrom(int value1, int value2)")
     void testMaxFrom() {
@@ -42,7 +49,7 @@ void testSum(int[] param, int expected) {
     @MethodSource("com.epam.izh.rd.online.Providers#testGetEvenDigitsProvider")
     @DisplayName("Тест метода MathService.getEvenDigits(int[] values)")
     void testGetEvenDigits(int[] param, int[] expected) {
-        assertEquals(expected, mathService.getEvenDigits(param));
+        assertArrayEquals(expected, mathService.getEvenDigits(param));
     }
 
     @ParameterizedTest
@@ -63,7 +70,7 @@ void testCalcFibonacci(int param, int expected) {
     @MethodSource("com.epam.izh.rd.online.Providers#testSortProvider")
     @DisplayName("Тест метода MathService.sort(int[] arr)")
     void testSort(int[] param, int[] expected) {
-        assertEquals(expected, mathService.sort(param));
+        assertArrayEquals(expected, mathService.sort(param));
     }
 
     @ParameterizedTest
@@ -77,6 +84,6 @@ void testIsPrimary(int param, boolean expected) {
     @MethodSource("com.epam.izh.rd.online.Providers#testReverseArrayProvider")
     @DisplayName("Тест метода MathService.reverseArray(int[] arr)")
     void testIsPrimary(int[] param, int[] expected) {
-        assertEquals(expected, mathService.reverseArray(param));
+        assertArrayEquals(expected, mathService.reverseArray(param));
     }
 }
diff --git a/src/test/java/com.epam.izh.rd.online/Providers.java b/src/test/java/com.epam.izh.rd.online/Providers.java
@@ -12,19 +12,30 @@ private Providers() {
 
     }
 
+    public static Stream<Arguments> testCompare() {
+        return Stream.of(
+                arguments(-10, -1, -1),
+                arguments(0, 0, 0),
+                arguments(3, 3, 0),
+                arguments(33, 2, 1)
+        );
+    }
+
     public static Stream<Arguments> testSumProvider() {
         return Stream.of(
+                arguments(new int[]{}, 0),
                 arguments(new int[]{1, 2}, 3),
                 arguments(new int[]{-6, 5, -2, 78}, 75),
-                arguments(new int[]{5, -7, 3, 7, 3, -1, 4, 3}, 15)
+                arguments(new int[]{5, -7, 3, 7, 3, -1, 4, 3}, 17)
         );
     }
 
     public static Stream<Arguments> testGetEvenDigitsProvider() {
         return Stream.of(
-                arguments(new int[]{2}, new int[]{1, 2}),
-                arguments(new int[]{-6, -2, 78}, new int[]{-6, 5, -2, 78}),
-                arguments(new int[]{}, new int[]{5, -7, 3, 7, 3, -1, 3})
+                arguments(new int[]{}, new int[]{}),
+                arguments(new int[]{1, 2}, new int[]{2}),
+                arguments(new int[]{-6, 5, -2, 78}, new int[]{-6, -2, 78}),
+                arguments(new int[]{5, -7, 3, 7, 3, -1, 3}, new int[]{})
         );
     }
 
@@ -39,6 +50,7 @@ public static Stream<Arguments> testCalcFactorialProvider() {
 
     public static Stream<Arguments> testCalcFibonacciProvider() {
         return Stream.of(
+                arguments(0, 0),
                 arguments(1, 1),
                 arguments(2, 1),
                 arguments(3, 2),
@@ -51,24 +63,27 @@ public static Stream<Arguments> testCalcFibonacciProvider() {
 
     public static Stream<Arguments> testSortProvider() {
         return Stream.of(
+                arguments(new int[]{}, new int[]{}),
                 arguments(new int[]{-1, -3, 4, 8, 5, 22, -5}, new int[]{-5, -3, -1, 4, 5, 8, 22}),
                 arguments(new int[]{3, 4, 3, 4}, new int[]{3, 3, 4, 4})
         );
     }
 
     public static Stream<Arguments> testIsPrimaryProvider() {
         return Stream.of(
-                arguments(1, true),
+                arguments(2, true),
                 arguments(4, false),
                 arguments(5, true),
                 arguments(10, false),
                 arguments(21, false),
-                arguments(23, true)
+                arguments(23, true),
+                arguments(7349, true)
         );
     }
 
     public static Stream<Arguments> testReverseArrayProvider() {
         return Stream.of(
+                arguments(new int[]{}, new int[]{}),
                 arguments(new int[]{-1, -3, 4, 8, 5, 22, -5}, new int[]{-5, 22, 5, 8, 4, -3, -1}),
                 arguments(new int[]{4, 4, 5, 5, 2, 6}, new int[]{6, 2, 5, 5, 4, 4})
         );
