Skip to content

Commit

Permalink
Fix redundant explicit type warnings in JSON test output
Browse files Browse the repository at this point in the history
Removed redundant explicit type warnings from multiple JSON test files. This cleanup ensures that the JSON outputs are free from unnecessary warnings, simplifying test result readability.
  • Loading branch information
zoobestik committed Oct 14, 2024
1 parent bb753da commit 2279468
Show file tree
Hide file tree
Showing 16 changed files with 16 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class KoansJUnitTestsRunnerTest : BaseJUnitTest() {
@Test
fun `koans test Default arguments`() {
runKoanTest(
"fun foo(name: String, number: Int = 42, toUpperCase: Boolean = false) =\n (if (toUpperCase) name.toUpperCase() else name) + number\n\nfun useFoo() = listOf(\n foo(\"a\"),\n foo(\"b\", number = 1),\n foo(\"c\", toUpperCase = true),\n foo(name = \"d\", number = 2, toUpperCase = true)\n)",
"fun foo(name: String, number: Int = 42, toUpperCase: Boolean = false) =\n (if (toUpperCase) name.uppercase() else name) + number\n\nfun useFoo() = listOf(\n foo(\"a\"),\n foo(\"b\", number = 1),\n foo(\"c\", toUpperCase = true),\n foo(name = \"d\", number = 2, toUpperCase = true)\n)",
"import org.junit.Test\nimport org.junit.Assert\n\nclass TestDefaultAndNamedParams() {\n\n @Test fun testDefaultAndNamedParams() {\n Assert.assertEquals(listOf(\"a42\", \"b1\", \"C42\", \"D2\"), useFoo())\n }\n}"
)
}
Expand Down

Large diffs are not rendered by default.

Large diffs are not rendered by default.

Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":11,"ch":19},"end":{"line":11,"ch":25}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":12,"ch":26},"end":{"line":12,"ch":32}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>Issue uniqueDescr4 registered correctly.\nIssue uniqueDescr2 marked as duplicate and rejected.\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>Issue uniqueDescr4 registered correctly.\nIssue uniqueDescr2 marked as duplicate and rejected.\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":4},"end":{"line":2,"ch":7}},"message":"The 'var' property is never written to, so it can be declared as 'val'.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":17}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":4,"ch":8},"end":{"line":4,"ch":9}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":4,"ch":11},"end":{"line":4,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":5,"ch":8},"end":{"line":5,"ch":9}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>initial\n</outStream>"}
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":4},"end":{"line":2,"ch":7}},"message":"The 'var' property is never written to, so it can be declared as 'val'.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":4,"ch":8},"end":{"line":4,"ch":9}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":5,"ch":8},"end":{"line":5,"ch":9}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>initial\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>a = 1, b = 2, c = 3\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>a = 1, b = 2, c = 3\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":6,"ch":11},"end":{"line":6,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":10,"ch":12},"end":{"line":10,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":11,"ch":12},"end":{"line":11,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\n</outStream>"}
{"errors":{"File.kt":[{"interval":{"start":{"line":10,"ch":12},"end":{"line":10,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":11,"ch":12},"end":{"line":11,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\ntrue\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\ntrue\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":16},"end":{"line":2,"ch":23}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":3,"ch":17},"end":{"line":3,"ch":24}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":4,"ch":8},"end":{"line":4,"ch":16}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\nfalse\n</outStream>"}
{"errors":{"File.kt":[{"interval":{"start":{"line":4,"ch":8},"end":{"line":4,"ch":16}},"message":"Variable is unused.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\nfalse\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":15},"end":{"line":2,"ch":19}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>a\n\n\n\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>a\n\n\n\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":16},"end":{"line":2,"ch":23}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":3,"ch":17},"end":{"line":3,"ch":24}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\nfalse\nnull\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\nfalse\nnull\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":15},"end":{"line":2,"ch":19}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>a\n\n\n\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>a\n\n\n\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":20},"end":{"line":2,"ch":22}},"message":"Initializer is redundant.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":7,"ch":14},"end":{"line":7,"ch":15}},"message":"Assignment can be replaced with operator assignment.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>10\n</outStream>"}
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":20},"end":{"line":2,"ch":22}},"message":"Initializer is redundant.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>10\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":8,"ch":11},"end":{"line":8,"ch":17}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>3\nhello\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>3\nhello\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":6,"ch":11},"end":{"line":6,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":10,"ch":12},"end":{"line":10,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":11,"ch":12},"end":{"line":11,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\n</outStream>"}
{"errors":{"File.kt":[{"interval":{"start":{"line":10,"ch":12},"end":{"line":10,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"},{"interval":{"start":{"line":11,"ch":12},"end":{"line":11,"ch":36}},"message":"Identity equality for arguments of types 'kotlin.Int?' and 'kotlin.Int?' is prohibited.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\nfalse\n</outStream>"}
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"errors":{"File.kt":[{"interval":{"start":{"line":2,"ch":11},"end":{"line":2,"ch":14}},"message":"Redundant explicit type.","severity":"WARNING","className":"WARNING"}]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\ntrue\n</outStream>"}
{"errors":{"File.kt":[]},"exception":null,"jvmByteCode":null,"text":"<outStream>true\ntrue\n</outStream>"}

0 comments on commit 2279468

Please sign in to comment.