From ddac01b1f40f589d53a9b1af3ee36cbab0f3413b Mon Sep 17 00:00:00 2001 From: octopusGarden Date: Fri, 4 Nov 2022 14:39:31 -0400 Subject: [PATCH] Fixed leap years to work with centuries --- 06_leapYears/leapYears.js | 2 +- 06_leapYears/leapYears.spec.js | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/06_leapYears/leapYears.js b/06_leapYears/leapYears.js index 5d8b840..224dbe6 100644 --- a/06_leapYears/leapYears.js +++ b/06_leapYears/leapYears.js @@ -4,7 +4,7 @@ const leapYears = function(year) { //else false if (year % 100 === 0 && year % 400 === 0) { return true; - } else if (year % 4 === 0) { + } else if (year % 4 === 0 && year % 100 !== 0) { return true; } else { return false; diff --git a/06_leapYears/leapYears.spec.js b/06_leapYears/leapYears.spec.js index 6fdaba9..2cd4110 100644 --- a/06_leapYears/leapYears.spec.js +++ b/06_leapYears/leapYears.spec.js @@ -4,19 +4,19 @@ describe('leapYears', () => { test('works with non century years', () => { expect(leapYears(1996)).toBe(true); }); - test.skip('works with non century years', () => { + test('works with non century years', () => { expect(leapYears(1997)).toBe(false); }); - test.skip('works with ridiculously futuristic non century years', () => { + test('works with ridiculously futuristic non century years', () => { expect(leapYears(34992)).toBe(true); }); - test.skip('works with century years', () => { + test('works with century years', () => { expect(leapYears(1900)).toBe(false); }); - test.skip('works with century years', () => { + test('works with century years', () => { expect(leapYears(1600)).toBe(true); }); - test.skip('works with century years', () => { + test('works with century years', () => { expect(leapYears(700)).toBe(false); }); });