safedep 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 132ffa3638720d8a67ed227359301f04ccd4e825
-  data.tar.gz: 9dfcd1b5e931ee79716bea0d5e967d6d034b8537
+  metadata.gz: f9a584134a92b66c3998292d52951e55df40452e
+  data.tar.gz: fa525a409445338e868b3e75e232d6414288e8e0
 SHA512:
-  metadata.gz: b24aadff525f1c319c3c02470ceb20b731f461c908758abbd42369b39381e36a7bf513b71ea62e78ff52b7eef026f2bdd0a230a2ed28b0a8d5693e760cefe38e
-  data.tar.gz: 3b21b4c4e858c753a1d3e0a37dc96ab35d2c68578b946d8a0b75187a53b1dd95d79e6e57fcc8bccdc9f00cec8bef4983b1ab22ccbfbbb874c9588bc18ae64e22
+  metadata.gz: 00bb90cf32c7a0b336129c9221d81487270b4c711e687c659e6d7bbc91ae8ad99850e54bc4aa3e059923f3711f0165e008fb4a80197c1e98218d17b2b0aab74a
+  data.tar.gz: 32ea3ddb8ffb4547e7540f79f9a6fee1bdcb2875e960c78916315f8df4feef9541337cb4f405fbf8dc9aa9cff83de686889190df119963f1e9f37e8033e491bb

data/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Development
 
+* Use extracted core gem [gemologist](https://github.com/yujinakayama/gemologist) internally.
+
 ## v0.2.0
 
 * Skip modification of dependencies that have `:git`, `:github` or `:path` option.

data/README.md

@@ -10,7 +10,7 @@
 
 * [Using >= Considered Harmful (or, What’s Wrong With >=) « Katz Got Your Tongue?](http://yehudakatz.com/2010/08/21/using-considered-harmful-or-whats-wrong-with/)
 
-Version specifier with `>=` is considered harmful, then dependencies without version speicifier must be super harmful. :)
+Version specifier with `>=` is considered harmful, then dependencies without version specifier must be super harmful. :)
 
 ## Example
 

data/lib/safedep/gemfile_lock.rb

@@ -1,31 +1,15 @@
+require 'gemologist/gemfile_lock'
 require 'bundler'
 require 'ostruct'
 
 module Safedep
-  class GemfileLock
-    attr_reader :path
-
-    def initialize(path)
-      @path = path
-    end
-
-    def find_dependency(name)
-      dependencies.find { |dep| dep.name == name }
-    end
-
+  class GemfileLock < Gemologist::GemfileLock
     def dependencies
       @dependencies ||= lockfile.specs + [bundler_dependency]
     end
 
     private
 
-    def lockfile
-      @lockfile ||= begin
-        content = File.read(path)
-        Bundler::LockfileParser.new(content)
-      end
-    end
-
     def bundler_dependency
       OpenStruct.new(name: 'bundler', version: Bundler::VERSION)
     end

data/lib/safedep/runner.rb

@@ -1,9 +1,8 @@
 require 'safedep/configuration'
-require 'safedep/gemspec'
-require 'safedep/gemfile'
+require 'safedep/error'
 require 'safedep/gemfile_lock'
 require 'safedep/policy/sem_ver'
-require 'safedep/error'
+require 'gemologist'
 
 module Safedep
   class Runner
@@ -51,27 +50,31 @@ module Safedep
     def gemspec
       @gemspec ||= begin
         path = Dir['*.gemspec'].first
-        Gemspec.new(path) if path
+        Gemologist::Gemspec.new(path) if path
       end
     end
 
     def gemfile
       @gemfile ||= begin
-        fail Error, "#{GEMFILE_PATH} is not found." unless File.exist?(GEMFILE_PATH)
-        Gemfile.new(GEMFILE_PATH)
+        check_file_existence!(GEMFILE_PATH)
+        Gemologist::Gemfile.new(GEMFILE_PATH)
       end
     end
 
     def gemfile_lock
       @gemfile_lock ||= begin
-        unless File.exist?(GEMFILE_LOCK_PATH)
-          fail Error, "#{GEMFILE_LOCK_PATH} is not found. Please run `bundle install`."
-        end
-
-        GemfileLock.new(GEMFILE_LOCK_PATH)
+        check_file_existence!(GEMFILE_LOCK_PATH, 'Please run `bundle install`.')
+        Safedep::GemfileLock.new(GEMFILE_LOCK_PATH)
       end
     end
 
