Skip to content

Commit fe5afad

Browse files
committed
refactor: clean up CheckTest by removing commented-out tests and unused account instance
1 parent c3f6c6b commit fe5afad

File tree

1 file changed

+1
-35
lines changed
  • lesson_17/bank/bank_app/src/test/java/com/codedifferently/lesson17/bank

1 file changed

+1
-35
lines changed

lesson_17/bank/bank_app/src/test/java/com/codedifferently/lesson17/bank/CheckTest.java

Lines changed: 1 addition & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -9,37 +9,15 @@
99
class CheckTest {
1010

1111
private CheckingAccount account1;
12-
private CheckingAccount account2;
1312
private Check classUnderTest;
1413

1514
@BeforeEach
1615
void setUp() {
1716
account1 = new CheckingAccount("123456789", null, 100.0);
18-
account2 = new CheckingAccount("987654321", null, 200.0);
17+
1918
classUnderTest = new Check("123456789", 50.0, account1);
2019
}
2120

22-
// @Test
23-
// void testDepositFunds() {
24-
// // Act
25-
// classUnderTest.depositFunds(account2);
26-
27-
// // Assert
28-
// assertThat(account1.getBalance()).isEqualTo(50.0);
29-
// assertThat(account2.getBalance()).isEqualTo(250.0);
30-
// }
31-
32-
// @Test
33-
// void testDepositFunds_CheckVoided() {
34-
// // Arrange
35-
// classUnderTest.voidCheck();
36-
37-
// // Act & Assert
38-
// assertThatExceptionOfType(CheckVoidedException.class)
39-
// .isThrownBy(() -> classUnderTest.depositFunds(account2))
40-
// .withMessage("Check is voided");
41-
// }
42-
4321
@Test
4422
void testConstructor_CantCreateCheckWithNegativeAmount() {
4523
// Act & Assert
@@ -74,16 +52,4 @@ void testToString() {
7452
assertThat(classUnderTest.toString())
7553
.isEqualTo("Check{checkNumber='123456789', amount=50.0, account=123456789}");
7654
}
77-
78-
/* @Test
79-
void testDepositFunds_DoesntDepositCheckTwice() {
80-
Check check = new Check("987654321", 100.0, account1);
81-
82-
classUnderTest.depositFunds(account2);
83-
84-
85-
assertThatExceptionOfType(CheckVoidedException.class)
86-
.isThrownBy(() -> classUnderTest.depositFunds(account2))
87-
.withMessage("Check is voided");
88-
}*/
8955
}

0 commit comments

Comments
 (0)