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); }); });