done

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

@@ -1,5 +1,8 @@
 package com.zipcodewilmington.gildedrose;
 
+import javax.xml.bind.SchemaOutputResolver;
+
+
 
 public class Inventory {
     private Item[] items;
@@ -10,57 +13,59 @@ public class Inventory {
     }
 
 
-    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 isAgedBrie(Item item) {
+        return item.getName().equals("Aged Brie");
+    }
 
-                    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 isBackstagePass(Item item) {
+        return item.getName().equals("Backstage passes to a TAFKAL80ETC concert");
+    }
 
-                        if (items[i].getSellIn() < 6) {
-                            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].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
-            }
+    public boolean isConjuredItem(Item item) {
+        return item.getName().equals("Sulfuros, Hand of Rognoros (not to be confused with Ragnaros)");
+    }
 
-            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);
-                    }
-                }
+    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);
+            }
+        } 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);
         }
     }
 }

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

@@ -5,8 +5,186 @@ 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);
     }
 }