Minor cleanup
This commit is contained in:
parent
5f84e96b11
commit
3ed294bf72
2 changed files with 12 additions and 11 deletions
|
@ -152,6 +152,7 @@ publishing {
|
||||||
id.set("gbevin")
|
id.set("gbevin")
|
||||||
name.set("Geert Bevin")
|
name.set("Geert Bevin")
|
||||||
email.set("gbevin@uwyn.com")
|
email.set("gbevin@uwyn.com")
|
||||||
|
url.set("https://github.com/gbevin")
|
||||||
}
|
}
|
||||||
developer {
|
developer {
|
||||||
id.set("ethauvin")
|
id.set("ethauvin")
|
||||||
|
|
|
@ -56,18 +56,18 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
@ParameterizedTest(name = "decode({0}) should be {1}")
|
@ParameterizedTest(name = "decode({0}) should be {1}")
|
||||||
@MethodSource("validMap")
|
@MethodSource("validMap")
|
||||||
fun `Decode Multiple URLs`(expected: String, source: String) {
|
fun `Decode URL`(expected: String, source: String) {
|
||||||
assertEquals(expected, decode(source))
|
assertEquals(expected, decode(source))
|
||||||
}
|
}
|
||||||
|
|
||||||
@ParameterizedTest(name = "decode({0})")
|
@ParameterizedTest(name = "decode({0})")
|
||||||
@MethodSource("invalid")
|
@MethodSource("invalid")
|
||||||
fun `Decode with Exceptions`(source: String) {
|
fun `Decode with Exception`(source: String) {
|
||||||
assertThrows(IllegalArgumentException::class.java, { decode(source) }, "decode($source)")
|
assertThrows(IllegalArgumentException::class.java, { decode(source) }, "decode($source)")
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun `Decode when none needed`() {
|
fun `Decode when None needed`() {
|
||||||
assertSame(same, decode(same))
|
assertSame(same, decode(same))
|
||||||
assertEquals("", decode(""), "decode('')")
|
assertEquals("", decode(""), "decode('')")
|
||||||
assertEquals(" ", decode(" "), "decode(' ')")
|
assertEquals(" ", decode(" "), "decode(' ')")
|
||||||
|
@ -75,7 +75,7 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
@ParameterizedTest(name = "encode({0}) should be {1}")
|
@ParameterizedTest(name = "encode({0}) should be {1}")
|
||||||
@MethodSource("validMap")
|
@MethodSource("validMap")
|
||||||
fun `Encode Multiple URLs`(source: String, expected: String) {
|
fun `Encode URL`(source: String, expected: String) {
|
||||||
assertEquals(expected, encode(source))
|
assertEquals(expected, encode(source))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -87,13 +87,13 @@ class UrlEncoderTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun `Encode when none needed`() {
|
fun `Encode when None needed`() {
|
||||||
assertSame(same, encode(same))
|
assertSame(same, encode(same))
|
||||||
assertSame(same, encode(same, ""), "with empty allow")
|
assertSame(same, encode(same, ""), "with empty allow")
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun `Encode with allow arg`() {
|
fun `Encode with Allow Arg`() {
|
||||||
assertEquals("?test=a%20test", encode("?test=a test", '=', '?'), "encode(x, =, ?)")
|
assertEquals("?test=a%20test", encode("?test=a test", '=', '?'), "encode(x, =, ?)")
|
||||||
assertEquals("?test=a%20test", encode("?test=a test", "=?"), "encode(x, =?)")
|
assertEquals("?test=a%20test", encode("?test=a test", "=?"), "encode(x, =?)")
|
||||||
assertEquals("aaa", encode("aaa", 'a'), "encode(aaa, a)")
|
assertEquals("aaa", encode("aaa", 'a'), "encode(aaa, a)")
|
||||||
|
@ -110,7 +110,7 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
@ParameterizedTest(name = "processMain(-d {0})")
|
@ParameterizedTest(name = "processMain(-d {0})")
|
||||||
@MethodSource("invalid")
|
@MethodSource("invalid")
|
||||||
fun `Main Decode with Exceptions`(source: String) {
|
fun `Main Decode with Exception`(source: String) {
|
||||||
assertThrows(IllegalArgumentException::class.java, { processMain(arrayOf("-d", source)) }, source)
|
assertThrows(IllegalArgumentException::class.java, { processMain(arrayOf("-d", source)) }, source)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -124,7 +124,7 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
@ParameterizedTest(name = "processMain(-e {0})")
|
@ParameterizedTest(name = "processMain(-e {0})")
|
||||||
@MethodSource("validMap")
|
@MethodSource("validMap")
|
||||||
fun `Main Encode with option`(source: String, expected: String) {
|
fun `Main Encode with Option`(source: String, expected: String) {
|
||||||
val result = processMain(arrayOf("-e", source))
|
val result = processMain(arrayOf("-e", source))
|
||||||
assertEquals(expected, result.output)
|
assertEquals(expected, result.output)
|
||||||
assertEquals(0, result.status, "processMain(-e $source).status")
|
assertEquals(0, result.status, "processMain(-e $source).status")
|
||||||
|
@ -132,7 +132,7 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun `Main Usage with Empty args`() {
|
fun `Main Usage with Empty Args`() {
|
||||||
assertEquals(usage, processMain(arrayOf(" ", " ")).output, "processMain(' ', ' ')")
|
assertEquals(usage, processMain(arrayOf(" ", " ")).output, "processMain(' ', ' ')")
|
||||||
assertEquals(usage, processMain(arrayOf("foo", " ")).output, "processMain('foo', ' ')")
|
assertEquals(usage, processMain(arrayOf("foo", " ")).output, "processMain('foo', ' ')")
|
||||||
assertEquals(usage, processMain(arrayOf(" ", "foo")).output, "processMain(' ', 'foo')")
|
assertEquals(usage, processMain(arrayOf(" ", "foo")).output, "processMain(' ', 'foo')")
|
||||||
|
@ -143,14 +143,14 @@ class UrlEncoderTest {
|
||||||
|
|
||||||
@ParameterizedTest
|
@ParameterizedTest
|
||||||
@ValueSource(strings = ["", "-d", "-e"])
|
@ValueSource(strings = ["", "-d", "-e"])
|
||||||
fun `Main Usage with invalid arg`(arg: String) {
|
fun `Main Usage with Invalid arg`(arg: String) {
|
||||||
val result = processMain(arrayOf(arg))
|
val result = processMain(arrayOf(arg))
|
||||||
assertEquals(usage, result.output, "processMain('$arg')")
|
assertEquals(usage, result.output, "processMain('$arg')")
|
||||||
assertEquals(1, result.status, "processMain('$arg').status")
|
assertEquals(1, result.status, "processMain('$arg').status")
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
fun `Main Usage with too many args`() {
|
fun `Main Usage with too Many Args`() {
|
||||||
assertEquals(usage, processMain(arrayOf("foo", "bar", "test")).output, "too many args")
|
assertEquals(usage, processMain(arrayOf("foo", "bar", "test")).output, "too many args")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue