Feature: Add Goal Icons and Tests

CommBank-Server/CommBank.csproj

@@ -13,7 +13,7 @@
     <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
     <PackageReference Include="BCrypt.Net-Next" Version="4.0.3" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="6.0.1" />
-    <PackageReference Include="MongoDB.Driver" Version="2.18.0" />
+    <PackageReference Include="MongoDB.Driver" Version="3.7.1" />
   </ItemGroup>
 
   <ItemGroup>

CommBank-Server/Models/Goal.cs

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

CommBank-Server/Program.cs

@@ -4,38 +4,41 @@
 
 var builder = WebApplication.CreateBuilder(args);
 
+// Add services
 builder.Services.AddControllers();
-
 builder.Services.AddEndpointsApiExplorer();
 builder.Services.AddSwaggerGen();
 
-builder.Configuration.SetBasePath(Directory.GetCurrentDirectory()).AddJsonFile("Secrets.json");
+// ✅ Load appsettings.json (IMPORTANT FIX)
+builder.Configuration
+    .SetBasePath(Directory.GetCurrentDirectory())
+    .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true);
 
-var mongoClient = new MongoClient(builder.Configuration.GetConnectionString("CommBank"));
-var mongoDatabase = mongoClient.GetDatabase("CommBank");
+// ✅ Get connection string properly
+var connectionString = builder.Configuration.GetConnectionString("CommBank");
 
-IAccountsService accountsService = new AccountsService(mongoDatabase);
-IAuthService authService = new AuthService(mongoDatabase);
-IGoalsService goalsService = new GoalsService(mongoDatabase);
-ITagsService tagsService = new TagsService(mongoDatabase);
-ITransactionsService transactionsService = new TransactionsService(mongoDatabase);
-IUsersService usersService = new UsersService(mongoDatabase);
+// ✅ Create MongoDB client
+var mongoClient = new MongoClient(connectionString);
+var mongoDatabase = mongoClient.GetDatabase("CommBank");
 
-builder.Services.AddSingleton(accountsService);
-builder.Services.AddSingleton(authService);
-builder.Services.AddSingleton(goalsService);
-builder.Services.AddSingleton(tagsService);
-builder.Services.AddSingleton(transactionsService);
-builder.Services.AddSingleton(usersService);
+// Register services
+builder.Services.AddSingleton<IAccountsService>(new AccountsService(mongoDatabase));
+builder.Services.AddSingleton<IAuthService>(new AuthService(mongoDatabase));
+builder.Services.AddSingleton<IGoalsService>(new GoalsService(mongoDatabase));
+builder.Services.AddSingleton<ITagsService>(new TagsService(mongoDatabase));
+builder.Services.AddSingleton<ITransactionsService>(new TransactionsService(mongoDatabase));
+builder.Services.AddSingleton<IUsersService>(new UsersService(mongoDatabase));
 
+// Enable CORS
 builder.Services.AddCors();
 
 var app = builder.Build();
 
-app.UseCors(builder => builder
-   .AllowAnyOrigin()
-   .AllowAnyMethod()
-   .AllowAnyHeader());
+// Configure middleware
+app.UseCors(policy =>
+    policy.AllowAnyOrigin()
+          .AllowAnyMethod()
+          .AllowAnyHeader());
 
 if (app.Environment.IsDevelopment())
 {
@@ -44,10 +47,8 @@
 }
 
 app.UseHttpsRedirection();
-
 app.UseAuthorization();
 
 app.MapControllers();
 
-app.Run();
-
+app.Run();

CommBank-Server/appsettings.json

@@ -1,10 +1,12 @@
 {
+  "ConnectionStrings": {
+    "CommBank": "mongodb+srv://joshna:joshna2006@cluster0.mqiyhab.mongodb.net/?retryWrites=true&w=majority"
+  },
   "Logging": {
     "LogLevel": {
       "Default": "Information",
       "Microsoft.AspNetCore": "Warning"
     }
   },
   "AllowedHosts": "*"
-}
-
+}

CommBank.Tests/GoalControllerTests.cs

@@ -1,4 +1,6 @@
-using CommBank.Controllers;
+using System.Collections.Generic;
+using Xunit;
+using CommBank.Controllers;
 using CommBank.Services;
 using CommBank.Models;
 using CommBank.Tests.Fake;
@@ -66,9 +68,28 @@ 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);
+
+        var userId = users[0].Id;
+
         // Act
-
+        var httpContext = new Microsoft.AspNetCore.Http.DefaultHttpContext();
+        controller.ControllerContext.HttpContext = httpContext;
+
+        var actionResult = await controller.Get(userId!);
+        var okResult = Assert.IsType<OkObjectResult>(actionResult.Result);
+        var result = Assert.IsAssignableFrom<IEnumerable<Goal>>(okResult.Value);
+
         // Assert
+        foreach (var goal in result)
+        {
+            Assert.Equal(userId, goal.UserId);
+        }
     }
 }