Improved tests.
This commit is contained in:
parent
5d5678aa79
commit
c2dfee4952
1 changed files with 37 additions and 31 deletions
|
@ -178,7 +178,7 @@ class AkismetTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun checkMockComment() {
|
fun mockCommentTest() {
|
||||||
with(mockComment) {
|
with(mockComment) {
|
||||||
assertEquals(userIp, comment.userIp, "userIp")
|
assertEquals(userIp, comment.userIp, "userIp")
|
||||||
assertEquals(userAgent, comment.userAgent, "userAgent")
|
assertEquals(userAgent, comment.userAgent, "userAgent")
|
||||||
|
@ -214,36 +214,40 @@ class AkismetTest {
|
||||||
assertEquals(blogCharset, "", "blogCharset")
|
assertEquals(blogCharset, "", "blogCharset")
|
||||||
assertEquals(userRole, "", "userRole")
|
assertEquals(userRole, "", "userRole")
|
||||||
assertEquals(recheckReason, "", "recheckReason")
|
assertEquals(recheckReason, "", "recheckReason")
|
||||||
assertEquals(serverEnv.size, 0, "serverEnv")
|
assertEquals(serverEnv.size, 0, "serverEnv size")
|
||||||
|
|
||||||
referrer = null
|
for (s in listOf("test", "", null)) {
|
||||||
permalink = null
|
referrer = s
|
||||||
type = null
|
permalink = s
|
||||||
authorEmail = null
|
type = s
|
||||||
author = null
|
authorEmail = s
|
||||||
authorUrl = null
|
author = s
|
||||||
content = null
|
authorUrl = s
|
||||||
dateGmt = null
|
content = s
|
||||||
postModifiedGmt = null
|
dateGmt = s
|
||||||
blogLang = null
|
postModifiedGmt = s
|
||||||
blogCharset = null
|
blogLang = s
|
||||||
userRole = null
|
blogCharset = s
|
||||||
recheckReason = null
|
userRole = s
|
||||||
|
recheckReason = s
|
||||||
|
|
||||||
assertEquals(referrer, "", "referrer null")
|
val expected = if (s.isNullOrEmpty()) "" else s
|
||||||
assertEquals(permalink, "", "permalink null")
|
|
||||||
assertEquals(type, "", "type null")
|
assertEquals(referrer, expected, "referrer: [$s]")
|
||||||
assertEquals(authorEmail, "", "authorEmail null")
|
assertEquals(permalink, expected, "permalink: [$s]")
|
||||||
assertEquals(author, "", "author null")
|
assertEquals(type, expected, "type: [$s]")
|
||||||
assertEquals(authorUrl, "", "authorUrl null")
|
assertEquals(authorEmail, expected, "authorEmail: [$s]")
|
||||||
assertEquals(content, "", "content null")
|
assertEquals(author, expected, "author: [$s]")
|
||||||
assertEquals(dateGmt, "", "dateGmt null")
|
assertEquals(authorUrl, expected, "authorUrl: [$s]")
|
||||||
assertEquals(postModifiedGmt, "", "postModifiedGmt null")
|
assertEquals(content, expected, "content: [$s]")
|
||||||
assertEquals(blogLang, "", "blogLang null")
|
assertEquals(dateGmt, expected, "dateGmt: [$s]")
|
||||||
assertEquals(blogCharset, "", "blogCharset null")
|
assertEquals(postModifiedGmt, expected, "postModifiedGmt: [$s]")
|
||||||
assertEquals(userRole, "", "userRole null")
|
assertEquals(blogLang, expected, "blogLang: [$s]")
|
||||||
assertEquals(recheckReason, "", "recheckReason null")
|
assertEquals(blogCharset, expected, "blogCharset: [$s]")
|
||||||
assertEquals(serverEnv.size, 0, "serverEnv null")
|
assertEquals(userRole, expected, "userRole: [$s]")
|
||||||
|
assertEquals(recheckReason, expected, "recheckReason: [$s]")
|
||||||
|
assertEquals(serverEnv.size, 0, "serverEnv size: [$s]")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -290,9 +294,11 @@ class AkismetTest {
|
||||||
assertTrue(checkComment(comment), "check_comment -> true")
|
assertTrue(checkComment(comment), "check_comment -> true")
|
||||||
assertEquals(response, "true", "response -> true")
|
assertEquals(response, "true", "response -> true")
|
||||||
|
|
||||||
assertFalse(checkComment(mockComment), "check_comment(request) -> false")
|
assertFalse(checkComment(mockComment), "check_comment(mock) -> false")
|
||||||
|
assertEquals(response, "false", "mock response -> false")
|
||||||
mockComment.userRole = ""
|
mockComment.userRole = ""
|
||||||
assertTrue(checkComment(mockComment), "check_comment(request) -> true")
|
assertTrue(checkComment(mockComment), "check_comment(mock) -> true")
|
||||||
|
assertEquals(response, "true", "mock response -> true")
|
||||||
|
|
||||||
assertEquals(httpStatusCode, 200, "status code")
|
assertEquals(httpStatusCode, 200, "status code")
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue