Added quality factor for the sellIn date and separate item updater classes

js-jest/src/gilded_rose.js

@@ -12,8 +12,21 @@ class ItemUpdater {
   }
 
   updateQuality() {
-    if (this.item.quality > 0 && this.item.quality <= 50) {
+
-      this.item.quality = this.item.quality - 1;
+    let qualityChangeFactor = 1;
+
+    //Once the sell by date has passed, Quality degrades twice as fast
+    if(this.item.sellIn < 0) {
+      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 - qualityChangeFactor;
     }
     return this.item;
   }

js-jest/src/item_updaters/aged_brie_updater.js

@@ -0,0 +1,21 @@
+const { ItemUpdater } = require("../gilded_rose");
+
+class AgedBrieUpdater extends ItemUpdater {
+    updateQuality() {
+        
+        //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 + 1;
+        }
+
+        return this.item;
+    }
+}
+
+module.exports = {
+    AgedBrieUpdater
+}

js-jest/test/gilded_rose.test.js

@@ -1,4 +1,5 @@
 const {Shop, Item, ItemUpdater} = require("../src/gilded_rose");
+const {AgedBrieUpdater} = require("../src/item_updaters/aged_brie_updater");
 
 describe("Gilded Rose check common rules", function () {
   it("Item quality should decrease by one", function () {
@@ -13,12 +14,34 @@ describe("Gilded Rose check common rules", function () {
     expect(item.quality).toBe(0);
   });
 
+  it("Item quality should decrease by two ", function () {
+    const gildedRose = new ItemUpdater(new Item("+5 Dexterity Vest", -10, 20));
+    const item = gildedRose.updateQuality();
+    expect(item.quality).toBe(18);
+  });
+
+});
+
+describe("Gilded Rose check Aged Brie rules", function () {
+  it("Item quality should increase by one", function () {
+    const updater = new AgedBrieUpdater(new Item("Aged Brie", 10, 20));
+    const item = updater.updateQuality();
+    expect(item.quality).toBe(21);
+  });
+
+  it("Item quality should not more than 50", function () {
+    const updater = new AgedBrieUpdater(new Item("Aged Brie", 10, 70));
+    const item = updater.updateQuality();
+    expect(item.quality).toBe(50);
+  });
+
 });
 
 describe("Gilded Rose system test", function() {
   it("should check list of itemUpater", function() {
     const itemUpdaters = [
       new ItemUpdater(new Item("+5 Dexterity Vest", 10, 20)),
+      new AgedBrieUpdater(new Item("Aged Brie", 2, 0)),
     ];
     
     const days = Number(process.argv[2]) || 2;