Lekce 06

ToDoList/src/ToDoList.Domain/DTOs/ToDoItemCreateRequestDto.cs

@@ -1,4 +1,5 @@
 namespace ToDoList.Domain.DTOs;
+
 using ToDoList.Domain.Models;
 
 public record ToDoItemCreateRequestDto(string Name, string Description, bool IsCompleted) //id is generated

ToDoList/src/ToDoList.Persistence/Repositories/IRepository.cs

@@ -3,4 +3,8 @@ namespace ToDoList.Persistence.Repositories;
 public interface IRepository<T> where T : class
 {
     public void Create(T item);
+    public IEnumerable<T> ReadAll();
+    public T? ReadById(int id);
+    public void Update(T item);
+    public void DeleteById(int id);
 }

ToDoList/src/ToDoList.Persistence/Repositories/ToDoItemsRepository.cs

@@ -1,5 +1,6 @@
 namespace ToDoList.Persistence.Repositories;
 
+using System.Collections.Generic;
 using ToDoList.Domain.Models;
 
 public class ToDoItemsRepository : IRepository<ToDoItem>
@@ -16,4 +17,19 @@ public void Create(ToDoItem item)
         context.ToDoItems.Add(item);
         context.SaveChanges();
     }
+    public IEnumerable<ToDoItem> ReadAll() => context.ToDoItems.ToList();
+    public ToDoItem? ReadById(int id) => context.ToDoItems.Find(id);
+    public void Update(ToDoItem item)
+    {
+        var foundItem = context.ToDoItems.Find(item.ToDoItemId) ?? throw new ArgumentOutOfRangeException($"ToDo item with ID {item.ToDoItemId} not found.");
+        context.Entry(foundItem).CurrentValues.SetValues(item);
+        context.SaveChanges();
+    }
+
+    public void DeleteById(int id)
+    {
+        var item = context.ToDoItems.Find(id) ?? throw new ArgumentOutOfRangeException($"ToDo item with ID {id} not found.");
+        context.ToDoItems.Remove(item);
+        context.SaveChanges();
+    }
 }

ToDoList/src/ToDoList.WebApi/Controllers/ToDoItemsController.cs

@@ -2,19 +2,16 @@ namespace ToDoList.WebApi.Controllers;
 using Microsoft.AspNetCore.Mvc;
 using ToDoList.Domain.DTOs;
 using ToDoList.Domain.Models;
