Fix ClassNotFoundException for com.google.firebase.remoteconfig.ktx.…

[b95505017]

…FirebaseConfigKtxRegistrar

Should be FirebaseRemoteConfigKtxRegistrar

[google-oss-bot]

Hi @b95505017. Thanks for your PR.

I'm waiting for a firebase member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[rlazo]

Hi @b95505017 Great catch! Please, also add to your PR the missing test that should have caught this earlier.

 @RunWith(RobolectricTestRunner::class)
  class LibraryVersionTest : BaseTestCase() {
      @Test
      fun `library version should be registered with runtime`() {
          val publisher = Firebase.app.get(UserAgentPublisher::class.java)
          assertThat(publisher.userAgent).contains(LIBRARY_NAME)
      }
  }

Thank you for contributing to Firebase!

[rlazo]

/ok-to-test

[codecov]

Codecov Report

Merging #1209 into master will increase coverage by 45.34%.
The diff coverage is n/a.

Flag	Coverage Δ	Complexity Δ
#Encoders_FirebaseEncodersJson	?	?
#Encoders_FirebaseEncodersProcessor	?	?
#Encoders_FirebaseEncodersReflective	?	?
#FirebaseAbt	?	?
#FirebaseCommon	?	?
#FirebaseCommon_DataCollectionTests	?	?
#FirebaseCommon_Ktx	?	?
#FirebaseComponents	?	?
#FirebaseConfig	?	?
#FirebaseConfig_Ktx	100% <ø> (ø)	0 <ø> (ø)	⬇️
#FirebaseCrashlytics	?	?
#FirebaseDatabase	?	?
#FirebaseDatabaseCollection	?	?
#FirebaseDatabase_Ktx	?	?
#FirebaseDatatransport	?	?
#FirebaseDynamicLinks	?	?
#FirebaseDynamicLinks_Ktx	?	?
#FirebaseFirestore	?	?
#FirebaseFirestore_Ktx	?	?
#FirebaseFunctions	?	?
#FirebaseFunctions_Ktx	?	?
#FirebaseInappmessaging	?	?
#FirebaseInappmessagingDisplay	?	?
#FirebaseInappmessagingDisplay_Ktx	?	?
#FirebaseInappmessaging_Ktx	?	?
#FirebaseInstallations	?	?
#FirebaseSegmentation	?	?
#FirebaseStorage	?	?
#FirebaseStorage_Ktx	?	?
#Tools_Errorprone	?	?
#Tools_Lint	?	?
#Transport_TransportBackendCct	?	?
#Transport_TransportRuntime	?	?

Impacted Files	Coverage Δ	Complexity Δ
.../inappmessaging/internal/AbtIntegrationHelper.java
...atabase/collection/ImmutableSortedMapIterator.java
...ndroid/datatransport/cct/internal/LogResponse.java
.../main/java/com/google/firebase/firestore/Blob.java
...ng/internal/injection/modules/RateLimitModule.java
...om/google/firebase/firestore/SnapshotMetadata.java
...ytics/internal/settings/SettingsCacheBehavior.java
...ndroid/datatransport/runtime/TransportContext.java
...cs/internal/settings/network/UpdateAppSpiCall.java
...le/firebase/storage/network/GetNetworkRequest.java
... and 688 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79f91d7...192457d. Read the comment docs.

[wbervoets]

any update when this will be fixed in a released version?

[rlazo]

Thanks for the heads up @wbervoets I'll approve and merge this CL as-is, and add the missing test in a follow-up PR. Thanks for your contribution to Firebase @b95505017 !