From 9a78a27b13fa412acb8e62efb9b501bc94959363 Mon Sep 17 00:00:00 2001 From: koneria Date: Sat, 30 Apr 2016 19:41:17 -0700 Subject: [PATCH] Added lombok and removed some unused classes --- myRetailApi/pom.xml | 11 +++++ .../myRetail/product/model/CatalogInfo.java | 26 ++--------- .../com/myRetail/product/model/Error.java | 29 ++++++------ .../com/myRetail/product/model/PriceInfo.java | 44 +++++-------------- .../myRetail/product/model/ProductInfo.java | 35 ++------------- .../product/proxy/TgtCatalogServiceImpl.java | 23 ---------- .../product/resources/ProductResource.java | 1 - .../myRetailBlackBoxTests/ProductApiTest.java | 1 - 8 files changed, 43 insertions(+), 127 deletions(-) delete mode 100644 myRetailApi/src/main/java/com/myRetail/product/proxy/TgtCatalogServiceImpl.java diff --git a/myRetailApi/pom.xml b/myRetailApi/pom.xml index fa67f0f..ea999f0 100644 --- a/myRetailApi/pom.xml +++ b/myRetailApi/pom.xml @@ -83,7 +83,18 @@ jersey-apache-client 1.19.1 + + org.projectlombok + lombok + 1.16.8 + + + + projectlombok.org + http://projectlombok.org/mavenrepo + + myRetailApi diff --git a/myRetailApi/src/main/java/com/myRetail/product/model/CatalogInfo.java b/myRetailApi/src/main/java/com/myRetail/product/model/CatalogInfo.java index d24ab41..11fe733 100644 --- a/myRetailApi/src/main/java/com/myRetail/product/model/CatalogInfo.java +++ b/myRetailApi/src/main/java/com/myRetail/product/model/CatalogInfo.java @@ -1,5 +1,7 @@ package com.myRetail.product.model; +import lombok.Data; + import javax.xml.bind.annotation.XmlElement; import javax.xml.bind.annotation.XmlRootElement; @@ -8,6 +10,7 @@ *

*/ @XmlRootElement +@Data public class CatalogInfo { @XmlElement @@ -22,27 +25,4 @@ public CatalogInfo(String id, String name) { this.id = id; this.name = name; } - public String getId() { - return id; - } - - public void setId(String id) { - this.id = id; - } - - public String getName() { - return name; - } - - public void setName(String name) { - this.name = name; - } - - @Override - public String toString() { - return "CatalogInfo{" + - "id='" + id + '\'' + - ", name='" + name + '\'' + - '}'; - } } diff --git a/myRetailApi/src/main/java/com/myRetail/product/model/Error.java b/myRetailApi/src/main/java/com/myRetail/product/model/Error.java index b6de28c..09ef734 100644 --- a/myRetailApi/src/main/java/com/myRetail/product/model/Error.java +++ b/myRetailApi/src/main/java/com/myRetail/product/model/Error.java @@ -1,5 +1,8 @@ package com.myRetail.product.model; +import lombok.Getter; +import lombok.Setter; + import java.util.Date; @@ -8,9 +11,21 @@ * in processing a valid or invalid request to the APIs

