commit 06f293942c47e27941ff9a3eda14ea91319f95e0
author Alice Kober-Sotzek <aliceks@google.com> Wed Jun 05 14:03:44 2019 +0200
committer David Pursehouse <dpursehouse@collab.net> Thu Jun 06 08:13:09 2019 +0900
tree 655955ac8d6dbff6e149d0fbde34e2aac5409a67
parent d2b61e2e0974092aec6fc5286de6a22c66795366

OptionalSubject: Remove now unnecessary method

All known usages were adapted.

Change-Id: I74c9d50d9e65dbdf00258c0eb46fd3ba9387b6f4

java/com/google/gerrit/truth/OptionalSubject.java

diff --git a/java/com/google/gerrit/truth/OptionalSubject.java b/java/com/google/gerrit/truth/OptionalSubject.java
index 1c288be..2023765 100644
--- a/java/com/google/gerrit/truth/OptionalSubject.java
+++ b/java/com/google/gerrit/truth/OptionalSubject.java
@@ -23,21 +23,12 @@
 import com.google.common.truth.Subject;
 import java.util.Optional;
 import java.util.function.BiFunction;
-import java.util.function.Function;
 
 public class OptionalSubject<S extends Subject, T> extends Subject {
 
   private final Optional<T> optional;
   private final BiFunction<StandardSubjectBuilder, ? super T, ? extends S> valueSubjectCreator;
 
-  // TODO(aliceks): Remove when all relevant usages are adapted to new check()/factory approach.
-  public static <S extends Subject, T> OptionalSubject<S, T> assertThat(
-      Optional<T> optional, Function<? super T, ? extends S> elementAssertThatFunction) {
-    Subject.Factory<S, T> valueSubjectFactory =
-        (metadata, value) -> elementAssertThatFunction.apply(value);
-    return assertThat(optional, valueSubjectFactory);
-  }
-
   public static <S extends Subject, T> OptionalSubject<S, T> assertThat(
       Optional<T> optional, Subject.Factory<? extends S, ? super T> valueSubjectFactory) {
     return assertAbout(optionals()).thatCustom(optional, valueSubjectFactory);