Skip to content

Commit 699aae5

Browse files
committed
added a test case and made some modifications
2 parents 0d63b1e + 4ac16ea commit 699aae5

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

src/main/java/UserModificationUI.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ public void actionPerformed(ActionEvent e) {
6464

6565
// ChangeController makes UI implement reportChange to invert the use-case --> UI dependency
6666
@Override
67-
// TODO: fix based on new UserDatabase/ getUser features
6867
public boolean reportChange(String username, String password, String feature, String newFeature) {
6968
User user = db.getUser(username);
7069
if (user.getPassword().equals(password) && user.getUsername().equals(username)){

src/test/java/TestUserChange.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
import org.junit.jupiter.api.Assertions;
2+
3+
import java.io.File;
4+
5+
public class TestUserChange {
6+
public void correctChange() {
7+
File accounts = new File("TestUserDatabase2.csv");
8+
UserDatabase accountDatabase = new UserDatabase(accounts);
9+
accountDatabase.createUser("parmism", "123", "[email protected]", "Basic");
10+
UserModificationUI ui = new UserModificationUI();
11+
ui.reportChange("parmism", "123", "Password", "456");
12+
String actual = accountDatabase.getUser("parmism").getPassword();
13+
Assertions.assertEquals("456", actual);
14+
}
15+
}

0 commit comments

Comments
 (0)