Make Node#condition? aware of case-match node

This PR makes `Node#condition?` aware of `case-match` node.

changelog/new_make_node_condition_p_aware_of_case_match.md

@@ -0,0 +1 @@
+* [#227](https://github.com/rubocop-hq/rubocop-ast/pull/227): Make `Node#condition?` aware of `case-match` node. ([@koic][])

lib/rubocop/ast/node.rb

@@ -57,7 +57,7 @@ class Node < Parser::AST::Node # rubocop:disable Metrics/ClassLength
  # @api private
  BASIC_CONDITIONALS = %i[if while until].to_set.freeze
  # @api private
- CONDITIONALS = (BASIC_CONDITIONALS + [:case]).freeze
+ CONDITIONALS = (BASIC_CONDITIONALS + %i[case case_match]).freeze
  # @api private
  POST_CONDITION_LOOP_TYPES = %i[while_post until_post].to_set.freeze
  # @api private

spec/rubocop/ast/node_spec.rb

@@ -836,4 +836,86 @@ class << expr
  end
  end
  end
+
+ describe '#conditional?' do
+ context 'when `if` node' do
+ let(:src) do
+ <<~RUBY
+ if condition
+ end
+ RUBY
+ end
+
+ it 'is true' do
+ expect(node).to be_conditional
+ end
+ end
+
+ context 'when `while` node' do
+ let(:src) do
+ <<~RUBY
+ while condition
+ end
+ RUBY
+ end
+
+ it 'is true' do
+ expect(node).to be_conditional
+ end
+ end
+
+ context 'when `until` node' do
+ let(:src) do
+ <<~RUBY
+ until condition
+ end
+ RUBY
+ end
+
+ it 'is true' do
+ expect(node).to be_conditional
+ end
+ end
+
+ context 'when `case` node' do
+ let(:src) do
+ <<~RUBY
+ case condition
+ when foo
+ end
+ RUBY
+ end
+
+ it 'is true' do
+ expect(node).to be_conditional
+ end
+ end
+
+ context 'when `case_match` node', :ruby27 do
+ let(:src) do
+ <<~RUBY
+ case pattern
+ in foo
+ end
+ RUBY
+ end
+
+ it 'is true' do
+ expect(node).to be_conditional
+ end
+ end
+
+ context 'when post condition loop node' do
+ let(:src) do
+ <<~RUBY
+ begin
+ end while condition
+ RUBY
+ end
+
+ it 'is false' do
+ expect(node).not_to be_conditional
+ end
+ end
+ end
 end