Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(contains): create exercise #442

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
dfca479
feat/ add new exercise
NikitaRevenco Mar 23, 2024
329d7d6
feat/ update README
NikitaRevenco Mar 23, 2024
897b2c1
feat/ create tests
NikitaRevenco Mar 23, 2024
59c6768
feat/ create solution
NikitaRevenco Mar 23, 2024
ac605aa
feat/ object is not nested
NikitaRevenco Mar 24, 2024
7146b0d
feat/ improve wording
NikitaRevenco Mar 24, 2024
e4e015d
feat/ wording
NikitaRevenco Mar 24, 2024
90afe10
feat/ wording
NikitaRevenco Mar 24, 2024
cd0a557
feat/ improve readability of solution
NikitaRevenco Mar 24, 2024
db402f7
Merge branch 'feat/exercise_14' of https://github.com/nikitarevenco/j…
NikitaRevenco Mar 24, 2024
15e4bec
feat/ mention that explicit NaN check is not required
NikitaRevenco Mar 24, 2024
c19c1bc
feat/ clarify that we would normally use math.isnan
NikitaRevenco Mar 24, 2024
4873c5a
Update 14_contains/solution/contains-solution.spec.js
NikitaRevenco Mar 24, 2024
360078c
feat/ null check
NikitaRevenco Mar 24, 2024
8c5851d
feat: improve clarity of suggestion
NikitaRevenco Mar 30, 2024
fc2fc72
feat: remove mention of IEEE
NikitaRevenco Mar 30, 2024
019af1d
feat: solution now accounts for children nested objects
NikitaRevenco Mar 30, 2024
ee12fc4
feat: add test to account for children nested objects
NikitaRevenco Mar 30, 2024
cc29bbd
feat: use Josh's solution that uses `some` method
NikitaRevenco Apr 7, 2024
0f87177
feat(contains): add some basic examples
NikitaRevenco Aug 18, 2024
e630a8b
feat(contains): add a more exhaustive test for reference in the object
NikitaRevenco Aug 18, 2024
8829dde
feat(contains): move same reference test to the beginning
NikitaRevenco Aug 20, 2024
de104e3
feat(contains): more concise wording
NikitaRevenco Aug 20, 2024
a204931
feat(contains): copy over tests from solution to spec file
NikitaRevenco Aug 20, 2024
d1c8f2c
fix(contains): change false to true for example which is true
NikitaRevenco Aug 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions 14_contains/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# Exercise 14 - contains

Write a function that searches for a value in a nested object. It returns true if the object contains that value.
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved
6 changes: 6 additions & 0 deletions 14_contains/contains.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
const contains = function() {

};

// Do not edit below this line
module.exports = contains;
15 changes: 15 additions & 0 deletions 14_contains/contains.spec.js
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved
MaoShizhong marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
const contains = require('./contains');

describe('contains', () => {
test('First test description', () => {
// Replace this comment with any other necessary code, and update the expect line as necessary

expect(contains()).toBe('');
});

test.skip('Second test description', () => {
// Replace this comment with any other necessary code, and update the expect line as necessary

expect(contains()).toBe('');
});
});
12 changes: 12 additions & 0 deletions 14_contains/solution/contains-solution.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
const contains = function(current, search) {
if (Object.values(current).includes(search)) return true;

for (const nestedObject of Object.values(current).filter((value) => typeof value === 'object')) {
return contains(nestedObject, search)
}

return false
};
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved

// Do not edit below this line
module.exports = contains;
45 changes: 45 additions & 0 deletions 14_contains/solution/contains-solution.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
const contains = require('./contains-solution');

describe('contains', () => {
const nestedObject = {
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved
data: {
banana: NaN,
info: {
magicNumber: 44,
stuff: {
thing: {
banana: NaN,
moreStuff: {
something: 'foo',
answer: [42]
}
}
}
}
}
}

test('true if property is number', () => {
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved
expect(contains(nestedObject, 44)).toBe(true);
});

test('true if property is string', () => {
expect(contains(nestedObject, 'foo')).toBe(true);
});

test('does not do type conversions', () => {
expect(contains(nestedObject, '44')).toBe(false);
});

test('false if property is not in object', () => {
expect(contains(nestedObject, 'bar')).toBe(false);
});

test('true if property is duplicated', () => {
expect(contains(nestedObject, NaN)).toBe(true);
NikitaRevenco marked this conversation as resolved.
Show resolved Hide resolved
});

test('false if property exists but is not primitive', () => {
expect(contains(nestedObject, [42])).toBe(false);
});
});