From 8bc6e47821b8df0ba544cf9bc9bf8e14eeeceb5b Mon Sep 17 00:00:00 2001 From: DonRehan <30264386+donRehan@users.noreply.github.com> Date: Mon, 12 Feb 2024 14:36:33 +0200 Subject: [PATCH] Update findTheOldest test to clear ambiguity --- 12_findTheOldest/findTheOldest.spec.js | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/12_findTheOldest/findTheOldest.spec.js b/12_findTheOldest/findTheOldest.spec.js index 4f07887..aac207c 100644 --- a/12_findTheOldest/findTheOldest.spec.js +++ b/12_findTheOldest/findTheOldest.spec.js @@ -21,7 +21,26 @@ describe('findTheOldest', () => { ] expect(findTheOldest(people).name).toBe('Ray'); }); - test('finds the oldest person if yearOfDeath field is not present', () => { + test.skip('finds the oldest person if yearOfDeath field is undefined on a non-oldest person', () => { + const people = [ + { + name: "Carly", + yearOfBirth: 2018, + }, + { + name: "Ray", + yearOfBirth: 1962, + yearOfDeath: 2011, + }, + { + name: "Jane", + yearOfBirth: 1912, + yearOfDeath: 1941, + }, + ] + expect(findTheOldest(people).name).toBe('Ray'); + }); + test.skip('finds the oldest person if yearOfDeath field is undefined for the oldest person', () => { const people = [ { name: "Carly",