Move unbound() shorthand to ScopeProvider itself

autodispose/src/main/java/com/uber/autodispose/ScopeProvider.java

@@ -24,6 +24,12 @@
  */
 public interface ScopeProvider {
 
+  ScopeProvider UNBOUND = new ScopeProvider() {
+    @Override public Maybe<?> requestScope() {
+      return Maybe.empty();
+    }
+  };
+
   /**
    * @return a Maybe that, upon emission, will trigger disposal.
    */

autodispose/src/main/java/com/uber/autodispose/TestScopeProvider.java

@@ -52,7 +52,9 @@ public static TestScopeProvider create(Maybe<?> delegate) {
    * scope is unbound.
    *
    * @return the created TestScopeProvider
+   * @deprecated in favor of {@link ScopeProvider#UNBOUND}. This method will be removed in 1.0.
    */
+  @Deprecated
   public static TestScopeProvider unbound() {
     return create(Maybe.empty());
   }

autodispose/src/test/java/com/uber/autodispose/AutoDisposeCompletableObserverTest.java

@@ -311,7 +311,7 @@ public CompletableObserver apply(Completable source, CompletableObserver observe
             }
           });
       Completable.complete()
-          .to(AutoDispose.with(Maybe.never())
+          .to(AutoDispose.with(ScopeProvider.UNBOUND)
               .forCompletable())
           .subscribe();
 

autodispose/src/test/java/com/uber/autodispose/AutoDisposeMaybeObserverTest.java

@@ -74,7 +74,7 @@ public class AutoDisposeMaybeObserverTest {
 
   @Test public void autoDispose_withSuperClassGenerics_compilesFine() {
     Maybe.just(new BClass())
-        .to(AutoDispose.with(Maybe.never()).<AClass>forMaybe())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND).<AClass>forMaybe())
         .subscribe(new Consumer<AClass>() {
           @Override public void accept(AClass aClass) throws Exception {
 
@@ -84,7 +84,7 @@ public class AutoDisposeMaybeObserverTest {
 
   @Test public void autoDispose_noGenericsOnEmpty_isFine() {
     Maybe.just(new BClass())
-        .to(AutoDispose.with(Maybe.never())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND)
             .forMaybe())
         .subscribe();
   }
@@ -343,7 +343,7 @@ public class AutoDisposeMaybeObserverTest {
         }
       });
       Maybe.just(1)
-          .to(AutoDispose.with(Maybe.never()).<Integer>forMaybe())
+          .to(AutoDispose.with(ScopeProvider.UNBOUND).<Integer>forMaybe())
           .subscribe();
 
       assertThat(atomicAutoDisposingObserver.get()).isNotNull();

autodispose/src/test/java/com/uber/autodispose/AutoDisposeObserverTest.java

@@ -77,7 +77,7 @@ public class AutoDisposeObserverTest {
 
   @Test public void autoDispose_withSuperClassGenerics_compilesFine() {
     Observable.just(new BClass())
-        .to(AutoDispose.with(Maybe.never()).<AClass>forObservable())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND).<AClass>forObservable())
         .subscribe(new Consumer<AClass>() {
           @Override public void accept(AClass aClass) throws Exception {
 
@@ -87,7 +87,7 @@ public class AutoDisposeObserverTest {
 
   @Test public void autoDispose_noGenericsOnEmpty_isFine() {
     Observable.just(new BClass())
-        .to(AutoDispose.with(Maybe.never())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND)
             .forObservable())
         .subscribe();
   }
@@ -278,7 +278,7 @@ public class AutoDisposeObserverTest {
         }
       });
       Observable.just(1)
-          .to(AutoDispose.with(Maybe.never()).<Integer>forObservable())
+          .to(AutoDispose.with(ScopeProvider.UNBOUND).<Integer>forObservable())
           .subscribe();
 
       assertThat(atomicAutoDisposingObserver.get()).isNotNull();

autodispose/src/test/java/com/uber/autodispose/AutoDisposeSingleObserverTest.java

@@ -76,7 +76,7 @@ public class AutoDisposeSingleObserverTest {
 
   @Test public void autoDispose_withSuperClassGenerics_compilesFine() {
     Single.just(new BClass())
-        .to(AutoDispose.with(Maybe.never()).<AClass>forSingle())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND).<AClass>forSingle())
         .subscribe(new Consumer<AClass>() {
           @Override public void accept(AClass aClass) throws Exception {
 
@@ -86,7 +86,7 @@ public class AutoDisposeSingleObserverTest {
 
   @Test public void autoDispose_noGenericsOnEmpty_isFine() {
     Single.just(new BClass())
-        .to(AutoDispose.with(Maybe.never())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND)
             .forSingle())
         .subscribe();
   }
@@ -314,7 +314,7 @@ public class AutoDisposeSingleObserverTest {
         }
       });
       Single.just(1)
-          .to(AutoDispose.with(Maybe.never()).<Integer>forSingle())
+          .to(AutoDispose.with(ScopeProvider.UNBOUND).<Integer>forSingle())
           .subscribe();
 
       assertThat(atomicAutoDisposingObserver.get()).isNotNull();

autodispose/src/test/java/com/uber/autodispose/AutoDisposeSubscriberTest.java

@@ -72,7 +72,7 @@ public class AutoDisposeSubscriberTest {
 
   @Test public void autoDispose_withSuperClassGenerics_compilesFine() {
     Flowable.just(new BClass())
-        .to(AutoDispose.with(Maybe.never()).<AClass>forFlowable())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND).<AClass>forFlowable())
         .subscribe(new Consumer<AClass>() {
           @Override public void accept(AClass aClass) throws Exception {
 
@@ -82,7 +82,7 @@ public class AutoDisposeSubscriberTest {
 
   @Test public void autoDispose_noGenericsOnEmpty_isFine() {
     Flowable.just(new BClass())
-        .to(AutoDispose.with(Maybe.never())
+        .to(AutoDispose.with(ScopeProvider.UNBOUND)
             .forFlowable())
         .subscribe();
   }
@@ -289,7 +289,7 @@ public class AutoDisposeSubscriberTest {
         }
       });
       Flowable.just(1)
-          .to(AutoDispose.with(Maybe.never()).<Integer>forFlowable())
+          .to(AutoDispose.with(ScopeProvider.UNBOUND).<Integer>forFlowable())
           .subscribe();
 
       assertThat(atomicAutoDisposingSubscriber.get()).isNotNull();