[APT-10345] Update Tests

Attempts to treat downloaded content license renewals similarly to how they are treated when streamed: attempt at renewing the license when playback starts.

Also moves the main thread observer to the point in ArmadilloPlayerChoreographer where it is returned, rather than using it in the Reducer. This addresses two problems: the state being inconsistent about which thread it was on, and preventing main thread ANRs.

Armadillo/src/main/java/com/scribd/armadillo/ArmadilloPlayerChoreographer.kt

@@ -41,6 +41,7 @@ import io.reactivex.Observable
 import io.reactivex.android.schedulers.AndroidSchedulers
 import io.reactivex.disposables.CompositeDisposable
 import io.reactivex.disposables.Disposable
+import io.reactivex.subjects.BehaviorSubject
 import java.util.concurrent.TimeUnit
 import javax.inject.Inject
 
@@ -214,8 +215,10 @@ internal class ArmadilloPlayerChoreographer : ArmadilloPlayer {
     /**
      * emits the most recently emitted state and all the subsequent states when an observer subscribes to it.
      */
+    val armadilloStateSubject: BehaviorSubject<ArmadilloState>
+        get() =  stateProvider.stateSubject
     override val armadilloStateObservable: Observable<ArmadilloState>
-        get() = stateProvider.stateSubject.observeOn(AndroidSchedulers.mainThread())
+        get() = armadilloStateSubject.observeOn(AndroidSchedulers.mainThread())
 
     private val pollingInterval =
         Observable.interval(observerPollIntervalMillis.longValue, TimeUnit.MILLISECONDS, AndroidSchedulers.mainThread())
@@ -439,7 +442,7 @@ internal class ArmadilloPlayerChoreographer : ArmadilloPlayer {
         data class PlaybackReadyData(val transportControls: MediaControllerCompat.TransportControls?, val engineIsReady: Boolean)
 
         Log.d(TAG, "Waiting for engine before invoking callback")
-        disposables.add(armadilloStateObservable
+        disposables.add(armadilloStateSubject
             .map {
                 PlaybackReadyData(playbackConnection?.transportControls, it.internalState.isPlaybackEngineReady)
             }

Armadillo/src/test/java/com/scribd/armadillo/ArmadilloPlayerChoreographerTest.kt

@@ -55,7 +55,7 @@ class ArmadilloPlayerChoreographerTest {
         whenever(stateProvider.stateSubject).thenReturn(stateSubject)
 
         choreographer.stateProvider = stateProvider
-        assertThat(choreographer.armadilloStateObservable.value).isEqualTo(state)
+        assertThat(choreographer.armadilloStateSubject.value).isEqualTo(state)
     }
 
     @Test