Что такое JUnit тесты и с чем они проводятся?

Question

[Виталий Фролов]

всем привет, в первый раз сюда пишу, поэтому особо не пинайте =)
в общем проблема такая, не понимаю как проводятся JUnit тесты с алгоритмами сортировки (и проводятся ли они вообще с сортировкой?)
В интернете, есть банальный пример на юнит-тест метода сложения, да с этим все ясно.
На основе этого примера сложения сделал юнит-тест сортировки в котором сначала вносил элементы в массив, затем делал сортировку
и

@Test 
public void testInsert() { 
ArrayIns arrayIns = new ArrayIns(10);
arrayIns.insert(5); 
arrayIns.insert(7); 
assertEquals(5,arrayIns.getFirst()); // сравнение ожидаемого результата с возвращаемым значением

на что мне на собеседовании сказали, что это хрень и не правильно

Так что вопрос такой =) как делать JUnit тесты и с чем их проводить,
поделитесь если кто знает ссылкой на сие великое для меня знание,
а то гуглю и натыкаюсь на один и тот же юнит-тест с сложением
P.S. Всем добра
мой гит: https://github.com/leVenTel8/SortirovkaVstavkoi/bl...

Comments

[MrShaggy]

Рекомендую почитать "Чистый код" Роберта Мартина. Там и про юнит тесты и про то, как не хорошо писать комменты после закрвающей скобки :)

Answer 1

[Сергей]

Просто поучительный пример.
Вам нужно протестировать класс. Что это значит? Протестировать его поведение, хотя бы основное. Т.е. желательно все методы которые могут использовать. Как у черного ящика. (скрытые не тестируем и можно не тестировать getter&setter).
Как это сделать?
Вы добавили один метод getFurst(). Вероятно, для тестирования. Но первым элементов не должна ограничиваться проверка. Поэтому нужен скорее метод getItem(int); Это приводит к изменению интерфейса вашего класса (замена не нужного метода нужным)
Сделали изменения, протестировали. Ок.
Далее пишете тест на insert. Несколько. И в конце у вас выскакивает исключение. Индекс превышает размер массива. Опа. Вы должны либо явно описать этот случай в документации (генерацию исключения), либо что-то еще сделать. Указать пользователю на этот скользкий случай.

Далее. Есть метод display
Как его протестировать автоматически? Никак. Это наводит на мысль "что-то не так". Не так, скорее потому, что этот метод не должен быть в этом классе. класс занимается сортировкой. Выводом на экран должен заниматься другой класс. Если вдруг понадобиться поменять вывод, то вам придется ковырять класс который уже работает и протестирован. И вы будете лапать снова весь код, с риском дернуть что-то что скрыто поменяет корректную работу.
Дисплей в другой класс.
итак. убрали getFirst. display. добавили getItem.
класс хорошо тестируется. и более удобный интерфейс.
Т.е. вывод из всего: Тестируемость класса зависит от хорошего интерфейса. Требует этого.
Пригодный к тестированию класс практически всегда (чаще всего) это класс который будет легко использовать. Класс который трудно протестировать, чаще всего трудно будет использовать.
Примерно так.
Конкретно по теме.
Просто по коду тестов, вопросов нет. Вопрос про тестирование класса.

Comments

[Виталий Фролов]

классный развернутый ответ(респект таким как Вы), посоветуйте что нибудь к прочтению, чтобы понимать как это все реализовывать, просто опыта разработки не ма, да и обратиться за советом не к кому

[Виталий Фролов]

все нашел способ вот часть кода, спасибо автору выше за развернутый ответ и наводящие вопросы

public long getItem(int i){
long item = a[i];
return item;

}

и сам тест сделал в блоке try catch, потому что кидало экзепшн на переполнение массива

