Bug 1880476 — Reduce visibility of fields in Message

app/src/main/java/org/mozilla/fenix/HomeActivity.kt

@@ -1229,8 +1229,8 @@ open class HomeActivity : LocaleAwareAppCompatActivity(), NavHostActivity {
 
         val fenixNimbusMessagingController = FenixNimbusMessagingController(messagingStorage)
         val researchSurfaceDialogFragment = ResearchSurfaceDialogFragment.newInstance(
-            keyMessageText = nextMessage.data.text,
+            keyMessageText = nextMessage.text,
-            keyAcceptButtonText = nextMessage.data.buttonLabel,
+            keyAcceptButtonText = nextMessage.buttonLabel,
             keyDismissButtonText = null,
         )
 

app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/onboarding/MessageCardViewHolder.kt

@@ -77,9 +77,9 @@ class MessageCardViewHolder(
         )
 
         MessageCard(
-            messageText = message.data.text,
+            messageText = message.text,
-            titleText = message.data.title,
+            titleText = message.title,
-            buttonText = message.data.buttonLabel,
+            buttonText = message.buttonLabel,
             messageColors = messageCardColors,
             onClick = { interactor.onMessageClicked(message) },
             onCloseButtonClick = { interactor.onMessageClosedClicked(message) },

app/src/main/java/org/mozilla/fenix/messaging/MessageNotificationWorker.kt

@@ -52,9 +52,8 @@ class MessageNotificationWorker(
                 ?: return Result.success()
 
         val currentBootUniqueIdentifier = BootUtils.getBootIdentifier(context)
-        val messageMetadata = nextMessage.metadata
         //  Device has NOT been power cycled.
-        if (messageMetadata.latestBootIdentifier == currentBootUniqueIdentifier) {
+        if (nextMessage.hasShownThisCycle(currentBootUniqueIdentifier)) {
             return Result.success()
         }
 
@@ -91,8 +90,8 @@ class MessageNotificationWorker(
         return createBaseNotification(
             context,
             ensureMarketingChannelExists(context),
-            message.data.title,
+            message.title,
-            message.data.text,
+            message.text,
             onClickPendingIntent,
             onDismissPendingIntent,
         )
@@ -189,7 +188,7 @@ class NotificationDismissedService : LifecycleService() {
 
                 if (message != null) {
                     // Update message as 'dismissed'.
-                    nimbusMessagingController.onMessageDismissed(message.metadata)
+                    nimbusMessagingController.onMessageDismissed(message)
                 }
             }
         }
@@ -220,7 +219,7 @@ class NotificationClickedReceiverActivity : ComponentActivity() {
 
             if (message != null) {
                 // Update message as 'clicked'.
-                nimbusMessagingController.onMessageClicked(message.metadata)
+                nimbusMessagingController.onMessageClicked(message)
 
                 // Create the intent.
                 val intent = nimbusMessagingController.getIntentForMessage(message)

app/src/main/java/org/mozilla/fenix/messaging/state/MessagingMiddleware.kt

@@ -93,7 +93,7 @@ class MessagingMiddleware(
         context.dispatch(UpdateMessages(newMessages))
         consumeMessageToShowIfNeeded(context, message)
         coroutineScope.launch {
-            controller.onMessageDismissed(message.metadata)
+            controller.onMessageDismissed(message)
         }
     }
 
@@ -103,7 +103,7 @@ class MessagingMiddleware(
     ) {
         // Update Nimbus storage.
         coroutineScope.launch {
-            controller.onMessageClicked(message.metadata)
+            controller.onMessageClicked(message)
         }
         // Update app state.
         val newMessages = removeMessage(context, message)

app/src/test/java/org/mozilla/fenix/messaging/state/MessagingMiddlewareTest.kt

@@ -221,7 +221,7 @@ class MessagingMiddlewareTest {
         store.dispatch(Evaluate(FenixMessageSurfaceId.HOMESCREEN))
         store.waitUntilIdle()
 
-        assertEquals(1, store.state.messaging.messages.first().metadata.displayCount)
+        assertEquals(1, store.state.messaging.messages.first().displayCount)
     }
 
     @Test
@@ -286,7 +286,7 @@ class MessagingMiddlewareTest {
         store.dispatch(Evaluate(FenixMessageSurfaceId.HOMESCREEN)).joinBlocking()
         store.waitUntilIdle()
 
-        assertEquals(messageDisplayed.metadata.displayCount, store.state.messaging.messages[0].metadata.displayCount)
+        assertEquals(messageDisplayed.displayCount, store.state.messaging.messages[0].displayCount)
     }
 
     @Test