-using ToDoList.Persistence;
 using ToDoList.Persistence.Repositories;
 
 [ApiController]
 [Route("api/[controller]")]
 public class ToDoItemsController : ControllerBase
 {
-    private readonly ToDoItemsContext context;
     private readonly IRepository<ToDoItem> repository;
 
-    public ToDoItemsController(ToDoItemsContext context, IRepository<ToDoItem> repository) // Az domigrujeme, zbyde nam tu jen repository :)
+    public ToDoItemsController(IRepository<ToDoItem> repository)
     {
-        this.context = context;
         this.repository = repository;
     }
 
@@ -44,18 +41,18 @@ public ActionResult<ToDoItemGetResponseDto> Create(ToDoItemCreateRequestDto requ
     [HttpGet]
     public ActionResult<IEnumerable<ToDoItemGetResponseDto>> Read()
     {
-        List<ToDoItem> itemsToGet;
+        IEnumerable<ToDoItem> itemsToGet;
         try
         {
-            itemsToGet = context.ToDoItems.ToList();
+            itemsToGet = repository.ReadAll();
         }
         catch (Exception ex)
         {
             return Problem(ex.Message, null, StatusCodes.Status500InternalServerError); //500
         }
 
         //respond to client
-        return (itemsToGet is null || itemsToGet.Count is 0)
+        return (itemsToGet is null || !itemsToGet.Any())
             ? NotFound() //404
             : Ok(itemsToGet.Select(ToDoItemGetResponseDto.FromDomain)); //200
     }
@@ -67,7 +64,7 @@ public ActionResult<ToDoItemGetResponseDto> ReadById(int toDoItemId)
         ToDoItem? itemToGet;
         try
         {
-            itemToGet = context.ToDoItems.Find(toDoItemId);
+            itemToGet = repository.ReadById(toDoItemId);
         }
         catch (Exception ex)
         {
@@ -91,14 +88,13 @@ public IActionResult UpdateById(int toDoItemId, [FromBody] ToDoItemUpdateRequest
         try
         {
             //retrieve the item
-            var itemToUpdate = context.ToDoItems.Find(toDoItemId);
+            var itemToUpdate = repository.ReadById(toDoItemId);
             if (itemToUpdate is null)
             {
                 return NotFound(); //404
             }
 
-            context.Entry(itemToUpdate).CurrentValues.SetValues(updatedItem);
-            context.SaveChanges();
+            repository.Update(updatedItem);
         }
         catch (Exception ex)
         {
@@ -115,14 +111,13 @@ public IActionResult DeleteById(int toDoItemId)
         //try to delete the item
         try
         {
-            var itemToDelete = context.ToDoItems.Find(toDoItemId);
+            var itemToDelete = repository.ReadById(toDoItemId);
             if (itemToDelete is null)
             {
                 return NotFound(); //404
             }
 
-            context.ToDoItems.Remove(itemToDelete);
-            context.SaveChanges();
+            repository.DeleteById(toDoItemId);
         }
         catch (Exception ex)
         {

ToDoList/tests/ToDoList.Test/IntegrationTests/DeleteTests.cs

@@ -3,16 +3,18 @@
 using Microsoft.AspNetCore.Mvc;
 using ToDoList.Domain.Models;
 using ToDoList.Persistence;
+using ToDoList.Persistence.Repositories;
 using ToDoList.WebApi.Controllers;
 
 public class DeleteTests
 {
     [Fact]
     public void Delete_ValidId_ReturnsNoContent()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context: context, repository: null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
         var toDoItem = new ToDoItem
         {
             Name = "Jmeno",
@@ -30,11 +32,12 @@
     }
 
     [Fact]
     public void Delete_InvalidId_ReturnsNotFound()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
         var toDoItem = new ToDoItem
         {
             Name = "Jmeno",

ToDoList/tests/ToDoList.Test/IntegrationTests/GetByIdTests.cs

@@ -3,16 +3,18 @@
 using Microsoft.AspNetCore.Mvc;
 using ToDoList.Domain.Models;
 using ToDoList.Persistence;
+using ToDoList.Persistence.Repositories;
 using ToDoList.WebApi.Controllers;
 
 public class GetByIdTests
 {
     [Fact]
     public void GetById_ValidId_ReturnsItem()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
         var toDoItem = new ToDoItem
         {
             Name = "Jmeno",
@@ -38,11 +40,12 @@
     }
 
     [Fact]
     public void GetById_InvalidId_ReturnsNotFound()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
 
         var toDoItem = new ToDoItem
         {

ToDoList/tests/ToDoList.Test/IntegrationTests/GetTests.cs

@@ -3,6 +3,7 @@ namespace ToDoList.Test.IntegrationTests;
 using Microsoft.AspNetCore.Mvc;
 using ToDoList.Domain.Models;
 using ToDoList.Persistence;
+using ToDoList.Persistence.Repositories;
 using ToDoList.WebApi.Controllers;
 
 public class GetTests
@@ -12,7 +13,8 @@ public void Get_AllItems_ReturnsAllItems()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
 
         var toDoItem = new ToDoItem
         {

ToDoList/tests/ToDoList.Test/IntegrationTests/PostTests.cs

@@ -9,12 +9,12 @@
 public class PostTests
 {
     [Fact]
     public void Post_ValidRequest_ReturnsNewItem()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
         var repository = new ToDoItemsRepository(context);
-        var controller = new ToDoItemsController(null, repository); // Docasny hack, nez z controlleru odstranime context.
+        var controller = new ToDoItemsController(repository);
         var request = new ToDoItemCreateRequestDto(
             Name: "Jmeno",
             Description: "Popis",

ToDoList/tests/ToDoList.Test/IntegrationTests/PutTests.cs

@@ -4,16 +4,18 @@
 using ToDoList.Domain.DTOs;
 using ToDoList.Domain.Models;
 using ToDoList.Persistence;
+using ToDoList.Persistence.Repositories;
 using ToDoList.WebApi.Controllers;
 
 public class PutTests
 {
     [Fact]
     public void Put_ValidId_ReturnsNoContent()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
         var toDoItem = new ToDoItem
         {
             Name = "Jmeno",
@@ -37,11 +39,12 @@
     }
 
     [Fact]
     public void Put_InvalidId_ReturnsNotFound()
     {
         // Arrange
         var context = new ToDoItemsContext("Data Source=../../../../../data/localdb.db");
-        var controller = new ToDoItemsController(context, null); // Docasny hack, nez z controlleru odstranime context.
+        var repository = new ToDoItemsRepository(context);
+        var controller = new ToDoItemsController(repository);
         var toDoItem = new ToDoItem
         {
             Name = "Jmeno",

ToDoList/tests/ToDoList.Test/UnitTests/PostTests.cs

@@ -11,11 +11,11 @@
 public class PostUnitTests
 {
     [Fact]
     public void Post_ValidRequest_ReturnsNewItem()
     {
         // Arrange
         var repositoryMock = Substitute.For<IRepository<ToDoItem>>();
-        var controller = new ToDoItemsController(null, repositoryMock); // Docasny hack, nez z controlleru odstranime context.
+        var controller = new ToDoItemsController(repositoryMock);
         var request = new ToDoItemCreateRequestDto(
             Name: "Jmeno",
             Description: "Popis",
@@ -29,6 +29,7 @@
 
         // Assert
         Assert.IsType<CreatedAtActionResult>(resultResult);
+        repositoryMock.Received(1).Create(Arg.Any<ToDoItem>());
         Assert.NotNull(value);
 
         Assert.Equal(request.Description, value.Description);
@@ -37,11 +38,11 @@
     }
 
     [Fact]
     public void Post_UnhandledException_Returns500()
     {
         // Arrange
         var repositoryMock = Substitute.For<IRepository<ToDoItem>>();
-        var controller = new ToDoItemsController(null, repositoryMock); // Docasny hack, nez z controlleru odstranime context.
+        var controller = new ToDoItemsController(repositoryMock);
         var request = new ToDoItemCreateRequestDto(
             Name: "Jmeno",
             Description: "Popis",