Remove Match#startsWith(), endsWith(). Don't seem to add a lot of value.

Author: fluentfuture

mug/src/main/java/com/google/mu/util/Substring.java

@@ -2006,26 +2006,6 @@ public String remove() {
       }
     }
 
-    /**
-     * Returns true if this match starts with {@code prefix}.
-     *
-     * @since 6.1
-     */
-    public boolean startsWith(String prefix) {
-      return prefix.length() <= length()
-          && context.regionMatches(startIndex, prefix, 0, prefix.length());
-    }
-
-    /**
-     * Returns true if this match ends with {@code suffix}.
-     *
-     * @since 6.1
-     */
-    public boolean endsWith(String suffix) {
-      return suffix.length() <= length()
-          && context.regionMatches(startIndex + (length() - suffix.length()), suffix, 0, suffix.length());
-    }
-
     /**
      * Returns a copy of the original string with the matched substring replaced with {@code
      * replacement}.

mug/src/test/java/com/google/mu/util/SubstringTest.java

@@ -2048,113 +2048,6 @@ public void skipFromEnd_toString() {
     assertThat(first("foo").skip(0, 2).toString()).isEqualTo("first('foo').skip(0, 2)");
   }
 
-  @Test
-  public void matchStartsWith_empty() {
-    Substring.Match match = Substring.first("").in("abc").get();
-    assertThat(match.startsWith("")).isTrue();
-    assertThat(match.startsWith("a")).isFalse();
-  }
-
-  @Test
-  public void matchStartsWith_singleChars() {
-    Substring.Match match = Substring.first("c").in("abcd").get();
-    assertThat(match.startsWith("")).isTrue();
-    assertThat(match.startsWith("c")).isTrue();
-    assertThat(match.startsWith("bc")).isFalse();
-    assertThat(match.startsWith("cd")).isFalse();
-    assertThat(match.startsWith("d")).isFalse();
-  }
-
-  @Test
-  public void matchStartsWith_twoChars() {
-    Substring.Match match = Substring.first("bc").in("abcd").get();
-    assertThat(match.startsWith("")).isTrue();
-    assertThat(match.startsWith("b")).isTrue();
-    assertThat(match.startsWith("bc")).isTrue();
-    assertThat(match.startsWith("bcd")).isFalse();
-    assertThat(match.startsWith("c")).isFalse();
-  }
-
-  @Test
-  public void matchStartsWith_threeChars() {
-    Substring.Match match = Substring.first("bcd").in("abcde").get();
-    assertThat(match.startsWith("")).isTrue();
-    assertThat(match.startsWith("b")).isTrue();
-    assertThat(match.startsWith("bc")).isTrue();
-    assertThat(match.startsWith("bcd")).isTrue();
-    assertThat(match.startsWith("bcde")).isFalse();
-    assertThat(match.startsWith("c")).isFalse();
-    assertThat(match.startsWith("d")).isFalse();
-    assertThat(match.startsWith("e")).isFalse();
-  }
-
-  @Test
-  public void matchStartsWith_withSkip() {
-    Substring.Match match = Substring.first("bc").in("abcd").get();
-    assertThat(match.skip(1, 0).startsWith("c")).isTrue();
-  }
-
-  @Test
-  public void matchStartsWith_withLimit() {
-    Substring.Match match = Substring.first("bc").in("abcd").get();
-    assertThat(match.limit(1).startsWith("b")).isTrue();
-    assertThat(match.limit(1).startsWith("bc")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_empty() {
-    Substring.Match match = Substring.first("").in("abc").get();
-    assertThat(match.endsWith("")).isTrue();
-    assertThat(match.endsWith("c")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_singleChars() {
-    Substring.Match match = Substring.first("b").in("abcd").get();
-    assertThat(match.endsWith("")).isTrue();
-    assertThat(match.endsWith("b")).isTrue();
-    assertThat(match.endsWith("bc")).isFalse();
-    assertThat(match.endsWith("ab")).isFalse();
-    assertThat(match.endsWith("d")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_twoChars() {
-    Substring.Match match = Substring.first("bc").in("abcd").get();
-    assertThat(match.endsWith("")).isTrue();
-    assertThat(match.endsWith("c")).isTrue();
-    assertThat(match.endsWith("bc")).isTrue();
-    assertThat(match.endsWith("bcd")).isFalse();
-    assertThat(match.endsWith("d")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_threeChars() {
-    Substring.Match match = Substring.first("bcd").in("abcde").get();
-    assertThat(match.endsWith("")).isTrue();
-    assertThat(match.endsWith("cd")).isTrue();
-    assertThat(match.endsWith("bcd")).isTrue();
-    assertThat(match.endsWith("abcd")).isFalse();
-    assertThat(match.endsWith("de")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_withLimit() {
-    Substring.Match match = Substring.first("bc").in("abcd").get();
-    assertThat(match.limit(1).endsWith("b")).isTrue();
-    assertThat(match.limit(1).endsWith("")).isTrue();
-    assertThat(match.limit(1).endsWith("bc")).isFalse();
-  }
-
-  @Test
-  public void matchEndsWith_withSkip() {
-    Substring.Match match = Substring.first("bcd").in("abcde").get();
-    assertThat(match.skip(1, 1).endsWith("")).isTrue();
-    assertThat(match.skip(1, 1).endsWith("c")).isTrue();
-    assertThat(match.skip(1, 1).endsWith("cd")).isFalse();
-    assertThat(match.skip(1, 1).endsWith("bc")).isFalse();
-  }
-
   @Test public void or_toString() {
     assertThat(first("foo").or(last("bar")).toString()).isEqualTo("first('foo').or(last('bar'))");
   }