Conversation
|
|
||
| @Test | ||
| public void testLengthOfSequenceIsEqualToItemsCount() throws Exception{ | ||
| final List<Integer> Result = ClassToBeTested.generateIntSequence(0,5); |
There was a problem hiding this comment.
Название локальных переменных начинается со строчной буквы (+camelCase, если слов в названии несколько).
С прописной принято называть классы (+camelCase, если слов в названии несколько).
Константы (static + final) принято называть прописными буквами с "_" в качестве разделителя между словами (напр. STATIC_VALUE)
| Assert.assertThat( | ||
| "The length of sequence isn't equal to 3", | ||
| Result, | ||
| hasItems(-2,-1,0,1) |
There was a problem hiding this comment.
hasItems игнорирует порядок следования элементов. Также, если в твоём Result будут какие-то лишние элементы, проверка пройдёт успешно, хотя на самом деле это был бы баг.
Надо использовать что-то другое, что проверяет списки на точное соответствие. Например, equalTo
| Assert.assertEquals(3, Result.size()); | ||
| Assert.assertEquals(4, (long)Result.get(0)); | ||
| Assert.assertEquals(5, (long)Result.get(1)); | ||
| Assert.assertEquals(6, (long)Result.get(2)); |
There was a problem hiding this comment.
Так как метод возвращает список Integer, то (long) Result.get(0) лучше заменить на Result.get(0).intValue()
И надо не забывать ставить пробелы -_-
|
|
||
| Assert.assertNotNull(result); | ||
| Assert.assertEquals(3, result.size()); | ||
| Assert.assertEquals(4, result.get(0).intValue()); |
There was a problem hiding this comment.
Очень маленькое замечание - при нескольких проверках в одном тесте лучше писать fail-message для каждой проверки. Так проще потом разгребать результаты падения.
2 participants
Здравствуйте, вот первая домашка. Надеюсь не очень критично, если не совсем до мая сдана)