From 74f35d6999930d5efa8ec9f8e99fcab99a47b5d4 Mon Sep 17 00:00:00 2001 From: Sophanarith Sok Date: Mon, 21 Aug 2017 21:15:54 -0700 Subject: [PATCH] test(tests): Add tests to Problem: Even Fibonacci Numbers (#15773) Add more specific tests and edit motivational comment for something more interesting. --- .../project-euler-problems.json | 24 +++++++++++-------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/challenges/08-coding-interview-questions-and-take-home-assignments/project-euler-problems.json b/challenges/08-coding-interview-questions-and-take-home-assignments/project-euler-problems.json index 1b06261e49..6759752480 100644 --- a/challenges/08-coding-interview-questions-and-take-home-assignments/project-euler-problems.json +++ b/challenges/08-coding-interview-questions-and-take-home-assignments/project-euler-problems.json @@ -34,19 +34,23 @@ "_id": "5900f36e1000cf542c50fe81", "challengeType": 5, "type": "bonfire", - "title": "Problem 2: Even Fibonacci numbers", + "title": "Problem 2: Even Fibonacci Numbers", "tests": [ - "assert.strictEqual(euler2(), 4613732, 'message: euler2() should return 4613732.');" + "assert.strictEqual(fiboEvenSum(10), 188, 'message: fiboEvenSum(10) should return 188.');", + "assert.strictEqual(fiboEvenSum(23), 60696, 'message: fiboEvenSum(23) should return 60696.');", + "assert.strictEqual(fiboEvenSum(43), 1485607536, 'message: fiboEvenSum(43) should return 1485607536.');", + "assert.strictEqual(fiboEvenSum(18), 3382, 'message: Your function is not returning the correct result using our tests values.');", + "assert.equal(fiboEvenSum(31) % 2 === 0, true, 'message: Your function should return an even value.');" ], "solutions": [], "translations": {}, "challengeSeed": [ - "function euler2() {", - " // Good luck!", + "function fiboEvenSum(number) {", + " // You can do it!", " return true;", "}", "", - "euler2();" + "fiboEvenSum(10);" ], "description": [ "Each new term in the Fibonacci sequence is generated by adding the previous two terms. By starting with 1 and 2, the first 10 terms will be:", @@ -640,11 +644,11 @@ "type": "bonfire", "title": "Problem 20: Factorial digit sum", "tests": [ - "assert.strictEqual(sumFactorialDigits(10), 27, 'message: \u003ccode\u003esumFactorialDigits(10)\u003c/code\u003e should return 27.');", - "assert.strictEqual(sumFactorialDigits(25), 72, 'message: \u003ccode\u003esumFactorialDigits(25)\u003c/code\u003e should return 72.');", - "assert.strictEqual(sumFactorialDigits(50), 216, 'message: \u003ccode\u003esumFactorialDigits(50)\u003c/code\u003e should return 216.');", - "assert.strictEqual(sumFactorialDigits(75), 432, 'message: \u003ccode\u003esumFactorialDigits(75)\u003c/code\u003e should return 432.');", - "assert.strictEqual(sumFactorialDigits(100), 648, 'message: \u003ccode\u003esumFactorialDigits(100)\u003c/code\u003e should return 648.');" + "assert.strictEqual(sumFactorialDigits(10), 27, 'message: sumFactorialDigits(10) should return 27.');", + "assert.strictEqual(sumFactorialDigits(25), 72, 'message: sumFactorialDigits(25) should return 72.');", + "assert.strictEqual(sumFactorialDigits(50), 216, 'message: sumFactorialDigits(50) should return 216.');", + "assert.strictEqual(sumFactorialDigits(75), 432, 'message: sumFactorialDigits(75) should return 432.');", + "assert.strictEqual(sumFactorialDigits(100), 648, 'message: sumFactorialDigits(100) should return 648.');" ], "solutions": [], "translations": {},