*/ public class Error { + @Getter + @Setter private Date date; + + @Getter + @Setter private String message; + + /** + *

+ * Constructur - creates an instance of this class without initializing any message. + *

+ * needed for JAXB serialization + */ public Error() {} public Error(Date d, String message ) { @@ -18,19 +33,5 @@ public Error(Date d, String message ) { this.message = message; } - public Date getDate() { - return date; - } - - public void setDate(Date date) { - this.date = date; - } - - public String getMessage() { - return message; - } - public void setMessage(String message) { - this.message = message; - } } diff --git a/myRetailApi/src/main/java/com/myRetail/product/model/PriceInfo.java b/myRetailApi/src/main/java/com/myRetail/product/model/PriceInfo.java index 40461cb..d1439af 100644 --- a/myRetailApi/src/main/java/com/myRetail/product/model/PriceInfo.java +++ b/myRetailApi/src/main/java/com/myRetail/product/model/PriceInfo.java @@ -1,5 +1,9 @@ package com.myRetail.product.model; +import lombok.Getter; +import lombok.Setter; +import lombok.ToString; + import javax.xml.bind.annotation.XmlElement; import javax.xml.bind.annotation.XmlRootElement; import javax.xml.bind.annotation.XmlTransient; @@ -10,16 +14,23 @@ * returned by subclasses of PricingDAO

*/ @XmlRootElement +@ToString public class PriceInfo { @XmlElement(name="value") + @Getter + @Setter private float price; @XmlElement(name="currency_code") + @Getter + @Setter private String currencyCode; + @XmlTransient + @Setter private String productId; public PriceInfo() {} @@ -30,37 +41,4 @@ public PriceInfo(String productId, float price, String currencyCode) { this.currencyCode = currencyCode; } - @XmlTransient - public String getProductId() { - return productId; - } - - public void setProductId(String productId) { - this.productId = productId; - } - - public float getPrice() { - return price; - } - - public void setPrice(float price) { - this.price = price; - } - - public String getCurrencyCode() { - return currencyCode; - } - - public void setCurrencyCode(String currencyCode) { - this.currencyCode = currencyCode; - } - - @Override - public String toString() { - return "PriceInfo{" + - "price=" + price + - ", currencyCode='" + currencyCode + '\'' + - ", productId='" + productId + '\'' + - '}'; - } } diff --git a/myRetailApi/src/main/java/com/myRetail/product/model/ProductInfo.java b/myRetailApi/src/main/java/com/myRetail/product/model/ProductInfo.java index 6a74336..1c8156e 100644 --- a/myRetailApi/src/main/java/com/myRetail/product/model/ProductInfo.java +++ b/myRetailApi/src/main/java/com/myRetail/product/model/ProductInfo.java @@ -1,5 +1,7 @@ package com.myRetail.product.model; +import lombok.Data; + import javax.xml.bind.annotation.XmlElement; import javax.xml.bind.annotation.XmlRootElement; @@ -9,6 +11,7 @@ * the catalog information and the pricing information. */ @XmlRootElement +@Data public class ProductInfo { @XmlElement private String id; @@ -29,36 +32,4 @@ public ProductInfo(String id, String name, PriceInfo priceInfo){ this.priceInfo = priceInfo; } - public String getId() { - return id; - } - - public void setId(String id) { - this.id = id; - } - - public String getName() { - return name; - } - - public void setName(String name) { - this.name = name; - } - - public PriceInfo getPriceInfo() { - return priceInfo; - } - - public void setPriceInfo(PriceInfo priceInfo) { - this.priceInfo = priceInfo; - } - - @Override - public String toString() { - return "ProductInfo{" + - "id='" + id + '\'' + - ", name='" + name + '\'' + - ", priceInfo=" + priceInfo + - '}'; - } } diff --git a/myRetailApi/src/main/java/com/myRetail/product/proxy/TgtCatalogServiceImpl.java b/myRetailApi/src/main/java/com/myRetail/product/proxy/TgtCatalogServiceImpl.java deleted file mode 100644 index b93da2f..0000000 --- a/myRetailApi/src/main/java/com/myRetail/product/proxy/TgtCatalogServiceImpl.java +++ /dev/null @@ -1,23 +0,0 @@ -package com.myRetail.product.proxy; - -import java.util.Optional; -import com.myRetail.product.model.CatalogInfo; - - -/** - * This class is not implemented yet. - */ -public class TgtCatalogServiceImpl extends CatalogServiceProxy { - - public Optional fetchCatalogInfo(String productId) { - return Optional.empty(); - } - - public void init() { - - } - - public void close() { - - } -} diff --git a/myRetailApi/src/main/java/com/myRetail/product/resources/ProductResource.java b/myRetailApi/src/main/java/com/myRetail/product/resources/ProductResource.java index e5c7d13..d24c8c9 100644 --- a/myRetailApi/src/main/java/com/myRetail/product/resources/ProductResource.java +++ b/myRetailApi/src/main/java/com/myRetail/product/resources/ProductResource.java @@ -99,7 +99,6 @@ public ProductInfo setProductPrice(@PathParam("id")String id, ProductInfo produc setupThreadContext(Optional.ofNullable(requestId)); List errors = findErrorsInRequest(id, productInfo); if(!errors.isEmpty()){ -// String errorMessage = errors.stream().map(i -> i.toString()).collect(Collectors.joining(", ")); String errorMessage = errors.stream().collect(Collectors.joining(", ")); errorMessage = String.format("Errors found in the request :%s",errorMessage); throw new BadRequestException(errorMessage,logger); diff --git a/myRetailBlackBoxTests/src/test/java/net/ak/myRetailBlackBoxTests/ProductApiTest.java b/myRetailBlackBoxTests/src/test/java/net/ak/myRetailBlackBoxTests/ProductApiTest.java index 77e707d..6b758af 100644 --- a/myRetailBlackBoxTests/src/test/java/net/ak/myRetailBlackBoxTests/ProductApiTest.java +++ b/myRetailBlackBoxTests/src/test/java/net/ak/myRetailBlackBoxTests/ProductApiTest.java @@ -4,7 +4,6 @@ import com.datastax.driver.core.Cluster; import com.datastax.driver.core.Session; import com.datastax.driver.core.policies.DefaultRetryPolicy; -import org.json.simple.JSONArray; import org.json.simple.parser.JSONParser; import org.junit.After; import org.junit.Before;