Merge branch 'master' of https://git.zipcode.rocks/woat/CleanCode-GildedRoseKata

.gitignore

@@ -0,0 +1,179 @@
+# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and Webstorm
+# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
+
+# User-specific stuff:
+.idea/**/workspace.xml
+.idea/**/tasks.xml
+.idea/dictionaries
+
+# Sensitive or high-churn files:
+.idea/**/dataSources/
+.idea/**/dataSources.ids
+.idea/**/dataSources.xml
+.idea/**/dataSources.local.xml
+.idea/**/sqlDataSources.xml
+.idea/**/dynamic.xml
+.idea/**/uiDesigner.xml
+
+# Gradle:
+.idea/**/gradle.xml
+.idea/**/libraries
+
+# CMake
+cmake-build-debug/
+
+# Mongo Explorer plugin:
+.idea/**/mongoSettings.xml
+
+## File-based project format:
+*.iws
+
+## Plugin-specific files:
+
+# IntelliJ
+/out/
+
+# mpeltonen/sbt-idea plugin
+.idea_modules/
+# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and Webstorm
+# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
+
+# User-specific stuff:
+*.iml
+.idea/**
+.idea/**/workspace.xml
+.idea/**/tasks.xml
+.idea/dictionaries
+
+# Sensitive or high-churn files:
+.idea/**/dataSources/
+.idea/**/dataSources.ids
+.idea/**/dataSources.xml
+.idea/**/dataSources.local.xml
+.idea/**/sqlDataSources.xml
+.idea/**/dynamic.xml
+.idea/**/uiDesigner.xml
+
+# Gradle:
+.idea/**/gradle.xml
+.idea/**/libraries
+
+# CMake
+cmake-build-debug/
+
+# Mongo Explorer plugin:
+.idea/**/mongoSettings.xml
+
+## File-based project format:
+*.iws
+
+## Plugin-specific files:
+
+# IntelliJ
+/out/
+
+# mpeltonen/sbt-idea plugin
+.idea_modules/
+
+# JIRA plugin
+atlassian-ide-plugin.xml
+
+# Cursive Clojure plugin
+.idea/replstate.xml
+
+# Crashlytics plugin (for Android Studio and IntelliJ)
+com_crashlytics_export_strings.xml
+crashlytics.properties
+crashlytics-build.properties
+fabric.properties# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio and Webstorm
+                 # Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
+
+                 # User-specific stuff:
+                 .idea/**/workspace.xml
+                 .idea/**/tasks.xml
+                 .idea/dictionaries
+
+                 # Sensitive or high-churn files:
+                 .idea/**/dataSources/
+                 .idea/**/dataSources.ids
+                 .idea/**/dataSources.xml
+                 .idea/**/dataSources.local.xml
+                 .idea/**/sqlDataSources.xml
+                 .idea/**/dynamic.xml
+                 .idea/**/uiDesigner.xml
+
+                 # Gradle:
+                 .idea/**/gradle.xml
+                 .idea/**/libraries
+
+                 # CMake
+                 cmake-build-debug/
+
+                 # Mongo Explorer plugin:
+                 .idea/**/mongoSettings.xml
+
+                 ## File-based project format:
+                 *.iws
+
+                 ## Plugin-specific files:
+
+                 # IntelliJ
+                 /out/
+
+                 # mpeltonen/sbt-idea plugin
+                 .idea_modules/
+
+                 # JIRA plugin
+                 atlassian-ide-plugin.xml
+
+                 # Cursive Clojure plugin
+                 .idea/replstate.xml
+
+                 # Crashlytics plugin (for Android Studio and IntelliJ)
+                 com_crashlytics_export_strings.xml
+                 crashlytics.properties
+                 crashlytics-build.properties
+                 fabric.properties
+# JIRA plugin
+atlassian-ide-plugin.xml
+
+# Cursive Clojure plugin
+.idea/replstate.xml
+
+# Crashlytics plugin (for Android Studio and IntelliJ)
+com_crashlytics_export_strings.xml
+crashlytics.properties
+crashlytics-build.properties
+fabric.properties
+
+.project
+.classpath
+.settings
+
+
+#maven build target
+target/
+
+# ---> Java
+# Compiled class file
+*.class
+
+# Log file
+*.log
+
+# BlueJ files
+*.ctxt
+
+# Mobile Tools for Java (J2ME)
+.mtj.tmp/
+
+# Package Files #
+*.jar
+*.war
+*.ear
+*.zip
+*.tar.gz
+*.rar
+
+# virtual machine crash logs, see http://www.java.com/en/download/help/error_hotspot.xml
+hs_err_pid*

Inventory.java

@@ -0,0 +1,39 @@
+
+public class Inventory {
+    private Item[] items;
+
+    public Inventory(Item[] items) {
+        super();
+        this.items = items;
+    }
+
+    public void updateQuality() {
+        for (int i = 0; i < items.length; i++) {
+            if (!items[i].getName().equals("Aged Brie")) {
+                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().equals("Sulfuras, Hand of Ragnaros")) {
+                items[i].setSellIn(items[i].getSellIn() - 1);
+            }
+
+            if (items[i].getSellIn() < 0) {
+                if (!items[i].getName().equals("Aged Brie")) {
+                  items[i].setQuality(items[i].getQuality() - 1);
+                } else {
+                    if (items[i].getQuality() < 50) {
+                        items[i].setQuality(items[i].getQuality() + 1);
+                    }
+                }
+            }
+        }
+}
+}

InventoryTest.java

