Remove deprecated ktx packages from sessions

[mrober]

No description provided.

[github-actions]

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

[google-oss-bot]

Coverage Report ¹

Affected Products

• firebase-sessions

  Overall coverage changed from ? (c033245) to 75.04% (b450898) by ?.

  20 individual files with coverage change

  Filename	Base (c033245)	Merge (b450898)	Diff
  ApplicationInfo.kt	?	100.00%	?
  AutoSessionEventEncoder.java	?	100.00%	?
  EventGDTLogger.kt	?	75.00%	?
  FirebaseSessions.kt	?	0.00%	?
  FirebaseSessionsDependencies.kt	?	91.30%	?
  FirebaseSessionsRegistrar.kt	?	76.00%	?
  LocalOverrideSettings.kt	?	100.00%	?
  RemoteSettings.kt	?	88.06%	?
  RemoteSettingsFetcher.kt	?	65.85%	?
  SessionCoordinator.kt	?	75.00%	?
  SessionEvent.kt	?	100.00%	?
  SessionEvents.kt	?	97.78%	?
  SessionGenerator.kt	?	91.67%	?
  SessionInitiateListener.kt	?	0.00%	?
  SessionInitiator.kt	?	74.19%	?
  SessionsSettings.kt	?	70.45%	?
  SessionSubscriber.kt	?	75.00%	?
  SettingsCache.kt	?	94.83%	?
  SettingsProvider.kt	?	50.00%	?
  Time.kt	?	0.00%	?

Test Logs

• Merge (b450898): https://github.com/firebase/firebase-android-sdk/actions/runs/6473795893

1. https://storage.googleapis.com/firebase-sdk-metric-reports/upHL779cdl.html

[github-actions]

Unit Test Results

   136 files  +     74     136 suites  +74   4m 30s ⏱️ - 3m 27s
1 044 tests +   571  1 044 ✔️ +   571  0 💤 ±0  0 ❌ ±0 
2 088 runs  +1 136  2 088 ✔️ +1 136  0 💤 ±0  0 ❌ ±0 

Results for commit 84b728f. ± Comparison against base commit c033245.

This pull request removes 473 and adds 1044 tests. Note that renamed tests count towards both.

com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testBindsService_oAndTargetingO
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testNoWrappedIntent
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testNullIntent
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_OTargetingO_highPriority
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_fallsBackToBindService
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[19]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[21]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[22]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[23]
…

com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_disabledAnrs_doesNotPersistsAppExitInfo
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_doesNotPersistsAppExitInfoIfItDoesntExist
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_persistsAppExitInfoIfItExists
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAnrBeforeSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAppExitInfoNotAnrButWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession_multipleAppExitInfo
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testconvertInputStreamToString_worksSuccessfully
com.google.firebase.crashlytics.internal.model.CrashlyticsReportTest ‑ testGetBinaryImageUuidUtf8Bytes_returnsNullWhenUuidIsNull
com.google.firebase.crashlytics.internal.model.CrashlyticsReportTest ‑ testGetBinaryImageUuidUtf8Bytes_returnsProperBytes
…

[google-oss-bot]

Startup Time Report ¹

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Startup time comparison between the CI merge commit (b450898) and the base commit (c033245) are not available.

No macrobenchmark data found for the base commit (c033245). Analysis for the CI merge commit (b450898) can be found at:

• https://storage.googleapis.com/firebase-sdk-metric-reports/BF8sRU8Iso/index.html

1. https://storage.googleapis.com/firebase-sdk-metric-reports/6IsvLk6d5F/index.html

[google-oss-bot]

Size Report ¹

Affected Products

• firebase-sessions

  Type	Base (c033245)	Merge (b450898)	Diff
  aar	108 kB	108 kB	-2 B (-0.0%)
  apk (release)	2.07 MB	2.07 MB	+8 B (+0.0%)

Test Logs

• Base (c033245): https://github.com/firebase/firebase-android-sdk/actions/runs/6473730451
• Merge (b450898): https://github.com/firebase/firebase-android-sdk/actions/runs/6473795893

1. https://storage.googleapis.com/firebase-sdk-metric-reports/ytYf5qlvWl.html

[VinayGuthal]

We would need dependency on both firebase-common and firebase-common-ktx. This will prevent issues wrt collision when used with the old version of firebase-common-ktx