criado metodo de edicao de produto por sku

Marcelo · Marcelo · commit 44220110843c · 2020-10-03T22:52:34.000-03:00
diff --git a/src/main/java/br/com/blz/testjava/business/ProductBusiness.java b/src/main/java/br/com/blz/testjava/business/ProductBusiness.java
@@ -29,4 +29,12 @@ public Product findBySku(Long sku) {
         return Optional.ofNullable(this.productService.findBySku(sku))
             .orElseThrow(() -> new SkuNotExistsException(ErrorEnum.SKU_NOT_EXISTS.getDescription()));
     }
+
+    public Product update(Long sku, Product product) {
+        var productSaved = Optional.ofNullable(this.productService.findBySku(sku))
+            .orElseThrow(() -> new SkuNotExistsException(ErrorEnum.SKU_NOT_EXISTS.getDescription()));
+        productSaved.setInventory(product.getInventory());
+        productSaved.setName(product.getName());
+        return this.productService.save(productSaved);
+    }
 }
diff --git a/src/main/java/br/com/blz/testjava/resources/v1/ProductResource.java b/src/main/java/br/com/blz/testjava/resources/v1/ProductResource.java
@@ -41,4 +41,17 @@ public ResponseEntity<Product> findBySku(@PathVariable(value = "sku") Long sku)
         return new ResponseEntity<>(this.productBusiness.findBySku(sku), HttpStatus.OK);
     }
 
+    @ApiOperation(value = "Api resposável por editar um produto por sku", nickname = "Edição de produto")
+    @ApiResponses({
+        @ApiResponse(code = 200, message = "Retorno de sucesso na edição de um produto"),
+        @ApiResponse(code = 422, message = "Retorno de erro de regras de negocio"),
+        @ApiResponse(code = 500, message = "Retorno de erro interno da aplicação")
+    })
+    @PutMapping(value = "/{sku}")
+    @ResponseStatus(HttpStatus.OK)
+    public ResponseEntity<Product> update(@PathVariable(value = "sku") Long sku,
+                                          @RequestBody Product product) {
+        return new ResponseEntity<>(this.productBusiness.update(sku, product), HttpStatus.OK);
+    }
+
 }
diff --git a/src/test/java/br/com/blz/testjava/business/ProductBusinessTest.java b/src/test/java/br/com/blz/testjava/business/ProductBusinessTest.java
@@ -1,6 +1,7 @@
 package br.com.blz.testjava.business;
 
 import br.com.blz.testjava.TestJavaApplication;
+import br.com.blz.testjava.domain.Product;
 import br.com.blz.testjava.enums.WarehouseType;
 import br.com.blz.testjava.exception.SkuExistsException;
 import br.com.blz.testjava.exception.SkuNotExistsException;
@@ -98,4 +99,33 @@ void mustThrowExceptionBecauseProductBySkuNotExistsInBase() {
         Assertions.assertThrows(SkuNotExistsException.class, () -> this.productBusiness.findBySku(12234l));
     }
 
+    @Test
+    void deveEstourarExcecaoAoFazerUpdatePoisProdutoNaoExiste() {
+        Assertions.assertThrows(SkuNotExistsException.class, () -> this.productBusiness.update(12234l,
+            Product.builder().build()));
+    }
+
+    @Test
+    void deveAlterarProdutoComSucesso() {
+        var warehouses = List.of(createWarehouse("SP", WarehouseType.PHYSICAL_STORE,
+            250l));
+        var inventory = createInventory(warehouses);
+        var product = createProduct("ps5 digital edition", 12234l, inventory);
+
+        var newWarehouses = List.of(createWarehouse("SP", WarehouseType.PHYSICAL_STORE,
+            250l), createWarehouse("SC", WarehouseType.PHYSICAL_STORE,
+            250l));
+        var newInventory = createInventory(newWarehouses);
+        var newProduct = createProduct("ps5 digital edition", 12234l, newInventory);
+
+        this.productBusiness.save(product);
+        var newProductSaved = this.productBusiness.update(12234l, newProduct);
+
+        Assertions.assertAll(
+            () -> Assertions.assertTrue(newProductSaved.isMarketable()),
+            () -> Assertions.assertEquals(500l, newProductSaved.getInventory().getQuantity()),
+            () -> Assertions.assertEquals(2, newProductSaved.getInventory().getWarehouses().size()),
+            () -> Assertions.assertEquals(newProduct.getName(), newProductSaved.getName()));
+    }
+
 }

Original file line number	Diff line number	Diff line change
`@@ -29,4 +29,12 @@ public Product findBySku(Long sku) {`
`29`	`29`	`return Optional.ofNullable(this.productService.findBySku(sku))`
`30`	`30`	`.orElseThrow(() -> new SkuNotExistsException(ErrorEnum.SKU_NOT_EXISTS.getDescription()));`
`31`	`31`	`}`
	`32`	`+`
	`33`	`+ public Product update(Long sku, Product product) {`
	`34`	`+ var productSaved = Optional.ofNullable(this.productService.findBySku(sku))`
	`35`	`+ .orElseThrow(() -> new SkuNotExistsException(ErrorEnum.SKU_NOT_EXISTS.getDescription()));`
	`36`	`+ productSaved.setInventory(product.getInventory());`
	`37`	`+ productSaved.setName(product.getName());`
	`38`	`+ return this.productService.save(productSaved);`
	`39`	`+ }`
`32`	`40`	`}`