Add ID param for PUT requests

NorthwindCRUD.Tests/CategroyServiceFixture.cs

@@ -28,7 +28,7 @@ public void ShouldUpdateCategory()
 
             createdCategory.Name = "Updated Category";
             createdCategory.Description = "Updated Description";
-            var updatedCategory = DataHelper.CategoryService.Update(createdCategory);
+            var updatedCategory = DataHelper.CategoryService.Update(createdCategory.CategoryId, createdCategory);
 
             Assert.IsNotNull(updatedCategory);
             updatedCategory = DataHelper2.CategoryService.GetById(updatedCategory.CategoryId);

NorthwindCRUD.Tests/EmployeeServiceFixture.cs

@@ -29,7 +29,7 @@ public void ShouldUpdateEmployee()
             var createdEmployee = DataHelper.EmployeeService.Create(employee);
 
             createdEmployee.Title = "Director";
-            var updatedEmployee = DataHelper.EmployeeService.Update(createdEmployee);
+            var updatedEmployee = DataHelper.EmployeeService.Update(createdEmployee.EmployeeId, createdEmployee);
 
             Assert.IsNotNull(updatedEmployee);
             updatedEmployee = DataHelper2.EmployeeService.GetById(updatedEmployee.EmployeeId);

NorthwindCRUD.Tests/OrderServiceFixture.cs

@@ -37,7 +37,7 @@ public void ShouldUpdateOrder()
             order.CustomerId = DataHelper.CreateCustomer().CustomerId;
             order.EmployeeId = DataHelper.CreateEmployee().EmployeeId;
 
-            var updatedOrder = DataHelper.OrderService.Update(order);
+            var updatedOrder = DataHelper.OrderService.Update(order.OrderId, order);
 
             Assert.IsNotNull(updatedOrder);
             updatedOrder = DataHelper2.OrderService.GetById(updatedOrder.OrderId);

NorthwindCRUD.Tests/ProductServiceFixture.cs

@@ -27,7 +27,7 @@ public void ShouldUpdateProduct()
             createdProduct.UnitPrice = 15;
             createdProduct.UnitsInStock = 50;
 
-            var updatedProduct = DataHelper.ProductService.Update(createdProduct);
+            var updatedProduct = DataHelper.ProductService.Update(createdProduct.ProductId, createdProduct);
 
             Assert.IsNotNull(updatedProduct);
             updatedProduct = DataHelper2.ProductService.GetById(updatedProduct.ProductId);

NorthwindCRUD.Tests/RegionServiceFixture.cs

@@ -27,7 +27,7 @@ public void ShouldUpdateRegion()
             string originalRegionDescription = region.RegionDescription;
             region.RegionDescription = "Updated Region";
 
-            var updatedRegion = DataHelper.RegionService.Update(region);
+            var updatedRegion = DataHelper.RegionService.Update(region.RegionId, region);
 
             Assert.IsNotNull(updatedRegion);
             updatedRegion = DataHelper2.RegionService.GetById(updatedRegion.RegionId);

NorthwindCRUD.Tests/ShipperServiceFixture.cs

@@ -31,7 +31,7 @@ public void ShouldUpdateShipper()
             shipper.CompanyName = "Updated shipper company";
             shipper.Phone = "555-555-5555";
 
-            var updatedShipper = DataHelper.ShipperService.Update(shipper);
+            var updatedShipper = DataHelper.ShipperService.Update(shipper.ShipperId, shipper);
 
             Assert.IsNotNull(updatedShipper);
             updatedShipper = DataHelper2.ShipperService.GetById(updatedShipper.ShipperId);

NorthwindCRUD.Tests/SupplierServiceFixture.cs

@@ -34,7 +34,7 @@ public void ShouldUpdateSupplier()
             supplier.ContactName = "Updated Contact";
             supplier.ContactTitle = "Updated Title";
 
-            var updatedSupplier = DataHelper.SupplierService.Update(supplier);
+            var updatedSupplier = DataHelper.SupplierService.Update(supplier.SupplierId, supplier);
             Assert.IsNotNull(updatedSupplier);
             updatedSupplier = DataHelper2.SupplierService.GetById(updatedSupplier.SupplierId);
             Assert.IsNotNull(updatedSupplier);

NorthwindCRUD.Tests/TerritoryServiceFixture.cs

@@ -29,7 +29,7 @@ public void ShouldUpdateTerritory()
 
             createdTerritory.TerritoryDescription = "Updated Territory";
 
-            var updatedTerritory = DataHelper.TerritoryService.Update(createdTerritory);
+            var updatedTerritory = DataHelper.TerritoryService.Update(createdTerritory.TerritoryId, createdTerritory);
 
             Assert.IsNotNull(updatedTerritory);
             updatedTerritory = DataHelper2.TerritoryService.GetById(updatedTerritory.TerritoryId);

NorthwindCRUD/Controllers/CategoriesController.cs

