Merge pull request #400 from viralpraxis/fix-not-with-binary-operator-precedence-unparsing

Fix `not` with binary operator unparsing

Author: mbj

lib/unparser/node_helpers.rb

@@ -40,6 +40,7 @@ def n_range?(node)
     end
 
     %i[
+      and
       arg
       args
       array
@@ -67,6 +68,7 @@ def n_range?(node)
       lambda
       lvar
       match_rest
+      or
       pair
       rescue
       send

lib/unparser/writer/send/unary.rb

@@ -12,11 +12,11 @@ class Unary < self
 
         private_constant(*constants(false))
 
-        def dispatch
+        def dispatch # rubocop:disable Metrics/AbcSize
           name = selector
           first_child = children.fetch(0)
 
-          if n_flipflop?(first_child)
+          if n_flipflop?(first_child) || n_and?(first_child) || n_or?(first_child)
             write 'not '
           else
             write(MAP.fetch(name, name).to_s)

test/corpus/semantic/not.rb

@@ -0,0 +1,16 @@
+!(foo and bar)
+!(foo or bar)
+not(foo and bar)
+not(foo or bar)
+! foo and bar
+! foo or bar
+not foo and bar
+not foo or bar
+!(foo && bar)
+!(foo || bar)
+not(foo && bar)
+not(foo || bar)
+! foo && bar
+! foo || bar
+not foo && bar
+not foo || bar