Add Lint/RequireParentheses

spec/ameba/rule/lint/logical_without_paren_spec.cr

@@ -0,0 +1,47 @@
+require "../../../spec_helper"
+
+module Ameba::Rule::Lint
+  describe LogicalWithoutParenthesis do
+    subject = LogicalWithoutParenthesis.new
+
+    it "passes if logical operator in call args has parenthesis" do
+      expect_no_issues subject, <<-CRYSTAL
+        if foo.includes?("bar") || foo.includes?("batz")
+          puts "this code is bug-free"
+        end
+
+        if foo.includes?("bar" || foo.includes? "batz")
+          puts "this code is bug-free"
+        end
+
+        form.add("query", "val_1" || "val_2")
+        form.add "query", "val_1" || "val_2"
+        form.add "query", ("val_1" || "val_2")
+        CRYSTAL
+    end
+
+    it "passes if logical operator in assignment call" do
+      expect_no_issues subject, <<-CRYSTAL
+        hello.there = "world" || method.call
+        hello.there ||= "world" || method.call
+        CRYSTAL
+    end
+
+    it "passes if logical operator in square bracket call" do
+      expect_no_issues subject, <<-CRYSTAL
+        hello["world" || :thing]
+        hello["world" || :thing]?
+        this.is[1 || method.call]
+        CRYSTAL
+    end
+
+    it "fails if logical operator in call args doesn't have parenthesis" do
+      expect_issue subject, <<-CRYSTAL
+        if foo.includes? "bar" || foo.includes? "batz"
+         # ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ error: Logical operator in method args without parenthesis is not allowed
+          puts "this code is not bug-free"
+        end
+        CRYSTAL
+    end
+  end
+end

src/ameba/rule/lint/logical_without_paren.cr

@@ -0,0 +1,48 @@
+module Ameba::Rule::Lint
+  # A rule that disallows logical operators in method args without parenthesis.
+  #
+  # For example, this is considered invalid:
+  #
+  # ```
+  # if foo.includes? "bar" || foo.includes? "batz"
+  # end
+  # ```
+  #
+  # And need to be written as:
+  #
+  # ```
+  # if foo.includes?("bar") || foo.includes?("batz")
+  # end
+  # ```
+  #
+  # YAML configuration example:
+  #
+  # ```
+  # Lint/LogicalWithoutParenthesis:
+  #   Enabled: true
+  # ```
+  class LogicalWithoutParenthesis < Base
+    properties do
+      description "Disallows logical operators in method args without parenthesis"
+    end
+
+    MSG = "Logical operator in method args without parenthesis is not allowed"
+
+    ALLOWED_CALL_NAMES = %w{[]? []}
+
+    def test(source, node : Crystal::Call)
+      return if node.args.empty? ||
+                node.has_parentheses? ||
+                node.name.ends_with?('=') ||
+                node.name.in?(ALLOWED_CALL_NAMES)
+
+      node.args.each do |arg|
+        if arg.is_a?(Crystal::BinaryOp)
+          if (right = arg.right).is_a?(Crystal::Call)
+            issue_for node, MSG unless right.args.empty?
+          end
+        end
+      end
+    end
+  end
+end