Making sameThreadScheduledExecutor's return type ListeningScheduledEx…

…ecutorService, to avoid leaking a package-private class through a public method. RELNOTES=`testing`: Changed the return type of `sameThreadScheduledExecutor` to `ListeningScheduledExecutorService` to avoid leaking a package-private class through a public method. This helps Kotlin callers. PiperOrigin-RevId: 488662036
google · Nov 15, 2022 · dafaa3e · dafaa3e
1 parent 6d7e326
commit dafaa3e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/android/guava-testlib/src/com/google/common/util/concurrent/testing/TestingExecutors.java b/android/guava-testlib/src/com/google/common/util/concurrent/testing/TestingExecutors.java
@@ -88,9 +88,9 @@ public static ListeningScheduledExecutorService noOpScheduledExecutor() {
    * invokeAll/invokeAny} throwing RejectedExecutionException, although a subset of the tasks may
    * already have been executed.
    *
-   * @since 15.0
+   * @since NEXT (taking the place of a method with a different return type from 15.0)
    */
-  public static SameThreadScheduledExecutorService sameThreadScheduledExecutor() {
+  public static ListeningScheduledExecutorService sameThreadScheduledExecutor() {
     return new SameThreadScheduledExecutorService();
   }
 

diff --git a/guava-testlib/src/com/google/common/util/concurrent/testing/TestingExecutors.java b/guava-testlib/src/com/google/common/util/concurrent/testing/TestingExecutors.java
@@ -88,9 +88,9 @@ public static ListeningScheduledExecutorService noOpScheduledExecutor() {
    * invokeAll/invokeAny} throwing RejectedExecutionException, although a subset of the tasks may
    * already have been executed.
    *
-   * @since 15.0
+   * @since NEXT (taking the place of a method with a different return type from 15.0)
    */
-  public static SameThreadScheduledExecutorService sameThreadScheduledExecutor() {
+  public static ListeningScheduledExecutorService sameThreadScheduledExecutor() {
     return new SameThreadScheduledExecutorService();
   }