@Test
public void testInsertionSort() throws Exception{
int maxSize = 10; // максимальный размер массива
ArrayIns arr;
arr = new ArrayIns(maxSize); // создание нового массива размера maxSize
for(int j=0; j

[Сергей]

Виталий Фролов: почитать. посмотрите Мартин Р.С., Мартин М. - Принципы, паттерны и методики гибкой разработки на языке C# - 2011.
или что-нибудь по TDD

[Сергей]

Виталий Фролов: в 36 строке комментарий вводит в заблуждение // вставка рандомныхэлемнтов в массив
Не нужно много комментариев. Лучшая документация это код. Комментарии нужны только в непонятных местах. Или в описании функций и классов.

[Сергей]

в 35 строке
в тесте должны быть либо однозначное исключение тогда пишем так
@Test(expected = ...
либо без исключений
все должно быть под контролем

[Виталий Фролов]

Сергей: спасибо что помогаете, сейчас изучаю Кента Бека TDD, как прочту на пишу =)
потому что полюбому появятся вопросы

Answer 2

[MrShaggy]

Вы хотите проверить правильность работы сортировки только через первое число в массиве? А если результатом сортировки будет "1 10 5 7 2 5", "1 5 7 10 2" или даже "1 2 5 10 7"? Это все разве верно?
Написание юнит теста у вас хорошее. Проверка условия отсортированности не правильная.

Comments

[Виталий Фролов]

как тогда правильно написать? я просто нубас полный и не знаю =)
буду признателен, если скажите как сделать.
Я понял про что Вы говорите, но как реализовать я ума неприложу

[Виталий Фролов]

ниже ссылка на исходник, как мне реализовать проверку алгоритма?

https://github.com/leVenTel8/SortirovkaVstavkoi/bl...

[MrShaggy]

Виталий Фролов: Как-то так:
В цикле пробегаетсь по массиву и проверяете условие assertEquals(true, arrayIns[i] <= arrayIns[i+1])
Я так понимаю у вас массив кастомный. Поэтому вместо [i] и [i+1] геттеры.

[Виталий Фролов]

во, хорошая мысль
слушайте, а тут можно в друзья добавлять или как? (просто первый день на тостере)Я смотрю Вы разбираетесь, если не трудо, можно приставать к Вам с вопросами, если я зависну)?

[MrShaggy]

Виталий Фролов: Можете добавить в GitHub в Collaborators на своем репозитории. Чекните issues у вас в репозитории. Я там подробно описал.

Answer 3

[Alexej Simakov]

по коду вообще не видно где проводится сортировка. Видно что проверяется что первый вставленный в массив элемент имеет то же значение что и первый элемент массива. Это совсем не то что мы хотим проверить.
Правильно например было бы проверять пары элементов на условие Xn <= Xn+1. Но опять же сперва нужно тест настроить т.е. создать неотсортированый массив, применить к нему сортировку и проверить результат.

Comments

[MrShaggy]

В репозитории есть доп. код

[Alexej Simakov]

Виталий Фролов : нужно было конечно testInsertionSort() привести в топике в качестве примера. В любом случае, тест должен быть универсальным, т.е. работать независимо от введенных данных. В вашем примере проверка работает только если единица первое число в отсортированном массиве.и если вдруг это окажется не так, то тест не пройдет. Однако может оказаться, что последовательность отсортированных чисел начинается с двойки. И у вас будет false negative. Т.е тест даст негативный результат и причина будет не в неверной сортировке а в неверной настройке проверки.

[Виталий Фролов]

как тогда реализовать проверку?

[Виталий Фролов]

Alexej Simakov: ниже ссылка на исходник, как мне реализовать проверку алгоритма?

https://github.com/leVenTel8/SortirovkaVstavkoi/bl...

[Alexej Simakov]

Виталий Фролов:
как-то так:

for(int i=0;i<arr.length-1;i++) assertTrue(arr[i] < arr[i+1])

[Виталий Фролов]

Alexej Simakov: спасибо огроменное =)

[Виталий Фролов]

Alexej Simakov: Idea ругается на Ваш код, не рабоает

[Alexej Simakov]

Виталий Фролов: кончено он не работает, это набросок самой логики. Синтаксис сами поправите.

Answer 4

[Виталий Фролов]

ниже ссылка на исходник, как мне реализовать JUnit тест к нему, если разбираетесь не проходите мимо плиз =)
помогите нубу разобраться, я понимаю про что говорят ребята сверху, что я проверяю только первое значение, а надо проверять весь массив, но как это реализовать в юнит тесте не знаю =)
P.S. надеюсь найдется гуру, который даст ответ или тыкнет куда посмотреть или чего почитать

https://github.com/leVenTel8/SortirovkaVstavkoi/bl...