@@ -0,0 +1,122 @@
+import static org.junit.Assert.*;
+import java.util.Arrays;
+import org.junit.Test;
+import org.junit.Before;
+
+public class InventoryTest {
+    private static final int MAX_QUALITY = 50;
+    private static final String AGED_BRIE_NAME = "Aged Brie";
+    private static final String BACKSTAGE_PASS_NAME = "Backstage passes to a TAFKAL80ETC concert";
+    private static final String NORMAL_ITEM_NAME = "+5 Dexterity Vest";
+
+    private Item sulfuras;
+    private Item normalItem;
+
+    @Before
+    public void setup(){
+        sulfuras = new Item("Sulfuras, Hand of Ragnaros", 0, 80);
+        normalItem = new Item(NORMAL_ITEM_NAME, 10, 20);
+    }
+
+    @Test
+    public void testUpdate_ForSulfurasQuality() throws Exception {
+        int expectedQuality = sulfuras.getQuality();
+        Item[] items = {sulfuras};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedQuality, sulfuras.getQuality());
+    }
+    @Test
+    public void testUpdate_ForSulfurasSellIn() throws Exception {
+        int expectedSellIn = sulfuras.getSellIn();
+        Item[] items = {sulfuras};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedSellIn, sulfuras.getSellIn());
+    }
+
+    @Test
+    public void testUpdate_ForNormalItemSellIn() throws Exception {
+        int expectedSellIn = normalItem.getSellIn() - 1;
+        Item[] items = {normalItem};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedSellIn, normalItem.getSellIn());
+    }
+
+    @Test
+    public void testUpdate_ForNormalItemQuantity() throws Exception {
+        int expectedQuality = normalItem.getQuality() - 1;
+        Item[] items = {normalItem};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedQuality, normalItem.getQuality());
+    }
+
+    @Test
+    public void testUpdate_ForSoldOutItem() throws Exception {
+        int minimumQuality = 0;
+        Item normalItemWithMinimumQuality = new Item(NORMAL_ITEM_NAME, 10, minimumQuality);
+        Item[] items = {normalItemWithMinimumQuality};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(minimumQuality, normalItemWithMinimumQuality.getQuality());
+    }
+
+    @Test
+    public void testUpdate_ForNormalItemOnceTheSellInDatePassed() throws Exception {
+        Item normalItemWithPassedSellDate = new Item(NORMAL_ITEM_NAME, -1, 25);
+        int expectedQuality = normalItemWithPassedSellDate.getQuality() - 2;
+        Item[] items = {normalItemWithPassedSellDate};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedQuality, normalItemWithPassedSellDate.getQuality());
+    }
+
+    @Test
+    public void testUpdate_AgedBrie() throws Exception {
+        Item agedBrie = new Item(AGED_BRIE_NAME, 10, 25);
+        int expectedQuality = agedBrie.getQuality() + 1;
+        Item[] items = {agedBrie};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedQuality, agedBrie.getQuality());
+    }
+
+    @Test
+    public void testUpdate_WhenAgedBrieIsAtMaxQuality() throws Exception {
+        Item agedBrie = new Item(AGED_BRIE_NAME, 10, MAX_QUALITY);
+        Item[] items = {agedBrie};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(MAX_QUALITY, agedBrie.getQuality());
+    }
+
+    @Test
+    public void testUpdate_WhenAgedBrieOnceTheSellInDatePassed() throws Exception {
+        Item agedBrie = new Item(AGED_BRIE_NAME, -1, 20);
+        int expectedQuality = agedBrie.getQuality() + 2;
+        Item[] items = {agedBrie};
+        Inventory sut = new Inventory(items);
+
+        sut.updateQuality();
+
+        assertEquals(expectedQuality, agedBrie.getQuality());
+    }
+}

Item.java

@@ -0,0 +1,36 @@
+public class Item {
+    private String name;
+    private int sellIn;
+    private int quality;
+
+    public Item(String name, int sellIn, int quality) {
+        super();
+        this.name = name;
+        this.sellIn = sellIn;
+        this.quality = quality;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public int getSellIn() {
+        return sellIn;
+    }
+
+    public void setSellIn(int sellIn) {
+        this.sellIn = sellIn;
+    }
+
+    public int getQuality() {
+        return quality;
+    }
+
+    public void setQuality(int quality) {
+        this.quality = quality;
+    }
+}

package.bluej

@@ -0,0 +1,53 @@
+#BlueJ package file
+dependency1.from=InventoryTest
+dependency1.to=Item
+dependency1.type=UsesDependency
+dependency2.from=InventoryTest
+dependency2.to=Inventory
+dependency2.type=UsesDependency
+editor.fx.0.height=722
+editor.fx.0.width=800
+editor.fx.0.x=320
+editor.fx.0.y=50
+objectbench.height=164
+objectbench.width=776
+package.divider.horizontal=0.6
+package.divider.vertical=0.6845018450184502
+package.editor.height=364
+package.editor.width=674
+package.editor.x=577
+package.editor.y=61
+package.frame.height=600
+package.frame.width=800
+package.numDependencies=2
+package.numTargets=3
+package.showExtends=true
+package.showUses=true
+project.charset=UTF-8
+readme.height=58
+readme.name=@README
+readme.width=47
+readme.x=10
+readme.y=10
+target1.height=50
+target1.name=Item
+target1.showInterface=false
+target1.type=ClassTarget
+target1.width=80
+target1.x=10
+target1.y=100
+target2.height=50
+target2.name=InventoryTest
+target2.showInterface=false
+target2.type=UnitTestTargetJunit4
+target2.width=80
+target2.x=40
+target2.y=160
+target3.association=InventoryTest
+target3.height=50
+target3.name=Inventory
+target3.showInterface=false
+target3.type=ClassTarget
+target3.width=80
+target3.x=10
+target3.y=190