komunikaty błędów

Controllers/ExpenseController.cs

@@ -16,6 +16,7 @@
  */
 
 using FirmTracker_Server.nHibernate.Expenses;
+using FirmTracker_Server.nHibernate.Products;
 using Microsoft.AspNetCore.Mvc;
 namespace FirmTracker_Server.Controllers
 {
@@ -34,6 +35,10 @@ namespace FirmTracker_Server.Controllers
         [ProducesResponseType(201)] // Created
         [ProducesResponseType(400)] // Bad Request
         public IActionResult CreateExpense([FromBody] Expense expense) {
+            if (expense.Value<=0)
+            {
+                throw new InvalidOperationException("Wydatek nie może posiadać kwoty mniejszej lub równej 0.");
+            }
             try
             {
                 _expenseCrud.AddExpense(expense);
@@ -66,7 +71,11 @@ namespace FirmTracker_Server.Controllers
         {
             if (id != expense.Id)
             {
-                return BadRequest("Expense ID mismatch");
+                return BadRequest("Nieprawidłowe ID wydatku");
+            }
+            if (expense.Value <= 0)
+            {
+                throw new InvalidOperationException("Wydatek nie może posiadać kwoty mniejszej lub równej 0.");
             }
             try
             {

Controllers/ProductController.cs

@@ -17,6 +17,8 @@
 
 using FirmTracker_Server.nHibernate.Products;
 using Microsoft.AspNetCore.Mvc;
+using System;
+
 namespace FirmTracker_Server.Controllers
 {
     [Route("api/[controller]")]
@@ -41,11 +43,18 @@ namespace FirmTracker_Server.Controllers
         {
             if (product.Type != 0 && product.Type != 1)
             {
-                return BadRequest("Kategoria produktu musi być ustawiona na 0 lub 1.");
+                throw new InvalidOperationException("Kategoria produktu musi być ustawiona na 0 lub 1.");
             }
             if (product.Type == 0 && product.Availability != 0)
             {
-                return BadRequest("Dostępność usługi musi być ustawiona na 0.");
+                throw new InvalidOperationException("Dostępność usługi musi być ustawiona na 0.");
+            }
+            if(product.Type ==1 && product.Availability < 0) {
+                throw new InvalidOperationException("Dostępność towaru nie może być ujemna.");
+            }
+            if (product.Price < 0)
+            {
+                throw new InvalidOperationException("Produkt nie może posiadać ujemnej ceny.");
             }
             try
             {
@@ -88,16 +97,23 @@ namespace FirmTracker_Server.Controllers
         public IActionResult UpdateProduct(int id, [FromBody] Product product)
         {
             if (id != product.Id)
-                return BadRequest("ID produktu nie zgadza się.");
+                throw new InvalidOperationException("ID produktu nie zgadza się.");
             if (product.Type != 0 && product.Type != 1)
             {
-                return BadRequest("Kategoria produktu musi być ustawiona na 0 lub 1.");
+                throw new InvalidOperationException("Kategoria produktu musi być ustawiona na 0 lub 1.");
             }
             if (product.Type == 0 && product.Availability != 0)
             {
-                return BadRequest("Dostępność usługi musi być ustawiona na 0.");
+                throw new InvalidOperationException("Dostępność usługi musi być ustawiona na 0.");
+            }
+            if (product.Type == 1 && product.Availability < 0)
+            {
+                throw new InvalidOperationException("Dostępność towaru nie może być ujemna.");
+            }
+            if (product.Price < 0)
+            {
+                throw new InvalidOperationException("Produkt nie może posiadać ujemnej ceny.");
             }
-
             try
             {
                 _productCrud.UpdateProduct(product);

TestClass.cs

@@ -54,7 +54,7 @@ namespace FirmTracker_Server
             var products = new List<Product>
             {
                 CreateProduct("Tarta_truskawka", "produkt", 31.99m, 1, 10),
-                CreateProduct("Tarta_czekolada", "produkt", 30.99m, 1, 8),
+                CreateProduct("Tarta_czekolada", "produkt", 30.99m, 1, 10),
                 CreateProduct("Tarta_agrest", "produkt", 32.90m, 1, 8),
                 CreateProduct("Tarta_pistacja", "produkt", 35.99m, 1, 12),
                 CreateProduct("Tarta_karmel", "produkt", 32.00m, 1, 12),

nHibernate/Transactions/Transaction2Mapping.cs

@@ -18,7 +18,7 @@ namespace FirmTracker_Server.nHibernate.Transactions
             HasMany(x => x.TransactionProducts)
                 .KeyColumn("TransactionId")
                 .Cascade.AllDeleteOrphan()
-                .Inverse(); // Ustawienie Inverse() wskazuje, że to `TransactionProduct` jest właścicielem relacji
+                .Inverse();
 
         }
     }

nHibernate/Transactions/TransactionMapping.cs

@@ -18,7 +18,7 @@ namespace FirmTracker_Server.nHibernate.Transactions
             HasMany(x => x.TransactionProducts)
                 .KeyColumn("TransactionId")
                 .Cascade.AllDeleteOrphan()
-                .Inverse(); // Ustawienie Inverse() wskazuje, że to `TransactionProduct` jest właścicielem relacji
+                .Inverse();
 
         }
     }