From a598ce77189a49755324f022ec2fbc626ccd712b Mon Sep 17 00:00:00 2001 From: zoengsw Date: Mon, 10 Nov 2025 17:30:51 +0000 Subject: [PATCH] refactor: unify using single quotation --- python/tests/test_gilded_rose.py | 33 ++++++++++++++++++-------------- 1 file changed, 19 insertions(+), 14 deletions(-) diff --git a/python/tests/test_gilded_rose.py b/python/tests/test_gilded_rose.py index 641b858c..8bae3cf6 100644 --- a/python/tests/test_gilded_rose.py +++ b/python/tests/test_gilded_rose.py @@ -35,16 +35,16 @@ class GildedRoseTest(unittest.TestCase): self.check_item_values(items[0], 'foo', 0, 1) def test_nonnegative_quality_after_degradation(self): - items = [Item("foo", 2, 0)] + items = [Item('foo', 2, 0)] self.generate_and_update_gilded_rose(items, 1) - self.check_item_values(items[0], "foo", 1, 0) + self.check_item_values(items[0], 'foo', 1, 0) def test_quality_degradation_after_sellby_date(self): - items = [Item("foo", 2, 10)] + items = [Item('foo', 2, 10)] gilded_rose = self.generate_and_update_gilded_rose(items, 2) - self.check_item_values(items[0], "foo", 0, 8) + self.check_item_values(items[0], 'foo', 0, 8) self.update_gilded_rose_days(gilded_rose, 2) - self.check_item_values(items[0], "foo", -2, 4) + self.check_item_values(items[0], 'foo', -2, 4) def test_aged_brie_after_4_days(self): items = [Item(AGED_BRIE, 2, 10)] @@ -78,26 +78,31 @@ class GildedRoseTest(unittest.TestCase): self.check_item_values(items[0], BACKSTAGE, expected_sell_in, expected_qual) def test_multiple_items_single_day_degradation(self): - items = [Item("foo", 0, 0), Item("bar", 1, 1), Item("baz", 2, 2)] + items = [Item('foo', 0, 0), Item('bar', 1, 1), Item('baz', 2, 2)] self.generate_and_update_gilded_rose(items, 1) - self.check_item_values(items[0], "foo", -1, 0) - self.check_item_values(items[1], "bar", 0, 0) - self.check_item_values(items[2], "baz", 1, 1) + self.check_item_values(items[0], 'foo', -1, 0) + self.check_item_values(items[1], 'bar', 0, 0) + self.check_item_values(items[2], 'baz', 1, 1) - def test_all_items_15_days(self): - items = [Item("foo", 10, 10), Item(SULFURAS, 10, 10), Item(BACKSTAGE, 10, 10), Item(AGED_BRIE, 10, 10)] + # @parameterized.expand([ + # ('normal_item_decrements_until_expired', 5, 5, 0, 0, -5, 0), + # ('') + # ]) + + def test_multiple_various_items_every_5_days_within_15_days(self): + items = [Item('foo', 10, 10), Item(SULFURAS, 10, 10), Item(BACKSTAGE, 10, 10), Item(AGED_BRIE, 10, 10)] gilded_rose = self.generate_and_update_gilded_rose(items, 5) - self.check_item_values(items[0], "foo", 5, 5) + self.check_item_values(items[0], 'foo', 5, 5) self.check_item_values(items[1], SULFURAS, 10, 10) self.check_item_values(items[2], BACKSTAGE, 5, 20) self.check_item_values(items[3], AGED_BRIE, 5, 15) self.update_gilded_rose_days(gilded_rose, 5) - self.check_item_values(items[0], "foo", 0, 0) + self.check_item_values(items[0], 'foo', 0, 0) self.check_item_values(items[1], SULFURAS, 10, 10) self.check_item_values(items[2], BACKSTAGE, 0, 35) self.check_item_values(items[3], AGED_BRIE, 0, 20) self.update_gilded_rose_days(gilded_rose, 5) - self.check_item_values(items[0], "foo", -5, 0) + self.check_item_values(items[0], 'foo', -5, 0) self.check_item_values(items[1], SULFURAS, 10, 10) self.check_item_values(items[2], BACKSTAGE, -5, 0) self.check_item_values(items[3], AGED_BRIE, -5, 30)