+    def check_file_existence!(path, additional_message = nil)
+      return if File.exist?(path)
+      message = "#{path} is not found."
+      message << ' ' + additional_message if additional_message
+      fail Error, message
+    end
+
     def should_ignore?(dependency)
       return true unless dependency.version_specifiers.empty?
       return true unless (dependency.groups & configuration.skipped_groups).empty?

data/lib/safedep/version.rb

@@ -2,7 +2,7 @@ module Safedep
   # http://semver.org/
   module Version
     MAJOR = 0
-    MINOR = 2
+    MINOR = 3
     PATCH = 0
 
     def self.to_s

data/safedep.gemspec

@@ -19,7 +19,6 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency 'parser', '~> 2.2'
-  spec.add_runtime_dependency 'astrolabe', '~> 1.3'
   spec.add_runtime_dependency 'bundler', '~> 1.7'
+  spec.add_runtime_dependency 'gemologist', '~> 0.1'
 end

data/spec/safedep/runner_spec.rb

@@ -60,9 +60,9 @@ module Safedep
 
     context 'when a dependency specified in Gemfile does not exist in Gemfile.lock', :gemfile, :lockfile do
       let!(:lockfile) do
-        require 'safedep/gemfile_lock'
+        require 'gemologist/gemfile_lock'
         create_file(lockfile_path, invalid_lockfile_source)
-        Safedep::GemfileLock.new(lockfile_path)
+        Gemologist::GemfileLock.new(lockfile_path)
       end
 
       let(:invalid_lockfile_source) { lockfile_source.gsub(/.*rspec.*\n/, '') }
@@ -72,6 +72,27 @@ module Safedep
       end
     end
 
+    context 'when bundler dependency is specified in gemspec', :gemfile, :lockfile do
+      let!(:gemspec) do
+        require 'gemologist/gemspec'
+        create_file(gemspec_path, gemspec_source)
+        Gemologist::Gemspec.new(gemspec_path)
+      end
+
+      let(:gemspec_path) { 'safedep.gemspec' }
+
+      let(:gemspec_source) { <<-END.strip_indent }
+        Gem::Specification.new do |spec|
+          spec.name = 'safedep'
+          spec.add_runtime_dependency 'bundler'
+        end
+      END
+
+      it 'does not raise error' do
+        expect { runner.run }.not_to raise_error
+      end
+    end
+
     context 'when Configuration#skipped_groups is specified', :gemspec, :gemfile, :lockfile do
       before do
         configuration.skipped_groups = ['development']

data/spec/support/shared_contexts.rb

@@ -12,9 +12,9 @@ end
 
 shared_context 'with gemspec', :gemspec do
   let!(:gemspec) do
-    require 'safedep/gemspec'
+    require 'gemologist/gemspec'
     create_file(gemspec_path, gemspec_source)
-    Safedep::Gemspec.new(gemspec_path)
+    Gemologist::Gemspec.new(gemspec_path)
   end
 
   let(:gemspec_path) { 'safedep.gemspec' }
@@ -32,9 +32,9 @@ end
 
 shared_context 'with Gemfile', :gemfile do
   let!(:gemfile) do
-    require 'safedep/gemfile'
+    require 'gemologist/gemfile'
     create_file(gemfile_path, gemfile_source)
-    Safedep::Gemfile.new(gemfile_path)
+    Gemologist::Gemfile.new(gemfile_path)
   end
 
   let(:gemfile_path) { 'Gemfile' }
@@ -56,9 +56,9 @@ end
 
 shared_context 'with Gemfile.lock', :lockfile do
   let!(:lockfile) do
-    require 'safedep/gemfile_lock'
+    require 'gemologist/gemfile_lock'
     create_file(lockfile_path, lockfile_source)
-    Safedep::GemfileLock.new(lockfile_path)
+    Gemologist::GemfileLock.new(lockfile_path)
   end
 
   let(:lockfile_path) { 'Gemfile.lock' }

metadata

@@ -1,57 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: safedep
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Yuji Nakayama
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-11 00:00:00.000000000 Z
+date: 2015-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: parser
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
-- !ruby/object:Gem::Dependency
-  name: astrolabe
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: gemologist
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '0.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '0.1'
 description: Make your Gemfile safe by adding dependency version specifiers automatically.
 email:
 - nkymyj@gmail.com
