cognizant code refactor exercise

js-jest/src/gilded_rose.js

@@ -1,67 +1,101 @@
 class Item {
-  constructor(name, sellIn, quality){
+  constructor(name, sellIn, quality) {
     this.name = name;
     this.sellIn = sellIn;
     this.quality = quality;
   }
 }
 
+const itemType = {
+  PASS: "Backstage passes to a TAFKAL80ETC concert",
+  BRIE: "Aged Brie",
+  SULFURAS: "Sulfuras, Hand of Ragnaros",
+  CONJURED: "Conjured Mana Cake",
+};
+
+function updateBrie(item) {
+  if (item.quality < 50) {
+    item.quality = item.quality + 1;
+  }
+  item.sellIn = item.sellIn - 1;
+  if (item.sellIn < 0 && item.quality < 50) {
+    item.quality = item.quality + 1;
+  }
+  lessAndGreaterThan(item);
+}
+function updateHand(item) {}
+function updateNormal(item) {
+  item.sellIn = item.sellIn - 1;
+  if (item.quality > 0) {
+    item.quality = item.quality - 1;
+  }
+
+  if (item.sellIn < 0 && item.quality > 0) {
+    item.quality = item.quality - 1;
+  }
+  lessAndGreaterThan(item);
+}
+
+function updatePass(item) {
+  if (item.quality < 50) {
+    item.quality = item.quality + 1;
+    if (item.sellIn < 11 && item.quality < 50) {
+      item.quality = item.quality + 1;
+    }
+    if (item.sellIn < 6 && item.quality < 50) {
+      item.quality = item.quality + 1;
+    }
+  }
+  item.sellIn = item.sellIn - 1;
+  if (item.sellIn < 0) {
+    item.quality = item.quality - item.quality;
+  }
+  lessAndGreaterThan(item);
+}
+
+const lessAndGreaterThan = (item) => {
+  if (item.quality > 50) item.quality = 50;
+  if (item.quality < 0) item.quality = 0;
+};
+
 class Shop {
-  constructor(items=[]){
+  constructor(items = []) {
     this.items = items;
   }
+
   updateQuality() {
-    for (let i = 0; i < this.items.length; i++) {
-      if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage passes to a TAFKAL80ETC concert') {
-        if (this.items[i].quality > 0) {
-          if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
-            this.items[i].quality = this.items[i].quality - 1;
-          }
-        }
-      } else {
-        if (this.items[i].quality < 50) {
-          this.items[i].quality = this.items[i].quality + 1;
-          if (this.items[i].name == 'Backstage passes to a TAFKAL80ETC concert') {
-            if (this.items[i].sellIn < 11) {
-              if (this.items[i].quality < 50) {
-                this.items[i].quality = this.items[i].quality + 1;
-              }
-            }
-            if (this.items[i].sellIn < 6) {
-              if (this.items[i].quality < 50) {
-                this.items[i].quality = this.items[i].quality + 1;
-              }
-            }
-          }
-        }
-      }
-      if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
-        this.items[i].sellIn = this.items[i].sellIn - 1;
-      }
-      if (this.items[i].sellIn < 0) {
-        if (this.items[i].name != 'Aged Brie') {
-          if (this.items[i].name != 'Backstage passes to a TAFKAL80ETC concert') {
-            if (this.items[i].quality > 0) {
-              if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
-                this.items[i].quality = this.items[i].quality - 1;
-              }
-            }
-          } else {
-            this.items[i].quality = this.items[i].quality - this.items[i].quality;
-          }
-        } else {
-          if (this.items[i].quality < 50) {
-            this.items[i].quality = this.items[i].quality + 1;
-          }
-        }
+    for (const item of this.items) {
+      // Set a degradation multiplier to 2 if expiration date has passed
+      // Otherwise set it to 1 (no multiplier)
+
+      let degradationMultiplier = item.sellIn < 0 ? 2 : 1;
+
+      switch (item.name) {
+        case itemType.BRIE:
+          updateBrie(item);
+          continue;
+        case itemType.PASS:
+          updatePass(item);
+          continue;
+        case itemType.SULFURAS:
+          continue;
+        case itemType.CONJURED:
+          item.quality -= 2 * degradationMultiplier;
+          item.sellIn--;
+          lessAndGreaterThan(item);
+          continue;
+        default:
+          item.quality -= 1 * degradationMultiplier;
+          item.sellIn--;
+          lessAndGreaterThan(item);
+          continue;
       }
     }
 
     return this.items;
   }
 }
