Merge pull request #103 from lakatoshv/v3.7.3_AddExceptionTestsForDeleteMethodsAndCodeRefactoringToAspNet10

V3.7.3 add exception tests for delete methods and code refactoring to asp net10

Author: lakatoshv

.gitignore

@@ -109,3 +109,5 @@
 /BlogWebApp/Blog.Web/ClientApp/.angular/cache/20.0.1/BlogAngular
 /BlogWebApp/Blog.Web/ClientApp/dist/BlogAngular
 /Clients/BlogRazor/bin/Debug/net9.0
+/BlogWebApp/BlogBlazor/bin/Debug/net9.0
+/BlogWebApp/BlogBlazor/obj

BlogWebApp/Blog.Web/Cache/CachedAttribute.cs

@@ -19,13 +19,13 @@
 /// <remarks>
 /// Initializes a new instance of the <see cref="CachedAttribute"/> class.
 /// </remarks>
-/// <param name="lifeTimeSeconds">The life time seconds.</param>
+/// <param name="lifeTimeSeconds">The lifetime seconds.</param>
 [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method)]
 public class CachedAttribute(int lifeTimeSeconds)
     : Attribute, IAsyncActionFilter
 {
     /// <summary>
-    /// The life time seconds.
+    /// The lifetime seconds.
     /// </summary>
     private readonly int _lifeTimeSeconds = lifeTimeSeconds;
 

BlogWebApp/Blog.Web/Controllers/V1/AccountsController.cs

@@ -222,7 +222,7 @@ public async Task<IActionResult> CreateAsync([FromBody] RegistrationRequest mode
             return Bad(result);
         }
 
-        model.Roles ??= new[] {"User"};
+        model.Roles ??= ["User"];
 
         foreach(var role in model.Roles)
         {

BlogWebApp/Blog.Web/Controllers/V1/CommentsController.cs

@@ -183,7 +183,7 @@ public async Task<IActionResult> CreateAsync([FromBody] CreateCommentRequest req
         comment.CreatedAt = Now;
         await _commentService.InsertAsync(comment);
         var response = new CreatedResponse<int> {Id = comment.Id};
-        var baseUrl = $@"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
+        var baseUrl = $"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
         var locationUrl = baseUrl + "/" +
                           ApiRoutes.CommentsController.Comments + "/" +
                           ApiRoutes.CommentsController.GetComment + "/" + comment.Id;

BlogWebApp/Blog.Web/Controllers/V1/PostsController.cs

@@ -223,7 +223,7 @@ public async Task<IActionResult> CreateAsync([FromBody] CreatePostRequest model)
 
         var response = new CreatedResponse<int> { Id = postToCreate.Id };
 
-        var baseUrl = $@"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
+        var baseUrl = $"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
         var locationUrl = baseUrl + "/" + ApiRoutes.PostsController.Show.Replace("{id}", postToCreate.Id.ToString());
 
         return Created(locationUrl, response);

BlogWebApp/Blog.Web/Controllers/V1/TagsController.cs

@@ -85,6 +85,9 @@ public async Task<ActionResult> GetTags()
     [Cached(600)]
     public async Task<ActionResult> GetTagsByFilter([FromBody] SearchParametersRequest searchParameters = null)
     {
+        if (searchParameters == null)
+            return BadRequest();
+        
         searchParameters.SortParameters ??= new SortParametersRequest();
 
         searchParameters.SortParameters.OrderBy ??= "asc";
@@ -179,12 +182,7 @@ public async Task<ActionResult> TagsActivity()
     [Authorize]
     public async Task<IActionResult> CreateAsync([FromBody] CreateTagRequest model)
     {
-        if (!ModelState.IsValid)
-        {
-            return Bad(ModelState);
-        }
-
-        if (await _tagsService.AnyAsync(new TagSpecification(x => x.Title.ToLower().Equals(model.Title.ToLower()))))
+        if (!ModelState.IsValid || await _tagsService.AnyAsync(new TagSpecification(x => x.Title.ToLower().Equals(model.Title.ToLower()))))
         {
             return Bad(ModelState);
         }
@@ -194,7 +192,7 @@ public async Task<IActionResult> CreateAsync([FromBody] CreateTagRequest model)
 
         var response = new CreatedResponse<int> { Id = tag.Id };
 
-        var baseUrl = $@"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
+        var baseUrl = $"{HttpContext.Request.Scheme}://{HttpContext.Request.Host.ToUriComponent()}";
         var locationUrl = baseUrl + "/" + ApiRoutes.TagsController.GetTag.Replace("{id}", tag.Id.ToString());
 
         return Created(locationUrl, response);

BlogWebApp/Blog.Web/Factories/CategoryRequestFactory.cs

@@ -49,7 +49,7 @@ public override CreateCategoryRequest GenerateForCreate() =>
     public override UpdateCategoryRequest GenerateForUpdate(int id)
     {
         var category = _unitOfWork.GetRepository<Category>().FirstOrDefault(new CategorySpecification(x => x.Id == id))
-            ?? throw new MicroserviceArgumentNullException();;
+            ?? throw new MicroserviceArgumentNullException();
 
 
         var mapped = _mapper.Map<UpdateCategoryRequest>(category);

BlogWebApp/Blog.Web/Filters/SwaggerFilters/LowercaseDocumentFilter.cs

@@ -17,7 +17,7 @@ public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context)
     }
 
     /// <summary>
-    /// Lowercase the everything but parameters.
+    /// Lowercase everything but parameters.
     /// </summary>
     /// <param name="key">The key.</param>
     /// <returns>string.</returns>

BlogWebApp/Blog.Web/Filters/SwaggerFilters/SwaggerGroupOperationFilter.cs

@@ -24,11 +24,11 @@ public void Apply(OpenApiOperation operation, OperationFilterContext context)
         if (attributes.Any())
         {
             var groupNameAttribute = attributes.First();
-            operation.Tags = new[] { new OpenApiTag { Name = groupNameAttribute.GroupName } };
+            operation.Tags = [new OpenApiTag { Name = groupNameAttribute.GroupName }];
         }
         else
         {
-            operation.Tags = new[] { new OpenApiTag { Name = controllerActionDescriptor.RouteValues["controller"] } };
+            operation.Tags = [new OpenApiTag { Name = controllerActionDescriptor.RouteValues["controller"] }];
         }
     }
 }

BlogWebApp/Blog.Web/StartupConfigureServicesInstallers/AuthenticationAndAuthorizationInstaller.cs

@@ -102,15 +102,15 @@ public void InstallServices(IServiceCollection services, IConfiguration configur
         });
         services.AddCors(options =>
         {
-            options.AddPolicy("AllowAll", bilder =>
+            options.AddPolicy("AllowAll", builder =>
             {
-                bilder.AllowAnyOrigin()
+                builder.AllowAnyOrigin()
                     .AllowAnyHeader()
                     .AllowAnyMethod();
             });
-            options.AddPolicy("AllowAllBlazor", bilder =>
+            options.AddPolicy("AllowAllBlazor", builder =>
             {
-                bilder.WithOrigins("https://localhost:44390").AllowAnyOrigin()
+                builder.WithOrigins("https://localhost:44390").AllowAnyOrigin()
                     .AllowAnyHeader()
                     .AllowAnyMethod();
             });