travis_check_rubies 0.1.0 → 0.2.0

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    ZjA1NTQ3ZjAwMjE4YmI4MzBkZGE2MGMxOWZiZmY0MDZiNzA3YWNjZQ==
-  data.tar.gz: !binary |-
-    YzMyMGIwYzk2NzM4NzI3NjUxMGE0MmU0NmVkOTEzN2U3NmU4YzkwOQ==
+SHA1:
+  metadata.gz: bfd262c7e986d7ca949f6811db98ef99f2aab55e
+  data.tar.gz: 65c395fa0c08ae67e8e0c35f526891091f02ab8d
 SHA512:
-  metadata.gz: !binary |-
-    MTk2MDVjYTJmYjY0MDA2MzdiMDFjZTE3Nzc5NjI3MzA2ZWVkMzkwNjc2MzJj
-    MTE3YmI3NjkxYzE2YjBlYzg5ZjFjYzQyZjVlZmJmYTViMGRiODUxNjg1MGRj
-    YWQ4Mzk0YmFjNDgzMDZlMDAyOGUyZmZhNzEyMWQxMWYzMmY3OWU=
-  data.tar.gz: !binary |-
-    NWM2YzI5YTExMzdkZmEyM2Y4ZTdjOGY5NjBkZDk4Mzc0YzJhNGFhMjc5NzE1
-    MmI4YTU2MTZjNGJmZDM5NDdhMWNiMGZmZGFjNTBjZjRhODQxOGVjNjI3YTNl
-    NmVlNTEzMWQwMjBiNmY1MTIzMzk5NzJjNmViODhmMjdjNjdjOWY=
+  metadata.gz: 86c99d9aa7055c649b3611d29181429f25b4997c253504c4af5740973606f8838eeb466355d9cb961f07e1e79556d76ef6accb3bc2ad42abcd5767a2054a6b44
+  data.tar.gz: d71305cf683e36b261330a182b2c77de094e4f6be19b795aed62908091323abb8ad319b3dd37db1b09a9f0879f30d576787be107815fe6e0f9ffd50f2ba7a8f4

data/.travis.yml

@@ -1,14 +1,20 @@
 sudo: false
 language: ruby
 rvm:
-  - '1.8.7-p371'
-  - '1.9.3-p551'
   - '2.0.0-p648'
   - '2.1.10'
   - '2.2.6'
   - '2.3.3'
   - '2.4.0'
-  - 'jruby-1.7.26'
   - 'jruby-9.0.5.0'
   - 'jruby-9.1.5.0'
-script: bundle exec travis_check_rubies
+script:
+  - bundle exec rspec
+  - bundle exec travis_check_rubies
+matrix:
+  allow_failures:
+    - rvm: 'jruby-9.0.5.0'
+  exclude:
+    - rvm: 'jruby-9.0.5.0'
+  include:
+    - rvm: 'jruby-9.0.5.0'

data/README.markdown

@@ -29,7 +29,7 @@ Merge into your `.travis.yml`:
 matrix:
   include:
     - env: CHECK_RUBIES=✓
