From c7863fc4de7be3470bd69e4e4305131cb3a3b04b Mon Sep 17 00:00:00 2001 From: Koleh David Date: Thu, 6 Feb 2025 16:38:04 +1030 Subject: [PATCH] t finish feature --- gilded_rose.py | 4 +- ...lded_rose.test_update_quality.approved.txt | 48 +++++++++---------- 2 files changed, 26 insertions(+), 26 deletions(-) diff --git a/gilded_rose.py b/gilded_rose.py index 0bbb8e84..e5d414a7 100755 --- a/gilded_rose.py +++ b/gilded_rose.py @@ -28,11 +28,11 @@ def update_item(item: Item): return if item.quality > 0: - item.quality = item.quality - 1 + item.quality = item.quality - (1 if item.name != "Conjured" else 2) item.sell_in = item.sell_in - 1 if item.sell_in < 0: if item.quality > 0: - item.quality = item.quality - 1 + item.quality = item.quality - (1 if item.name != "Conjured" else 2) class GildedRose: diff --git a/test_gilded_rose.test_update_quality.approved.txt b/test_gilded_rose.test_update_quality.approved.txt index 15ed8040..ab7bffb0 100644 --- a/test_gilded_rose.test_update_quality.approved.txt +++ b/test_gilded_rose.test_update_quality.approved.txt @@ -36,40 +36,40 @@ args: ('Normal', 12, 50) => 'Normal, 11, 49' args: ('Normal', 12, 51) => 'Normal, 11, 50' args: ('Conjured', -1, -1) => 'Conjured, -2, -1' args: ('Conjured', -1, 0) => 'Conjured, -2, 0' -args: ('Conjured', -1, 1) => 'Conjured, -2, 0' -args: ('Conjured', -1, 49) => 'Conjured, -2, 47' -args: ('Conjured', -1, 50) => 'Conjured, -2, 48' -args: ('Conjured', -1, 51) => 'Conjured, -2, 49' +args: ('Conjured', -1, 1) => 'Conjured, -2, -1' +args: ('Conjured', -1, 49) => 'Conjured, -2, 45' +args: ('Conjured', -1, 50) => 'Conjured, -2, 46' +args: ('Conjured', -1, 51) => 'Conjured, -2, 47' args: ('Conjured', 0, -1) => 'Conjured, -1, -1' args: ('Conjured', 0, 0) => 'Conjured, -1, 0' -args: ('Conjured', 0, 1) => 'Conjured, -1, 0' -args: ('Conjured', 0, 49) => 'Conjured, -1, 47' -args: ('Conjured', 0, 50) => 'Conjured, -1, 48' -args: ('Conjured', 0, 51) => 'Conjured, -1, 49' +args: ('Conjured', 0, 1) => 'Conjured, -1, -1' +args: ('Conjured', 0, 49) => 'Conjured, -1, 45' +args: ('Conjured', 0, 50) => 'Conjured, -1, 46' +args: ('Conjured', 0, 51) => 'Conjured, -1, 47' args: ('Conjured', 1, -1) => 'Conjured, 0, -1' args: ('Conjured', 1, 0) => 'Conjured, 0, 0' -args: ('Conjured', 1, 1) => 'Conjured, 0, 0' -args: ('Conjured', 1, 49) => 'Conjured, 0, 48' -args: ('Conjured', 1, 50) => 'Conjured, 0, 49' -args: ('Conjured', 1, 51) => 'Conjured, 0, 50' +args: ('Conjured', 1, 1) => 'Conjured, 0, -1' +args: ('Conjured', 1, 49) => 'Conjured, 0, 47' +args: ('Conjured', 1, 50) => 'Conjured, 0, 48' +args: ('Conjured', 1, 51) => 'Conjured, 0, 49' args: ('Conjured', 10, -1) => 'Conjured, 9, -1' args: ('Conjured', 10, 0) => 'Conjured, 9, 0' -args: ('Conjured', 10, 1) => 'Conjured, 9, 0' -args: ('Conjured', 10, 49) => 'Conjured, 9, 48' -args: ('Conjured', 10, 50) => 'Conjured, 9, 49' -args: ('Conjured', 10, 51) => 'Conjured, 9, 50' +args: ('Conjured', 10, 1) => 'Conjured, 9, -1' +args: ('Conjured', 10, 49) => 'Conjured, 9, 47' +args: ('Conjured', 10, 50) => 'Conjured, 9, 48' +args: ('Conjured', 10, 51) => 'Conjured, 9, 49' args: ('Conjured', 11, -1) => 'Conjured, 10, -1' args: ('Conjured', 11, 0) => 'Conjured, 10, 0' -args: ('Conjured', 11, 1) => 'Conjured, 10, 0' -args: ('Conjured', 11, 49) => 'Conjured, 10, 48' -args: ('Conjured', 11, 50) => 'Conjured, 10, 49' -args: ('Conjured', 11, 51) => 'Conjured, 10, 50' +args: ('Conjured', 11, 1) => 'Conjured, 10, -1' +args: ('Conjured', 11, 49) => 'Conjured, 10, 47' +args: ('Conjured', 11, 50) => 'Conjured, 10, 48' +args: ('Conjured', 11, 51) => 'Conjured, 10, 49' args: ('Conjured', 12, -1) => 'Conjured, 11, -1' args: ('Conjured', 12, 0) => 'Conjured, 11, 0' -args: ('Conjured', 12, 1) => 'Conjured, 11, 0' -args: ('Conjured', 12, 49) => 'Conjured, 11, 48' -args: ('Conjured', 12, 50) => 'Conjured, 11, 49' -args: ('Conjured', 12, 51) => 'Conjured, 11, 50' +args: ('Conjured', 12, 1) => 'Conjured, 11, -1' +args: ('Conjured', 12, 49) => 'Conjured, 11, 47' +args: ('Conjured', 12, 50) => 'Conjured, 11, 48' +args: ('Conjured', 12, 51) => 'Conjured, 11, 49' args: ('Aged Brie', -1, -1) => 'Aged Brie, -2, 1' args: ('Aged Brie', -1, 0) => 'Aged Brie, -2, 2' args: ('Aged Brie', -1, 1) => 'Aged Brie, -2, 3'