-
 module.exports = {
   Item,
-  Shop
-}
+  Shop,
+};

js-jest/test/gilded_rose.test.js

@@ -1,9 +1,94 @@
-const {Shop, Item} = require("../src/gilded_rose");
+const { Shop, Item } = require("../src/gilded_rose");
 
-describe("Gilded Rose", function() {
-  it("should foo", function() {
-    const gildedRose = new Shop([new Item("foo", 0, 0)]);
-    const items = gildedRose.updateQuality();
-    expect(items[0].name).toBe("fixme");
+const gildedRose = new Shop([
+  new Item("+5 Dexterity Vest", 10, 20),
+  new Item("Aged Brie", 2, 0),
+  new Item("Sulfuras, Hand of Ragnaros", 0, 35),
+  new Item("Backstage passes to a TAFKAL80ETC concert", 15, 20),
+  new Item("Backstage passes to a TAFKAL80ETC concert", 10, 45),
+  new Item("Backstage passes to a TAFKAL80ETC concert", 5, 32),
+  new Item("Conjured Mana Cake", 3, 6),
+  new Item("Expired Pears", -1, 11),
+  new Item("Overpriced Trinket", 10, 55),
+  new Item("Conjured Mana Cake", -3, 6),
+]);
+
+const items = gildedRose.updateQuality();
+
+describe("After one updateQuality call", function () {
+  it("Miscellaneous item should have decremented sellIn by 1", function () {
+    expect(items[0].sellIn).toBe(9);
+  });
+
+  it("Miscellaneous item should have decremented quality by 1", function () {
+    expect(items[0].quality).toBe(19);
+  });
+
+  it("Expired item should have decremented quality by 2", function () {
+    expect(items[7].quality).toBe(9);
+  });
+
+  it("Overpriced item should have reset quality to 50", function () {
+    expect(items[8].quality).toBe(50);
+  });
+});
+
+describe("After one updateQuality call, Aged Brie", function () {
+  it("should have decremented sellIn by 1", function () {
+    expect(items[1].sellIn).toBe(1);
+  });
+
+  it("should have incremented quality by 1", function () {
+    expect(items[1].quality).toBe(1);
+  });
+});
+
+describe("After one updateQuality call, Sulfuras", function () {
+  it("should have same sellIn", function () {
+    expect(items[2].sellIn).toBe(0);
+  });
+
+  it("should have same quality", function () {
+    expect(items[2].quality).toBe(35);
+  });
+});
+
+describe("After one updateQuality call", function () {
+  it("First backstage pass should have decremented sellIn by 1", function () {
+    expect(items[3].sellIn).toBe(14);
+  });
+
+  it("First backstage pass should have incremented quality by 1", function () {
+    expect(items[3].quality).toBe(21);
+  });
+
+  it("Second backstage pass should have decremented sellIn by 1", function () {
+    expect(items[4].sellIn).toBe(9);
+  });
+
+  it("Second backstage pass should have incremented quality by 2", function () {
+    expect(items[4].quality).toBe(47);
+  });
+
+  it("Third backstage pass should have decremented sellIn by 1", function () {
+    expect(items[5].sellIn).toBe(4);
+  });
+
+  it("Third backstage pass should have incremented quality by 3", function () {
+    expect(items[5].quality).toBe(35);
+  });
+});
+
+describe("After one updateQuality call, Conjured item", function () {
+  it("should have decremented sellIn by 1", function () {
+    expect(items[6].sellIn).toBe(2);
+  });
+
+  it("should have decremented quality by 2", function () {
+    expect(items[6].quality).toBe(4);
+  });
+
+  it("should have decremented quality by 4 if expired", function () {
+    expect(items[9].quality).toBe(2);
   });
 });