@@ -198,16 +198,16 @@ public ActionResult<CategoryDetailsDto> Create(CategoryDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<CategoryDto> Update(CategoryDto model)
+        public ActionResult<CategoryDto> Update(int id, CategoryDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<CategoryDto, CategoryDb>(model);
-                    var category = this.categoryService.Update(mappedModel);
+                    var category = this.categoryService.Update(id, mappedModel);
 
                     if (category != null)
                     {

NorthwindCRUD/Controllers/CustomersController.cs

@@ -209,16 +209,16 @@ public ActionResult<CustomerDto> Create(CustomerDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<CustomerDto> Update(CustomerDto model)
+        public ActionResult<CustomerDto> Update(string id, CustomerDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<CustomerDto, CustomerDb>(model);
-                    var customer = this.customerService.Update(mappedModel);
+                    var customer = this.customerService.Update(id, mappedModel);
 
                     if (customer != null)
                     {

NorthwindCRUD/Controllers/EmployeesController.cs

@@ -304,16 +304,16 @@ public ActionResult<EmployeeDto> Create(EmployeeDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<EmployeeDto> Update(EmployeeDto model)
+        public ActionResult<EmployeeDto> Update(int id, EmployeeDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<EmployeeDto, EmployeeDb>(model);
-                    var employee = this.employeeService.Update(mappedModel);
+                    var employee = this.employeeService.Update(id, mappedModel);
 
                     if (employee != null)
                     {

NorthwindCRUD/Controllers/OrdersController.cs

@@ -311,16 +311,16 @@ public ActionResult<OrderDto> Create(OrderDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<OrderDto> Update(OrderDto model)
+        public ActionResult<OrderDto> Update(int id, OrderDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<OrderDto, OrderDb>(model);
-                    var order = this.orderService.Update(mappedModel);
+                    var order = this.orderService.Update(id, mappedModel);
                     if (order != null)
                     {
                         return Ok(this.mapper.Map<OrderDb, OrderDto>(order));

NorthwindCRUD/Controllers/ProductsController.cs

@@ -277,16 +277,16 @@ public ActionResult<ProductDto> Create(ProductDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<ProductDto> Update(ProductDto model)
+        public ActionResult<ProductDto> Update(int id, ProductDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<ProductDto, ProductDb>(model);
-                    var product = this.productService.Update(mappedModel);
+                    var product = this.productService.Update(id, mappedModel);
 
                     if (product != null)
                     {

NorthwindCRUD/Controllers/RegionsController.cs

@@ -183,16 +183,16 @@ public ActionResult<RegionDto> Create(RegionDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<RegionDto> Update(RegionDto model)
+        public ActionResult<RegionDto> Update(int id, RegionDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<RegionDto, RegionDb>(model);
-                    var region = this.regionService.Update(mappedModel);
+                    var region = this.regionService.Update(id, mappedModel);
 
                     if (region != null)
                     {

NorthwindCRUD/Controllers/ShippersController.cs

@@ -178,16 +178,16 @@ public ActionResult<ShipperDto> Create(ShipperDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<ShipperDto> Update(ShipperDto model)
+        public ActionResult<ShipperDto> Update(int id, ShipperDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<ShipperDto, ShipperDb>(model);
-                    var shipper = this.shipperService.Update(mappedModel);
+                    var shipper = this.shipperService.Update(id, mappedModel);
 
                     if (shipper != null)
                     {

NorthwindCRUD/Controllers/SuppliersController.cs

@@ -178,16 +178,16 @@ public ActionResult<SupplierDto> Create(SupplierDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<SupplierDto> Update(SupplierDto model)
+        public ActionResult<SupplierDto> Update(int id, SupplierDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<SupplierDto, SupplierDb>(model);
-                    var supplier = this.supplierService.Update(mappedModel);
+                    var supplier = this.supplierService.Update(id, mappedModel);
 
                     if (supplier != null)
                     {

NorthwindCRUD/Controllers/TerritoriesController.cs

@@ -214,16 +214,16 @@ public ActionResult<TerritoryDto> Create(TerritoryDto model)
             }
         }
 
-        [HttpPut]
+        [HttpPut("{id}")]
         [Authorize]
-        public ActionResult<TerritoryDto> Update(TerritoryDto model)
+        public ActionResult<TerritoryDto> Update(string id, TerritoryDto model)
         {
             try
             {
                 if (ModelState.IsValid)
                 {
                     var mappedModel = this.mapper.Map<TerritoryDto, TerritoryDb>(model);
-                    var territory = this.territoryService.Update(mappedModel);
+                    var territory = this.territoryService.Update(id, mappedModel);
 
                     if (territory != null)
                     {

NorthwindCRUD/ControllersGraphQL/CategoryController.cs

@@ -50,10 +50,10 @@ public CategoryDto Create(CategoryDto model)
         }
 
         [Mutation]
-        public CategoryDto? Update(CategoryDto model)
+        public CategoryDto? Update(int id, CategoryDto model)
         {
             var mappedModel = this.mapper.Map<CategoryDto, CategoryDb>(model);
-            var category = this.categoryService.Update(mappedModel);
+            var category = this.categoryService.Update(id, mappedModel);
 
             return category != null ? this.mapper.Map<CategoryDb, CategoryDto>(category) : null;
         }

NorthwindCRUD/ControllersGraphQL/CustomerController.cs

@@ -51,10 +51,10 @@ public CustomerDto Create(CustomerDto model)
         }
 
         [Mutation]
-        public CustomerDto? Update(CustomerDto model)
+        public CustomerDto? Update(string id, CustomerDto model)
         {
             var mappedModel = this.mapper.Map<CustomerDto, CustomerDb>(model);
-            var customer = this.customerService.Update(mappedModel);
+            var customer = this.customerService.Update(id, mappedModel);
             return customer != null ? this.mapper.Map<CustomerDb, CustomerDto>(customer) : null;
         }
 

NorthwindCRUD/ControllersGraphQL/EmployeeController.cs

@@ -51,10 +51,10 @@ public EmployeeDto Create(EmployeeDto model)
         }
 
         [Mutation]
-        public EmployeeDto? Update(EmployeeDto model)
+        public EmployeeDto? Update(int id, EmployeeDto model)
         {
             var mappedModel = this.mapper.Map<EmployeeDto, EmployeeDb>(model);
-            var employee = this.employeeService.Update(mappedModel);
+            var employee = this.employeeService.Update(id, mappedModel);
             return employee != null ? this.mapper.Map<EmployeeDb, EmployeeDto>(employee) : null;
         }
 

NorthwindCRUD/ControllersGraphQL/OrderController.cs

@@ -50,10 +50,10 @@ public OrderDto Create(OrderDto model)
         }
 
         [Mutation]
-        public OrderDto? Update(OrderDto model)
+        public OrderDto? Update(int id, OrderDto model)
         {
             var mappedModel = this.mapper.Map<OrderDto, OrderDb>(model);
-            var order = this.orderService.Update(mappedModel);
+            var order = this.orderService.Update(id, mappedModel);
             return order != null ? this.mapper.Map<OrderDb, OrderDto>(order) : null;
         }
 

NorthwindCRUD/Models/Contracts/ICategory.cs

@@ -2,7 +2,7 @@
 {
     public interface ICategory
     {
-        int CategoryId { get; set; }
+        int CategoryId { get; }
 
         string Description { get; set; }
 

NorthwindCRUD/Models/Contracts/ICustomer.cs

@@ -4,7 +4,7 @@ namespace NorthwindCRUD.Models.Contracts
 {
     public interface ICustomer
     {
-        string CustomerId { get; set; }
+        string CustomerId { get; }
 
         string CompanyName { get; set; }
 

NorthwindCRUD/Models/Contracts/IEmployee.cs

@@ -4,7 +4,7 @@ namespace NorthwindCRUD.Models.Contracts
 {
     public interface IEmployee
     {
-        int EmployeeId { get; set; }
+        int EmployeeId { get; }
 
         string LastName { get; set; }
 

NorthwindCRUD/Models/Contracts/IOrder.cs

@@ -5,7 +5,7 @@ namespace NorthwindCRUD.Models.Contracts
 {
     public interface IOrder
     {
-        int OrderId { get; set; }
+        int OrderId { get; }
 
         string? CustomerId { get; set; }
 

NorthwindCRUD/Models/Contracts/IProduct.cs

@@ -2,7 +2,7 @@
 {
     public interface IProduct
     {
-        int ProductId { get; set; }
+        int ProductId { get; }
 
         int? SupplierId { get; set; }
 

NorthwindCRUD/Models/Contracts/IRegion.cs

@@ -2,7 +2,7 @@
 {
     public interface IRegion
     {
-        int RegionId { get; set; }
+        int RegionId { get; }
 
         string RegionDescription { get; set; }
     }

NorthwindCRUD/Models/Contracts/IShipper.cs

@@ -2,7 +2,7 @@
 {
     public interface IShipper
     {
-         int ShipperId { get; set; }
+         int ShipperId { get; }
 
          string CompanyName { get; set; }
 

NorthwindCRUD/Models/Contracts/ISupplier.cs

@@ -2,7 +2,7 @@
 {
     public interface ISupplier
     {
-        int SupplierId { get; set; }
+        int SupplierId { get; }
 
         string? CompanyName { get; set; }
 

NorthwindCRUD/Models/Contracts/ITerritory.cs

@@ -2,7 +2,7 @@
 {
     public interface ITerritory
     {
-        string TerritoryId { get; set; }
+        string TerritoryId { get; }
 
         string TerritoryDescription { get; set; }
 

NorthwindCRUD/Models/Dtos/CategoryDto.cs

@@ -5,7 +5,7 @@ namespace NorthwindCRUD.Models.Dtos
 {
     public class CategoryDto : ICategory
     {
-        public int CategoryId { get; set; }
+        public int CategoryId { get; private set; }
 
         [StringLength(500, ErrorMessage = "Description cannot exceed 500 characters.")]
         public string Description { get; set; }