-      rvm: '2'
+      rvm: '2.4.0'
       script: bundle exec travis_check_rubies
 ```
 

data/bin/travis_check_rubies

@@ -1,24 +1,36 @@
 #!/usr/bin/env ruby
 
-require 'travis_check_rubies/version'
+require 'travis_check_rubies/travis_yml'
+require 'optparse'
 
-matches = TravisCheckRubies::Version.matches
-if matches.values.any?
-  ansi_red = `tput setaf 1 2>/dev/null`
-  ansi_green = `tput setaf 2 2>/dev/null`
-  ansi_reset = `tput sgr0 2>/dev/null`
-  puts 'Suggested changes:'
-  matches.each do |version, matched_versions|
-    if !matched_versions || matched_versions.include?(version)
-      puts " #{version}"
-    else
-      puts "#{ansi_red}-#{version}#{ansi_reset}"
-    end
-    if matched_versions
-      (matched_versions - [version]).each do |matched_version|
-        puts "#{ansi_green}+#{matched_version}#{ansi_reset}"
-      end
-    end
+options = {}
+
+op = OptionParser.new
+op.on('--parts N|M..N', 'Match updates by how many parts (0..2 by default)') do |parts|
+  options[:parts] = case parts
+  when /\A\d+\z/
+    parts.to_i
+  when /\A(\d+)..(\d+)\z/
+    $1.to_i..$2.to_i
+  else
+    fail "Expected number N or range M..N, got #{parts}"
   end
-  abort
 end
+op.on('--[no-]allow-pre', 'Allow matching pre releases (false by default)') do |allow_pre|
+  options[:allow_pre] = allow_pre
+end
+op.on('--[no-]intermediary', 'Include all latest version distinct by maximum matchable '\
+    'parts, like 2.3.X when current is 2.2.X and latest is 2.4.X (true by default)') do |intermediary|
+  options[:intermediary] = intermediary
+end
+op.on('--exclude V,V,V', Array, 'Exclude matching versions') do |exclude|
+  options[:exclude] = exclude
+end
+begin
+  op.parse!
+rescue => e
+  abort "#{e}\n\n#{op.help}"
+end
+
+suggestions = TravisCheckRubies::TravisYml.new.suggestions(options)
+abort suggestions if suggestions

data/lib/travis_check_rubies/travis_yml.rb

@@ -0,0 +1,70 @@
+require 'yaml'
+require 'travis_check_rubies/version'
+
+module TravisCheckRubies
+  class TravisYml
+    attr_reader :path
+    attr_reader :versions, :allow_failures_versions, :exclude_versions, :include_versions
+
+    def initialize(path = '.travis.yml')
+      @path = path
+
+      yaml = YAML.load_file(path)
+      @versions = Array(yaml['rvm']).map(&Version.method(:new))
+      @allow_failures_versions = matrix_versions(yaml, 'allow_failures')
+      @exclude_versions = matrix_versions(yaml, 'exclude')
+      @include_versions = matrix_versions(yaml, 'include')
+    end
+
+    def suggestions(options)
+      suggestions = Hash.new{ |h, k| h[k] = [] }
+
+      updates = Version.updates(versions, options)
+
+      versions.each do |version|
+        next unless (for_version = updates[version])
+        suggestions['rvm'] << "#{version} -> #{for_version.join(', ')}"
+      end
+
+      allow_failures_versions.each do |version|
+        next if versions.include?(version)
+        next if include_versions.include?(version)
+        suggestions['matrix.allow_failures'] << "#{version} in matrix.allow_failures is not in rvm or include list"
+      end
+
+      exclude_versions.each do |version|
+        next if versions.include?(version)
+        suggestions['matrix.exclude'] << "#{version} in matrix.exclude is not in rvm list"
+      end
+
+      {
+        'matrix.allow_failures' => allow_failures_versions,
+        'matrix.exclude' => exclude_versions,
+      }.each do |section, versions|
+        versions.each do |version|
+          next unless (for_version = updates[version])
+          suggestions[section] << "#{version} -> #{for_version.join(', ')}"
+        end
+      end
+
+      include_versions.each do |version|
+        next unless (for_version = Version.update(version, options))
+        next if for_version.include?(version)
+        suggestions['matrix.include'] << "#{version} -> #{for_version.join(', ')}"
+      end
+
+      return if suggestions.empty?
+      suggestions.map do |section, lines|
+        "#{section}:\n#{lines.map{ |line| "  #{line}" }.join("\n")}"
+      end.join("\n")
+    end
+
+  private
+
+    def matrix_versions(yaml, key)
+      return [] unless (matrix = yaml['matrix'])
+      return [] unless (list = matrix[key])
+      Array(list).map{ |attrs| attrs['rvm'] }.compact.map(&Version.method(:new))
+    end
+  end
+end

data/lib/travis_check_rubies/version.rb

@@ -1,12 +1,17 @@
 require 'net/http'
+require 'fspath'
 require 'set'
 require 'uri'
-require 'yaml'
 
 module TravisCheckRubies
   class Version
+    ROOT_URL = 'http://rubies.travis-ci.org/'
+    CACHE_TIME = 24 * 60 * 60
+
     class << self
-      ROOT_URL = 'http://rubies.travis-ci.org/'
+      def convert(version_or_string)
+        version_or_string.is_a?(self) ? version_or_string : new(version_or_string)
+      end
 
       def available
         @available ||= begin
@@ -14,36 +19,81 @@ module TravisCheckRubies
             url.start_with?(base_url)
           end.map do |url|
             new(url[%r{([^/]+)\.tar\.(?:gz|bz2)$}, 1])
-          end
+          end.sort
         end
       end
 
-      def selected
-        @selected ||= begin
-          Array(YAML.load_file('.travis.yml')['rvm']).map do |string|
-            new(string)
+      def update(version, parts: 0..2, allow_pre: false, intermediary: true, exclude: [])
+        version = convert(version)
+        return unless version.version_parts
+
+        parts = Array(parts)
+        exclude = exclude.map{ |ev| convert(ev) }
+        ordered = allow_pre ? available : available.partition(&:pre).inject(:+)
+        candidates = ordered.reverse.select do |v|
+          next unless v.version_parts
+          next unless v.match?(version, parts.min)
+          next unless v >= version
+          next if exclude.any?{ |ev| ev.match?(v, ev.version_parts.length) }
+          true
+        end
+
+        updates = if intermediary
+          candidates.group_by do |v|
+            v.version_parts.take(parts.max)
+          end.flat_map do |_, group|
+            parts.map do |n|
+              group.find{ |v| v.match?(version, n) }
+            end
+          end
+        else
+          parts.map do |n|
+            candidates.find{ |v| v.match?(version, n) }
           end
         end
+
+        updates.compact!
+        updates.uniq!
+        updates.sort!
+
+        updates unless [version] == updates
       end
 
-      def matches
-        matches = {}
-        deduplicate = Set.new
-        available_versioned = available.select(&:version_parts).sort.reverse
-        selected.select(&:version_parts).sort.reverse.each do |version|
-          matches[version] = (0..version.version_parts.length).map do |n|
-            available_versioned.find{ |v| version.match?(v, n) }
-          end.reverse.select{ |v| deduplicate.add?(v) }
+      def updates(versions, **options)
+        versions = versions.map{ |v| convert(v) }
+
+        updates = {}
+        has = Set.new
+        versions.uniq.sort.reverse_each do |version|
+          deduplicated = (update(version, options) || [version]).select{ |v| has.add?(v) }
+          updates[version] = [version] == deduplicated ? nil : deduplicated
         end
-        Hash[selected.map do |v|
-          [v, [v] == matches[v] ? nil : matches[v]]
-        end]
+
+        Hash[versions.map{ |v| [v, updates[v]] }]
       end
 
     private
 
+      def cache_path
+        @cache_path ||= FSPath(ENV['XDG_CACHE_HOME'] || '~/.cache').expand_path / 'travis_check_rubies.txt'
+      end
+
       def index_urls
-        @index_urls ||= Net::HTTP.get(URI(ROOT_URL + 'index.txt')).split("\n")
+        @index_urls ||= begin
+          if cache_path.size? && cache_path.mtime + CACHE_TIME > Time.now
+            cache_path.read
+          else
+            data = Net::HTTP.get(URI(ROOT_URL + 'index.txt'))
+
+            cache_path.dirname.mkpath
+            FSPath.temp_file('travis_check_rubies', cache_path.dirname) do |f|
+              f.write(data)
+              f.path.rename(cache_path)
+            end
+
+            data
+          end
+        end.split("\n")
       end
 
       def base_url
@@ -91,6 +141,10 @@ module TravisCheckRubies
       match_by(n) == other.match_by(n)
     end
 
+    def inspect
+      "#<#{self.class.name} #{str}>"
+    end
+
   protected
 
     def compare_by
@@ -103,7 +157,7 @@ module TravisCheckRubies
     end
 
     def match_by(n)
-      [type, variant, version_parts.take(n)]
+      [type, variant, version_parts && version_parts.take(n)]
     end
   end
 end

data/spec/travis_check_rubies/version_spec.rb

@@ -0,0 +1,425 @@
+require 'rspec'
+require 'travis_check_rubies/version'
+
+describe TravisCheckRubies::Version do
+  def v(str)
+    described_class.new(str)
+  end
+
+  def vs(strs)
+    strs.map{ |str| described_class.new(str) }
+  end
+
+  def cleanup_instance_variables(o)
+    o.instance_variables.each do |name|
+      o.remove_instance_variable(name)
+    end
+  end
+
+  describe 'parsing' do
+    {
+      '1.2.3-pre1' => {
+        version_parts: [1, 2, 3],
+        pre: 'pre1',
+      },
+      'jruby' => {
+        type: 'jruby',
+      },
+      'ruby-2.3.4.5-clang' => {
+        str: '2.3.4.5-clang',
+        version_parts: [2, 3, 4, 5],
+        variant: 'clang',
+      },
+      'xrb-1000' => {
+        type: 'xrb',
+        version_parts: [1000],
+      },
+      '1.5' => {
+        version_parts: [1, 5],
+      },
+      '1.8.4-p616' => {
+        version_parts: [1, 8, 4, 616],
+      },
+    }.each do |version_string, defined_attributes|
+      context "of #{version_string}" do
+        subject{ v(version_string) }
+
+        let(:default_attributes) do
+          {
+            str: version_string,
+            type: 'ruby',
+            version_parts: nil,
+            pre: nil,
+            variant: nil,
+          }
+        end
+
+        it{ is_expected.to have_attributes(default_attributes.merge(defined_attributes)) }
+      end
+    end
+  end
+
+  describe '#eql?' do
+    it 'returns false for different version' do
+      expect(v('1.8')).not_to eql(v('1.9'))
+    end
+
+    it 'returns true for equal versions' do
+      expect(v('1.8')).to eql(v('1.8'))
+    end
+
+    it 'returns true for same version in different format' do
+      expect(v('1.8')).to eql(v('ruby-1.8'))
+    end
+
+    it 'returns true for same version string' do
+      expect(v('1.8')).to eql('1.8')
+    end
+  end
+
+  describe '#<=>' do
+    let(:versions) do
+      vs(%w[
+        jruby-head
+        jruby-1.7.7
+        jruby-1.7.26
+        jruby-9.0.0.0.pre1
+        jruby-9.0.0.0.pre2
+        jruby-9.0.0.0
+        ruby-1.8.7
+        ruby-1.8.7-p371
+        ruby-2.0.0-p645
+        ruby-2.0.0-p647
+        ruby-2.0.0-p647-clang
+      ])
+    end
+
+    it 'oders versions using all parts' do
+      expect(versions.reverse.sort).to eq(versions)
+    end
+  end
+
+  describe '#match?' do
+    subject{ v('ruby-1.7.7') }
+
+    context 'for same version' do
+      let(:version){ v('ruby-1.7.7') }
+
+      it 'matches with any number of parts' do
+        (0..4).each{ |n| is_expected.to be_match(version, n) }
+      end
+    end
+
+    context 'for different patch version' do
+      let(:version){ v('ruby-1.7.8') }
+
+      it 'matches with 0..2 parts' do
+        (0..2).each{ |n| is_expected.to be_match(version, n) }
+      end
+
+      it 'does not match with more parts' do
+        (3..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+
+    context 'for different minor version' do
+      let(:version){ v('ruby-1.8.7') }
+
+      it 'matches with 0..1 parts' do
+        (0..1).each{ |n| is_expected.to be_match(version, n) }
+      end
+
+      it 'does not match with more parts' do
+        (2..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+
+    context 'for different major version' do
+      let(:version){ v('ruby-2.7.7') }
+
+      it 'matches with 0 parts' do
+        is_expected.to be_match(version, 0)
+      end
+
+      it 'does not match with more parts' do
+        (1..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+
+    context 'for version with additional part' do
+      let(:version){ v('ruby-1.7.7.1') }
+
+      it 'matches with 0..3 parts' do
+        (0..3).each{ |n| is_expected.to be_match(version, n) }
+      end
+
+      it 'does not match with more parts' do
+        is_expected.not_to be_match(version, 4)
+      end
+    end
+
+    context 'for pre release of same version' do
+      let(:version){ v('ruby-1.7.7-pre1') }
+
+      it 'matches with any number of parts' do
+        (0..4).each{ |n| is_expected.to be_match(version, n) }
+      end
+    end
+
+    context 'for different type of same version' do
+      let(:version){ v('jruby-1.7.7') }
+
+      it 'does not match' do
+        (0..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+
+    context 'for different variant of same version' do
+      let(:version){ v('ruby-1.7.7-clang') }
+
+      it 'does not match' do
+        (0..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+
+    context 'for version without version parts' do
+      let(:version){ v('ruby') }
+
+      it 'does not match' do
+        (0..4).each{ |n| is_expected.not_to be_match(version, n) }
+      end
+    end
+  end
+
+  describe '#inspect' do
+    {
+      '1.2.3-pre1' => '#<TravisCheckRubies::Version 1.2.3-pre1>',
+      'jruby' => '#<TravisCheckRubies::Version jruby>',
+    }.each do |version_string, inspect|
+      context "for #{version_string}" do
+        it{ expect(v(version_string).inspect).to eq(inspect) }
+      end
+    end
+  end
+
+  describe '.index_urls' do
+    let(:cache_path){ FSPath.temp_file_path }
+
+    before do
+      cleanup_instance_variables(described_class)
+      allow(described_class).to receive(:cache_path).and_return(cache_path)
+    end
+
+    it 'returns urls from text index of rubies.travis-ci.org' do
+      allow(Net::HTTP).to receive(:get).with(URI('http://rubies.travis-ci.org/index.txt')).
+        and_return("one\ntwo\nthree")
+
+      expect(described_class.send(:index_urls)).to eq(%w[one two three])
+    end
+
+    it 'caches result' do
+      allow(Net::HTTP).to receive(:get).with(URI('http://rubies.travis-ci.org/index.txt')).
+        once.and_return("a\nb\nc")
+
+      3.times{ expect(described_class.send(:index_urls)).to eq(%w[a b c]) }
+    end
+
+    it 'reads cache from file if it is new' do
+      allow(cache_path).to receive(:size?).and_return(616)
+      allow(cache_path).to receive(:mtime).and_return(Time.now - described_class::CACHE_TIME / 2)
+      allow(cache_path).to receive(:read).and_return("foo\nbar")
+
+      expect(Net::HTTP).not_to receive(:get)
+      expect(described_class.send(:index_urls)).to eq(%w[foo bar])
+    end
+
+    it 'writes cache file if it is stale' do
+      allow(cache_path).to receive(:size?).and_return(616)
+      allow(cache_path).to receive(:mtime).and_return(Time.now - described_class::CACHE_TIME * 2)
+      allow(Net::HTTP).to receive(:get).with(URI('http://rubies.travis-ci.org/index.txt')).
+        once.and_return("brave\nnew\nworld")
+
+      expect(described_class.send(:index_urls)).to eq(%w[brave new world])
+      expect(cache_path.read).to eq("brave\nnew\nworld")
+    end
+  end
+
+  describe '.base_url' do
+    before do
+      cleanup_instance_variables(described_class)
+      allow(ENV).to receive(:[]).with('TRAVIS').and_return(env_travis)
+    end
+
+    context 'when env variable TRAVIS is set' do
+      let(:env_travis){ 'true' }
+
+      it 'gets base_url from rvm debug' do
+        allow(described_class).to receive(:`).with('rvm debug').
+          and_return(%Q{  foo: "xxx"  \n  system: "XXX/YYY"  \n  bar: "yyy"  })
+
+        expect(described_class.send(:base_url)).to eq('http://rubies.travis-ci.org/XXX/YYY/')
+      end
+    end
+
+    context 'when env variable TRAVIS is not set' do
+      let(:env_travis){ nil }
+
+      it 'gets base_url from first ubuntu url in index' do
+        allow(described_class).to receive(:index_urls).and_return(%w[
+          http://rubies.travis-ci.org/osx/AAA/1.tar.gz
+          http://rubies.travis-ci.org/ubuntu/ZZZ/2.tar.gz
+          http://rubies.travis-ci.org/ubuntu/BBB/3.tar.gz
+        ])
+
+        expect(described_class.send(:base_url)).to eq('http://rubies.travis-ci.org/ubuntu/BBB/')
+      end
+    end
+  end
+
+  describe '.available' do
+    before do
+      cleanup_instance_variables(described_class)
+    end
+
+    it 'gets sorted versions from index urls matching base_url' do
+      allow(described_class).to receive(:index_urls).and_return(%w[
+        http://rubies.travis-ci.org/osx/AAA/1.tar.gz
+        http://rubies.travis-ci.org/ubuntu/ZZZ/2.tar.gz
+        http://rubies.travis-ci.org/ubuntu/BBB/4.tar.gz
+        http://rubies.travis-ci.org/ubuntu/BBB/3.tar.bz2
+      ])
+      allow(described_class).to receive(:base_url).and_return('http://rubies.travis-ci.org/ubuntu/BBB/')
+
+      expect(described_class.available).to eq([v('3'), v('4')])
+    end
+
+    it 'caches result' do
+      allow(described_class).to receive(:index_urls).once.and_return(%w[
+        http://rubies.travis-ci.org/ubuntu/CCC/a.tar.gz
+        http://rubies.travis-ci.org/ubuntu/CCC/b.tar.bz2
+      ])
+      allow(described_class).to receive(:base_url).and_return('http://rubies.travis-ci.org/ubuntu/CCC/')
+
+      3.times{ expect(described_class.available).to eq([v('a'), v('b')]) }
+    end
+  end
+
+  describe '.update' do
+    before do
+      allow(described_class).to receive(:available).and_return(vs(available_strs).sort)
+    end
+
+    context 'for version without version parts' do
+      let(:available_strs){ %w[ruby 1.2.3 2.3.4] }
+
+      it{ expect(described_class.update(v('ruby'))).to eq nil }
+    end
+
+    context 'when there are no updates' do
+      let(:available_strs){ %w[ruby 1.2.3 2.3.4] }
+
+      it{ expect(described_class.update(v('2.3.4'))).to eq nil }
+    end
+
+    context 'when version is not present' do
+      let(:available_strs){ %w[ruby 1.2.3 2.3.4] }
+
+      it{ expect(described_class.update(v('3.4.5'))).to eq [] }
+    end
+
+    describe 'parts and intermediary' do
+      let(:version){ v('1.7.1') }
+
+      let(:available_strs){ %w[
+        1.7.1 1.7.2  1.8.6 1.8.7  1.9.2 1.9.3
+        2.0.8 2.0.9  2.1.6 2.1.7  2.2.4 2.2.5
+        3.0.4 3.0.5  3.1.2 3.1.3  3.2.0 3.2.1
+      ] }
+
+      context 'when intermediary is true' do
+        it 'returns all latest with distinct 0..2 (by default) matching version parts' do
+          expect(described_class.update(version)).to eq vs(%w[1.7.2 1.8.7 1.9.3 2.0.9 2.1.7 2.2.5 3.0.5 3.1.3 3.2.1])
+        end
+
+        {
+          0 => %w[3.2.1],
+          0..1 => %w[1.9.3 2.2.5 3.2.1],
+          0..2 => %w[1.7.2 1.8.7 1.9.3 2.0.9 2.1.7 2.2.5 3.0.5 3.1.3 3.2.1],
+          1..2 => %w[1.7.2 1.8.7 1.9.3],
+          2 => %w[1.7.2],
+        }.each do |parts, expected|
+          it "returns all latest with distinct #{parts} matching version parts" do
+            expect(described_class.update(version, parts: parts)).to eq vs(expected)
+          end
+        end
+      end
+
+      context 'when intermediary is false' do
+        it 'returns latest for each 0..2 (by default) matching version parts' do
+          expect(described_class.update(version, intermediary: false)).to eq vs(%w[1.7.2 1.9.3 3.2.1])
+        end
+
+        {
+          0 => %w[3.2.1],
+          0..1 => %w[1.9.3 3.2.1],
+          0..2 => %w[1.7.2 1.9.3 3.2.1],
+          1..2 => %w[1.7.2 1.9.3],
+          2 => %w[1.7.2],
+        }.each do |parts, expected|
+          it "returns latest for each #{parts} matching version parts" do
+            expect(described_class.update(version, parts: parts, intermediary: false)).to eq vs(expected)
+          end
+        end
+      end
+    end
+
+    describe 'pre releases' do
+      let(:available_strs){ %w[2.4.0-pre 2.4.0 2.4.1 2.4.2-pre1 2.4.2-pre2] }
+
+      context 'when for pre release there is only newer pre release' do
+        it 'returns next pre release' do
+          expect(described_class.update(v('2.4.2-pre1'))).to eq [v('2.4.2-pre2')]
+        end
+      end
+
+      context 'when there is a newer release and even newer pre release' do
+        context 'for release' do
+          it 'returns newer release when allow_pre is false' do
+            expect(described_class.update(v('2.4.0'))).to eq [v('2.4.1')]
+          end
+
+          it 'returns even newer pre release when allow_pre is true' do
+            expect(described_class.update(v('2.4.0'), allow_pre: true)).to eq [v('2.4.2-pre2')]
+          end
+        end
+
+        context 'for pre release' do
+          it 'returns release when allow_pre is false' do
+            expect(described_class.update(v('2.4.0-pre'))).to eq [v('2.4.1')]
+          end
+
+          it 'returns even newer pre release when allow_pre is true' do
+            expect(described_class.update(v('2.4.0-pre'), allow_pre: true)).to eq [v('2.4.2-pre2')]
+          end
+        end
+      end
+    end
+  end
+
+  describe '.updates' do
+    before do
+      allow(described_class).to receive(:available).
+        and_return(vs(%w[1.8.7 1.9.3 2.0.9 2.1.7]).sort)
+    end
+
+    it 'returns a hash with updates for higher versions removed from updates for lower versions' do
+      expect(described_class.updates(vs(%w[2.0.8 2.1.7 1.8.6 1.9.3])).to_a).
+        to eq({
+          v('2.0.8') => vs(%w[2.0.9]),
+          v('2.1.7') => nil,
+          v('1.8.6') => vs(%w[1.8.7]),
+          v('1.9.3') => nil,
+        }.to_a)
+    end
+  end
+end

