Conversation
| private void checkConditions(String ownerName, String accountNumber, double balance, boolean isBlocked) { | ||
| if (ownerName == null || ownerName.isBlank()) { | ||
| throw new RuntimeException("Имя владельца не может быть пустым"); | ||
| throw new NullPointerException("Имя владельца не может быть пустым"); |
There was a problem hiding this comment.
npe неприятная штука, давай везде illegallargument кидать
|
|
||
| @DisplayName("Тесты конструктора BankAccount") | ||
| public class Main { | ||
| public static void main(String[] args) { |
There was a problem hiding this comment.
main должен отстаься без изменений, посмотри где пишут тесты
| } | ||
| } No newline at end of file | ||
|
|
||
| @Test |
There was a problem hiding this comment.
Все тесты нужны перенести в нужно место
|
|
||
| BankAccount bankAccount = new BankAccount(ownerName, accountNumber, balance, isBlocked); | ||
|
|
||
| assertEquals("Алексеев", bankAccount.getOwnerName()); |
There was a problem hiding this comment.
у тебя уже есть перменная которая хранит в себе "Алексеев" ты можншь испоьзовать ее, как и везде где так, тоже исправь
|
|
||
| BankAccount bankAccount = new BankAccount(ownerName, accountNumber, balance, isBlocked); | ||
|
|
||
| assertEquals(0, bankAccount.getBalance(), 0); |
| @Test | ||
| @DisplayName("Создаёт объект с пустой строкой номера счёта") | ||
| public void createBankAccountWitchAccountNumberIsBlank() { | ||
| assertThrows(NullPointerException.class, |
There was a problem hiding this comment.
А почему тут не проверили что за сообщение будет?
|
|
||
| @Test | ||
| @DisplayName("Создаёт объект с пустой строкой номера счёта и проверяет текст ошибки") | ||
| public void createBankAccountWitchAccountNumberIsBlankAndExceptionMessage() { |
There was a problem hiding this comment.
Не понял чем отличается от прошлого теста?
|
|
||
| @Test | ||
| @DisplayName("Имя владельца сохраняется точно как передано") | ||
| public void shouldPreserveNameExactlyAsPassed() { |
There was a problem hiding this comment.
Для чего этот тест? Не хватает проверки просто успешного кейса, что имя равняется тому что мы задали
| 12323.00, | ||
| false); | ||
|
|
||
| assertNotNull(bankAccount); |
There was a problem hiding this comment.
Тоже не понял зачем этот тест, когда мы создаем объект в успешном тесте, мы потом дергает поля из объекта, и если бы объект был null, то мы получили бы там npe и тест упал
|
|
||
| assertEquals(" Капутов ", bankAccount.getOwnerName()); | ||
| } | ||
| } |
There was a problem hiding this comment.
А где тесты на другие конструкторы?
2 participants
No description provided.