Updated instances of assert() in LatLngTest.kt to asserThat().

New imports:
 - org.hamcrest.CoreMatchers.equalTo
 - import org.hamcrest.CoreMatchers.not
 - org.hamcrest.CoreMatchers.assertThat
This commit is contained in:
u7683345 2024-10-15 20:47:10 +11:00
parent e19dcd9af2
commit 336ee946c2

View file

@ -2,6 +2,9 @@ package fr.free.nrw.commons.location
import org.junit.Before import org.junit.Before
import org.junit.Test import org.junit.Test
import org.hamcrest.MatcherAssert.assertThat
import org.hamcrest.CoreMatchers.equalTo
import org.hamcrest.CoreMatchers.not
class LatLngTest { class LatLngTest {
private lateinit var latLng1: LatLng private lateinit var latLng1: LatLng
@ -14,51 +17,51 @@ class LatLngTest {
@Test @Test
fun testConstructorSmallLongitude() { fun testConstructorSmallLongitude() {
latLng1 = LatLng(0.0, -181.0, 0.0f) latLng1 = LatLng(0.0, -181.0, 0.0f)
assert(latLng1.longitude == 179.0) assertThat(latLng1.longitude, equalTo(179.0))
} }
@Test @Test
fun testConstructorBigLongitude() { fun testConstructorBigLongitude() {
latLng1 = LatLng(0.0, 181.0, 0.0f) latLng1 = LatLng(0.0, 181.0, 0.0f)
assert(latLng1.longitude == -179.0) assertThat(latLng1.longitude, equalTo(179.0))
} }
@Test @Test
fun testConstructorSmallLatitude() { fun testConstructorSmallLatitude() {
latLng1 = LatLng(-91.0, 0.0, 0.0f) latLng1 = LatLng(-91.0, 0.0, 0.0f)
assert(latLng1.latitude == -90.0) assertThat(latLng1.latitude, equalTo(-90.0))
} }
@Test @Test
fun testConstructorBigLatitude() { fun testConstructorBigLatitude() {
latLng1 = LatLng(91.0, 0.0, 0.0f) latLng1 = LatLng(91.0, 0.0, 0.0f)
assert(latLng1.latitude == 90.0) assertThat(latLng1.latitude, equalTo(90.0))
} }
@Test @Test
fun testHashCodeDiffersWenLngZero() { fun testHashCodeDiffersWenLngZero() {
latLng1 = LatLng(2.0, 0.0, 0.0f) latLng1 = LatLng(2.0, 0.0, 0.0f)
latLng2 = LatLng(1.0, 0.0, 0.0f) latLng2 = LatLng(1.0, 0.0, 0.0f)
assert(latLng1.hashCode() != latLng2.hashCode()) assertThat(latLng1.hashCode(), not(equalTo(latLng2.hashCode())))
} }
@Test @Test
fun testHashCodeDiffersWenLatZero() { fun testHashCodeDiffersWenLatZero() {
latLng1 = LatLng(0.0, 1.0, 0.0f) latLng1 = LatLng(0.0, 1.0, 0.0f)
latLng2 = LatLng(0.0, 2.0, 0.0f) latLng2 = LatLng(0.0, 2.0, 0.0f)
assert(latLng1.hashCode() != latLng2.hashCode()) assertThat(latLng1.hashCode(), not(equalTo(latLng2.hashCode())))
} }
@Test @Test
fun testEqualsWorks() { fun testEqualsWorks() {
latLng1 = LatLng(1.0, 2.0, 5.0f) latLng1 = LatLng(1.0, 2.0, 5.0f)
latLng2 = LatLng(1.0, 2.0, 0.0f) latLng2 = LatLng(1.0, 2.0, 0.0f)
assert(latLng1.equals(latLng2)) assertThat(latLng1, equalTo(latLng2))
} }
@Test @Test
fun testToString() { fun testToString() {
latLng1 = LatLng(1.0, 2.0, 5.0f) latLng1 = LatLng(1.0, 2.0, 5.0f)
assert(latLng1.toString().equals("lat/lng: (1.0,2.0)")) assertThat(latLng1.toString(), equalTo("lat/lng: (1.0,2.0)"))
} }
} }