data/travis_check_rubies.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = 'travis_check_rubies'
-  s.version     = '0.1.0'
+  s.version     = '0.2.0'
   s.summary     = 'Are you using the latest rubies in .travis.yml?'
   s.description = 'Check if `.travis.yml` specifies latest available rubies from listed on http://rubies.travis-ci.org and propose changes'
   s.homepage    = "http://github.com/toy/#{s.name}"
@@ -13,4 +13,10 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = %w[lib]
+
+  s.required_ruby_version = '>= 2.0.0'
+
+  s.add_runtime_dependency 'fspath', '~> 3.0'
+
+  s.add_development_dependency 'rspec', '~> 3.0'
 end

metadata

@@ -1,15 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: travis_check_rubies
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Ivan Kuchin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-22 00:00:00.000000000 Z
-dependencies: []
+date: 2017-03-11 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: fspath
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.0'
 description: Check if `.travis.yml` specifies latest available rubies from listed
   on http://rubies.travis-ci.org and propose changes
 email: 
@@ -18,14 +46,16 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.markdown
 - bin/travis_check_rubies
 - lib/travis_check_rubies.rb
+- lib/travis_check_rubies/travis_yml.rb
 - lib/travis_check_rubies/version.rb
+- spec/travis_check_rubies/version_spec.rb
 - travis_check_rubies.gemspec
 homepage: http://github.com/toy/travis_check_rubies
 licenses:
@@ -37,19 +67,19 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.4
+rubygems_version: 2.6.10
 signing_key: 
 specification_version: 4
 summary: Are you using the latest rubies in .travis.yml?
-test_files: []
-has_rdoc: 
+test_files:
+- spec/travis_check_rubies/version_spec.rb