[APT-10344] Initialization State Smoothness

- Ensured that ArmadilloPlayer.armadilloStateObservable has a state as soon as the player is initialized
- Fixed UnknownHostException being mapped to a HTTP status code issue rather than a Connectivity issue.

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

@@ -33,7 +33,9 @@ internal class ArmadilloStateStore(private val reducer: Reducer, private val han
         const val TAG = "ArmadilloStateStore"
     }
 
-    private val armadilloStateObservable = BehaviorSubject.create<ArmadilloState>()
+    private val armadilloStateObservable = BehaviorSubject.create<ArmadilloState>().also {
+       it.onNext(ArmadilloState(downloadInfo = emptyList()))
+    }
 
     override fun init(state: ArmadilloState) = armadilloStateObservable.onNext(state)
 

Armadillo/src/main/java/com/scribd/armadillo/error/ArmadilloException.kt

@@ -3,6 +3,7 @@ package com.scribd.armadillo.error
 import com.google.android.exoplayer2.upstream.HttpDataSource.HttpDataSourceException
 import com.scribd.armadillo.actions.Action
 import java.net.SocketTimeoutException
+import java.net.UnknownHostException
 
 sealed class ArmadilloException(cause: Throwable? = null,
                                 isNetworkRelatedError: Boolean = false,
@@ -181,7 +182,7 @@ class DrmDownloadException(cause: Exception)
     : ArmadilloException(
     cause = cause,
     message = "Failed to process DRM license for downloading.",
-    isNetworkRelatedError = (cause is HttpDataSourceException) || (cause is SocketTimeoutException)) {
+    isNetworkRelatedError = (cause is HttpDataSourceException) || (cause is SocketTimeoutException) || (cause is UnknownHostException)) {
     override val errorCode = 701
 }
 

Armadillo/src/main/java/com/scribd/armadillo/playback/ExoPlaybackExceptionExt.kt

@@ -30,9 +30,8 @@ internal fun ExoPlaybackException.toArmadilloException(): ArmadilloException {
                     val httpCause = source.cause as? HttpDataSource.InvalidResponseCodeException
                     HttpResponseCodeException(httpCause?.responseCode ?: 0, httpCause?.dataSpec?.uri.toString(), source)
                 }
+                is UnknownHostException,
                 is SocketTimeoutException -> ConnectivityException(source)
-                is UnknownHostException ->
-                    HttpResponseCodeException(0, source.message, source) // Message is supposed to be the host for UnknownHostException
                 else -> ArmadilloIOException(cause = this, actionThatFailedMessage = "Exoplayer error.")
             }
         }

RELEASE.md

@@ -1,5 +1,9 @@
 # Project Armadillo Release Notes
 
+## 1.5.4
+- Ensured that ArmadilloPlayer.armadilloStateObservable has a state as soon as the player is initialized
+- Fixed UnknownHostException being mapped to a HTTP status code issue rather than a Connectivity issue.
+
 ## 1.5.3
 - Attempts to renew the widevine license of downloaded DRM content when playback begins, similarly to how streaming does it.
 - Fixes ANR issue in the Reducer.

gradle.properties

@@ -13,7 +13,7 @@ org.gradle.jvmargs=-Xmx1536m
 # org.gradle.parallel=true
 PACKAGE_NAME=com.scribd.armadillo
 GRADLE_PLUGIN_VERSION=7.2.0
-LIBRARY_VERSION=1.5.3
+LIBRARY_VERSION=1.5.4
 EXOPLAYER_VERSION=2.19.1
 RXJAVA_VERSION=2.2.4
 RXANDROID_VERSION=2.0.1