NaturalNode / natural / 40285332419

87%

push

github

fix: JaroWinklerDistance of single character strings that differ in case (#786)

* fix: JaroWinklerDistance of single character strings that differ in case

For strings of length 1 the match window floor(1 / 2) - 1 is negative, so
no character matched and the Jaro distance was 0. The prefix loop also ran
past the end of the strings, so JaroWinklerDistance('A', 'a', { ignoreCase:
true }) returned 0.4 instead of 1. Clamp the match window to 0 and bound
the prefix loop by the lengths of the strings.

2402 of 3112 branches covered (77.19%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

9097 of 10073 relevant lines covered (90.31%)

12843.97 hits per line