mirror of
https://github.com/emilybache/GildedRose-Refactoring-Kata.git
synced 2026-02-16 06:51:27 +00:00
Fix test
update variable naming use String.includes for choosing a category from item name
This commit is contained in:
parent
9653512287
commit
ac49fc2888
@ -19,46 +19,51 @@ export class GildedRose {
|
||||
|
||||
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
|
||||
let currentItem = this.items[i];
|
||||
let itemName = currentItem.name;
|
||||
let itemQuality = currentItem.quality;
|
||||
let itemSellin = currentItem.sellIn;
|
||||
if (!itemName.includes('Sulfuras')) {
|
||||
itemSellin = itemSellin - 1;
|
||||
}
|
||||
|
||||
if (!itemName.includes('Aged Brie') && !itemName.includes('Backstage passes')) {
|
||||
if (itemQuality > 0) {
|
||||
if (!itemName.includes('Sulfuras')) {
|
||||
itemQuality = itemQuality - 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 (itemQuality < 50) {
|
||||
itemQuality = itemQuality + 1
|
||||
if (itemName.includes('Backstage passes')) {
|
||||
if (itemSellin < 11) {
|
||||
if (itemQuality < 50) {
|
||||
itemQuality = itemQuality + 1
|
||||
}
|
||||
}
|
||||
if (this.items[i].sellIn < 6) {
|
||||
if (this.items[i].quality < 50) {
|
||||
this.items[i].quality = this.items[i].quality + 1
|
||||
if (itemSellin < 6) {
|
||||
if (itemQuality < 50) {
|
||||
itemQuality = itemQuality + 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
|
||||
if (itemSellin < 0) {
|
||||
if (!itemName.includes('Aged Brie')) {
|
||||
if (!itemName.includes('Backstage passes')) {
|
||||
if (itemQuality > 0) {
|
||||
if (!itemName.includes('Sulfuras, Hand of Ragnaros')) {
|
||||
itemQuality = itemQuality - 1
|
||||
}
|
||||
}
|
||||
} else {
|
||||
this.items[i].quality = this.items[i].quality - this.items[i].quality
|
||||
itemQuality = itemQuality - itemQuality
|
||||
}
|
||||
} else {
|
||||
if (this.items[i].quality < 50) {
|
||||
this.items[i].quality = this.items[i].quality + 1
|
||||
if (itemQuality < 50) {
|
||||
itemQuality = itemQuality + 1
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@ -10,7 +10,8 @@ const items = [
|
||||
new Item("Backstage passes to a TAFKAL80ETC concert", 10, 49),
|
||||
new Item("Backstage passes to a TAFKAL80ETC concert", 5, 49),
|
||||
// this conjured item does not work properly yet
|
||||
new Item("Conjured Mana Cake", 3, 6)];
|
||||
new Item("Conjured Mana Cake", 3, 6)
|
||||
];
|
||||
|
||||
|
||||
const gildedRose = new GildedRose(items);
|
||||
|
||||
@ -4,6 +4,6 @@ describe('Gilded Rose', () => {
|
||||
it('should foo', () => {
|
||||
const gildedRose = new GildedRose([new Item('foo', 0, 0)]);
|
||||
const items = gildedRose.updateQuality();
|
||||
expect(items[0].name).toBe('fixme');
|
||||
expect(items[0].name).toBe('foo');
|
||||
});
|
||||
});
|
||||
|
||||
Loading…
Reference in New Issue
Block a user