Changed implementation of UserDatabase.UserExists and UserDatabase.getList. There still seems to be an issue with one of the tests

Author: Madhavan7

src/test/java/TestUserDatabase2.csv renamed to src/main/java/TestUserDatabase2.csv

@@ -3,12 +3,14 @@
 import java.util.List;
 public class UserDatabase implements UserExists, UserRetriever, UserCreator, IRetrieveList{
     File accounts;
+    List<User> accountList;
     public UserDatabase(File accounts){
         this.accounts = accounts;
+        this.accountList = this.getList();
     }
     @Override
     public boolean UserExists(String username, String email) {
-        User user = null;
+        /*User user = null;
         try(FileInputStream fileIn = new FileInputStream(accounts);
         ObjectInputStream in = new ObjectInputStream(fileIn)){
             do{
@@ -24,7 +26,13 @@ public boolean UserExists(String username, String email) {
         } catch (ClassNotFoundException e) {
             throw new RuntimeException(e);
         }
-        return user != null;
+        return user != null;*/
+        for(User user: this.accountList){
+            if(user.getUsername().equals(username) || user.getEmail().equals(email)){
+                return true;
+            }
+        }
+        return false;
     }
 
     // Creates a new user with a username and password, and an email address
@@ -36,6 +44,7 @@ public void createUser(String username, String password, String email, String ty
         try(FileOutputStream fileOut = new FileOutputStream(accounts)){
             try(ObjectOutputStream out = new ObjectOutputStream(fileOut)){
                 out.writeObject(newUser);
+                this.accountList.add(newUser);
                 out.close();
                 fileOut.close();
             }catch(Exception e){
@@ -64,10 +73,14 @@ public List<User> getList() {
         List<User> users = new ArrayList<>();
         try(FileInputStream fileIn = new FileInputStream(accounts);
             ObjectInputStream in = new ObjectInputStream(fileIn)){
-            User user = (User) in.readObject();
-            while(user != null){
-                users.add(user);
-                user = (User)in.readObject();
+            /*User user = (User) in.readObject();*/
+            while(true){
+                try{
+                    User user = (User) in.readObject();
+                    users.add(user);}
+                catch(EOFException e){
+                    break;
+                }
             }
         }catch(Exception e){
             e.printStackTrace();

src/main/java/UserDatabase.java

@@ -2,7 +2,6 @@
 import org.junit.jupiter.api.Test;
 
 import java.io.File;
-import java.util.ArrayList;
 import java.util.List;
 
 public class UserDatabaseTest {
@@ -24,6 +23,7 @@ public void addingMultipleFiles(){
     public void rightEmailWrongUser(){
         File accounts = new File("TestUserDatabase2.csv");
         UserDatabase accountDatabase = new UserDatabase(accounts);
+        accountDatabase.createUser("MadhavGopakumar", "123", "[email protected]", "Basic");
         Assertions.assertTrue(accountDatabase.UserExists("MadG", "[email protected]"));
     }
     @Test