@@ -71,31 +57,17 @@ files:
 - README.md
 - Rakefile
 - bin/safedep
-- lib/safedep/abstract_dependency.rb
-- lib/safedep/abstract_gemfile.rb
 - lib/safedep/cli.rb
 - lib/safedep/cli/option_parser.rb
 - lib/safedep/configuration.rb
 - lib/safedep/error.rb
-- lib/safedep/gemfile.rb
-- lib/safedep/gemfile/dependency.rb
 - lib/safedep/gemfile_lock.rb
-- lib/safedep/gemspec.rb
-- lib/safedep/gemspec/dependency.rb
-- lib/safedep/literal.rb
 - lib/safedep/policy/sem_ver.rb
 - lib/safedep/runner.rb
-- lib/safedep/util.rb
 - lib/safedep/version.rb
 - safedep.gemspec
 - spec/.rubocop.yml
 - spec/safedep/cli_spec.rb
-- spec/safedep/gemfile/dependency_spec.rb
-- spec/safedep/gemfile_lock_spec.rb
-- spec/safedep/gemfile_spec.rb
-- spec/safedep/gemspec/dependency_spec.rb
-- spec/safedep/gemspec_spec.rb
-- spec/safedep/literal_spec.rb
 - spec/safedep/policy/sem_ver_spec.rb
 - spec/safedep/runner_spec.rb
 - spec/spec_helper.rb
@@ -121,19 +93,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: Make your Gemfile safe by adding dependency version specifiers automatically.
 test_files:
 - spec/.rubocop.yml
 - spec/safedep/cli_spec.rb
-- spec/safedep/gemfile/dependency_spec.rb
-- spec/safedep/gemfile_lock_spec.rb
-- spec/safedep/gemfile_spec.rb
-- spec/safedep/gemspec/dependency_spec.rb
-- spec/safedep/gemspec_spec.rb
-- spec/safedep/literal_spec.rb
 - spec/safedep/policy/sem_ver_spec.rb
 - spec/safedep/runner_spec.rb
 - spec/spec_helper.rb

data/lib/safedep/abstract_dependency.rb

@@ -1,80 +0,0 @@
-require 'safedep/literal'
-
-module Safedep
-  class AbstractDependency
-    attr_reader :node, :rewriter
-
-    def self.valid_node?(node)
-      return false unless node.send_type?
-      _receiver_node, message, name_node, = *node
-      method_names.include?(message) && name_node.str_type?
-    end
-
-    def self.method_names
-      fail NotImplementedError
-    end
-
-    def initialize(node, rewriter)
-      fail 'Invalid node.' unless self.class.valid_node?(node)
-      @node = node
-      @rewriter = rewriter
-    end
-
-    def name
-      name_node.children.first
-    end
-
-    def groups
-      fail NotImplementedError
-    end
-
-    def version_specifiers
-      @version_specifiers ||= version_nodes.map { |node| Literal.value(node) }.flatten
-    end
-
-    def version_specifiers=(*specifiers)
-      source = specifiers.flatten.map { |specifier| "'#{specifier}'" }.join(', ')
-
-      if version_nodes.empty?
-        rewriter.insert_after(name_node.loc.expression, ", #{source}")
-      else
-        rewriter.replace(version_range, source)
-      end
-    end
-
-    def options
-      {}
-    end
-
-    private
-
-    def method_name
-      node.children[1]
-    end
-
-    def name_node
-      node.children[2]
-    end
-
-    def trailing_nodes
-      node.children[3..-1]
-    end
-
-    def version_nodes
-      fail NotImplementedError
-    end
-
-    def version_range
-      version_nodes.first.loc.expression.join(version_nodes.last.loc.expression)
-    end
-
-    def content_range_of_str_node(str_node)
-      map = str_node.loc
-      Parser::Source::Range.new(
-        map.expression.source_buffer,
-        map.begin.end_pos,
-        map.end.begin_pos
-      )
-    end
-  end
-end

data/lib/safedep/abstract_gemfile.rb

