Merge pull request #428 from donRehan/main

12_findTheOldest fix ambiguity and remove redundant test description
This commit is contained in:
Eric Olkowski
2024-02-13 17:24:09 -05:00
committed by GitHub
+2 -2
View File
@@ -21,7 +21,7 @@ describe('findTheOldest', () => {
] ]
expect(findTheOldest(people).name).toBe('Ray'); expect(findTheOldest(people).name).toBe('Ray');
}); });
test.skip('finds the person with the greatest age if someone is still living', () => { test.skip('finds the oldest person if yearOfDeath field is undefined on a non-oldest person', () => {
const people = [ const people = [
{ {
name: "Carly", name: "Carly",
@@ -40,7 +40,7 @@ describe('findTheOldest', () => {
] ]
expect(findTheOldest(people).name).toBe('Ray'); expect(findTheOldest(people).name).toBe('Ray');
}); });
test.skip('finds the person with the greatest age if the OLDEST is still living', () => { test.skip('finds the oldest person if yearOfDeath field is undefined for the oldest person', () => {
const people = [ const people = [
{ {
name: "Carly", name: "Carly",