Make the spell checker a bit more conservative.

Reduce the max spell-checking distance for matching words.
"target" will not match "range" anymore.

--
PiperOrigin-RevId: 150638073
MOS_MIGRATED_REVID=150638073

1 files changed, 15 insertions, 9 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/util/SpellCheckerTest.java b/src/test/java/com/google/devtools/build/lib/util/SpellCheckerTest.java
index 5b4e55d4df..5ff97e8715 100644
--- a/src/test/java/com/google/devtools/build/lib/util/SpellCheckerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/util/SpellCheckerTest.java
@@ -84,8 +84,9 @@ public class SpellCheckerTest {
   @Test
   public void suggest() throws Exception {
     List<String> dict = Lists.newArrayList(
-        "isalnum", "isalpha", "isdigit", "islower", "isupper", "find", "join",
-        "rsplit", "rstrip", "split", "splitlines", "startswith", "strip", "title", "upper");
+        "isalnum", "isalpha", "isdigit", "islower", "isupper", "find", "join", "range",
+        "rsplit", "rstrip", "split", "splitlines", "startswith", "strip", "title", "upper",
+        "x", "xyz");
 
     assertThat(SpellChecker.suggest("isdfit", dict)).isEqualTo("isdigit");
     assertThat(SpellChecker.suggest("rspit", dict)).isEqualTo("rsplit");
@@ -95,12 +96,17 @@ public class SpellCheckerTest {
     assertThat(SpellChecker.suggest("fird", dict)).isEqualTo("find");
     assertThat(SpellChecker.suggest("stip", dict)).isEqualTo("strip");
     assertThat(SpellChecker.suggest("isAln", dict)).isEqualTo("isalnum");
-
-    assertThat(SpellChecker.suggest("isAl", dict)).isEqualTo(null);
-    assertThat(SpellChecker.suggest("", dict)).isEqualTo(null);
-    assertThat(SpellChecker.suggest("f", dict)).isEqualTo(null);
-    assertThat(SpellChecker.suggest("fir", dict)).isEqualTo(null);
-    assertThat(SpellChecker.suggest("wqevxc", dict)).isEqualTo(null);
-    assertThat(SpellChecker.suggest("ialsnuaip", dict)).isEqualTo(null);
+    assertThat(SpellChecker.suggest("targe", dict)).isEqualTo("range");
+    assertThat(SpellChecker.suggest("rarget", dict)).isEqualTo("range");
+    assertThat(SpellChecker.suggest("xyw", dict)).isEqualTo("xyz");
+
+    assertThat(SpellChecker.suggest("target", dict)).isNull();
+    assertThat(SpellChecker.suggest("isAl", dict)).isNull();
+    assertThat(SpellChecker.suggest("", dict)).isNull();
+    assertThat(SpellChecker.suggest("f", dict)).isNull();
+    assertThat(SpellChecker.suggest("fir", dict)).isNull();
+    assertThat(SpellChecker.suggest("wqevxc", dict)).isNull();
+    assertThat(SpellChecker.suggest("ialsnuaip", dict)).isNull();
+    assertThat(SpellChecker.suggest("xy", dict)).isNull();
   }
 }