WIP

### What's done: * WIP
saveourtool · Sep 13, 2021 · e469d96 · e469d96
1 parent 56168af
commit e469d96
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 2 deletions.
diff --git a/...at-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/chapter3/BooleanExpressionsRule.kt b/...at-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/chapter3/BooleanExpressionsRule.kt
@@ -155,15 +155,22 @@ class BooleanExpressionsRule(configRules: List<RulesConfig>) : DiktatRule(
         node: ASTNode,
         simplifiedExpr: Expression<String>,
         mapOfExpressionToChar: HashMap<String, Char>) {
+        println("simplifiedExpr ${simplifiedExpr}")
         var correctKotlinBooleanExpression = simplifiedExpr
             .toString()
             .replace("&", "&&")
             .replace("|", "||")
-            .drop(1)  // dropping first (
-            .dropLast(1)  // dropping last )
+
+        if (simplifiedExpr.toString().first() == '(' && simplifiedExpr.toString().last() == ')') {
+            correctKotlinBooleanExpression = correctKotlinBooleanExpression
+                .drop(1)
+                .dropLast(1)
+        }
+
         mapOfExpressionToChar.forEach { (key, value) ->
             correctKotlinBooleanExpression = correctKotlinBooleanExpression.replace(value.toString(), key)
         }
+        println("correct node ${correctKotlinBooleanExpression}")
         node.replaceChild(node.firstChildNode, KotlinParser().createNode(correctKotlinBooleanExpression))
     }
 

diff --git a/...t-rules/src/test/kotlin/org/cqfn/diktat/ruleset/chapter3/BooleanExpressionsRuleFixTest.kt b/...t-rules/src/test/kotlin/org/cqfn/diktat/ruleset/chapter3/BooleanExpressionsRuleFixTest.kt
@@ -19,4 +19,10 @@ class BooleanExpressionsRuleFixTest : FixTestBase("test/paragraph3/boolean_expre
     fun `check distributive law fixing`() {
         fixAndCompare("DistributiveLawExpected.kt", "DistributiveLawTest.kt")
     }
+
+    @Test
+    @Tag(WarningNames.COMPLEX_BOOLEAN_EXPRESSION)
+    fun `check same expressions`() {
+        fixAndCompare("SameExpressionsInConditionExpected.kt", "SameExpressionsInConditionTest.kt")
+    }
 }
diff --git a/.../test/resources/test/paragraph3/boolean_expressions/SameExpressionsInConditionExpected.kt b/.../test/resources/test/paragraph3/boolean_expressions/SameExpressionsInConditionExpected.kt
@@ -0,0 +1,11 @@
+fun foo() {
+    if (a) {}
+    if (a) {}
+    if (a) {}
+
+    return if (node is TomlKeyValueSimple) {
+        decodeValue().toString().toLowerCase() != "null"
+    } else {
+        true
+    }
+}
diff --git a/.../src/test/resources/test/paragraph3/boolean_expressions/SameExpressionsInConditionTest.kt b/.../src/test/resources/test/paragraph3/boolean_expressions/SameExpressionsInConditionTest.kt
@@ -0,0 +1,11 @@
+fun foo() {
+    if (a || a) {}
+    if (a && a) {}
+    if ((((a && a)))) {}
+
+    return if ((node is TomlKeyValueSimple) || (node is TomlKeyValueSimple)) {
+        decodeValue().toString().toLowerCase() != "null"
+    } else {
+        true
+    }
+}