done

src/main/java/com/zipcodewilmington/gildedrose/Inventory.java

@@ -1,6 +1,8 @@
 package com.zipcodewilmington.gildedrose;
 
 
+import javax.xml.bind.SchemaOutputResolver;
+
 public class Inventory {
     private Item[] items;
 
@@ -9,58 +11,114 @@ public class Inventory {
         this.items = items;
     }
 
+    public boolean isAgedBrie(Item item) {
+        return item.getName().equals("Aged Brie");
+    }
 
-    public void updateQuality() {
-        for (int i = 0; i < items.length; i++) {
-            if (!items[i].getName().equals("Aged Brie")
-                    && !items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                if (items[i].getQuality() > 0) {
-                    if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                        items[i].setQuality(items[i].getQuality() - 1);
-                    }
-                }
-            } else {
-                if (items[i].getQuality() < 50) {
-                    items[i].setQuality(items[i].getQuality() + 1);
+    public boolean isBackstagePass(Item item) {
+        return item.getName().equals("Backstage passes to a TAFKAL80ETC concert");
+    }
 
-                    if (items[i].getName() == "Backstage passes to a TAFKAL80ETC concert") {
-                        if (items[i].getSellIn() < 11) {
-                            if (items[i].getQuality() < 50) {
-                                items[i].setQuality(items[i].getQuality() + 1);
-                            }
-                        }
+    public boolean isSulfuras(Item item) {
+        return item.getName().equals("Sulfuras, Hand of Ragnaros");
+    }
 
-                        if (items[i].getSellIn() < 6) {
-                            if (items[i].getQuality() < 50) {
-                                items[i].setQuality(items[i].getQuality() + 1);
-                            }
-                        }
-                    }
-                }
-            }
+    public boolean isConjuredItem(Item item) {
+        return item.getName().equals("Sulfuros, Hand of Rognoros (not to be confused with Ragnaros)");
+    }
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
+    public void updateQuality(Item item) {
+        if (isAgedBrie(item)) {
+            if (item.getSellIn() > 0 && item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 1);
+            } else if (item.getSellIn() < 0 && item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 2);
             }
-
-            if (items[i].getSellIn() < 0) {
-                if (!items[i].getName().equals("Aged Brie")) {
-                    if (!items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        if (items[i].getQuality() > 0) {
-                            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                                items[i].setQuality(items[i].getQuality() - 1);
-                            }
-                        }
-                    } else {
-                        items[i].setQuality(items[i].getQuality()
-                                - items[i].getQuality());
-                    }
-                } else {
-                    if (items[i].getQuality() < 50) {
-                        items[i].setQuality(items[i].getQuality() + 1);
-                    }
-                }
+        } else if (isBackstagePass(item)) {
+            if (item.getSellIn() > 0 && item.getSellIn() > 10 && item.getQuality() >= 0) {
+                item.setQuality(item.getQuality() - 1);
+            } else if (item.getSellIn() <= 10 && item.getSellIn() > 5 && item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 2);
+            } else if (item.getSellIn() <= 5 && item.getSellIn() > 0 && item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 3);
+            } else {
+                item.setQuality(0);
+            }
+        } else if (isConjuredItem(item)) {
+            if (item.getSellIn() > 0 && item.getQuality() >= 0) {
+                item.setQuality(item.getQuality() - 2);
+            } else if (item.getQuality() >= 0) {
+                item.setQuality(item.getQuality() - 4);
             }
+        } else if (isSulfuras(item)) {
+            item.setQuality(80);
+        } else {
+            if (item.getSellIn() > 0 && item.getQuality() >= 0) {
+                item.setQuality(item.getQuality() - 1);
+            } else if (item.getQuality() >= 0) {
+                item.setQuality(item.getQuality() - 2);
+            }
+        }
+    }
+
+    public void updateSellIn(Item item) {
+        if (!isSulfuras(item)) {
+            item.setSellIn(item.getSellIn() - 1);
         }
     }
-}
+}
+
+
+//    public void updateQuality1() {
+//        for (int i = 0; i < items.length; i++) {
+//            if (!items[i].getName().equals("Aged Brie") && !items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
+//                if (items[i].getQuality() > 0) {
+//                    if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                        items[i].setQuality(items[i].getQuality() - 1);
+//                    }
+//                }
+//            } else {
+//                if (items[i].getQuality() < 50) {
+//                    items[i].setQuality(items[i].getQuality() + 1);
+//
+//                    if (items[i].getName() == "Backstage passes to a TAFKAL80ETC concert") {
+//                        if (items[i].getSellIn() < 11) {
+//                            if (items[i].getQuality() < 50) {
+//                                items[i].setQuality(items[i].getQuality() + 1);
+//                            }
+//                        }
+//
+//                        if (items[i].getSellIn() < 6) {
+//                            if (items[i].getQuality() < 50) {
+//                                items[i].setQuality(items[i].getQuality() + 1);
+//                            }
+//                        }
+//                    }
+//                }
+//            }
+//
+//            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                items[i].setSellIn(items[i].getSellIn() - 1);
+//            }
+//
+//            if (items[i].getSellIn() < 0) {
+//                if (!items[i].getName().equals("Aged Brie")) {
+//                    if (!items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
+//                        if (items[i].getQuality() > 0) {
+//                            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                                items[i].setQuality(items[i].getQuality() - 1);
+//                            }
+//                        }
+//                    } else {
+//                        items[i].setQuality(items[i].getQuality()
+//                                - items[i].getQuality());
+//                    }
+//                } else {
+//                    if (items[i].getQuality() < 50) {
+//                        items[i].setQuality(items[i].getQuality() + 1);
+//                    }
+//                }
+//            }
+//        }
+//    }
+//}

