Skip to content

Conversation

iRipVanWinkle
Copy link
Contributor

Method Trie::doesWordExist() return true when word is complete otherwise false

Method Trie::doesWordExist() return `true` when word is complete otherwise `false`
trie.addWord('caption');

expect(trie.doesWordExist('cat')).toBe(true);
expect(trie.doesWordExist('cap')).toBe(true);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong test case

@codecov-io
Copy link

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #175   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         134    134           
  Lines        2485   2486    +1     
  Branches      415    416    +1     
=====================================
+ Hits         2485   2486    +1
Impacted Files Coverage Δ
src/data-structures/trie/Trie.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5eb1195...a8929a0. Read the comment docs.

@trekhleb trekhleb changed the base branch from master to trie-fix August 21, 2018 19:59
@trekhleb
Copy link
Owner

@iRipVanWinkle
Олежка, спасибо за фикс!

@trekhleb trekhleb merged commit 392cd98 into trekhleb:trie-fix Aug 21, 2018
harshes53 pushed a commit to harshes53/javascript-algorithms that referenced this pull request Dec 6, 2018
Method Trie::doesWordExist() return `true` when word is complete otherwise `false`
shoredata pushed a commit to shoredata/javascript-algorithms that referenced this pull request Mar 28, 2019
Method Trie::doesWordExist() return `true` when word is complete otherwise `false`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants