fixed comments, logs messages

AndriyBorkovich · May 6, 2024 · 9d6168e · 9d6168e
1 parent fe18898
commit 9d6168e
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 10 deletions.
diff --git a/src/BSMS.API/BackgroundJobs/ScheduleTripsJob.cs b/src/BSMS.API/BackgroundJobs/ScheduleTripsJob.cs
@@ -8,6 +8,7 @@ namespace BSMS.API.BackgroundJobs;
 /// Schedule new trips based on current date
 /// </summary>
 /// <param name="serviceProvider"></param>
+/// <param name="logger"></param>
 public class ScheduleTripsJob(
     IServiceProvider serviceProvider,
     ILogger<ScheduleTripsJob> logger) : IHostedService

diff --git a/src/BSMS.API/BackgroundJobs/TripStartOrStopPeriodicJob.cs b/src/BSMS.API/BackgroundJobs/TripStartOrStopPeriodicJob.cs
@@ -1,6 +1,4 @@
-using System.Collections.Concurrent;
-using Bogus;
-using BSMS.Core.Entities;
+using BSMS.Core.Entities;
 using BSMS.Core.Enums;
 using BSMS.Infrastructure.Persistence;
 using Microsoft.EntityFrameworkCore;
@@ -90,7 +88,8 @@ private async Task HandleDepartingAsync(
         if (tripsToSetInTransit.Count is not 0)
         {
             await dbContext.BulkUpdateAsync(tripsToSetInTransit, opt => opt.IncludeGraph = true, stoppingToken);
-            logger.LogInformation("{0} trips has started", tripsToSetInTransit.Count);
+            logger.LogInformation("{0} trips have started", tripsToSetInTransit.Count(t => t.Status == TripStatus.InTransit));
+             logger.LogInformation("{0} trips have been delayed", tripsToSetInTransit.Count(t => t.Status == TripStatus.Delayed));
         }
     }
 
@@ -115,7 +114,7 @@ private async Task HandleArrivingAsync(
 
         if (updatedRows > 0)
         {
-            logger.LogInformation("{0} trips has completed", updatedRows);
+            logger.LogInformation("{0} trips have completed", updatedRows);
         }
     }
 
@@ -157,6 +156,8 @@ private async Task TryRestartAsync(BusStationContext dbContext, DateTime current
         if (tripsToRestart.Count is not 0)
         {
             await dbContext.BulkUpdateAsync(tripsToRestart, opt => opt.IncludeGraph = true,  stoppingToken);
+            logger.LogInformation("{0} trips have been restarted", tripsToRestart.Count(t => t.Status == TripStatus.InTransit));
+            logger.LogInformation("{0} trips are still delaying", tripsToRestart.Count(t => t.Status == TripStatus.Delayed));
         }
     }
 

diff --git a/src/BSMS.Infrastructure/Seed/BusStationSeeder.cs b/src/BSMS.Infrastructure/Seed/BusStationSeeder.cs
@@ -128,7 +128,7 @@ public async Task GenerateTicketsAndPaymentsForTrip(int tripId)
         var tripScheduleEntries = dbContext.BusScheduleEntries.Where(be => be.Trips.Any(t => t.TripId == tripId));
 
         var seatIds = await tripScheduleEntries
-                           .Select(be => be.Bus)
+                           .Select(e => e.Bus)
                            .SelectMany(b => b.Seats)
                            .Where(s => s.IsFree)
                            .Select(s => s.SeatId)
@@ -164,9 +164,9 @@ public async Task GenerateTicketsAndPaymentsForTrip(int tripId)
 
                 var endStopId = f.PickRandom(orderedStopsIds);
 
-                // Pick a random seat that hasn't been chosen yet and remove the chosen seat from available seats
-                var chosenSeatId = f.PickRandom(availableSeatIds.ToList());
-                availableSeatIds.Remove(chosenSeatId);
+                // Pick a random seat that hasn't been choosen yet and remove the choosen seat from available seats
+                var choosenSeatId = f.PickRandom(availableSeatIds.ToList());
+                availableSeatIds.Remove(choosenSeatId);
 
                 var newTicket = new Ticket
                 {
@@ -191,7 +191,7 @@ public async Task GenerateTicketsAndPaymentsForTrip(int tripId)
 
             await dbContext.BulkInsertOptimizedAsync(tickets, opt => opt.IncludeGraph = true);
 
-            logger.LogInformation("Passengers bought {0} tickets", ticketsToGenerate);
+            logger.LogInformation("Passengers have bought {0} tickets", ticketsToGenerate);
         }
         else
         {