Merge pull request #390 from viralpraxis/fix-and-or-operators-round-trip-assoc

Fix `and` and `or` operators RT broken associativity

Author: mbj

lib/unparser/writer/binary.rb

@@ -52,9 +52,13 @@ def symbol_name
       end
 
       def dispatch
-        left_emitter.write_to_buffer
-        write(' ', MAP.fetch(effective_symbol), ' ')
-        visit(right)
+        if node.type.eql?(:and) && left.type.equal?(:or)
+          emit_with(KEYWORD_TOKENS)
+        else
+          left_emitter.write_to_buffer
+          write(' ', MAP.fetch(effective_symbol), ' ')
+          visit(right)
+        end
       end
 
     private

test/corpus/semantic/and.rb

@@ -6,3 +6,13 @@
 
 if a...b and c...d
 end
+
+a and b and c
+(a and b) and c
+a and (b and c)
+a and b or c
+(a and b) or c
+a and (b or c)
+a or b and c
+(a or b) and c
+a or (b and c)

test/corpus/semantic/or.rb

@@ -0,0 +1 @@
+a[:x] = b[:x] || c[:x] || d(:new)