Merge 84c8dde into 367c740

blevesearch · Dec 3, 2018 · d240a15 · d240a15
2 parents 367c740 + 84c8dde
commit d240a15
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/search/searcher/search_fuzzy.go b/search/searcher/search_fuzzy.go
@@ -31,6 +31,10 @@ func NewFuzzySearcher(indexReader index.IndexReader, term string,
 		return nil, fmt.Errorf("fuzziness exceeds max (%d)", MaxFuzziness)
 	}
 
+	if fuzziness < 0 {
+		return nil, fmt.Errorf("invalid fuzziness, negative")
+	}
+
 	// Note: we don't byte slice the term for a prefix because of runes.
 	prefixTerm := ""
 	for i, r := range term {

diff --git a/search/searcher/search_fuzzy_test.go b/search/searcher/search_fuzzy_test.go
@@ -140,3 +140,16 @@ func TestFuzzySearch(t *testing.T) {
 		}
 	}
 }
+
+func TestFuzzySearchLimitErrors(t *testing.T) {
+	explainTrue := search.SearcherOptions{Explain: true}
+	_, err := NewFuzzySearcher(nil, "water", 3, 3, "desc", 1.0, explainTrue)
+	if err == nil {
+		t.Fatal("`fuzziness exceeds max (2)` error expected")
+	}
+
+	_, err = NewFuzzySearcher(nil, "water", 3, -1, "desc", 1.0, explainTrue)
+	if err == nil {
+		t.Fatal("`invalid fuzziness, negative` error expected")
+	}
+}