@@ -1,50 +0,0 @@
-require 'astrolabe/builder'
-require 'parser/current'
-
-module Safedep
-  class AbstractGemfile
-    attr_reader :path
-
-    def initialize(path)
-      @path = path
-    end
-
-    def find_dependency(name)
-      dependencies.find { |dep| dep.name == name }
-    end
-
-    def dependencies
-      @dependencies ||= ast.each_node.with_object([]) do |node, dependencies|
-        next unless dependency_class.valid_node?(node)
-        dependencies << dependency_class.new(node, source_rewriter)
-      end
-    end
-
-    def rewrite!
-      rewritten_source = source_rewriter.process
-      File.write(path, rewritten_source)
-    end
-
-    private
-
-    def dependency_class
-      fail NotImplementedError
-    end
-
-    def ast
-      @ast ||= begin
-        builder = Astrolabe::Builder.new
-        parser = Parser::CurrentRuby.new(builder)
-        parser.parse(source_buffer)
-      end
-    end
-
-    def source_rewriter
-      @source_rewriter ||= Parser::Source::Rewriter.new(source_buffer)
-    end
-
-    def source_buffer
-      @source_buffer ||= Parser::Source::Buffer.new(path).read
-    end
-  end
-end

data/lib/safedep/gemfile/dependency.rb

@@ -1,61 +0,0 @@
-require 'safedep/abstract_dependency'
-require 'safedep/util'
-require 'astrolabe/sexp'
-
-module Safedep
-  class Gemfile
-    class Dependency < AbstractDependency
-      include Util, Astrolabe::Sexp
-
-      METHOD_NAMES = [:gem].freeze
-
-      def self.method_names
-        METHOD_NAMES
-      end
-
-      def groups
-        @groups ||= (groups_via_block + groups_via_option).map(&:to_sym)
-      end
-
-      def options
-        @options ||= symbolize_keys(Literal.value(options_node) || {})
-      end
-
-      private
-
-      def groups_via_block
-        return [] unless group_node
-        _receiver_node, _message, *arg_nodes = *group_node
-        arg_nodes.map { |node| Literal.value(node) }
-      end
-
-      def groups_via_option
-        Array(options[:group])
-      end
-
-      # https://github.com/bundler/bundler/blob/v1.7.11/lib/bundler/dsl.rb#L68-L70
-      def version_nodes
-        @version_nodes ||= trailing_nodes - [options_node]
-      end
-
-      def options_node
-        node = trailing_nodes.last
-
-        if node && node.hash_type?
-          node
-        else
-          nil
-        end
-      end
-
-      def group_node
-        candidates = node.each_ancestor(:block).map { |block_node| block_node.children.first }
-
-        candidates.find do |send_node|
-          receiver_node, message, = *send_node
-          receiver_node.nil? && message == :group
-        end
-      end
-    end
-  end
-end

data/lib/safedep/gemfile.rb

@@ -1,13 +0,0 @@
-require 'safedep/abstract_gemfile'
-
-module Safedep
-  class Gemfile < AbstractGemfile
-    require 'safedep/gemfile/dependency'
-
-    private
-
-    def dependency_class
-      Gemfile::Dependency
-    end
-  end
-end

data/lib/safedep/gemspec/dependency.rb

@@ -1,32 +0,0 @@
-require 'safedep/abstract_dependency'
-require 'safedep/util'
-
-module Safedep
-  class Gemspec
-    class Dependency < AbstractDependency
-      include Util
-
-      METHOD_NAMES = [:add_runtime_dependency, :add_development_dependency, :add_dependency].freeze
-
-      def self.method_names
-        METHOD_NAMES
-      end
-
-      def groups
-        case method_name
-        when :add_development_dependency
-          [:development]
-        else
-          []
-        end
-      end
-
-      private
-
-      # https://github.com/rubygems/rubygems/blob/v2.4.5/lib/rubygems/specification.rb#L449-L473
-      def version_nodes
-        trailing_nodes
-      end
-    end
-  end
-end

data/lib/safedep/gemspec.rb

@@ -1,13 +0,0 @@
-require 'safedep/abstract_gemfile'
-
-module Safedep
-  class Gemspec < AbstractGemfile
-    require 'safedep/gemspec/dependency'
-
-    private
-
-    def dependency_class
-      Gemspec::Dependency
-    end
-  end
-end

data/lib/safedep/literal.rb

