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

.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();
             });

BlogWebApp/Blog.Web/StartupConfigureServicesInstallers/CorsInstaller.cs

@@ -29,7 +29,7 @@ public void InstallServices(IServiceCollection services, IConfiguration configur
                 {
                     builder.AllowAnyHeader();
                     builder.AllowAnyMethod();
-                    builder.SetIsOriginAllowed(host => true);
+                    builder.SetIsOriginAllowed(_ => true);
                     builder.AllowCredentials();
                 }
                 else

BlogWebApp/Blog.Web/StartupConfigures/ConfigureSpa.cs

@@ -26,7 +26,7 @@ public static void Configure(IApplicationBuilder app, IWebHostEnvironment env)
 
             if (env.IsDevelopment())
             {
-                spa.UseProxyToSpaDevelopmentServer("http://localhost:4200"); ;
+                spa.UseProxyToSpaDevelopmentServer("http://localhost:4200");
             }
             else
             {

BlogWebApp/Blog.Web/SwaggerExamples/Responses/CreatedResponseExample.cs

@@ -6,7 +6,9 @@
 /// <summary>
 /// Created response example.
 /// </summary>
-/// <seealso cref="IExamplesProvider{CreatedResponse{int}}" />
+/// <seealso>
+///     <cref>IExamplesProvider{CreatedResponse{int}}</cref>
+/// </seealso>
 public class CreatedResponseExample : IExamplesProvider<CreatedResponse<int>>
 {
     /// <inheritdoc cref="IExamplesProvider{T}"/>

BlogWebApp/Blog.Web/VIewModels/AspNetUser/TwoFactorAuthenticationViewModel.cs

@@ -1,7 +1,7 @@
 namespace BLog.Web.ViewModels.AspNetUser;
 
 /// <summary>
-/// Two factor authentication view model
+/// Two-factor authentication view model
 /// </summary>
 public class TwoFactorAuthenticationViewModel
 {

BlogWebApp/Blog.sln.DotSettings.user

@@ -1,4 +1,7 @@
 <wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
+	<s:String x:Key="/Default/Environment/AssemblyExplorer/XmlDocument/@EntryValue">&lt;AssemblyExplorer&gt;&#xD;
+  &lt;Assembly Path="C:\Users\user\.nuget\packages\microsoft.aspnetcore.mvc.core\2.2.5\lib\netstandard2.0\Microsoft.AspNetCore.Mvc.Core.dll" /&gt;&#xD;
+&lt;/AssemblyExplorer&gt;</s:String>
 	<s:String x:Key="/Default/Environment/UnitTesting/UnitTestSessionStore/Sessions/=780df159_002D8db1_002D4d42_002D9c85_002D23db90ee4434/@EntryIndexedValue">&lt;SessionState ContinuousTestingMode="0" Name="PostTagRelationsServiceTests" xmlns="urn:schemas-jetbrains-com:jetbrains-ut-session"&gt;&#xD;
   &lt;TestAncestor&gt;&#xD;
     &lt;TestId&gt;xUnit::8AE98490-2264-4CBC-8B61-62DA16A2EC82::net5.0::Blog.ServicesTests.EntityServices.PostTagRelationsServiceTests&lt;/TestId&gt;&#xD;

BlogWebApp/Blog.slnLaunch.user

@@ -0,0 +1,15 @@
+[
+  {
+    "Name": "New Profile",
+    "Projects": [
+      {
+        "Path": "Blog.Web\\Blog.Web.csproj",
+        "Action": "Start"
+      },
+      {
+        "Path": "BlogBlazor\\BlogBlazor.csproj",
+        "Action": "Start"
+      }
+    ]
+  }
+]

BlogWebApp/Data/Blog.Data.Core/Entity.cs

@@ -4,8 +4,8 @@
 
 namespace Blog.Data.Core;
 
-using System;
 using Blog.Core;
+using System;
 
 /// <summary>
 /// Entity.
@@ -74,13 +74,13 @@ public virtual bool Equals(Entity other)
         var otherType = other.GetUnproxiedType();
         var thisType = this.GetUnproxiedType();
 
-        return thisType.IsAssignableFrom(otherType) || otherType.IsAssignableFrom(thisType);
+        return otherType != null && (thisType.IsAssignableFrom(otherType) || otherType.IsAssignableFrom(thisType));
     }
 
     /// <inheritdoc cref="object"/>
     public override int GetHashCode()
     {
-        return Equals(this.Id, default(int)) ? base.GetHashCode() : this.Id.GetHashCode();
+        return Equals(this.Id, 0) ? base.GetHashCode() : this.Id.GetHashCode();
     }
 
     /// <summary>
@@ -90,7 +90,7 @@ public override int GetHashCode()
     /// <returns>bool.</returns>
     private static bool IsTransient(Entity obj)
     {
-        return obj != null && Equals(obj.Id, default(int));
+        return obj != null && Equals(obj.Id, 0);
     }
 
     /// <summary>

BlogWebApp/Data/Blog.Data.Core/Models/BaseDeletableModel.cs

@@ -4,9 +4,9 @@
 
 namespace Blog.Data.Core.Models;
 
-using System;
 using Blog.Core;
 using Interfaces;
+using System;
 
 /// <summary>
 /// Base deletable model.
@@ -89,13 +89,13 @@ public virtual bool Equals(BaseDeletableModel<TKey> other)
         var otherType = other.GetUnproxiedType();
         var thisType = this.GetUnproxiedType();
 
-        return thisType.IsAssignableFrom(otherType) || otherType.IsAssignableFrom(thisType);
+        return otherType != null && (thisType.IsAssignableFrom(otherType) || otherType.IsAssignableFrom(thisType));
     }
 
     /// <inheritdoc cref="object"/>
     public override int GetHashCode()
     {
-        return Equals(this.Id, default(int)) ? base.GetHashCode() : this.Id.GetHashCode();
+        return Equals(this.Id, 0) ? base.GetHashCode() : this.Id.GetHashCode();
     }
 
     /// <summary>
@@ -105,7 +105,7 @@ public override int GetHashCode()
     /// <returns>bool.</returns>
     private static bool IsTransient(BaseDeletableModel<TKey> obj)
     {
-        return obj != null && Equals(obj.Id, default(int));
+        return obj != null && Equals(obj.Id, 0);
     }
 
     /// <summary>

BlogWebApp/Data/Blog.Data.Core/Models/BaseModel.cs

@@ -4,26 +4,22 @@
 
 namespace Blog.Data.Core.Models;
 
+using Interfaces;
 using System;
 using System.ComponentModel.DataAnnotations;
-using Interfaces;
 
 /// <summary>
 /// Base model.
 /// </summary>
 /// <typeparam name="TKey">TKey.</typeparam>
-public abstract class BaseModel<TKey> : IAuditInfo
+public abstract class BaseModel<TKey>(TKey id)
+    : IAuditInfo
 {
-    protected BaseModel(TKey id)
-    {
-        Id = id;
-    }
-
     /// <summary>
     /// Gets or sets id.
     /// </summary>
     [Key]
-    public TKey Id { get; }
+    public TKey Id { get; } = id;
 
     /// <summary>
     /// Gets or sets created on.

BlogWebApp/Data/Blog.Data.Models/ApplicationUser.cs

@@ -4,11 +4,11 @@
 
 namespace Blog.Data.Models;
 
-using System;
-using System.Collections.Generic;
 using Blog.Core;
 using Blog.Data.Core.Models.Interfaces;
 using Microsoft.AspNetCore.Identity;
+using System;
+using System.Collections.Generic;
 
 /// <summary>
 /// Application user entity.

BlogWebApp/Data/Blog.Data.Models/Message.cs

@@ -2,10 +2,9 @@
 // Copyright (c) PlaceholderCompany. All rights reserved.
 // </copyright>
 
-using Blog.Core.Enums;
-
 namespace Blog.Data.Models;
 
+using Blog.Core.Enums;
 using Core;
 
 /// <summary>

BlogWebApp/Data/Blog.Data.Models/Post.cs

@@ -2,8 +2,8 @@
 
 namespace Blog.Data.Models;
 
-using System;
 using Core;
+using System;
 
 /// <summary>
 /// Post entity.

BlogWebApp/Data/Blog.Data.Models/PostsTagsRelations.cs

@@ -7,7 +7,7 @@ namespace Blog.Data.Models;
 using Core;
 
 /// <summary>
-/// Posts Tags Many to Many Relations entity.
+/// Posts Tags Many-to-Many Relations entity.
 /// </summary>
 /// <seealso cref="Entity" />
 public class PostsTagsRelations : Entity

BlogWebApp/Data/Blog.Data.Models/Setting.cs

@@ -9,7 +9,6 @@ namespace Blog.Data.Models;
 /// <summary>
 /// Setting.
 /// </summary>
-
 public class Setting : Entity
 {
     /// <summary>

BlogWebApp/Data/Blog.Data/ApplicationDbContext.cs

@@ -4,15 +4,15 @@
 
 namespace Blog.Data;
 
+using Blog.Data.Core.Models.Interfaces;
+using Microsoft.AspNetCore.Identity.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore;
+using Models;
 using System;
 using System.Linq;
 using System.Reflection;
 using System.Threading;
 using System.Threading.Tasks;
-using Blog.Data.Core.Models.Interfaces;
-using Models;
-using Microsoft.AspNetCore.Identity.EntityFrameworkCore;
-using Microsoft.EntityFrameworkCore;
 
 /// <summary>
 /// Application database context.
@@ -159,7 +159,7 @@ protected override void OnModelCreating(ModelBuilder builder)
         foreach (var deletableEntityType in deletableEntityTypes)
         {
             var method = SetIsDeletedQueryFilterMethod.MakeGenericMethod(deletableEntityType.ClrType);
-            method.Invoke(null, new object[] { builder });
+            method.Invoke(null, [builder]);
         }
 
         // Disable cascade delete