NaturalNode / natural / 40188291853

87%

Build Type

push

github

Committed by web-flow

Commit Message

fix: do not look up TfIdf terms on the prototype of documents and cache (#785)

* fix: do not look up TfIdf terms on the prototype of documents and cache

Terms that match a property of Object, such as "constructor", were looked
up on the prototype chain. TfIdf.tf returned the Object function for a
document that lacks the term, so tfidf() returned NaN, and the IDF cache
created by the constructor and addFileSync was a plain object, so idf()
returned the Object function for instances created from deserialized data
or filled with addFileSync.

Use own property lookups for the term frequency and use a cache without a
prototype in the constructor and in addFileSync, like addDocument does.

* chore: trigger CI

* chore: trigger CI

Coverage Stats

2397 of 3110 branches covered (77.07%)

Branch coverage included in aggregate %.

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

8 existing lines in 2 files now uncovered.

9087 of 10064 relevant lines covered (90.29%)

12761.56 hits per line