fix(back): ServiceProvider not found during tests

oxymore-tech · Aug 10, 2023 · 3f903c0 · 3f903c0
1 parent 0f42918
commit 3f903c0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/back/src/Liane/Liane.Service/Internal/Trip/Event/LianeMemberPingHandler.cs b/back/src/Liane/Liane.Service/Internal/Trip/Event/LianeMemberPingHandler.cs
@@ -20,14 +20,16 @@ public sealed class LianeMemberPingHandler : IEventListener<LianeEvent.MemberPin
   private readonly ConcurrentDictionary<string, Task<LianeTracker>> trackers = new();
   private readonly ILianeService lianeService;
   private readonly ICurrentContext currentContext;
-  private readonly ServiceProvider serviceProvider;
-  public LianeMemberPingHandler(IMongoDatabase db, ILianeMemberTracker lianeMemberTracker, ILianeService lianeService, ICurrentContext currentContext, IOsrmService osrmService, IPostgisService postgisService, ServiceProvider serviceProvider)
+  private readonly IOsrmService osrmService;
+  private readonly IPostgisService postgisService;
+  public LianeMemberPingHandler(IMongoDatabase db, ILianeMemberTracker lianeMemberTracker, ILianeService lianeService, ICurrentContext currentContext, IOsrmService osrmService, IPostgisService postgisService)
   {
     mongo = db;
     this.lianeMemberTracker = lianeMemberTracker;
     this.lianeService = lianeService;
     this.currentContext = currentContext;
-    this.serviceProvider = serviceProvider;
+    this.osrmService = osrmService;
+    this.postgisService = postgisService;
   }
 
   private async Task EndTrip(Ref<Api.Trip.Liane> liane)
@@ -77,7 +79,7 @@ public async Task OnEvent(LianeEvent.MemberPing e, Ref<Api.User.User>? sender =
         })
         .SetAutoDisposeTimeout(() => EndTrip(liane.Id)
         , 3600 * 1000)
-        .Build(serviceProvider);
+        .Build(osrmService, postgisService);
     });
 
     await tracker.Push(ping);

diff --git a/back/src/Liane/Liane.Service/Internal/Trip/LianeTracker.cs b/back/src/Liane/Liane.Service/Internal/Trip/LianeTracker.cs
@@ -9,7 +9,6 @@
 using Liane.Service.Internal.Osrm;
 using Liane.Service.Internal.Postgis;
 using Liane.Service.Internal.Util;
-using Microsoft.Extensions.DependencyInjection;
 
 namespace Liane.Service.Internal.Trip;
 
@@ -63,10 +62,8 @@ public Builder SetAutoDisposeTimeout(Func<Task> onTimeout, int delayMillis)
       return this;
     }
 
-    public async Task<LianeTracker> Build(ServiceProvider serviceProvider)
+    public async Task<LianeTracker> Build(IOsrmService osrmService, IPostgisService postgisService)
     {
-      var osrmService = serviceProvider.GetService<IOsrmService>()!;
-      var postgisService = serviceProvider.GetService<IPostgisService>()!;
       var route = await osrmService.Route(liane.WayPoints.Select(w => w.RallyingPoint.Location));
       var routeAsLineString = route.Routes[0].Geometry.Coordinates.ToLineString();
       tripSession  = await postgisService.CreateOngoingTrip(liane.Id, routeAsLineString);

diff --git a/back/src/Liane/Liane.Test/Integration/LianeTrackerTest.cs b/back/src/Liane/Liane.Test/Integration/LianeTrackerTest.cs
@@ -10,8 +10,11 @@
 using Liane.Api.Trip;
 using Liane.Api.User;
 using Liane.Api.Util.Ref;
+using Liane.Service.Internal.Osrm;
+using Liane.Service.Internal.Postgis;
 using Liane.Service.Internal.Trip;
 using Liane.Test.Util;
+using Microsoft.Extensions.DependencyInjection;
 using MongoDB.Driver;
 using NUnit.Framework;
 
@@ -58,7 +61,7 @@ public async Task ShouldFinishTrip()
       {
         finished = true;
       })
-      .Build(ServiceProvider);
+      .Build(ServiceProvider.GetService<IOsrmService>()!, ServiceProvider.GetService<IPostgisService>()!);
 
     var pings = geojsonPings.Features.Select(f =>
     {