Clean up the code

js-jest/src/gilded_rose.js

@@ -5,32 +5,6 @@ class Item {
     this.quality = quality;
   }
 }
-
-class ItemUpdater {
-  constructor(item) {
-    this.item = item;
-    this.qualityChangeFactor = 1;
-  }
-
-  updateQuality() {
-
-    //Once the sell by date has passed, Quality degrades twice as fast
-    if(this.item.sellIn < 0) {
-      this.qualityChangeFactor = 2;
-    }
-
-    //The Quality of an item is never more than 50
-    if (this.item.quality > 50) {
-      this.item.quality = 50;
-    }
-
-    if (this.item.quality > 0 && this.item.quality < 50) {
-      this.item.quality = this.item.quality - this.qualityChangeFactor;
-    }
-    return this.item;
-  }
-}
-
 class Shop {
   constructor(itemUpdaters=[]){
     this.itemUpdaters = itemUpdaters;
@@ -50,6 +24,5 @@ class Shop {
 
 module.exports = {
   Item,
-  Shop,
+  Shop
-  ItemUpdater
 }

js-jest/src/item_updater.js

@@ -0,0 +1,28 @@
+class ItemUpdater {
+    constructor(item) {
+      this.item = item;
+      this.qualityChangeFactor = 1;
+    }
+  
+    updateQuality() {
+  
+      //Once the sell by date has passed, Quality degrades twice as fast
+      if(this.item.sellIn < 0) {
+        this.qualityChangeFactor = 2;
+      }
+  
+      //The Quality of an item is never more than 50
+      if (this.item.quality > 50) {
+        this.item.quality = 50;
+      }
+  
+      if (this.item.quality > 0 && this.item.quality < 50) {
+        this.item.quality = this.item.quality - this.qualityChangeFactor;
+      }
+      return this.item;
+    }
+  }
+
+  module.exports = {
+    ItemUpdater
+  }

js-jest/src/item_updaters/aged_brie_updater.js

@@ -1,4 +1,4 @@
-const { ItemUpdater } = require("../gilded_rose");
+const { ItemUpdater } = require("../item_updater");
 
 class AgedBrieUpdater extends ItemUpdater {
     updateQuality() {

js-jest/src/item_updaters/backstage_passes_updater.js

@@ -1,4 +1,4 @@
-const { ItemUpdater } = require("../gilded_rose");
+const { ItemUpdater } = require("../item_updater");
 
 class BackStagePassesUpdater extends ItemUpdater {
     updateQuality() {

js-jest/src/item_updaters/conjured_updater.js

@@ -1,4 +1,4 @@
-const { ItemUpdater } = require("../gilded_rose");
+const { ItemUpdater } = require("../item_updater");
 
 class ConjuredUpdater extends ItemUpdater {
     updateQuality() {

js-jest/test/gilded_rose.test.js

@@ -1,8 +1,9 @@
-const {Shop, Item, ItemUpdater} = require("../src/gilded_rose");
+const {Shop, Item} = require("../src/gilded_rose");
 const {AgedBrieUpdater} = require("../src/item_updaters/aged_brie_updater");
 const {SulfurasUpdater } = require("../src/item_updaters/sulfuras_updater");
 const {BackStagePassesUpdater } = require("../src/item_updaters/backstage_passes_updater");
 const {ConjuredUpdater } = require("../src/item_updaters/conjured_updater");
+const {ItemUpdater } = require("../src/item_updater");
 
 describe("Gilded Rose check common rules", function () {
   it("Item quality should decrease by one", function () {
@@ -86,10 +87,17 @@ describe("Gilded Rose check Conjured rules", function () {
 });
 
 describe("Gilded Rose system test", function() {
-  it("should check list of itemUpater", function() {
+  it("should check list of itemUpaters", function () {
     const itemUpdaters = [
       new ItemUpdater(new Item("+5 Dexterity Vest", 10, 20)),
       new AgedBrieUpdater(new Item("Aged Brie", 2, 0)),
+      new ItemUpdater(new Item("Elixir of the Mongoose", 5, 7)),
+      new SulfurasUpdater(new Item("Sulfuras, Hand of Ragnaros", 0, 80)),
+      new SulfurasUpdater(new Item("Sulfuras, Hand of Ragnaros", -1, 80)),
+      new BackStagePassesUpdater(new Item("Backstage passes to a TAFKAL80ETC concert", 15, 20)),
+      new BackStagePassesUpdater(new Item("Backstage passes to a TAFKAL80ETC concert", 10, 49)),
+      new BackStagePassesUpdater(new Item("Backstage passes to a TAFKAL80ETC concert", 5, 49)),
+      new ConjuredUpdater(new Item("Conjured Mana Cake", 3, 6)),
     ];
 
     const days = Number(process.argv[2]) || 2;