Add press-back command push action

core-android/src/main/java/com/jraska/github/client/core/android/TopActivityProvider.kt

@@ -14,8 +14,8 @@ class TopActivityProvider(
 ) {
 
   private val pendingActions = mutableListOf<(Activity) -> Unit>()
-  private var topActivity: Activity? = null
-    set(activity) {
+  var topActivity: Activity? = null
+    private set(activity) {
       field = activity
       executePendingActions()
     }

feature/push/src/main/java/com/jraska/github/client/push/PressBackButtonCommand.kt

@@ -0,0 +1,16 @@
+package com.jraska.github.client.push
+
+import com.jraska.github.client.common.BooleanResult
+import com.jraska.github.client.common.BooleanResult.SUCCESS
+import com.jraska.github.client.core.android.TopActivityProvider
+import javax.inject.Inject
+
+internal class PressBackButtonCommand @Inject constructor(
+  private val topActivityProvider: TopActivityProvider
+) : PushActionCommand {
+  override fun execute(action: PushAction): BooleanResult {
+    val activity = topActivityProvider.topActivity ?: return SUCCESS // no-op if in background
+    activity.onBackPressed()
+    return SUCCESS
+  }
+}

feature/push/src/main/java/com/jraska/github/client/push/PushModule.kt

@@ -3,6 +3,7 @@ package com.jraska.github.client.push
 import com.google.firebase.database.FirebaseDatabase
 import com.jraska.github.client.core.android.OnAppCreate
 import com.jraska.github.client.core.android.ServiceModel
+import dagger.Binds
 import dagger.Module
 import dagger.Provides
 import dagger.multibindings.ClassKey
@@ -11,66 +12,59 @@ import dagger.multibindings.IntoSet
 import dagger.multibindings.StringKey
 import javax.inject.Singleton
 
-@Module
+@Module(includes = [PushModule.Declarations::class])
 object PushModule {
 
   @Provides
-  @IntoSet
-  internal fun bindObserverSetup(observerSetup: PushIntentObserver.CallbacksSetup): OnAppCreate {
-    return observerSetup
+  @Singleton
+  internal fun firebaseDatabase(): FirebaseDatabase {
+    return FirebaseDatabase.getInstance()
   }
 
-  @Provides
-  @IntoSet
-  internal fun setupNotificationsOnCreate(notificationSetup: NotificationSetup): OnAppCreate {
-    return notificationSetup
-  }
+  @Module
+  abstract class Declarations {
 
-  @Provides
-  @IntoMap
-  @ClassKey(PushHandleModel::class)
-  internal fun bindServiceModel(pushHandleModel: PushHandleModel): ServiceModel {
-    return pushHandleModel
-  }
+    @Binds
+    @IntoSet
+    internal abstract fun bindObserverSetup(observerSetup: PushIntentObserver.CallbacksSetup): OnAppCreate
 
-  @Provides
-  @IntoMap
-  @StringKey("refresh_config")
-  internal fun refreshConfigCommand(command: RefreshConfigCommand): PushActionCommand {
-    return command
-  }
+    @Binds
+    @IntoSet
+    internal abstract fun setupNotificationsOnCreate(notificationSetup: NotificationSetup): OnAppCreate
 
-  @Provides
-  @IntoMap
-  @StringKey("set_config_as_property")
-  internal fun configAsPropertyCommand(command: ConfigAsPropertyCommand): PushActionCommand {
-    return command
-  }
+    @Binds
+    @IntoMap
+    @ClassKey(PushHandleModel::class)
+    internal abstract fun bindServiceModel(pushHandleModel: PushHandleModel): ServiceModel
 
-  @Provides
-  @IntoMap
-  @StringKey("set_analytics_property")
-  internal fun setAnalyticsProperty(command: SetAnalyticsPropertyCommand): PushActionCommand {
-    return command
-  }
+    @Binds
+    @IntoMap
+    @StringKey("refresh_config")
+    internal abstract fun refreshConfigCommand(command: RefreshConfigCommand): PushActionCommand
 
-  @Provides
-  @IntoMap
-  @StringKey("notification")
-  internal fun notificationCommand(command: ShowNotificationCommand): PushActionCommand {
-    return command
-  }
+    @Binds
+    @IntoMap
+    @StringKey("set_config_as_property")
+    internal abstract fun configAsPropertyCommand(command: ConfigAsPropertyCommand): PushActionCommand
 
-  @Provides
-  @IntoMap
-  @StringKey("launch_deep_link")
-  internal fun deepLinkCommand(command: LaunchDeepLinkCommand): PushActionCommand {
-    return command
-  }
+    @Binds
+    @IntoMap
+    @StringKey("set_analytics_property")
+    internal abstract fun setAnalyticsProperty(command: SetAnalyticsPropertyCommand): PushActionCommand 
 
-  @Provides
-  @Singleton
-  internal fun firebaseDatabase(): FirebaseDatabase {
-    return FirebaseDatabase.getInstance()
+    @Binds
+    @IntoMap
+    @StringKey("notification")
+    internal abstract fun notificationCommand(command: ShowNotificationCommand): PushActionCommand 
+
+    @Binds
+    @IntoMap
+    @StringKey("launch_deep_link")
+    internal abstract fun deepLinkCommand(command: LaunchDeepLinkCommand): PushActionCommand 
+
+    @Binds
+    @IntoMap
+    @StringKey("press_back")
+    internal abstract fun pressBackCommand(command: PressBackButtonCommand): PushActionCommand
   }
 }