finished lab with 100% line coverage

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

@@ -12,55 +12,83 @@ 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);
 
-                    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);
-                            }
-                        }
+            switch(items[i].getName()){
+                case "Aged Brie":
+                        updateBrie(items[i]);
+                        break;
+                case "Backstage passes to a TAFKAL80ETC concert" :
+                        updateBackstagePass(items[i]);
+                        break;
+                case "Sulfuras, Hand of Ragnaros" :
+                        updateSulfra(items[i]);
+                        break;
 
-                        if (items[i].getSellIn() < 6) {
-                            if (items[i].getQuality() < 50) {
-                                items[i].setQuality(items[i].getQuality() + 1);
-                            }
-                        }
-                    }
-                }
+                case "Conjured Item" :
+                        updateConjure(items[i]);
+                        break;
+                default :
+                        updateDefault(items[i]);
+                        break;
             }
+        }
+    }
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
-            }
+    public void updateBrie(Item item){
+        if(!sellInExpired(item)){
+            incrementQuality(item,1);
+        } else{
+            incrementQuality(item,2);
+        }
+        decrementSellIn(item);
+    }
 
-            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 updateBackstagePass(Item item){
+        if(sellInExpired(item)){
+            item.setQuality(0);
+        } else if(item.getSellIn() <= 5){
+            incrementQuality(item, 3);
+        } else if(item.getSellIn() <= 10){
+            incrementQuality(item, 2);
+        } else{
+            incrementQuality(item, 1);
+        }
+        decrementSellIn(item);
+    }
+
+    public void updateSulfra(Item item){
+        item.setQuality(80);
+        item.setSellIn(item.getSellIn());
+    }
+
+    public void updateConjure(Item item){
+        incrementQuality(item, -2);
+        decrementSellIn(item);
+    }
+
+    public void updateDefault(Item item){
+        incrementQuality(item, -1);
+        decrementSellIn(item);
+    }
+
+    public boolean sellInExpired(Item item){
+        if(item.getSellIn()<=0){
+            return true;
+        } else {
+            return false;
         }
     }
+
+    public void incrementQuality(Item item, int incr){
+        int currentQual = item.getQuality() + incr;
+        if(currentQual < 50 && currentQual >= 0) {
+            item.setQuality(currentQual);
+        } else{
+            item.setQuality(50);
+        }
+    }
+
+    public void decrementSellIn(Item item){
+        item.setSellIn(item.getSellIn()-1);
+    }
 }

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

@@ -4,9 +4,189 @@ import org.junit.Assert;
 import org.junit.Test;
 
 public class InventoryTest {
+
+    @Test
+    public void setQualityTest(){
+        //Given
+        Item rand = new Item("rand", 10, 10);
+        int expVal = 42;
+
+        //When
+        rand.setQuality(42);
+
+        //Then
+        int actVal = rand.getQuality();
+        Assert.assertEquals(expVal, actVal);
+    }
+
+    @Test
+    public void setNameTest(){
+        //Given
+        Item rand = new Item("rand", 10, 10);
+        String expString = "random";
+
+        //When
+        rand.setName(expString);
+
+        //Then
+        String actString = rand.getName();
+        Assert.assertEquals(expString, actString);
+    }
+
+    @Test
+    public void agedBrie50Quality(){
+        //Given
+        Item brie = new Item("Aged Brie", 2, 50);
+        Item[] items = new Item[1];
+        items[0] = brie;
+        Inventory inv = new Inventory(items);
+        int exp = 50;
+
+        //When
+        inv.updateQuality();
+        int act = brie.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+
+    @Test
+    public void agedBrieExpireTest(){
+        //Given
+        Item brie = new Item("Aged Brie", -1, 39);
+        Item[] items = new Item[1];
+        items[0] = brie;
+        Inventory inv = new Inventory(items);
+        int exp = 41;
+
+        //When
+        inv.updateQuality();
+        int act = brie.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+    public void passQualityWith5Days(){
+        //Given
+        Item pass = new Item("Backstage passes to a TAFKAL80ETC concert", 5, 39);
+        Item[] items = new Item[1];
+        items[0] = pass;
+        Inventory inv = new Inventory(items);
+        int exp = 42;
+
+        //When
+        inv.updateQuality();
+        int act = pass.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+    public void passQualityWith11Days(){
+        //Given
+        Item pass = new Item("Backstage passes to a TAFKAL80ETC concert", 11, 39);
+        Item[] items = new Item[1];
+        items[0] = pass;
+        Inventory inv = new Inventory(items);
+        int exp = 40;
+
+        //When
+        inv.updateQuality();
+        int act = pass.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+    public void passQualityWith0Days(){
+        //Given
+        Item pass = new Item("Backstage passes to a TAFKAL80ETC concert", 0, 39);
+        Item[] items = new Item[1];
+        items[0] = pass;
+        Inventory inv = new Inventory(items);
+        int exp = 0;
+
+        //When
+        inv.updateQuality();
+        int act = pass.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+    public void passQualityWith10Days(){
+        //Given
+        Item pass = new Item("Backstage passes to a TAFKAL80ETC concert", 10, 39);
+        Item[] items = new Item[1];
+        items[0] = pass;
+        Inventory inv = new Inventory(items);
+        int exp = 41;
+
+        //When
+        inv.updateQuality();
+        int act = pass.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
     @Test
-    public void updateQuantityTest(){
-        Assert.assertEquals(1, 1);
+    public void sulfurasTest(){
+        //Given
+        Item sulfuras = new Item("Sulfuras, Hand of Ragnaros", 17, 80);
+        Item[] items = new Item[1];
+        items[0] =sulfuras;
+        Inventory inv = new Inventory(items);
+        int exp = 80;
+
+        //When
+        inv.updateQuality();
+        int act = sulfuras.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+
+    public void conjureTest(){
+        //Given
+        Item conjure = new Item("Conjured Item", 17, 25);
+        Item[] items = new Item[1];
+        items[0] = conjure;
+        Inventory inv = new Inventory(items);
+        int exp = 23;
+
+        //When
+        inv.updateQuality();
+        int act = conjure.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
+    }
+
+    @Test
+
+    public void defaultTest(){
+        //Given
+        Item rand = new Item("randName", 17, 25);
+        Item[] items = new Item[1];
+        items[0] = rand;
+        Inventory inv = new Inventory(items);
+        int exp = 24;
+
+        //When
+        inv.updateQuality();
+        int act = rand.getQuality();
+
+        //Then
+        Assert.assertEquals(exp, act);
     }
 }