From fe2a19749bfbe1db8cec4fd8dd7b86cd6b99b8fd Mon Sep 17 00:00:00 2001 From: Konstantin Pavlov Date: Mon, 24 Jun 2019 07:06:37 +0200 Subject: [PATCH] test: The Quality of an item is never more than 50 --- Java/src/test/java/com/gildedrose/GildedRoseTest.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Java/src/test/java/com/gildedrose/GildedRoseTest.java b/Java/src/test/java/com/gildedrose/GildedRoseTest.java index a23614c1..cc2a23b8 100644 --- a/Java/src/test/java/com/gildedrose/GildedRoseTest.java +++ b/Java/src/test/java/com/gildedrose/GildedRoseTest.java @@ -8,6 +8,7 @@ import org.junit.jupiter.params.provider.ValueSource; import static com.gildedrose.TestHelper.assertItem; import static com.gildedrose.TestHelper.prepareApp; +import static java.lang.Math.min; import static org.apache.commons.lang3.RandomUtils.nextInt; @@ -58,8 +59,9 @@ class GildedRoseTest { } @Feature("\"Aged Brie\" actually increases in Quality the older it gets") + @Feature("The Quality of an item is never more than 50") @ParameterizedTest(name = "Initial quality: {arguments}") - @ValueSource(ints = {0, 1, 49}) + @ValueSource(ints = {0, 1, 49, 50}) void shouldIncreaseQualityForAgedBrie(int initialQuality) { // given GildedRose app = prepareApp(new Item("Aged Brie", 1, initialQuality)); @@ -69,6 +71,7 @@ class GildedRoseTest { // then final Item item = app.items[0]; - assertItem(item, "Aged Brie", 0, initialQuality + 1); + final int expectedQuality = min(initialQuality + 1, 50); + assertItem(item, "Aged Brie", 0, expectedQuality); } }