@@ -1,48 +0,0 @@
-module Safedep
-  module Literal
-    module_function
-
-    def value(node)
-      concretize(node)
-    end
-
-    def concretize(node) # rubocop:disable CyclomaticComplexity
-      return nil unless node
-
-      case node.type
-      when :true                    then true
-      when :false                   then false
-      when :nil                     then nil
-      when :int, :float, :str, :sym then node.children.first
-      when :regexp                  then concretize_regexp(node)
-      when :array                   then concretize_array(node)
-      when :hash                    then concretize_hash(node)
-      when :irange, :erange         then concretize_range(node)
-      end
-    end
-
-    def concretize_regexp(regexp_node)
-      str_node, *_interporated_nodes, regopt_node = *regexp_node
-      string = str_node.children.first
-      options = regopt_node.children.map(&:to_s).reduce(:+)
-      eval("/#{string}/#{options}") # rubocop:disable Eval
-    end
-
-    def concretize_array(array_node)
-      array_node.children.map { |child_node| concretize(child_node) }
-    end
-
-    def concretize_hash(hash_node)
-      hash_node.children.each_with_object({}) do |pair_node, hash|
-        key_node, value_node = *pair_node
-        key = concretize(key_node)
-        hash[key] = concretize(value_node) if key
-      end
-    end
-
-    def concretize_range(range_node)
-      values = range_node.children.map { |child_node| concretize(child_node) }
-      Range.new(*values, range_node.type == :erange)
-    end
-  end
-end

data/lib/safedep/util.rb

@@ -1,16 +0,0 @@
-module Safedep
-  module Util
-    module_function
-
-    def symbolize_keys(original_hash)
-      hash = original_hash.dup
-
-      original_hash.each do |key, value|
-        hash[key] = symbolize_keys(value) if value.is_a?(Hash)
-        hash[key.to_sym] = hash.delete(key) if key.is_a?(String)
-      end
-
-      hash
-    end
-  end
-end

data/spec/safedep/gemfile/dependency_spec.rb

