From 93681509e5eaecfa4b0585314bd8a51759632d2d Mon Sep 17 00:00:00 2001 From: Aniruddh Agarwal Date: Thu, 24 Sep 2015 15:49:32 +0800 Subject: [PATCH] Add tests to intermediate bonfire The bonfire only tested the camper's solution for numbers upto 29. This commit adds tests that verify the solution for numbers upto 1000. This should motivate the campers to create a more sound logic that will work for more numbers. --- seed/challenges/intermediate-bonfires.json | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/seed/challenges/intermediate-bonfires.json b/seed/challenges/intermediate-bonfires.json index d8b008208db..0e635d60d3f 100644 --- a/seed/challenges/intermediate-bonfires.json +++ b/seed/challenges/intermediate-bonfires.json @@ -93,11 +93,25 @@ "id": "a7f4d8f2483413a6ce226cac", "title": "Roman Numeral Converter", "tests": [ - "assert.deepEqual(convert(12), \"XII\", 'message: convert(12) should return \"XII\".');", "assert.deepEqual(convert(5), \"V\", 'message: convert(5) should return \"V\".');", "assert.deepEqual(convert(9), \"IX\", 'message: convert(9) should return \"IX\".');", + "assert.deepEqual(convert(12), \"XII\", 'message: convert(12) should return \"XII\".');", + "assert.deepEqual(convert(16), \"XVI\", 'message: convert(16) should return \"XVI\".');", "assert.deepEqual(convert(29), \"XXIX\", 'message: convert(29) should return \"XXIX\".');", - "assert.deepEqual(convert(16), \"XVI\", 'message: convert(16) should return \"XVI\".');" + "assert.deepEqual(convert(44), \"XLIV\", 'message: convert(44) should return \"XLIV\".');", + "assert.deepEqual(convert(45), \"XLV\", 'convert(45) should return \"XLV\"');", + "assert.deepEqual(convert(68), \"LXVIII\", 'convert(68) should return \"LXVIII\"');", + "assert.deepEqual(convert(83), \"LXXXIII\", 'convert(83) should return \"LXXXIII\"');", + "assert.deepEqual(convert(97), \"XCVII\", 'convert(97) should return \"XCVII\"');", + "assert.deepEqual(convert(99), \"XCIX\", 'convert(99) should return \"XCIX\"');", + "assert.deepEqual(convert(500), \"D\", 'convert(500) should return \"D\"');", + "assert.deepEqual(convert(501), \"DI\", 'convert(501) should return \"DI\"');", + "assert.deepEqual(convert(649), \"DCXLIX\", 'convert(649) should return \"DCXLIX\"');", + "assert.deepEqual(convert(798), \"DCCXCVIII\", 'convert(798) should return \"DCCXCVIII\"');", + "assert.deepEqual(convert(891), \"DCCCXCI\", 'convert(891) should return \"DCCCXCI\"');", + "assert.deepEqual(convert(1000), \"M\", 'convert(1000) should return \"M\"');", + "assert.deepEqual(convert(1004), \"MIV\", 'convert(1004) should return \"MIV\"');", + "assert.deepEqual(convert(1006), \"MVI\", 'convert(1006) should return \"MVI\"');" ], "description": [ "Convert the given number into a roman numeral.",