Remove collectEvents from SessionDetails, use sessionGenerator.collectEvents

Author: mrober

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/FirebaseSessions.kt

@@ -62,7 +62,7 @@ internal constructor(
     val sessionDetails = sessionGenerator.generateNewSession()
     val sessionEvent = SessionEvents.startSession(firebaseApp, sessionDetails, sessionSettings)
 
-    if (!sessionDetails.collectEvents) {
+    if (!sessionGenerator.collectEvents) {
       Log.d(TAG, "Session Start event sampled.")
       return
     }

firebase-sessions/src/main/kotlin/com/google/firebase/sessions/SessionGenerator.kt

@@ -26,7 +26,6 @@ import java.util.UUID
 internal data class SessionDetails(
   val sessionId: String,
   val firstSessionId: String,
-  val collectEvents: Boolean,
   val sessionIndex: Int,
 )
 
@@ -37,7 +36,7 @@ internal data class SessionDetails(
  * @hide
  */
 internal class SessionGenerator(
-  private val collectEvents: Boolean,
+  val collectEvents: Boolean,
   private val uuidGenerator: () -> UUID = UUID::randomUUID
 ) {
   private val firstSessionId = generateSessionId()
@@ -54,7 +53,6 @@ internal class SessionGenerator(
       SessionDetails(
         sessionId = if (sessionIndex == 0) firstSessionId else generateSessionId(),
         firstSessionId,
-        collectEvents,
         sessionIndex,
       )
     return currentSession

firebase-sessions/src/test/kotlin/com/google/firebase/sessions/SessionGeneratorTest.kt

@@ -72,7 +72,6 @@ class SessionGeneratorTest {
         SessionDetails(
           sessionId = SESSION_ID_1,
           firstSessionId = SESSION_ID_1,
-          collectEvents = true,
           sessionIndex = 0,
         )
       )
@@ -94,7 +93,6 @@ class SessionGeneratorTest {
         SessionDetails(
           sessionId = SESSION_ID_1,
           firstSessionId = SESSION_ID_1,
-          collectEvents = true,
           sessionIndex = 0,
         )
       )
@@ -110,7 +108,6 @@ class SessionGeneratorTest {
         SessionDetails(
           sessionId = SESSION_ID_2,
           firstSessionId = SESSION_ID_1,
-          collectEvents = true,
           sessionIndex = 1,
         )
       )
@@ -126,7 +123,6 @@ class SessionGeneratorTest {
         SessionDetails(
           sessionId = SESSION_ID_3,
           firstSessionId = SESSION_ID_1,
-          collectEvents = true,
           sessionIndex = 2,
         )
       )

firebase-sessions/src/test/kotlin/com/google/firebase/sessions/testing/TestSessionEventData.kt

@@ -25,7 +25,6 @@ internal object TestSessionEventData {
     SessionDetails(
       sessionId = "a1b2c3",
       firstSessionId = "a1a1a1",
-      collectEvents = true,
       sessionIndex = 3,
     )