Skip to content

Commit 55d0db1

Browse files
cortinicofacebook-github-bot
authored andcommitted
Minor cleanup of AbstractLayoutAnimationTest
Summary: This test was not using AssertJ. I've updated it then. Changelog: [Internal] [Changed] - Reviewed By: javache Differential Revision: D74065455
1 parent ec38728 commit 55d0db1

1 file changed

Lines changed: 5 additions & 8 deletions

File tree

packages/react-native/ReactAndroid/src/test/java/com/facebook/react/uimanager/layoutanimation/AbstractLayoutAnimationTest.kt

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ import android.view.animation.LinearInterpolator
1616
import com.facebook.react.bridge.ReadableMap
1717
import com.facebook.react.uimanager.IllegalViewOperationException
1818
import org.assertj.core.api.Assertions.assertThat
19+
import org.assertj.core.api.Assertions.assertThatThrownBy
1920
import org.junit.After
2021
import org.junit.Assert.assertThrows
2122
import org.junit.Before
@@ -53,9 +54,7 @@ class AbstractLayoutAnimationTest {
5354
y: Int,
5455
width: Int,
5556
height: Int
56-
): Animation {
57-
return mock()
58-
}
57+
): Animation = mock()
5958
}
6059
}
6160

@@ -129,11 +128,9 @@ class AbstractLayoutAnimationTest {
129128
): Animation = mock()
130129
}
131130

132-
val exception =
133-
assertThrows(IllegalViewOperationException::class.java) {
134-
invalidAnimation.initializeFromConfig(config, 300)
135-
}
136-
assertThat(exception.message).contains("Invalid layout animation")
131+
assertThatThrownBy { invalidAnimation.initializeFromConfig(config, 300) }
132+
.isInstanceOf(IllegalViewOperationException::class.java)
133+
.hasMessageContaining("Invalid layout animation")
137134
}
138135

139136
@Test

0 commit comments

Comments
 (0)