diff --git a/CommBank-Server/Models/Goal.cs b/CommBank-Server/Models/Goal.cs
index 77ff1ad5..81f01923 100644
--- a/CommBank-Server/Models/Goal.cs
+++ b/CommBank-Server/Models/Goal.cs
@@ -11,6 +11,8 @@ public class Goal
 
     public string? Name { get; set; }
 
+    public string? Icon { get; set; }
+
     public UInt64 TargetAmount { get; set; } = 0;
 
     public DateTime TargetDate { get; set; }
diff --git a/CommBank.Tests/GoalControllerTests.cs b/CommBank.Tests/GoalControllerTests.cs
index 8380181f..431543df 100644
--- a/CommBank.Tests/GoalControllerTests.cs
+++ b/CommBank.Tests/GoalControllerTests.cs
@@ -66,9 +66,34 @@ public async void Get()
     public async void 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 httpContext =
+            new Microsoft.AspNetCore.Http.DefaultHttpContext();
+
+        controller.ControllerContext.HttpContext = httpContext;
+
+        var result =
+            await controller.GetForUser(users[0].Id!);
+
         // Assert
+        Assert.NotNull(result);
+
+        foreach (Goal goal in result!)
+        {
+            Assert.IsAssignableFrom<Goal>(goal);
+            Assert.Equal(users[0].Id, goal.UserId);
+        }
     }
 }
\ No newline at end of file