Skip to content

Commit

Permalink
fix: remove currentRoute parameter in in-app event listener (customer…
Browse files Browse the repository at this point in the history
  • Loading branch information
levibostian committed Jan 20, 2023
1 parent 9fa6f04 commit 688e4a5
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion messaginginapp/api/messaginginapp.api
Expand Up @@ -26,7 +26,7 @@ public final class io/customer/messaginginapp/ModuleMessagingInApp : io/customer

public abstract interface class io/customer/messaginginapp/type/InAppEventListener {
public abstract fun errorWithMessage (Lio/customer/messaginginapp/type/InAppMessage;)V
public abstract fun messageActionTaken (Lio/customer/messaginginapp/type/InAppMessage;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)V
public abstract fun messageActionTaken (Lio/customer/messaginginapp/type/InAppMessage;Ljava/lang/String;Ljava/lang/String;)V
public abstract fun messageDismissed (Lio/customer/messaginginapp/type/InAppMessage;)V
public abstract fun messageShown (Lio/customer/messaginginapp/type/InAppMessage;)V
}
Expand Down
Expand Up @@ -74,7 +74,7 @@ internal class GistInAppMessagesProvider(private val provider: GistApi) :
override fun embedMessage(message: Message, elementId: String) {}

override fun onAction(message: Message, currentRoute: String, action: String, name: String) {
listener?.messageActionTaken(InAppMessage.getFromGistMessage(message), currentRoute = currentRoute, action = action, name = name)
listener?.messageActionTaken(InAppMessage.getFromGistMessage(message), action = action, name = name)
}

override fun onError(message: Message) {
Expand Down
Expand Up @@ -4,5 +4,5 @@ interface InAppEventListener {
fun messageShown(message: InAppMessage)
fun messageDismissed(message: InAppMessage)
fun errorWithMessage(message: InAppMessage)
fun messageActionTaken(message: InAppMessage, currentRoute: String, action: String, name: String)
fun messageActionTaken(message: InAppMessage, action: String, name: String)
}
Expand Up @@ -179,7 +179,7 @@ internal class InAppMessagesProviderTest : BaseTest() {
val givenAction = String.random
val givenName = String.random
gistInAppMessagesProvider.onAction(givenMessage, givenCurrentRoute, givenAction, givenName)
verify(eventListenerMock).messageActionTaken(expectedInAppMessage, currentRoute = givenCurrentRoute, action = givenAction, name = givenName)
verify(eventListenerMock).messageActionTaken(expectedInAppMessage, action = givenAction, name = givenName)
}

@Test
Expand Down

0 comments on commit 688e4a5

Please sign in to comment.