@@ -1,231 +0,0 @@
-require 'safedep/gemfile'
-
-module Safedep
-  class Gemfile
-    describe Dependency, :gemfile do
-      include FileHelper
-      include_context 'isolated environment'
-
-      let(:gemfile_source) { <<-END.strip_indent }
-        source 'https://rubygems.org'
-
-        gemspec
-
-        group :development, :test do
-          gem 'rake'
-          gem 'rspec', '~> 3.1'
-          gem 'rubocop', require: false
-        end
-
-        group :development do
-          gem 'guard', '>= 2.1', '< 3.0'
-        end
-      END
-
-      describe '#name' do
-        let(:dependency) { gemfile.dependencies.first }
-
-        it 'returns the name of the gem' do
-          expect(dependency.name).to eq('rake')
-        end
-      end
-
-      describe '#groups' do
-        subject { dependency.groups }
-
-        let(:gemfile_source) { <<-END.strip_indent }
-          source 'https://rubygems.org'
-
-          gem 'parser'
-
-          group :development, :test do
-            gem 'rake'
-          end
-
-          group :development do
-            gem 'guard'
-            gem 'rubocop', group: :test
-          end
-
-          group 'development' do
-            gem 'fuubar'
-          end
-
-          gem 'rspec', 'group' => [:test, :development]
-        END
-
-        context 'when the dependency is specified in top level' do
-          let(:dependency) { gemfile.find_dependency('parser') }
-          it { should be_empty }
-        end
-
-        context 'when the dependency is specified in :development block' do
-          let(:dependency) { gemfile.find_dependency('guard') }
-          it { should eq([:development]) }
-        end
-
-        context 'when the dependency is specified in "development" block' do
-          let(:dependency) { gemfile.find_dependency('fuubar') }
-          it { should eq([:development]) }
-        end
-
-        context 'when the dependency is specified in :development and :test group' do
-          let(:dependency) { gemfile.find_dependency('rake') }
-          it { should eq([:development, :test]) }
-        end
-
-        context 'when the dependency is specified with group: [:test, :development] option' do
-          let(:dependency) { gemfile.find_dependency('rspec') }
-          it { should eq([:test, :development]) }
-        end
-
-        context 'when the dependency is specified with group: :test option in :development group' do
-          let(:dependency) { gemfile.find_dependency('rubocop') }
-          it { should eq([:development, :test]) }
-        end
-      end
-
-      describe '#version_specifiers' do
-        subject(:version_specifiers) { dependency.version_specifiers }
-
-        context 'when the dependency has a version specifier' do
-          let(:dependency) { gemfile.find_dependency('rspec') }
-
-          it 'returns the specifier' do
-            expect(version_specifiers).to eq(['~> 3.1'])
-          end
-        end
-
-        context 'when the dependency has multiple specifiers' do
-          let(:dependency) { gemfile.find_dependency('guard') }
-
-          it 'returns the specifiers' do
-            expect(version_specifiers).to eq(['>= 2.1', '< 3.0'])
-          end
-        end
-
-        context 'when the dependency has no version specifier' do
-          context 'and has no options' do
-            let(:dependency) { gemfile.find_dependency('rake') }
-            it { should be_empty }
-          end
-
-          context 'but has options' do
-            let(:dependency) { gemfile.find_dependency('rubocop') }
-            it { should be_empty }
-          end
-        end
-      end
-
-      describe '#version_specifiers=' do
-        let(:rewritten_source) { File.read(gemfile.path) }
-
-        context 'when the dependency has a version specifier' do
-          let(:dependency) { gemfile.find_dependency('rspec') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            source 'https://rubygems.org'
-
-            gemspec
-
-            group :development, :test do
-              gem 'rake'
-              gem 'rspec', '> 4.0'
-              gem 'rubocop', require: false
-            end
-
-            group :development do
-              gem 'guard', '>= 2.1', '< 3.0'
-            end
-          END
-
-          it 'replaces the existing specifier with the passed specifier' do
-            dependency.version_specifiers = '> 4.0'
-            gemfile.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-
-        context 'when the dependency has no version specifier' do
-          context 'and has no options' do
-            let(:dependency) { gemfile.find_dependency('rake') }
-
-            let(:expected_source) { <<-END.strip_indent }
-              source 'https://rubygems.org'
-
-              gemspec
-
-              group :development, :test do
-                gem 'rake', '~> 10.1'
-                gem 'rspec', '~> 3.1'
-                gem 'rubocop', require: false
-              end
-
-              group :development do
-                gem 'guard', '>= 2.1', '< 3.0'
-              end
-            END
-
-            it 'adds the passed specifier' do
-              dependency.version_specifiers = '~> 10.1'
-              gemfile.rewrite!
-              expect(rewritten_source).to eq(expected_source)
-            end
-          end
-
-          context 'but has options' do
-            let(:dependency) { gemfile.find_dependency('rubocop') }
-
-            let(:expected_source) { <<-END.strip_indent }
-              source 'https://rubygems.org'
-
-              gemspec
-
-              group :development, :test do
-                gem 'rake'
-                gem 'rspec', '~> 3.1'
-                gem 'rubocop', '~> 0.28', require: false
-              end
-
-              group :development do
-                gem 'guard', '>= 2.1', '< 3.0'
-              end
-            END
-
-            it 'adds the passed specifier' do
-              dependency.version_specifiers = '~> 0.28'
-              gemfile.rewrite!
-              expect(rewritten_source).to eq(expected_source)
-            end
-          end
-        end
-
-        context 'when multiple specifiers are passed' do
-          let(:dependency) { gemfile.find_dependency('rspec') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            source 'https://rubygems.org'
-
-            gemspec
-
-            group :development, :test do
-              gem 'rake'
-              gem 'rspec', '>= 4.0', '< 5.0'
-              gem 'rubocop', require: false
-            end
-
-            group :development do
-              gem 'guard', '>= 2.1', '< 3.0'
-            end
-          END
-
-          it 'replaces the existing specifier with the passed specifiers' do
-            dependency.version_specifiers = ['>= 4.0', '< 5.0']
-            gemfile.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-      end
-    end
-  end
-end

data/spec/safedep/gemfile_lock_spec.rb

@@ -1,29 +0,0 @@
-require 'safedep/gemfile_lock'
-
-module Safedep
-  describe GemfileLock, :lockfile do
-    include FileHelper
-    include_context 'isolated environment'
-
-    describe '#find_dependency' do
-      it 'returns the dependency matching the passed name' do
-        dependency = lockfile.find_dependency('rspec')
-        expect(dependency.name).to eq('rspec')
-      end
-    end
-
-    describe '#dependencies' do
-      let(:dep_names) { lockfile.dependencies.map(&:name) }
-
-      it 'returns an array of the specifications' do
-        expect(dep_names).to include('rake', 'rspec', 'rubocop')
-      end
-
-      it 'includes "bundler" dependency' do
-        dependency = lockfile.find_dependency('bundler')
-        expect(dependency.name).to eq('bundler')
-        expect(dependency.version).to eq(Bundler::VERSION)
-      end
-    end
-  end
-end

data/spec/safedep/gemfile_spec.rb

@@ -1,22 +0,0 @@
-require 'safedep/gemfile'
-
-module Safedep
-  describe Gemfile, :gemfile do
-    include FileHelper
-    include_context 'isolated environment'
-
-    describe '#find_dependency' do
-      it 'returns the dependency matching the passed name' do
-        dependency = gemfile.find_dependency('rspec')
-        expect(dependency.name).to eq('rspec')
-      end
-    end
-
-    describe '#dependencies' do
-      it 'returns an array of the dependencies' do
-        expect(gemfile.dependencies.size).to eq(3)
-        expect(gemfile.dependencies).to all be_a(Gemfile::Dependency)
-      end
-    end
-  end
-end

data/spec/safedep/gemspec/dependency_spec.rb

@@ -1,184 +0,0 @@
-require 'safedep/gemspec'
-
-module Safedep
-  class Gemspec
-    describe Dependency, :gemspec do
-      include FileHelper
-      include_context 'isolated environment'
-
-      let(:gemspec_source) { <<-END.strip_indent }
-        Gem::Specification.new do |spec|
-          spec.name = 'safedep'
-          spec.add_dependency 'parser'
-          spec.add_runtime_dependency 'bundler', '>= 1.3.1', '< 2.0'
-          spec.add_runtime_dependency 'astrolabe', ['>= 1.3', '< 2.0']
-          spec.add_development_dependency 'rspec', '~> 3.1'
-        end
-      END
-
-      describe '#name' do
-        let(:dependency) { gemspec.dependencies.first }
-
-        it 'returns the name of the gem' do
-          expect(dependency.name).to eq('parser')
-        end
-      end
-
-      describe '#groups' do
-        subject { dependency.groups }
-
-        context 'when the dependency is specified via #add_dependency' do
-          let(:dependency) { gemspec.find_dependency('parser') }
-          it { should be_empty }
-        end
-
-        context 'when the dependency is specified via #add_runtime_dependency' do
-          let(:dependency) { gemspec.find_dependency('astrolabe') }
-          it { should be_empty }
-        end
-
-        context 'when the dependency is specified via #add_runtime_dependency' do
-          let(:dependency) { gemspec.find_dependency('rspec') }
-          it { should eq([:development]) }
-        end
-      end
-
-      describe '#version_specifiers' do
-        subject(:version_specifier) { dependency.version_specifiers }
-
-        context 'when the dependency has no version specifier' do
-          let(:dependency) { gemspec.find_dependency('parser') }
-          it { should be_empty }
-        end
-
-        context 'when the dependency has a version specifier' do
-          let(:dependency) { gemspec.find_dependency('rspec') }
-
-          it 'returns the specifier' do
-            expect(version_specifier).to eq(['~> 3.1'])
-          end
-        end
-
-        context 'when the dependency has multiple version specifiers' do
-          let(:dependency) { gemspec.find_dependency('bundler') }
-
-          it 'returns the specifiers' do
-            expect(version_specifier).to eq(['>= 1.3.1', '< 2.0'])
-          end
-        end
-
-        context 'when the dependency has multiple version specifiers in an array' do
-          let(:dependency) { gemspec.find_dependency('astrolabe') }
-
-          it 'returns the specifiers' do
-            expect(version_specifier).to eq(['>= 1.3', '< 2.0'])
-          end
-        end
-      end
-
-      describe '#version_specifiers=' do
-        let(:rewritten_source) { File.read(gemspec.path) }
-
-        context 'when the dependency has no version specifier' do
-          let(:dependency) { gemspec.find_dependency('parser') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            Gem::Specification.new do |spec|
-              spec.name = 'safedep'
-              spec.add_dependency 'parser', '~> 2.2'
-              spec.add_runtime_dependency 'bundler', '>= 1.3.1', '< 2.0'
-              spec.add_runtime_dependency 'astrolabe', ['>= 1.3', '< 2.0']
-              spec.add_development_dependency 'rspec', '~> 3.1'
-            end
-          END
-
-          it 'adds the passed specifier' do
-            dependency.version_specifiers = '~> 2.2'
-            gemspec.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-
-        context 'when the dependency has a version specifier' do
-          let(:dependency) { gemspec.find_dependency('rspec') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            Gem::Specification.new do |spec|
-              spec.name = 'safedep'
-              spec.add_dependency 'parser'
-              spec.add_runtime_dependency 'bundler', '>= 1.3.1', '< 2.0'
-              spec.add_runtime_dependency 'astrolabe', ['>= 1.3', '< 2.0']
-              spec.add_development_dependency 'rspec', '> 4.0'
-            end
-          END
-
-          it 'replaces the existing specifier with the passed specifier' do
-            dependency.version_specifiers = '> 4.0'
-            gemspec.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-
-        context 'when the dependency has multiple version specifiers' do
-          let(:dependency) { gemspec.find_dependency('bundler') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            Gem::Specification.new do |spec|
-              spec.name = 'safedep'
-              spec.add_dependency 'parser'
-              spec.add_runtime_dependency 'bundler', '~> 2.0'
-              spec.add_runtime_dependency 'astrolabe', ['>= 1.3', '< 2.0']
-              spec.add_development_dependency 'rspec', '~> 3.1'
-            end
-          END
-
-          it 'replaces the existing specifiers with the passed specifier' do
-            dependency.version_specifiers = '~> 2.0'
-            gemspec.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-
-        context 'when the dependency has multiple version specifiers in an array' do
-          let(:dependency) { gemspec.find_dependency('astrolabe') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            Gem::Specification.new do |spec|
-              spec.name = 'safedep'
-              spec.add_dependency 'parser'
-              spec.add_runtime_dependency 'bundler', '>= 1.3.1', '< 2.0'
-              spec.add_runtime_dependency 'astrolabe', '~> 2.0'
-              spec.add_development_dependency 'rspec', '~> 3.1'
-            end
-          END
-
-          it 'replaces the existing specifiers with the passed specifier' do
-            dependency.version_specifiers = '~> 2.0'
-            gemspec.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-
-        context 'when multiple specifiers are passed' do
-          let(:dependency) { gemspec.find_dependency('rspec') }
-
-          let(:expected_source) { <<-END.strip_indent }
-            Gem::Specification.new do |spec|
-              spec.name = 'safedep'
-              spec.add_dependency 'parser'
-              spec.add_runtime_dependency 'bundler', '>= 1.3.1', '< 2.0'
-              spec.add_runtime_dependency 'astrolabe', ['>= 1.3', '< 2.0']
-              spec.add_development_dependency 'rspec', '>= 3.1', '< 4.0'
-            end
-          END
-
-          it 'replaces the existing specifier with the passed specifier' do
-            dependency.version_specifiers = ['>= 3.1', '< 4.0']
-            gemspec.rewrite!
-            expect(rewritten_source).to eq(expected_source)
-          end
-        end
-      end
-    end
-  end
-end

data/spec/safedep/gemspec_spec.rb

@@ -1,22 +0,0 @@
-require 'safedep/gemspec'
-
-module Safedep
-  describe Gemspec, :gemspec do
-    include FileHelper
-    include_context 'isolated environment'
-
-    describe '#find_dependency' do
-      it 'returns the dependency matching the passed name' do
-        dependency = gemspec.find_dependency('parser')
-        expect(dependency.name).to eq('parser')
-      end
-    end
-
-    describe '#dependencies' do
-      it 'returns an array of the dependencies' do
-        expect(gemspec.dependencies.size).to eq(2)
-        expect(gemspec.dependencies).to all be_a(Gemspec::Dependency)
-      end
-    end
-  end
-end

data/spec/safedep/literal_spec.rb

@@ -1,49 +0,0 @@
-require 'safedep/literal'
-require 'astrolabe/builder'
-require 'parser/current'
-
-module Safedep
-  describe Literal do
-    let(:node) do
-      builder = Astrolabe::Builder.new
-      parser = Parser::CurrentRuby.new(builder)
-      parser.parse(source_buffer)
-    end
-
-    let(:source_buffer) do
-      Parser::Source::Buffer.new('(string)').tap do |buffer|
-        buffer.source = source
-      end
-    end
-
-    describe '.value' do
-      subject { Literal.value(node) }
-
-      [
-        true,
-        false,
-        nil,
-        123,
-        3.14,
-        'foo',
-        :foo,
-        /foo/im,
-        ['foo', 123],
-        { 'foo' => 123, bar: false },
-        1..3,
-        1...3,
-        { 'foo' => [:bar, { baz: 3.14 }] }
-      ].each do |value|
-        context "with #{value.inspect} node" do
-          let(:source) { value.inspect }
-          it { should eq(value) }
-        end
-      end
-
-      context 'when nil is passed' do
-        let(:node) { nil }
-        it { should be_nil }
-      end
-    end
-  end
-end