Browse Source

Small quality fixes

pull/3012/head
Benoit Marty 3 months ago
parent
commit
f87047b899
  1. 2
      features/call/impl/src/main/kotlin/io/element/android/features/call/impl/utils/ElementCallBaseUrlProvider.kt
  2. 6
      features/call/impl/src/test/kotlin/io/element/android/features/call/utils/DefaultCallWidgetProviderTest.kt

2
features/call/impl/src/main/kotlin/io/element/android/features/call/impl/utils/ElementCallBaseUrlProvider.kt

@ -46,7 +46,7 @@ class DefaultElementCallBaseUrlProvider @Inject constructor( @@ -46,7 +46,7 @@ class DefaultElementCallBaseUrlProvider @Inject constructor(
} catch (e: HttpException) {
Timber.w(e, "Failed to fetch wellknown data")
// Ignore Http 404, but re-throws any other exceptions
if (e.code() != HttpURLConnection.HTTP_NOT_FOUND /* 404 */) {
if (e.code() != HttpURLConnection.HTTP_NOT_FOUND) {
throw e
}
null

6
features/call/impl/src/test/kotlin/io/element/android/features/call/utils/DefaultCallWidgetProviderTest.kt

@ -116,9 +116,9 @@ class DefaultCallWidgetProviderTest { @@ -116,9 +116,9 @@ class DefaultCallWidgetProviderTest {
@Test
fun `getWidget - will use a wellknown base url if it exists`() = runTest {
val aCustomUrl = "https://custom.element.io"
val provideLambda = lambdaRecorder<SessionId, String?> { String -> aCustomUrl }
val providesLambda = lambdaRecorder<SessionId, String?> { _ -> aCustomUrl }
val elementCallBaseUrlProvider = FakeElementCallBaseUrlProvider { sessionId ->
provideLambda(sessionId)
providesLambda(sessionId)
}
val room = FakeMatrixRoom().apply {
givenGenerateWidgetWebViewUrlResult(Result.success("url"))
@ -135,7 +135,7 @@ class DefaultCallWidgetProviderTest { @@ -135,7 +135,7 @@ class DefaultCallWidgetProviderTest {
)
provider.getWidget(A_SESSION_ID, A_ROOM_ID, "clientId", "languageTag", "theme")
assertThat(settingsProvider.providedBaseUrls).containsExactly(aCustomUrl)
provideLambda.assertions()
providesLambda.assertions()
.isCalledOnce()
.with(value(A_SESSION_ID))
}

Loading…
Cancel
Save