Merge pull request #6 from vicheanath/git-repo

Add Test and restructure folder project

SearchBugs.Api/SearchBugs.Api.csproj → src/SearchBugs.Api/SearchBugs.Api.csproj

@@ -8,9 +8,12 @@
   </PropertyGroup>
 
   <ItemGroup>
-	  <PackageReference Include="Microsoft.AspNetCore.OpenApi" Version="8.0.7" />
-	  <PackageReference Include="Swashbuckle.AspNetCore" Version="6.6.2" />
-	  <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="8.0.7" />
+	  <PackageReference Include="Microsoft.AspNetCore.OpenApi" Version="8.0.8" />
+	  <PackageReference Include="Swashbuckle.AspNetCore" Version="6.7.3" />
+	  <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="8.0.8">
+	    <PrivateAssets>all</PrivateAssets>
+	    <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+	  </PackageReference>
   </ItemGroup>
 
   <ItemGroup>

SearchBugs.Application/BugTracking/GetBugs/GetBugsQueryHandler.cs → src/SearchBugs.Application/BugTracking/GetBugs/GetBugsQueryHandler.cs

@@ -1,5 +1,4 @@
-
-using Shared.Data;
+using Shared.Data;
 using Shared.Messaging;
 using Shared.Results;
 
@@ -21,20 +20,20 @@ public Task<Result<List<BugsResponse>>> Handle(GetBugsQuery request, Cancellatio
     public async Task<IEnumerable<BugsResponse>?> GetBugsByProjectIdAsync(Guid projectId) =>
         await _sqlQueryExecutor.QueryAsync<BugsResponse>(@"
             SELECT 
-    b.id as Id,
-    b.title as Title,
-    b.description as Description,
-    s.name as Status,
-    p.name as Priority,
-    b.severity as Severity,
-    p.name as ProjectName,
-    CONCAT(u.name_first_name,' ',u.name_last_name) as Assignee,
-    CONCAT(u.name_first_name,' ',u.name_last_name) as Reporter,
-    b.created_on_utc as CreatedOnUtc,
-    b.modified_on_utc as UpdatedOnUtc
-FROM bug b
-JOIN bug_status s ON b.status_id = s.id
-JOIN bug_priority p ON b.priority_id = p.id
-JOIN ""user"" u ON b.assignee_id = u.id
-ORDER BY b.created_on_utc DESC", new { projectId });
+                b.id as Id,
+                b.title as Title,
+                b.description as Description,
+                s.name as Status,
+                p.name as Priority,
+                b.severity as Severity,
+                p.name as ProjectName,
+                CONCAT(u.name_first_name,' ',u.name_last_name) as Assignee,
+                CONCAT(u.name_first_name,' ',u.name_last_name) as Reporter,
+                b.created_on_utc as CreatedOnUtc,
+                b.modified_on_utc as UpdatedOnUtc
+            FROM bug b
+            JOIN bug_status s ON b.status_id = s.id
+            JOIN bug_priority p ON b.priority_id = p.id
+            JOIN ""user"" u ON b.assignee_id = u.id
+            ORDER BY b.created_on_utc DESC", new { projectId });
 }

SearchBugs.Application/DependencyInjection.cs → src/SearchBugs.Application/DependencyInjection.cs

@@ -9,12 +9,12 @@ public static IServiceCollection AddApplication(this IServiceCollection services
     {
         services.AddMediatR(config =>
         {
-            config.RegisterServicesFromAssemblyContaining<ApplicationAssemblyReference>();
+            config.RegisterServicesFromAssembly(ApplicationAssemblyReference.Assembly);
 
             config.AddOpenBehavior(typeof(ValidationPipelineBehavior<,>));
         });
 
-        services.AddValidatorsFromAssembly(ApplicationAssemblyReference.Assembly);
+        services.AddValidatorsFromAssembly(ApplicationAssemblyReference.Assembly, includeInternalTypes: true);
 
         return services;
     }

src/SearchBugs.Application/Git/DeleteGitRepo/DeleteGitCommandValidator.cs

@@ -0,0 +1,14 @@
+using FluentValidation;
+using Shared.Extensions;
+
+namespace SearchBugs.Application.Git.DeleteGitRepo;
+
+internal sealed class DeleteGitCommandValidator : AbstractValidator<DeleteGitRepoCommand>
+{
+    public DeleteGitCommandValidator()
+    {
+        RuleFor(x => x.Url)
+            .NotEmpty()
+            .WithError(GitValidationErrors.UrlIsRequired);
+    }
+}

SearchBugs.Application/Git/DeleteGitRepo/DeleteGitRepoCommand.cs → src/SearchBugs.Application/Git/DeleteGitRepo/DeleteGitRepoCommand.cs

@@ -2,4 +2,4 @@
 
 namespace SearchBugs.Application.Git.DeleteGitRepo;
 
-public sealed class DeleteGitRepoCommand(string Url) : ICommand;
+public record DeleteGitRepoCommand(string Url) : ICommand;

src/SearchBugs.Application/Git/DeleteGitRepo/DeleteGitRepoCommandHandler.cs

@@ -0,0 +1,34 @@
+using SearchBugs.Domain;
+using SearchBugs.Domain.Repositories;
+using Shared.Messaging;
+using Shared.Results;
+
+namespace SearchBugs.Application.Git.DeleteGitRepo;
+
+public sealed class DeleteGitRepoCommandHandler : ICommandHandler<DeleteGitRepoCommand>
+{
+    private readonly IGitRepository _gitRepository;
+    private readonly IUnitOfWork _unitOfWork;
+
+    public DeleteGitRepoCommandHandler(IGitRepository gitRepository, IUnitOfWork unitOfWork)
+    {
+        _gitRepository = gitRepository;
+        _unitOfWork = unitOfWork;
+    }
+
+    public async Task<Result> Handle(DeleteGitRepoCommand request, CancellationToken cancellationToken)
+    {
+        var gitRepo = await _gitRepository.GetByUrlAsync(request.Url, cancellationToken);
+
+        if (gitRepo.IsFailure)
+        {
+            return Result.Failure(GitValidationErrors.GitRepoNotFound);
+        }
+
+        await _gitRepository.Remove(gitRepo.Value);
+
+        await _unitOfWork.SaveChangesAsync(cancellationToken);
+
+        return Result.Success();
+    }
+}

SearchBugs.Application/Git/GitValidationErrors.cs → src/SearchBugs.Application/Git/GitValidationErrors.cs

@@ -8,4 +8,6 @@ internal static class GitValidationErrors
     internal static Error DescriptionIsRequired => new("Git.DescriptionIsRequired", "The git's description is required.");
     internal static Error UrlIsRequired => new("Git.UrlIsRequired", "The git's url is required.");
 
+    internal static Error GitRepoNotFound => new("Git.GitRepoNotFound", "The git repository was not found.");
+
 }

SearchBugs.Application/SearchBugs.Application.csproj → src/SearchBugs.Application/SearchBugs.Application.csproj

@@ -7,8 +7,8 @@
   </PropertyGroup>
 	<ItemGroup>
 		<PackageReference Include="FluentValidation.DependencyInjectionExtensions" Version="11.9.2" />
-		<PackageReference Include="MediatR" Version="12.4.0" />
-		<PackageReference Include="Microsoft.EntityFrameworkCore" Version="8.0.7" />
+		<PackageReference Include="MediatR" Version="12.4.1" />
+		<PackageReference Include="Microsoft.EntityFrameworkCore" Version="8.0.8" />
 	</ItemGroup>
 	<ItemGroup>
 		<ProjectReference Include="..\SearchBugs.Domain\SearchBugs.Domain.csproj" />

SearchBugs.Application/Users/UpdateUser/UpdateUserCommandHandler.cs → src/SearchBugs.Application/Users/UpdateUser/UpdateUserCommandHandler.cs

@@ -18,7 +18,7 @@ public UpdateUserCommandHandler(IUnitOfWork unitOfWork, IUserRepository userRepo
 
     public async Task<Result> Handle(UpdateUserCommand request, CancellationToken cancellationToken)
     {
-        var user = await _userRepository.GetByIdAsync(request.Id, cancellationToken);
+        var user = await _userRepository.GetByIdAsync(new UserId(request.Id), cancellationToken);
         if (user.IsFailure)
         {
             return Result.Failure(user.Error);

src/SearchBugs.Domain/Bugs/BugsErrors.cs

@@ -0,0 +1,21 @@
+using Shared.Errors;
+
+namespace SearchBugs.Domain.Bugs;
+
+internal static class BugsErrors
+{
+    internal static Error TitleIsRequired => new Error("Bug.TitleIsRequired", "Title is required");
+    internal static Error InvalidBugStatus => new Error("Bug.InvalidStatus", "Invalid bug status");
+
+    internal static Error InvalidBugPriority => new Error("Bug.InvalidPriority", "Invalid bug priority");
+
+    internal static Error InvalidBugSeverity => new Error("Bug.InvalidSeverity", "Invalid bug severity");
+
+    internal static Error InvalidProjectId => new Error("Bug.InvalidProjectId", "Invalid project id");
+
+    internal static Error InvalidAssigneeId => new Error("Bug.InvalidAssigneeId", "Invalid assignee id");
+
+    internal static Error InvalidReporterId => new Error("Bug.InvalidReporterId", "Invalid reporter id");
+
+    internal static Error InvalidTitle => new Error("Bug.InvalidTitle", "Invalid title");
+}

SearchBugs.Domain/Bugs/IBugRepository.cs → src/SearchBugs.Domain/Bugs/IBugRepository.cs

@@ -2,13 +2,9 @@
 
 namespace SearchBugs.Domain.Bugs;
 
-public interface IBugRepository
+public interface IBugRepository : IRepository<Bug, BugId>
 {
-    Task<Result<Bug>> GetByIdAsync(BugId id, CancellationToken cancellationToken);
-    Task Add(Bug bug);
-
     Task<Result<BugStatus>> GetBugStatusByName(string name, CancellationToken cancellationToken);
 
     Task<Result<BugPriority>> GetBugPriorityByName(string name, CancellationToken cancellationToken);
-
 }

src/SearchBugs.Domain/Bugs/TimeTracingId.cs

@@ -0,0 +1,8 @@
+using Shared.Primitives;
+
+namespace SearchBugs.Domain.Bugs;
+
+public record TimeTracingId(Guid Value) : IEntityId
+{
+    public static TimeTracingId New => new(Guid.NewGuid());
+}

src/SearchBugs.Domain/IRepository.cs

@@ -0,0 +1,18 @@
+using Shared.Primitives;
+using Shared.Results;
+using System.Linq.Expressions;
+
+namespace SearchBugs.Domain;
+
+public interface IRepository<TEntity, TEntityId>
+    where TEntity : Entity<TEntityId>
+    where TEntityId : class, IEntityId
+{
+    Task<Result> Add(TEntity entity);
+
+    Task<Result<TEntity>> GetByIdAsync(IEntityId id, CancellationToken cancellationToken = default, params Expression<Func<TEntity, object>>[] includes);
+
+    Task<Result> Remove(TEntity entity);
+
+    Task<Result> Update(TEntity entity);
+}

src/SearchBugs.Domain/Projects/IProjectRepository.cs

@@ -0,0 +1,5 @@
+namespace SearchBugs.Domain.Projects;
+
+public interface IProjectRepository : IRepository<Project, ProjectId>
+{
+}

src/SearchBugs.Domain/Repositories/IGitRepository.cs

@@ -0,0 +1,8 @@
+using Shared.Results;
+
+namespace SearchBugs.Domain.Repositories;
+
+public interface IGitRepository : IRepository<Repository, RepositoryId>
+{
+    Task<Result<Repository>> GetByUrlAsync(string url, CancellationToken cancellationToken = default);
+}

src/SearchBugs.Domain/Services/Ensure.cs

@@ -0,0 +1,32 @@
+using Shared.Errors;
+
+namespace SearchBugs.Domain.Services;
+
+public static class Ensure
+{
+    public static Error NotNull<T>(T value, Error error)
+    {
+        return value == null ? error : Error.None;
+    }
+
+    public static Error NotNullOrEmpty<T>(T value, Error error)
+    {
+        return value == null || string.IsNullOrEmpty(value.ToString()) ? error : Error.None;
+    }
+
+    public static Error NotNullOrWhiteSpace(string value, Error error)
+    {
+        return string.IsNullOrWhiteSpace(value) ? error : Error.None;
+    }
+
+    public static Error NotNegative<T>(T value, Error error)
+    {
+        return value switch
+        {
+            int i when i < 0 => error,
+            long l when l < 0 => error,
+            decimal d when d < 0 => error,
+            _ => Error.None
+        };
+    }
+}

SearchBugs.Domain/Users/IUserRepository.cs → src/SearchBugs.Domain/Users/IUserRepository.cs

@@ -3,15 +3,11 @@
 
 namespace SearchBugs.Domain.Users;
 
-public interface IUserRepository
+public interface IUserRepository : IRepository<User, UserId>
 {
     Task<Result<User>> GetUserByEmailAsync(string email, CancellationToken cancellationToken);
 
-    Task Add(User user);
-
     Task<Result<Role>> GetRoleByIdAsync(int roleId, CancellationToken cancellationToken);
 
     Task<Result<User>> IsEmailUniqueAsync(Email email, CancellationToken cancellationToken);
-
-    Task<Result<User>> GetByIdAsync(Guid id, CancellationToken cancellationToken = default);
 }