Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Add Konsist test to use isNull() instead of isEqualTo(null), and fix existing issue.

Browse source
This commit is contained in:
Benoit Marty 2023-12-04 15:30:57 +01:00
parent bd58d85b69
commit a77ad88001
5 changed files with 19 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

4
features/messages/impl/src/test/kotlin/io/element/android/features/messages/impl/timeline/components/reactionsummary/ReactionSummaryPresenterTests.kt
View file

@ -53,7 +53,7 @@ class ReactionSummaryPresenterTests {
presenter.present()
}.test {
val initialState = awaitItem()
assertThat(initialState.target).isEqualTo(null)
assertThat(initialState.target).isNull()
initialState.eventSink(summaryEvent)
assertThat(awaitItem().target).isNotNull()
@ -69,7 +69,7 @@ class ReactionSummaryPresenterTests {
presenter.present()
}.test {
val initialState = awaitItem()
assertThat(initialState.target).isEqualTo(null)
assertThat(initialState.target).isNull()
initialState.eventSink(summaryEvent)
val reactions = awaitItem().target?.reactions

2
features/messages/impl/src/test/kotlin/io/element/android/features/messages/impl/timeline/model/InReplyToMetadataKtTest.kt
View file

@ -284,7 +284,7 @@ class InReplyToMetadataKtTest {
).metadata()
}.test {
awaitItem().let {
assertThat(it).isEqualTo(null)
assertThat(it).isNull()
}
}
}

8
features/messages/impl/src/test/kotlin/io/element/android/features/messages/impl/voicemessages/timeline/DefaultVoiceMessagePlayerTest.kt
View file

@ -149,7 +149,7 @@ class DefaultVoiceMessagePlayerTest {
assertThat(it.isPlaying).isFalse()
assertThat(it.isEnded).isFalse()
assertThat(it.currentPosition).isEqualTo(0)
assertThat(it.duration).isEqualTo(null)
assertThat(it.duration).isNull()
}
matchReadyState(1_000L)
player2.play()
@ -177,7 +177,7 @@ class DefaultVoiceMessagePlayerTest {
assertThat(it.isPlaying).isFalse()
assertThat(it.isEnded).isFalse()
assertThat(it.currentPosition).isEqualTo(0)
assertThat(it.duration).isEqualTo(null)
assertThat(it.duration).isNull()
}
matchReadyState(1_000L)
player1.play()
@ -238,7 +238,7 @@ class DefaultVoiceMessagePlayerTest {
assertThat(it.isPlaying).isFalse()
assertThat(it.isEnded).isFalse()
assertThat(it.currentPosition).isEqualTo(2000)
assertThat(it.duration).isEqualTo(null)
assertThat(it.duration).isNull()
}
assertThat(player.prepare().isSuccess).isTrue()
awaitItem().let {
@ -300,7 +300,7 @@ private suspend fun TurbineTestContext<VoiceMessagePlayer.State>.matchInitialSta
assertThat(it.isPlaying).isFalse()
assertThat(it.isEnded).isFalse()
assertThat(it.currentPosition).isEqualTo(0)
assertThat(it.duration).isEqualTo(null)
assertThat(it.duration).isNull()
}
}