diff --git a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundColonRuleTest.kt b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundColonRuleTest.kt index f702ec7765..09c4362160 100644 --- a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundColonRuleTest.kt +++ b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundColonRuleTest.kt @@ -215,7 +215,6 @@ class SpacingAroundColonRuleTest { fun foo(bar: String) {} """.trimIndent() spacingAroundColonRuleAssertThat(code) - // TODO: Offset col is not correct. .hasLintViolation(1, 13, "Unexpected spacing before \":\"") .isFormattedAs(formattedCode) } diff --git a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCommaRuleTest.kt b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCommaRuleTest.kt index af2c5b7e07..e3603b28ce 100644 --- a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCommaRuleTest.kt +++ b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCommaRuleTest.kt @@ -20,7 +20,6 @@ class SpacingAroundCommaRuleTest { val foo2 = Foo(1, 3) """.trimIndent() spacingAroundCommaRuleAssertThat(code) - // TODO: Col offset is not correct .hasLintViolation(1, 18, "Missing spacing after \",\"") .isFormattedAs(formattedCode) } @@ -50,7 +49,6 @@ class SpacingAroundCommaRuleTest { } """.trimIndent() spacingAroundCommaRuleAssertThat(code) - // TODO: Col offset is not correct .hasLintViolation(2, 10, "Missing spacing after \",\"") .isFormattedAs(formattedCode) } diff --git a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCurlyRuleTest.kt b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCurlyRuleTest.kt index daf4cbd817..01a2bc4366 100644 --- a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCurlyRuleTest.kt +++ b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundCurlyRuleTest.kt @@ -81,7 +81,6 @@ class SpacingAroundCurlyRuleTest { """.trimIndent() spacingAroundCurlyRuleAssertThat(code) .hasLintViolations( - // TODO: Col offset is not correct LintViolation(1, 17, "Missing spacing after \"{\""), LintViolation(1, 23, "Missing spacing before \"}\""), ).isFormattedAs(formattedCode) @@ -176,7 +175,6 @@ class SpacingAroundCurlyRuleTest { fun Array.getFoo(): T = this[this.count { it == "foo" }] """.trimIndent() spacingAroundCurlyRuleAssertThat(code) - // TODO: Col offset is not correct .hasLintViolation(1, 62, "Unexpected space after \"}\"") .isFormattedAs(formattedCode) } @@ -213,12 +211,10 @@ class SpacingAroundCurlyRuleTest { LintViolation(2, 15, "Missing spacing around \"{\""), LintViolation(2, 26, "Missing spacing around \"}\""), LintViolation(2, 47, "Missing spacing before \"{\""), - // TODO: Col offset is not correct LintViolation(3, 13, "Missing spacing after \"{\""), LintViolation(3, 31, "Missing spacing around \"{\""), LintViolation(3, 40, "Missing spacing around \"}\""), LintViolation(3, 61, "Missing spacing before \"{\""), - // TODO: Col offset is not correct LintViolation(3, 63, "Missing spacing after \"}\""), LintViolation(3, 63, "Missing spacing before \"}\""), ).isFormattedAs(formattedCode) diff --git a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundUnaryOperatorRuleTest.kt b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundUnaryOperatorRuleTest.kt index a7dd19700e..1b137a3cca 100644 --- a/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundUnaryOperatorRuleTest.kt +++ b/ktlint-ruleset-standard/src/test/kotlin/com/pinterest/ktlint/ruleset/standard/SpacingAroundUnaryOperatorRuleTest.kt @@ -149,7 +149,7 @@ class SpacingAroundUnaryOperatorRuleTest { spacingAroundUnaryOperatorRuleAssertThat(code) .hasLintViolations( LintViolation(2, 17, "Unexpected spacing in \"foo\" !!"), - // TODO: "foo3" should also be disallowed + // Space between the unary operator (!!) and the . is cleared by 'dot-spacing' rule ).isFormattedAs(formattedCode) }