Merge pull request #86 from LachlanMcKee/issue-85-improve-interop-vie…

…w-error-message

[#85] Improved InteropViewImpl error messaging

CHANGELOG.md

@@ -9,6 +9,7 @@
 - [#81](https://github.com/bumble-tech/appyx/issues/81) – Added: Support integration point for multiple roots
 - [#65](https://github.com/bumble-tech/appyx/pull/65) – Added: `InteropBuilderStub` and `InteropSimpleBuilderStub` testing util classes
 - [#47](https://github.com/bumble-tech/appyx/issues/47) – Updated: The 'customisations' module is now pure Java/Kotlin.
+- [#85](https://github.com/bumble-tech/appyx/issues/85) – Updated: Improved `InteropView` error messaging when `Activity` does not implement `IntegrationPointAppyxProvider`
 
 
 ## 1.0-alpha03

interop-ribs/src/main/java/com/bumble/appyx/interop/ribs/InteropView.kt

@@ -1,7 +1,7 @@
 package com.bumble.appyx.interop.ribs
 
+import android.app.Activity
 import android.content.Context
-import androidx.appcompat.app.AppCompatActivity
 import androidx.compose.runtime.Composable
 import com.badoo.ribs.compose.ComposeRibView
 import com.badoo.ribs.compose.ComposeView
@@ -46,9 +46,13 @@ internal class InteropViewImpl private constructor(
     }
 
     private fun retrieveIntegrationPoint(): IntegrationPoint {
-        val activity = context.findActivity<AppCompatActivity>()
+        val activity = context.findActivity<Activity>()
+        check(activity != null) {
+            "Could not find an activity from the context: $context"
+        }
         check(activity is IntegrationPointAppyxProvider) {
-            "Activity where InteropNode is used must implement IntegrationPointAppyxProvider"
+            "Activity where InteropNode is used must implement IntegrationPointAppyxProvider. " +
+                    "Activity: '${activity::class.java.name}', Node: '${appyxNode::class.java.name}'"
         }
         return activity.integrationPointAppyx
     }