Add goal icons and GetGoalsForUser test

.gitignore

@@ -405,3 +405,5 @@ ASALocalRun/
 .localhistory/
 
 
+# Local secrets
+Secrets.json

CommBank-Server/Models/Goal.cs

@@ -27,4 +27,6 @@ public class Goal
 
     [BsonRepresentation(BsonType.ObjectId)]
     public string? UserId { get; set; }
+
+    public string? Icon { get; set; }
 }

CommBank.Tests/GoalControllerTests.cs

@@ -2,7 +2,6 @@
 using CommBank.Services;
 using CommBank.Models;
 using CommBank.Tests.Fake;
-using Microsoft.AspNetCore.Mvc;
 
 namespace CommBank.Tests;
 
@@ -16,44 +15,52 @@ public GoalControllerTests()
     }
 
     [Fact]
-    public async void GetAll()
+    public async Task GetAll()
     {
         // Arrange
         var goals = collections.GetGoals();
         var users = collections.GetUsers();
+
         IGoalsService goalsService = new FakeGoalsService(goals, goals[0]);
         IUsersService usersService = new FakeUsersService(users, users[0]);
+
         GoalController controller = new(goalsService, usersService);
 
         // Act
         var httpContext = new Microsoft.AspNetCore.Http.DefaultHttpContext();
         controller.ControllerContext.HttpContext = httpContext;
+
         var result = await controller.Get();
 
         // Assert
         var index = 0;
+
         foreach (Goal goal in result)
         {
             Assert.IsAssignableFrom<Goal>(goal);
             Assert.Equal(goals[index].Id, goal.Id);
             Assert.Equal(goals[index].Name, goal.Name);
+
             index++;
         }
     }
 
     [Fact]
-    public async void Get()
+    public async Task Get()
     {
         // Arrange
         var goals = collections.GetGoals();
         var users = collections.GetUsers();
+
         IGoalsService goalsService = new FakeGoalsService(goals, goals[0]);
         IUsersService usersService = new FakeUsersService(users, users[0]);
+
         GoalController controller = new(goalsService, usersService);
 
         // Act
         var httpContext = new Microsoft.AspNetCore.Http.DefaultHttpContext();
         controller.ControllerContext.HttpContext = httpContext;
+
         var result = await controller.Get(goals[0].Id!);
 
         // Assert
@@ -63,12 +70,31 @@ public async void Get()
     }
 
     [Fact]
-    public async void GetForUser()
+    public async Task GetForUser()
     {
         // Arrange
-
+        var goals = collections.GetGoals();
+        var users = collections.GetUsers();
+
+        IGoalsService goalsService = new FakeGoalsService(goals, goals[0]);
+        IUsersService usersService = new FakeUsersService(users, users[0]);
+
+        GoalController controller = new(goalsService, usersService);
+
         // Act
-
+        var result = await controller.GetForUser(users[0].Id!);
+
         // Assert
+        Assert.NotNull(result);
+        Assert.Equal(goals.Count, result.Count);
+
+        for (var index = 0; index < goals.Count; index++)
+        {
+            Assert.Equal(goals[index].Id, result[index].Id);
+            Assert.Equal(goals[index].Name, result[index].Name);
+            Assert.Equal(goals[index].TargetAmount, result[index].TargetAmount);
+            Assert.Equal(goals[index].Balance, result[index].Balance);
+            Assert.Equal(goals[index].UserId, result[index].UserId);
+        }
     }
 }