src/test/java/com/zipcodewilmington/gildedrose/InventoryTest.java

@@ -5,8 +5,234 @@ import org.junit.Test;
 
 public class InventoryTest {
     @Test
-    public void updateQuantityTest(){
-        Assert.assertEquals(1, 1);
+    public void testBrieQuality() {
+        Item brie = new Item("Aged Brie", 10, 45);
+        Item[] item = {brie};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(brie);
+
+        int expected = 46;
+        int actual = brie.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testBrieQuality_PastSellIn() {
+        Item brie = new Item("Aged Brie", -2, 45);
+        Item[] item = {brie};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(brie);
+
+        int expected = 47;
+        int actual = brie.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testBackstageQuality() {
+        Item backstage = new Item("Backstage passes to a TAFKAL80ETC concert", 15, 45);
+        Item[] item = {backstage};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(backstage);
+
+        int expected = 44;
+        int actual = backstage.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testBackstageQuality_LessThanTenDays() {
+        Item backstage = new Item("Backstage passes to a TAFKAL80ETC concert", 8, 45);
+        Item[] item = {backstage};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(backstage);
+
+        int expected = 47;
+        int actual = backstage.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testBackstageQuality_LessThanFiveDays() {
+        Item backstage = new Item("Backstage passes to a TAFKAL80ETC concert", 4, 45);
+        Item[] item = {backstage};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(backstage);
+
+        int expected = 48;
+        int actual = backstage.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testBackstageQuality_AfterConcert() {
+        Item backstage = new Item("Backstage passes to a TAFKAL80ETC concert", 0, 50);
+        Item[] item = {backstage};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(backstage);
+
+        int expected = 0;
+        int actual = backstage.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testSulfurasQuality() {
+        Item sulfuras = new Item("Sulfuras, Hand of Ragnaros", 10, 80);
+        Item[] item = {sulfuras};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(sulfuras);
+
+        int expected = 80;
+        int actual = sulfuras.getQuality();
+
+        Assert.assertEquals(expected, actual);
     }
+
+    @Test
+    public void testNormalItemQuality() {
+        Item normal = new Item("Generic Mana Potion", 10, 45);
+        Item[] item = {normal};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(normal);
+
+        int expected = 44;
+        int actual = normal.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+
+    @Test
+    public void testNormalItemQuality_PastSellIn() {
+        Item normal = new Item("Generic Mana Potion", -1, 10);
+        Item[] item = {normal};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(normal);
+
+        int expected = 8;
+        int actual = normal.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testConjuredItemQuality () {
+        Item conjured = new Item("Sulfuros, Hand of Rognoros (not to be confused with Ragnaros)", 5, 45);
+        Item[] item = {conjured};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(conjured);
+
+        int expected = 43;
+        int actual = conjured.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testConjuredItemQuality_PastSellIn () {
+        Item conjured = new Item("Sulfuros, Hand of Rognoros (not to be confused with Ragnaros)", -1, 10);
+        Item[] item = {conjured};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateQuality(conjured);
+
+        int expected = 6;
+        int actual = conjured.getQuality();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testNormalItemSellIn () {
+        Item normal = new Item("Generic Mana Potion", 5, 45);
+        Item[] item = {normal};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateSellIn(normal);
+
+        int expected = 4;
+        int actual = normal.getSellIn();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+    @Test
+    public void testSulfurasItemSellIn () {
+        Item sulfuras = new Item("Sulfuras, Hand of Ragnaros", 15, 45);
+        Item[] item = {sulfuras};
+        Inventory inventory = new Inventory(item);
+
+        inventory.updateSellIn(sulfuras);
+
+        int expected = 15;
+        int actual = sulfuras.getSellIn();
+
+        Assert.assertEquals(expected, actual);
+    }
+
+//    @Test
+//    public void updateBrieQualityTest_PastSellIn() {
+//        Item brie = new Item("Aged Brie", -2, 45);
+//        Item[] item = {brie};
+//        Inventory inventory = new Inventory(item);
+//
+//        inventory.updateQuality();
+//
+//        int expected = 47;
+//        int actual = brie.getQuality();
+//
+//        Assert.assertEquals(expected, actual);
+//    }
+
+//    @Test
+//    public void testIsAgedBrie() {
+//        Item item = new Item("Aged Brie", 6, 50);
+//
+//        String expected = "Aged Brie";
+//
+//        String actual = item.getName();
+//
+//        Assert.assertEquals(expected, actual);
+//    }
+//
+//    @Test
+//    public void testIsBackstagePass() {
+//        Item item = new Item("Backstage passes to a TAFKAL80ETC concert", 6, 50);
+//
+//        String expected = "Backstage passes to a TAFKAL80ETC concert";
+//
+//        String actual = item.getName();
+//
+//        Assert.assertEquals(expected, actual);
+//    }
+//
+//    @Test
+//    public void testIsSulfuras() {
+//        Item item = new Item("Sulfuras, Hand of Ragnaros", 6, 50);
+//
+//        String expected = "Sulfuras, Hand of Ragnaros";
+//
+//        String actual = item.getName();
+//
+//        Assert.assertEquals(expected, actual);
+//
+//    }
 }