Skip to content

Тестирование метода generateIntSequence#10

Open
XeniaIva wants to merge 1 commit into48x:masterfrom
XeniaIva:myhomework
Open

Тестирование метода generateIntSequence#10
XeniaIva wants to merge 1 commit into48x:masterfrom
XeniaIva:myhomework

Conversation

@XeniaIva
Copy link

Чек-лист проверок метода ClassToBeTested.generateIntSequence:
1)Проверка корректной работы метода при передаче двух аргументов, сумма которых меньше максимально допустимого значения типа Int
2)Проверка корректной работы метода при передаче двух аргументов, сумма которых равна максимально допустимому значению типа int
3)Проверка корректной работы программы при передаче отрицательного значения itemsCoun
4)Проверка корректной работы программы при передаче itemsCount=0
5)Проверка корректной работы программы при передаче двух аргументов, сумма которых больше максимально допустимого значения типа int

Copy link
Owner

@48x 48x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По кейсам - ОК.
Не помешали бы сообщения о падении в проверках, особенно когда их несколько в одном тесте

* сумма которых меньше максимально допустимого значения типа Int
*/
@Test
public void CorrectGenerationWhenSumIsLessThanIntegerMaxValue() {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Тестовые методы принято называть с test
  2. Названия методов начинаются на маленькую букву

public void WhenitemsCountIsLessThanZero() {
try {
ClassToBeTested.generateIntSequence(0, -3);
Assert.fail();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше fail-message всё же указывать

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants