synvert-core 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6ed207188d4ec855b1b3e7f89bbd8fe237fdcfd4
-  data.tar.gz: dbe3386bcd2c13ab431afd27111290979ab20e62
+  metadata.gz: 8ee7de00fd1f401d3d4ea506c941871da5714229
+  data.tar.gz: 406b2f2ecdfdb8b837ad0e8839c5e37b1c9d821f
 SHA512:
-  metadata.gz: fef1237ad08c20bee81b382f67efa36e0f2d137eb7f4fb5289daf35dfe6053b57a99020e127394d1b7f5f12b9ab4b4983feb01ad6f0ce680db175574cd9d687e
-  data.tar.gz: 80c1a8239eea4c895a3c5f5fac37c30f919c70daa60fc4f7f55b789f4e5e43f7329bc5fe3f9aafdebfab6ac3441957eea5c851ccbd0bf7fd9dc73614e9258c98
+  metadata.gz: d1db8ef41e1b179c38df52c22091418183e40b879921b462bc47d9f9821a47cb4d4823983bb54024624717cbcf39bfa5b4a86575c4cd69a6ae9b179c7eb1bab6
+  data.tar.gz: f394394861d482865963fccd5024a340ddabf9e43077a0467a79ef9de77285348905e09d70f5e334d1e1ae759737dfd709bb5c49a955ff7b331eaacb13082482

data/CHANGELOG.md

@@ -1,12 +1,15 @@
 # CHANGELOG
 
+## 0.3.0
+
+* Rename node.source(instance) to node.to_source
+* Add has_key? and hash_value helper methods for hash node
+* Fix Instance#check_conflict_actions
+
 ## 0.2.0
 
 * Add remove_file dsl
 * Add warn dsl
-
-## 0.1.1
-
 * Return empty array if no available rewriters
 
 ## 0.1.0

data/lib/synvert/core/node_ext.rb

@@ -119,6 +119,33 @@ class Parser::AST::Node
     end
   end
 
+  # Test if hash node contains specified key.
+  #
+  # @param [Object] key value.
+  # @return [Boolean] true if specified key exists.
+  # @raise [Synvert::Core::MethodNotSupported] if calls on other node.
+  def has_key?(key)
+    if :hash == self.type
+      self.children.any? { |pair_node| pair_node.key.to_value == key }
+    else
+      raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{self.inspect}"
+    end
+  end
+
+  # Get hash value node according to specified key.
+  #
+  # @param [Object] key value.
+  # @return [Parser::AST::Node] value node.
+  # @raise [Synvert::Core::MethodNotSupported] if calls on other node.
+  def hash_value(key)
+    if :hash == self.type
+      value_node = self.children.find { |pair_node| pair_node.key.to_value == key }
+      value_node ? value_node.value : nil
+    else
+      raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{self.inspect}"
+    end
+  end
+
   # Get key node of hash :pair node.
   #
   # @return [Parser::AST::Node] key node.
@@ -143,11 +170,26 @@ class Parser::AST::Node
     end
   end
 
+  # Return the exact value.
+  #
+  # @return [Object] exact value.
+  # @raise [Synvert::Core::MethodNotSupported] if calls on other node.
+  def to_value
+    case self.type
+    when :str, :sym
+      self.children.last
+    when :array
+      self.children.map(&:to_value)
+    else
+      raise Synvert::Core::MethodNotSupported.new "to_value is not handled for #{self.inspect}"
+    end
+  end
+
   # Get the source code of current node.
   #
-  # @param instance [Synvert::Core::Rewriter::Instance]
   # @return [String] source code.
-  def source(instance)
+  def to_source
+    instance = Synvert::Rewriter::Instance.current
     if self.loc.expression
       instance.current_source[self.loc.expression.begin_pos...self.loc.expression.end_pos]
     end
@@ -175,23 +217,22 @@ class Parser::AST::Node
 
   # Match current node with rules.
   #
-  # @param instance [Synvert::Core::Rewriter::Instance] used to get crrent source code.
   # @param rules [Hash] rules to match.
   # @return true if matches.
-  def match?(instance, rules)
+  def match?(rules)
     flat_hash(rules).keys.all? do |multi_keys|
       if multi_keys.last == :any
-        actual_values = actual_value(self, instance, multi_keys[0...-1])
+        actual_values = actual_value(self, multi_keys[0...-1])
         expected = expected_value(rules, multi_keys)
-        actual_values.any? { |actual| match_value?(instance, actual, expected) }
+        actual_values.any? { |actual| match_value?(actual, expected) }
       elsif multi_keys.last == :not
-        actual = actual_value(self, instance, multi_keys[0...-1])
+        actual = actual_value(self, multi_keys[0...-1])
         expected = expected_value(rules, multi_keys)
-        !match_value?(instance, actual, expected)
+        !match_value?(actual, expected)
       else
-        actual = actual_value(self, instance, multi_keys)
+        actual = actual_value(self, multi_keys)
         expected = expected_value(rules, multi_keys)
-        match_value?(instance, actual, expected)
+        match_value?(actual, expected)
       end
     end
   end
@@ -229,33 +270,32 @@ private
 
   # Compare actual value with expected value.
   #
-  # @param instance [Synvert::Core::Rewriter::Instance] used to get source code.
   # @param actual [Object] actual value.
   # @param expected [Object] expected value.
   # @return [Integer] -1, 0 or 1.
   # @raise [Synvert::Core::MethodNotSupported] if expected class is not supported.
-  def match_value?(instance, actual, expected)
+  def match_value?(actual, expected)
     case expected
     when Symbol
       if Parser::AST::Node === actual
-        actual.source(instance) == ":#{expected}"
+        actual.to_source == ":#{expected}"
       else
         actual.to_sym == expected
       end
     when String
       if Parser::AST::Node === actual
-        actual.source(instance) == expected || actual.source(instance)[1...-1] == expected
+        actual.to_source == expected || actual.to_source[1...-1] == expected
       else
         actual.to_s == expected
       end
     when Regexp
       if Parser::AST::Node === actual
-        actual.source(instance) =~ Regexp.new(expected.to_s, Regexp::MULTILINE)
+        actual.to_source =~ Regexp.new(expected.to_s, Regexp::MULTILINE)
       else
         actual.to_s =~ Regexp.new(expected.to_s, Regexp::MULTILINE)
       end
     when Array
-      actual.zip(expected).all? { |a, e| match_value?(instance, a, e) }
+      actual.zip(expected).all? { |a, e| match_value?(a, e) }
     when NilClass
       actual.nil?
     when Numeric
@@ -297,13 +337,12 @@ private
   # Get actual value from the node.
   #
   # @param node [Parser::AST::Node]
-  # @param instance [Synvert::Core::Rewriter::Instance]
   # @param multi_keys [Array<Symbol>]
   # @return [Object] actual value.
-  def actual_value(node, instance, multi_keys)
+  def actual_value(node, multi_keys)
     multi_keys.inject(node) { |n, key|
       if n
-        key == :source ? n.send(key, instance) : n.send(key)
+        key == :source ? n.send(key) : n.send(key)
       end
     }
   end

data/lib/synvert/core/rewriter/condition.rb

@@ -27,7 +27,7 @@ module Synvert::Core
     def match?
       match = false
       @instance.current_node.recursive_children do |child_node|
-        match = match || (child_node && child_node.match?(@instance, @rules))
+        match = match || (child_node && child_node.match?(@rules))
       end
       match
     end
@@ -39,7 +39,7 @@ module Synvert::Core
     def match?
       match = false
       @instance.current_node.recursive_children do |child_node|
-        match = match || (child_node && child_node.match?(@instance, @rules))
+        match = match || (child_node && child_node.match?(@rules))
       end
       !match
     end
@@ -50,7 +50,7 @@ module Synvert::Core
     # check if only have one child node and the child node matches rules.
     def match?
       @instance.current_node.body.size == 1 &&
-        @instance.current_node.body.first.match?(@instance, @rules)
+        @instance.current_node.body.first.match?(@rules)
     end
   end
 end

data/lib/synvert/core/rewriter/instance.rb

@@ -6,6 +6,10 @@ module Synvert::Core
   #
   # One instance can contains one or many [Synvert::Core::Rewriter::Scope] and [Synvert::Rewriter::Condition].
   class Rewriter::Instance
+    class <<self
+      attr_accessor :current
+    end
+
     # @!attribute [rw] current_node
     #   @return current parsing node
     # @!attribute [rw] current_source
@@ -32,6 +36,8 @@ module Synvert::Core
     # It finds all files, for each file, it executes the block code, gets all rewrite actions,
     # and rewrite source code back to original file.
     def process
+      self.class.current = self
+
       parser = Parser::CurrentRuby.new
       file_pattern = File.join(Configuration.instance.get(:path), @file_pattern)
       Dir.glob(file_pattern).each do |file_path|
@@ -164,12 +170,19 @@ module Synvert::Core
     # So if there is an overlap between two actions, it removes the conflict actions and operate them in the next loop.
     def check_conflict_actions
       i = @actions.length - 1
+      j = i - 1
       @conflict_actions = []
-      while i > 0
-        if @actions[i].begin_pos <= @actions[i - 1].end_pos
-          @conflict_actions << @actions.delete_at(i)
+      return if i < 0
+
+      begin_pos = @actions[i].begin_pos
+      while j > -1
+        if begin_pos <= @actions[j].end_pos
+          @conflict_actions << @actions.delete_at(j)
+        else
+          i = j
+          begin_pos = @actions[i].begin_pos
         end
-        i -= 1
+        j -= 1
       end
       @conflict_actions
     end

data/lib/synvert/core/rewriter/scope.rb

@@ -21,9 +21,9 @@ module Synvert::Core
       return unless current_node
       process_with_node current_node do
         matching_nodes = []
-        matching_nodes << current_node if current_node.match? @instance, @rules
+        matching_nodes << current_node if current_node.match? @rules
         current_node.recursive_children do |child_node|
-          matching_nodes << child_node if child_node.match? @instance, @rules
+          matching_nodes << child_node if child_node.match? @rules
         end
         matching_nodes.each do |matching_node|
           process_with_node matching_node do

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = "0.2.0"
+    VERSION = "0.3.0"
   end
 end

data/spec/synvert/core/node_ext_spec.rb

@@ -6,8 +6,8 @@ describe Parser::AST::Node do
       node = parse('class Synvert; end')
       expect(node.name).to eq parse('Synvert')
 
-      node = parse('class Synvert::Core::Rewriter::Instance; end')
-      expect(node.name).to eq parse('Synvert::Core::Rewriter::Instance')
+      node = parse('class Synvert::Rewriter::Instance; end')
+      expect(node.name).to eq parse('Synvert::Rewriter::Instance')
     end
 
     it 'gets for module node' do
@@ -50,7 +50,8 @@ describe Parser::AST::Node do
       source = 'RSpec.configure do |config|; end'
       node = parse(source)
       instance = double(current_source: source)
-      expect(node.arguments.map { |argument| argument.source(instance) }).to eq ['config']
+      Synvert::Rewriter::Instance.current = instance
+      expect(node.arguments.map { |argument| argument.to_source }).to eq ['config']
     end
 
     it 'gets for defined? node' do
@@ -91,28 +92,52 @@ describe Parser::AST::Node do
   describe "#keys" do
     it 'gets for hash node' do
       node = parse("{:foo => :bar, 'foo' => 'bar'}")
-      expect(node.keys).to eq [Parser::CurrentRuby.parse(':foo'), Parser::CurrentRuby.parse("'foo'")]
+      expect(node.keys).to eq [parse(':foo'), parse("'foo'")]
     end
   end
 
   describe "#values" do
     it 'gets for hash node' do
       node = parse("{:foo => :bar, 'foo' => 'bar'}")
-      expect(node.values).to eq [Parser::CurrentRuby.parse(':bar'), Parser::CurrentRuby.parse("'bar'")]
+      expect(node.values).to eq [parse(':bar'), parse("'bar'")]
+    end
+  end
+
+  describe "#has_key?" do
+    it "gets true if key exists" do
+      node = parse("{:foo => :bar}")
+      expect(node.has_key?(:foo)).to be_truthy
+    end
+
+    it "gets false if key does not exist" do
+      node = parse("{:foo => :bar}")
+      expect(node.has_key?('foo')).to be_falsey
+    end
+  end
+
+  describe "#hash_value" do
+    it "gets value of specified key" do
+      node = parse("{:foo => :bar}")
+      expect(node.hash_value(:foo)).to eq parse(':bar')
+    end
+
+    it "gets nil if key does not exist" do
+      node = parse("{:foo => :bar}")
+      expect(node.hash_value(:bar)).to be_nil
     end
   end
 
   describe "#key" do
     it 'gets for pair node' do
       node = parse("{:foo => 'bar'}").children[0]
-      expect(node.key).to eq Parser::CurrentRuby.parse(':foo')
+      expect(node.key).to eq parse(':foo')
     end
   end
 
   describe "#value" do
     it 'gets for hash node' do
       node = parse("{:foo => 'bar'}").children[0]
-      expect(node.value).to eq Parser::CurrentRuby.parse("'bar'")
+      expect(node.value).to eq parse("'bar'")
     end
   end
 
@@ -123,12 +148,30 @@ describe Parser::AST::Node do
     end
   end
 
-  describe '#source' do
+  describe "#to_value" do
+    it 'gets for string' do
+      node = parse("'str'")
+      expect(node.to_value).to eq "str"
+    end
+
+    it 'gets for symbol' do
+      node = parse(":str")
+      expect(node.to_value).to eq :str
+    end
+
+    it 'gets for array' do
+      node = parse("['str', :str]")
+      expect(node.to_value).to eq ['str', :str]
+    end
+  end
+
+  describe '#to_source' do
     it 'gets for node' do
       source = 'params[:user][:email]'
       instance = double(current_source: source)
+      Synvert::Rewriter::Instance.current = instance
       node = parse(source)
-      expect(node.source(instance)).to eq 'params[:user][:email]'
+      expect(node.to_source).to eq 'params[:user][:email]'
     end
   end
 
@@ -153,57 +196,58 @@ describe Parser::AST::Node do
 
   describe '#match?' do
     let(:instance) {
-      rewriter = Synvert::Core::Rewriter.new('foobar')
-      Synvert::Core::Rewriter::Instance.new(rewriter, 'file pattern')
+      rewriter = Synvert::Rewriter.new('foobar')
+      Synvert::Rewriter::Instance.new(rewriter, 'file pattern')
     }
+    before { Synvert::Rewriter::Instance.current = instance }
 
     it 'matches class name' do
       source = 'class Synvert; end'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'class', name: 'Synvert')
+      expect(node).to be_match(type: 'class', name: 'Synvert')
     end
 
     it 'matches message with regexp' do
       source = 'User.find_by_login(login)'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'send', message: /^find_by_/)
+      expect(node).to be_match(type: 'send', message: /^find_by_/)
     end
 
     it 'matches arguments with symbol' do
       source = 'params[:user]'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'send', receiver: 'params', message: '[]', arguments: [:user])
+      expect(node).to be_match(type: 'send', receiver: 'params', message: '[]', arguments: [:user])
     end
 
     it 'matches assign number' do
       source = 'at_least(0)'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'send', arguments: [0])
+      expect(node).to be_match(type: 'send', arguments: [0])
     end
 
     it 'matches arguments with string' do
       source = 'params["user"]'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'send', receiver: 'params', message: '[]', arguments: ['user'])
+      expect(node).to be_match(type: 'send', receiver: 'params', message: '[]', arguments: ['user'])
     end
 
     it 'matches arguments any' do
       source = 'config.middleware.insert_after ActiveRecord::QueryCache, Lifo::Cache, page_cache: false'
       instance.current_source = source
       node = parse(source)
-      expect(node).to be_match(instance, type: 'send', arguments: {any: 'Lifo::Cache'})
+      expect(node).to be_match(type: 'send', arguments: {any: 'Lifo::Cache'})
     end
 
     it 'matches not' do
       source = 'class Synvert; end'
       instance.current_source = source
       node = parse(source)
-      expect(node).not_to be_match(instance, type: 'class', name: {not: 'Synvert'})
+      expect(node).not_to be_match(type: 'class', name: {not: 'Synvert'})
     end
   end
 end

data/spec/synvert/core/rewriter/condition_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 module Synvert::Core
-  describe Rewriter::IfExistCondition do
+  describe Rewriter::Condition do
     let(:source) {
       """
       RSpec.configure do |config|
@@ -12,94 +12,79 @@ module Synvert::Core
     }
     let(:node) { Parser::CurrentRuby.parse(source) }
     let(:instance) { double(:current_node => node, :current_source => source) }
+    before { Rewriter::Instance.current = instance }
 
-    describe '#process' do
-      it 'call block if match anything' do
-        run = false
-        condition = Rewriter::IfExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
-          run = true
+    describe Rewriter::IfExistCondition do
+      describe '#process' do
+        it 'call block if match anything' do
+          run = false
+          condition = Rewriter::IfExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
+            run = true
+          end
+          condition.process
+          expect(run).to be_truthy
         end
-        condition.process
-        expect(run).to be_true
-      end
 
-      it 'not call block if not match anything' do
-        run = false
-        condition = Rewriter::IfExistCondition.new instance, type: 'send', message: 'include', arguments: ['FactoryGirl::SyntaxMethods'] do
-          run = true
+        it 'not call block if not match anything' do
+          run = false
+          condition = Rewriter::IfExistCondition.new instance, type: 'send', message: 'include', arguments: ['FactoryGirl::SyntaxMethods'] do
+            run = true
+          end
+          condition.process
+          expect(run).to be_falsey
         end
-        condition.process
-        expect(run).to be_false
       end
     end
-  end
-
-  describe Rewriter::UnlessExistCondition do
-    let(:source) {
-      """
-      RSpec.configure do |config|
-        config.include EmailSpec::Helpers
-        config.include EmailSpec::Methods
-      end
-      """
-    }
-    let(:node) { Parser::CurrentRuby.parse(source) }
-    let(:instance) { double(:current_node => node, :current_source => source) }
 
-    describe '#process' do
-      it 'call block if match anything' do
-        run = false
-        condition = Rewriter::UnlessExistCondition.new instance, type: 'send', message: 'include', arguments: ['FactoryGirl::Syntax::Methods'] do
-          run = true
+    describe Rewriter::UnlessExistCondition do
+      describe '#process' do
+        it 'call block if match anything' do
+          run = false
+          condition = Rewriter::UnlessExistCondition.new instance, type: 'send', message: 'include', arguments: ['FactoryGirl::Syntax::Methods'] do
+            run = true
+          end
+          condition.process
+          expect(run).to be_truthy
         end
-        condition.process
-        expect(run).to be_true
-      end
 
-      it 'not call block if not match anything' do
-        run = false
-        condition = Rewriter::UnlessExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
-          run = true
+        it 'not call block if not match anything' do
+          run = false
+          condition = Rewriter::UnlessExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
+            run = true
+          end
+          condition.process
+          expect(run).to be_falsey
         end
-        condition.process
-        expect(run).to be_false
       end
     end
-  end
 
-  describe Rewriter::IfOnlyExistCondition do
-    describe '#process' do
-      it 'gets matching nodes' do
-        source = """
-          RSpec.configure do |config|
-            config.include EmailSpec::Helpers
+    describe Rewriter::IfOnlyExistCondition do
+      describe '#process' do
+        it 'gets matching nodes' do
+          source = """
+            RSpec.configure do |config|
+              config.include EmailSpec::Helpers
+            end
+          """
+          node = Parser::CurrentRuby.parse(source)
+          instance = double(:current_node => node, :current_source => source)
+          Rewriter::Instance.current = instance
+          run = false
+          condition = Rewriter::IfOnlyExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
+            run = true
           end
-        """
-        node = Parser::CurrentRuby.parse(source)
-        instance = double(:current_node => node, :current_source => source)
-        run = false
-        condition = Rewriter::IfOnlyExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
-          run = true
+          condition.process
+          expect(run).to be_truthy
         end
-        condition.process
-        expect(run).to be_true
-      end
 
-      it 'not call block if does not match' do
-        source = """
-          RSpec.configure do |config|
-            config.include EmailSpec::Helpers
-            config.include EmailSpec::Methods
+        it 'not call block if does not match' do
+          run = false
+          condition = Rewriter::IfOnlyExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
+            run = true
           end
-        """
-        node = Parser::CurrentRuby.parse(source)
-        instance = double(:current_node => node, :current_source => source)
-        run = false
-        condition = Rewriter::IfOnlyExistCondition.new instance, type: 'send', message: 'include', arguments: ['EmailSpec::Helpers'] do
-          run = true
+          condition.process
+          expect(run).to be_falsey
         end
-        condition.process
-        expect(run).to be_false
       end
     end
   end

data/spec/synvert/core/rewriter/instance_spec.rb

@@ -166,6 +166,30 @@ end
         expect(File).to receive(:write).with('config/environments/production.rb', output)
         instance.process
       end
+
+      describe "#check_conflict_actions" do
+        it "has no conflict" do
+          action1 = double(begin_pos: 10, end_pos: 20)
+          action2 = double(begin_pos: 30, end_pos: 40)
+          action3 = double(begin_pos: 50, end_pos: 60)
+          instance = Rewriter::Instance.new rewriter, 'spec/spec_helper.rb'
+          instance.instance_variable_set :@actions, [action1, action2, action3]
+          instance.send(:check_conflict_actions)
+          expect(instance.instance_variable_get :@conflict_actions).to eq []
+          expect(instance.instance_variable_get :@actions).to eq [action1, action2, action3]
+        end
+
+        it "has no conflict" do
+          action1 = double(begin_pos: 30, end_pos: 40)
+          action2 = double(begin_pos: 50, end_pos: 60)
+          action3 = double(begin_pos: 10, end_pos: 20)
+          instance = Rewriter::Instance.new rewriter, 'spec/spec_helper.rb'
+          instance.instance_variable_set :@actions, [action1, action2, action3]
+          instance.send(:check_conflict_actions)
+          expect(instance.instance_variable_get :@conflict_actions).to eq [action2, action1]
+          expect(instance.instance_variable_get :@actions).to eq [action3]
+        end
+      end
     end
   end
 end

data/spec/synvert/core/rewriter/scope_spec.rb

@@ -18,6 +18,7 @@ end
     }
     let(:node) { Parser::CurrentRuby.parse(source) }
     let(:instance) { double(:current_node => node, :current_node= => node, :current_source => source) }
+    before { Rewriter::Instance.current = instance }
 
     describe '#process' do
       it 'not call block if no matching node' do
@@ -26,7 +27,7 @@ end
           run = true
         end
         scope.process
-        expect(run).to be_false
+        expect(run).to be_falsey
       end
 
       it 'call block if there is matching node' do
@@ -35,7 +36,7 @@ end
           run = true
         end
         scope.process
-        expect(run).to be_true
+        expect(run).to be_truthy
       end
     end
   end

data/spec/synvert/core/rewriter_spec.rb

@@ -73,7 +73,7 @@ module Synvert::Core
           add_file 'foo.bar', 'FooBar'
         end
         rewriter.process_with_sandbox
-        expect(File.exist?('./foo.bar')).to be_false
+        expect(File.exist?('./foo.bar')).to be_falsey
       end
     end
 
@@ -84,7 +84,7 @@ module Synvert::Core
           remove_file 'foo.bar'
         end
         rewriter.process
-        expect(File.exist? './foo.bar').to be_false
+        expect(File.exist? './foo.bar').to be_falsey
       end
 
       it 'does nothing if file not exist' do
@@ -92,7 +92,7 @@ module Synvert::Core
           remove_file 'foo.bar'
         end
         rewriter.process
-        expect(File.exist? './foo.bar').to be_false
+        expect(File.exist? './foo.bar').to be_falsey
       end
 
       it 'does nothing in sandbox mode' do
@@ -101,7 +101,7 @@ module Synvert::Core
           add_file 'foo.bar', 'FooBar'
         end
         rewriter.process_with_sandbox
-        expect(File.exist?('./foo.bar')).to be_true
+        expect(File.exist?('./foo.bar')).to be_truthy
         FileUtils.rm './foo.bar'
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-16 00:00:00.000000000 Z
+date: 2014-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser