TIAM.Database/DataLayers/Admins/AdminDal.cs

@@ -48,7 +48,7 @@ namespace TIAM.Database.DataLayers.Admins
         public List<Car> GetCarByUserProductMappingId(Guid userProductMappingId) => Session(ctx => ctx.Cars.Where(x => x.UserProductMappingId == userProductMappingId).ToList());
         public Task<List<Car>> GetCarsByUserProductMappingIdAsync(Guid userProductMappingId) => SessionAsync(ctx => ctx.Cars.Where(x => x.UserProductMappingId == userProductMappingId).ToList());
         public Task<bool> AddCarAsync(Car car) => TransactionAsync(ctx => ctx.Cars.Add(car).State == EntityState.Added);
-        public Task<Car?> UpdateCarAsync(Car car) => UpdateSafeAsync(car);
+        public Task<bool> UpdateCarAsync(Car car) => TransactionAsync(ctx => ctx.Cars.Update(car).State == EntityState.Modified);
         public Task<bool> RemoveCarAsync(Car car) => TransactionAsync(ctx => ctx.Cars.Remove(car).State == EntityState.Deleted);
 
         #endregion Car

TIAM.Services.Server.Tests/LoginServices/LoginServiceServerTests.cs

@@ -1,4 +1,3 @@
-using AyCode.Core.Consts;
 using AyCode.Core.Server.Loggers;
 using AyCode.Services.Server.Tests.LoginServices;
 using TIAM.Database.DataLayers.Users;
@@ -8,7 +7,6 @@ using TIAM.Entities.Emails;
 using TIAM.Entities.Profiles;
 using TIAM.Entities.ServiceProviders;
 using TIAM.Entities.Users;
-using TIAM.Models.Dtos.Users;
 using TIAM.Models.Server.Logins;
 using TIAM.Services.Server.Logins;
 
@@ -30,19 +28,6 @@ namespace TIAM.Services.Server.Tests.LoginServices
         {
             GlobalLogger.Debug("Test started0...");
 
-            //var loginService = Activator.CreateInstance(typeof(LoginService), Dal, AcEnv.AppConfiguration) as LoginService;
-
-            //var userId = Guid.Parse(RegisterUserIdString);
-            //var errorCode = await loginService.RegistrationAsync(userId, RegisterEmail, RegisterPassword, null);
-            //var userModelDto = await Dal.GetUserModelDtoByIdAsync<UserModelDtoDetail>(userId, false);
-
-            //if (userModelDto != null)
-            //{
-            //    var userProperties = new Dictionary<Guid, string>();
-            //    //get user's properties
-            //    var hasProperties = userModelDto.ServiceProviders.Count > 0 ? true : (userModelDto.ServiceProviders.Count == 0 ? false : false);
-            //}
-
             return base.AcBase_RegisterUser_ReturnsUser_WhenUserExist(userIdEmailPasswordStrings);
         }
 

TIAMWebApp/Server/Controllers/ServiceProviderAPIController.cs

@@ -316,14 +316,14 @@ namespace TIAMWebApp.Server.Controllers
 
         [NonAction]
         [ApiExplorerSettings(IgnoreApi = true)]
-        private async Task<Car?> CarDataChanging(Car car, TrackingState trackingState)
+        private async Task<bool> CarDataChanging(Car car, TrackingState trackingState)
         {
             var logText = $"[{trackingState.ToString().ToUpper()}] CarDataChanging called; Id: {car.Id}; OwnerId: {car.UserProductMappingId}; LicensePlate: {car.LicencePlate}";
 
             if (car.UserProductMappingId.IsNullOrEmpty() || car.LicencePlate.IsNullOrWhiteSpace())
             {
                 _logger.Error(logText);
-                return null;
+                return false;
             }
 
             _logger.Info(logText);
@@ -333,11 +333,11 @@ namespace TIAMWebApp.Server.Controllers
                 case TrackingState.Add:
                     if (car.Id.IsNullOrEmpty()) car.Id = Guid.NewGuid();
 
-                    return await adminDal.AddCarAsync(car) ? car : null;
+                    return await adminDal.AddCarAsync(car);
                 case TrackingState.Update:
                     return await adminDal.UpdateCarAsync(car);
                 case TrackingState.Remove:
-                    return await adminDal.RemoveCarAsync(car) ? car : null;
+                    return await adminDal.RemoveCarAsync(car);
 
                 case TrackingState.Get:
                 case TrackingState.GetAll:
@@ -352,7 +352,11 @@ namespace TIAMWebApp.Server.Controllers
         [Tags("Finished", "Cars")]
         [EndpointSummary("Create car")]
         [SignalR(SignalRTags.CreateCar)]
-        public async Task<Car?> CreateCar(Car car) => await CarDataChanging(car, TrackingState.Add);
+        public async Task<Car?> CreateCar(Car car)
+        {
+            var result = await CarDataChanging(car, TrackingState.Add);
+            return result ? car : null;
+        }
 
         [AllowAnonymous]
         [HttpPost]
@@ -360,7 +364,12 @@ namespace TIAMWebApp.Server.Controllers
         [Tags("Finished", "Cars")]
         [EndpointSummary("Update car")]
         [SignalR(SignalRTags.UpdateCar)]
-        public async Task<Car?> UpdateCar(Car car) => await CarDataChanging(car, TrackingState.Update);
+        public async Task<Car?> UpdateCar(Car car)
+        {
+            var result = await CarDataChanging(car, TrackingState.Update);
+            return result ? car : null;
+        }
+            
 
         [AllowAnonymous]
         [HttpPost]
@@ -368,7 +377,8 @@ namespace TIAMWebApp.Server.Controllers
         [Tags("Finished", "Cars")]
         [EndpointSummary("Delete car")]
         [SignalR(SignalRTags.DeleteCar)]
-        public async Task<Car?> DeleteCar(Car car) => await CarDataChanging(car, TrackingState.Remove);
+        public async Task<IActionResult> DeleteCar(Car car)
+            => await CarDataChanging(car, TrackingState.Remove) ? Ok(car) : BadRequest("Invalid request");
 
         [HttpPost]
         [Route(APIUrls.AddProductRouteName)]