Add Konsist test to use isEmpty() instead of isEqualTo(empty), and fix existing issue.
This commit is contained in:
parent
7c58a843dc
commit
bd58d85b69
2 changed files with 13 additions and 1 deletions
|
|
@ -39,7 +39,7 @@ class MapWebkitPermissionsTest {
|
||||||
@Test
|
@Test
|
||||||
fun `given any other permission, it returns nothing`() {
|
fun `given any other permission, it returns nothing`() {
|
||||||
val permission = mapWebkitPermissions(arrayOf(PermissionRequest.RESOURCE_PROTECTED_MEDIA_ID))
|
val permission = mapWebkitPermissions(arrayOf(PermissionRequest.RESOURCE_PROTECTED_MEDIA_ID))
|
||||||
assertThat(permission).isEqualTo(emptyList<String>())
|
assertThat(permission).isEmpty()
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -86,4 +86,16 @@ class KonsistTestTest {
|
||||||
functionDeclaration.text.contains("isEqualTo(true)")
|
functionDeclaration.text.contains("isEqualTo(true)")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `use isEmpty() instead of isEqualTo(empty)`() {
|
||||||
|
Konsist
|
||||||
|
.scopeFromTest()
|
||||||
|
.functions()
|
||||||
|
// Exclude self
|
||||||
|
.withoutName("use isEmpty() instead of isEqualTo(empty)")
|
||||||
|
.assertFalse { functionDeclaration ->
|
||||||
|
functionDeclaration.text.contains("isEqualTo(empty")
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue