bundler-stats 2.0.1 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: effbf5fc1dcb265737923b548467df2b831f444638f673e37622a2efd564cb54
-  data.tar.gz: 23cae592f904d2cdaef97b5597f953b0a927f24368d5f6e1b8763224391d4884
+  metadata.gz: 38f866872f6ba724ee82db46ba1715c05bcf8f22272f0ae6a23a96b1760232ce
+  data.tar.gz: ee9b4655d4c8faa1da7f23d3602820ab70eeaf10aad836f4bc7c544cd10bf795
 SHA512:
-  metadata.gz: de70cf8fedea1260329b24fea38e1f564215ae7f6bf67d5d0d87d6aa1b639cad39ced3625069ea6ecb38e315a3fec7de561cde0a331e951d550234f90e6ed0c7
-  data.tar.gz: 606dc092473b823593dc2baec837c91956a6554c31f66990afc1f89052dac95f9c0209324b7e643429e04fe922025bcb090264f9730a7b6d900706bf6138ccfd
+  metadata.gz: e31cce716bfae8bdc92e525d1047678b96c4ea18f80aee2875718f84c690a8a76dbd34ee923b445724716f45eb0e9cdbeab69e13b0d6479284cb4a1185beeecf
+  data.tar.gz: 2644320effea69f6e117a92154e3ddf0e0a3327a23d8141262dc4138fefa8a8a84eab049e68686ded2f778d2b271cbf8a00c43a6eb2b9fa06c09d01b11ce694d

data/.github/workflows/main.yml

@@ -0,0 +1,27 @@
+name: CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+        ruby-version: [3.1, 3.0, 2.7, 2.6, 2.5, 2.4, 2.3]
+    runs-on: ${{ matrix.os }}
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby ${{ matrix.ruby-version }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+      - name: Install dependencies
+        run: bundle install
+      - name: Run tests
+        run: bundle exec rspec

data/.gitignore

@@ -1 +1,2 @@
 Gemfile.lock
+bundler-stats-*.gem

data/.ruby-version

@@ -0,0 +1 @@
+2.7.3

data/CHANGELOG.md

@@ -1,19 +1,112 @@
 Changelog
 =============
 
-## [1.1.0] - 2018-03-16
+## [2.3.0] - 2022-02-12
+
+### Added
+- Skiplist now correctly removes dependencies from 1st and onward levels
+- Wildcard ability for specifying gems for skiplist
+- Document `versions` in README
+
+## [2.2.0] - 2022-02-09
+
+### Added
+- Support for gems.rb and gems.locked files.
+- Proper exit codes from thor when errors occur
+- Looser dependency on thor
+
+## [2.1.1] - 2021-11-30
+
+### Changed
+- Symlink isn't the recommended way to alias a command. Patterned current
+  solution after bundle-audit. Boy that naming convention was unfortunate.
+- Pin down dependencies a bit.
+
+### Added
+- Moved to Github test workflow, by @etagwerker.
+- New default ruby version to be most current non-3 ruby.
+- Build artifacts now ignored.
+- Moved to a real spec_helper so tests can be filtered during runs if desired.
+
 ### Fixed
+- Failing test from previous version.
 
-- Remove unintentional inclusion of pry outside of dev environment
+## [2.1.0] - 2021-11-29
+
+### Changed
+- Add Travis targets for more modern rubies, by @etagwerker.
+- Make sorting predictable across platforms, by @etagwerker.
+
+### Fixed
+- Fix error in CI when `tput` isn't available, by @etagwerker.
+
+## [2.0.1] - 2018-05-04
 
-## [1.1.0] - 2018-03-15
 ### Added
+- Complete custom table printer for some nicer output.
+
+## [2.0.0] - 2018-05-04
+Broken as hell.
+
+## [1.3.4] - 2019-04-18
+
+### Changed
+- Allow use of either `bundle-stats` or `bundler-stats` since the gem name was
+  a confusing choice. Live and learn.
+
+### Added
+- Display resolved version of a gem when using `bundler-stats show`.
+
+## [1.3.3] - 2019-04-18
+
+### Changed
+- Only print missing system dependency warning once per target gem, rather than
+  blowing up the console when a complicated gem is affected.
 
+## [1.3.2] - 2019-04-17
+
+### Fixed
+- Fix issue when testing removability and a system gem from another platform
+  is "required", thx @rwojnarowski.
+
+## [1.3.1] - 2019-04-05
+
+### Changed
+- Nicer table printing, still committed to not adding a table printing gem.
+
+## [1.3.0] - 2019-04-05
+
+### Changed
+- Reversed the order in which gems are printed to worst-offenders-first.
+
+## [1.2.1] - 2019-04-05
+
+### Fixed
+- When a system gem is missing from the lockfile (but is depended upon), warn
+  the user rather than exploding.
+
+## [1.2.0] - 2019-04-05
+
+### Fixed
+- Loosen dependency on thor gem, by localhostdotdev.
+
+## [1.1.2] - 2018-03-16
+Wonkiness w/ versioning. Apparently I was bad at this.
+
+## [1.1.0] - 2018-03-16
+Eventually superseded by 1.1.2 for reasons.
+
+### Fixed
+- Remove unintentional inclusion of pry outside of dev environment, per @Tuxified
+
+## [1.1.0] - 2018-03-15
+
+### Added
 - Adds a way to view dependency version restrictions for a given gem, by @olivierlacan
 
 ## [1.0.0] - 2016-04-13
-### Added
 
+### Added
 - Base library, woo!
 - List all transitive dependencies and how many other deps rely on them
 - View list of Github-specified dependencies

data/README.md

@@ -34,8 +34,10 @@ Usage
         bundle-stats version          # Prints the bundler-stats version
         bundle-stats versions TARGET  # Shows versions requirements for target in other dependencies
 
+### Auditing Your Gemfile
+
 The most obvious thing to do is run the command by itself, which should help
-identify problem areas:
+identify problem areas.
 
     > bundle-stats
 
@@ -62,7 +64,7 @@ identify problem areas:
       Unpinned Versions   54
             Github Refs   0
 
-It looks like rails_admin is a huge problem. Use `show` to investigate:
+It looks like rails_admin is a huge problem. Use `show` to investigate.
 
     > bundle-stats show rails_admin
     bundle-stats for rails_admin
@@ -98,22 +100,117 @@ It looks like rails_admin is a huge problem. Use `show` to investigate:
     |                                | bundler                                |
     +--------------------------------|----------------------------------------+
 
-Removing the dep will only actually remove 9 gems. The rest are shared
-dependencies with other gems like rails. We can also omit trees we aren't going
-to remove (hi rails) by not following them:
+### Skipping Common Dependencies
+
+Let's take a look at another common gem to rails codebases. In this case,
+we have some unique dependencies, but also many dependencies on rails, and on
+its constituent gems.
+
+    > bundle-stats show compass-rails
+    bundle-stats for compass-rails
+
+    +--------------------------------|----------------------------------------+
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (35) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
+    |                                | railties, sprockets-rails, tilt        |
+    |                                | actionpack, activesupport, method_source |
+    |                                | rake, thor, actionview, rack, rack-test |
+    |                                | rails-dom-testing, rails-html-sanitizer |
+    |                                | builder, erubi, concurrent-ruby, i18n  |
+    |                                | minitest, tzinfo, thread_safe, nokogiri |
+    |                                | mini_portile2, loofah, crass           |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
+    +--------------------------------|----------------------------------------+
+
+We're not looking to remove rails, so there's not much point in including it
+within this output. Instead, we can use the `nofollow` flag to skip it in all
+output lists.
 
-    > bundle-stats show sass-rails --nofollow="railties,activesupport,actionpack"
-    bundle-stats for sass-rails
+    > bundle-stats show compass-rails --nofollow="railties,activeupport,actionview,actionpack"
+    bundle-stats for compass-rails
 
     +--------------------------------|----------------------------------------+
-    |           Depended Upon By (2) | compass-rails, rails_admin             |
-    |                 Depends On (9) | railties, sass, sprockets              |
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (20) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
     |                                | sprockets-rails, tilt, concurrent-ruby |
-    |                                | rack, actionpack, activesupport        |
-    |             Unique to This (0) |                                        |
+    |                                | rack, activesupport, i18n, minitest    |
+    |                                | tzinfo, thread_safe                    |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
     +--------------------------------|----------------------------------------+
 
-To consume information with a build job or somesuch, all commands can emit JSON:
+This is better, but for other codebases it's common for gems to depend on each
+of the _many many_ child gems of rails individually. Rather than specifying each
+by itself, we can use wildcards to remove them in bulk.
+
+    > bundle-stats show compass-rails --nofollow="rail*,action*,active*"
+    bundle-stats for compass-rails
+
+    +--------------------------------|----------------------------------------+
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (15) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
+    |                                | sprockets-rails, tilt, concurrent-ruby |
+    |                                | rack                                   |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
+    +--------------------------------|----------------------------------------+
+
+### Showing Required Versions
+
+Sometimes you try to upgrade a gem, you ask bundler _very nicely_, but you
+find that the version hasn't changed. When that happens, sometimes it's not
+clear why the gem version isn't changing.
+
+In that case, use `versions` to list all required dependencies on your gem.
+
+    > bundle-stats versions actionpack
+    version dependencies for actionpack
+
+    Depended Upon By   24
+    Resolved Version   5.2.0
+
+    +----------------------------|------------------+
+    |                       Name | Required Version |
+    +----------------------------|------------------+
+    |                  webpacker | = 5.2.0          |
+    |               actionmailer | = 5.2.0          |
+    |              activestorage | = 5.2.0          |
+    |               coffee-rails | = 5.2.0          |
+    |              compass-rails | = 5.2.0          |
+    |                     devise | = 5.2.0          |
+    |          factory_bot_rails | = 5.2.0          |
+    |         font-awesome-rails | = 5.2.0          |
+    |                rails_admin | = 5.2.0          |
+    |                   railties | = 5.2.0          |
+    |                 sass-rails | = 5.2.0          |
+    |                     scenic | = 5.2.0          |
+    |                 versionist | = 5.2.0          |
+    |                actioncable | = 5.2.0          |
+    |               jquery-rails | = 5.2.0          |
+    |            jquery-ui-rails | = 5.2.0          |
+    | jquery_file_download-rails | = 5.2.0          |
+    |                      rails | = 5.2.0          |
+    |                rspec-rails | >= 3.0           |
+    |            sprockets-rails | >= 4.0           |
+    |                 haml-rails | >= 4.0.1         |
+    |                 responders | >= 4.2.0, < 5.3  |
+    |                simple_form | >= 5.0           |
+    |   rails-controller-testing | ~> 5.x, >= 5.0.1 |
+    +----------------------------|------------------+
+
+### Output Formats
+
+To consume information with a build job or somesuch, all commands can emit JSON.
 
     > bundle-stats show sass-rails --nofollow="railties,activesupport,actionpack" -f json
     {
@@ -154,7 +251,7 @@ Credits
 Thanks to the many kind people at [RailsCamp East
 2016](http://east.railscamp.com) for the help, the ideas, and the support.
 
-Thanks to Isaac Bowen for being pedantic about speeling.
+Also, many other folks for their feature / fix contributions. ❤️
 
 License
 -------

data/bin/bundle-stats

@@ -1,6 +1,5 @@
 #!/usr/bin/env ruby
 
-
 require 'rubygems'
 
 lib_dir = File.expand_path(File.join(File.dirname(__FILE__),'..','lib'))

data/bin/bundler-stats

@@ -0,0 +1,3 @@
+#!/usr/bin/env ruby
+
+load File.expand_path('../bundle-stats', __FILE__)

data/bundler-stats.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |gem|
   gem.licenses    = "MIT"
   gem.authors     = "Joseph Mastey"
   gem.email       = "jmmastey@gmail.com"
-  gem.homepage    = ""
+  gem.homepage    = "http://github.com/jmmastey/bundler-stats"
 
   gem.metadata    = {
     "homepage_uri" => "http://github.com/jmmastey/bundler-stats",
@@ -35,12 +35,12 @@ Gem::Specification.new do |gem|
 
   gem.require_paths = %w[ext lib].select { |dir| File.directory?(dir) }
 
-  gem.add_dependency "bundler", ">= 1.9"
-  gem.add_dependency "thor", "~> 0.19"
+  gem.add_dependency "bundler", ">= 1.9", "< 3"
+  gem.add_dependency "thor", ">= 0.19.0", "< 2.0"
 
   gem.add_development_dependency "rspec", "~> 3.4"
   gem.add_development_dependency "guard", "~> 2.13"
-  gem.add_development_dependency "guard-rspec"
+  gem.add_development_dependency "guard-rspec", "< 5"
   gem.add_development_dependency "pry", "~> 0.10"
-  gem.add_development_dependency "rb-readline"
+  gem.add_development_dependency "rb-readline", "< 1"
 end

data/lib/bundler/stats/calculator.rb

@@ -5,7 +5,7 @@ module Bundler
     class Calculator
       attr_reader :parser, :tree, :gemfile, :remover
 
-      def initialize(gemfile_path, lockfile_path, options = {})
+      def initialize(gemfile_path, lockfile_path, skiplist = nil)
         raise ArgumentError unless File.readable?(lockfile_path)
         raise ArgumentError unless File.readable?(gemfile_path)
 
@@ -17,9 +17,7 @@ module Bundler
         lock_contents = File.read(lockfile_path)
         @parser = Bundler::LockfileParser.new(lock_contents)
 
-        skiplist = options.fetch(:skiplist, [])
         @tree = Bundler::Stats::Tree.new(@parser, skiplist: skiplist)
-
         @remover = Bundler::Stats::Remover.new(@tree, @gemfile)
       end
 
@@ -64,7 +62,7 @@ module Bundler
         stats = @gemfile.map do |gem|
           @tree.summarize(gem.name)
         end
-        stats.sort_by { |row| row[:total_dependencies] }.reverse
+        stats.sort_by { |row| [row[:total_dependencies] * -1, row[:name]] }
       end
     end
   end

data/lib/bundler/stats/cli.rb

@@ -59,6 +59,10 @@ module Bundler
 
       private
 
+      def self.exit_on_failure?
+        true
+      end
+
       def draw_stats(gem_stats, summary)
         max_name_length = gem_stats.map { |gem| gem[:name].length }.max
 
@@ -95,7 +99,7 @@ module Bundler
 
       def draw_versions(stats, target)
         dependers = stats[:top_level_dependencies] # they do the depending
-        say "bundle-stats for #{target}"
+        say "version dependencies for #{target}"
         say Printer.new(
           headers: nil,
           borders: false,
@@ -117,31 +121,30 @@ module Bundler
       end
 
       def build_calculator(options)
-        if !options[:nofollow].nil?
-          skiplist = options[:nofollow].gsub(/\s+/, '').split(",")
-        else
-          skiplist = []
-        end
-
-        @calculator ||= Bundler::Stats::Calculator.new(gemfile_path, lockfile_path, skiplist: skiplist)
+        skiplist = Bundler::Stats::Skiplist.new(options[:nofollow])
+        @calculator ||= Bundler::Stats::Calculator.new(gemfile_path, lockfile_path, skiplist)
       end
 
       def gemfile_path
         cwd = Pathname.new("./")
         until cwd.realdirpath.root? do
-          return (cwd + "Gemfile") if File.exist?(cwd + "Gemfile")
+          %w(gems.rb Gemfile).each do |gemfile|
+            return (cwd + gemfile) if File.exist?(cwd + gemfile)
+          end
           cwd = cwd.parent
         end
-        raise ArgumentError, "Couldn't find Gemfile in this directory or parents"
+        raise ArgumentError, "Couldn't find gems.rb nor Gemfile in this directory or parents"
       end
 
       def lockfile_path
         cwd = Pathname.new(".")
         until cwd.realdirpath.root? do
-          return (cwd + "Gemfile.lock") if File.exist?(cwd + "Gemfile.lock")
+          %w(gems.locked Gemfile.lock).each do |lockfile|
+            return (cwd + lockfile) if File.exist?(cwd + lockfile)
+          end
           cwd = cwd.parent
         end
-        raise ArgumentError, "Couldn't find Gemfile in this directory or parents"
+        raise ArgumentError, "Couldn't find gems.locked nor Gemfile.lock in this directory or parents"
       end
     end
   end

data/lib/bundler/stats/printer.rb

@@ -38,7 +38,7 @@ class Bundler::Stats::Printer
 
   def terminal_width
     Integer(Kernel.send(:"`", "tput cols"))
-  rescue Errno::ENOENT, TypeError, Errno::ENOENT
+  rescue StandardError
     80
   end
 

data/lib/bundler/stats/skiplist.rb

@@ -0,0 +1,32 @@
+class Bundler::Stats::Skiplist
+  attr_reader :regex_list
+
+  def initialize(list = '')
+    @regex_list = if list.respond_to?(:regex_list)
+      list.regex_list
+    elsif list.respond_to?(:split)
+      list
+        .gsub(/\s+/, '')
+        .split(",")
+        .map { |str| glob_to_regex(str) }
+    else
+      (list || []).map { |str| glob_to_regex(str) }
+    end
+  end
+
+  def filter(gem_list)
+    gem_list.reject { |gem| self.include?(gem) }
+  end
+
+  def include?(target)
+    @regex_list.any? { |pattern| pattern =~ target.name }
+  end
+
+  private
+
+  def glob_to_regex(str)
+    str.gsub!('*','.*') # weird globbish behavior
+
+    /^#{str}$/
+  end
+end

data/lib/bundler/stats/tree.rb

@@ -1,20 +1,21 @@
 class Bundler::Stats::Tree
-  attr_accessor :tree
+  attr_accessor :tree, :skiplist
 
   ERR_MESSAGE = "The dependency `%s` wasn't found. It may not be present in " \
                 "your Gemfile.lock. This often happens when a dependency isn't " \
                 "installed on your platform."
 
-  def initialize(parser, skiplist: [])
+  def initialize(parser, skiplist: '')
     raise ArgumentError unless parser.respond_to?(:specs)
 
     @parser   = parser
     @tree     = specs_as_tree(@parser.specs)
-    @skiplist = skiplist
+    @skiplist = Bundler::Stats::Skiplist.new(skiplist)
   end
 
   def summarize(target)
     transitive_dependencies = transitive_dependencies(target)
+
     { name: target,
       total_dependencies: transitive_dependencies.count,
       first_level_dependencies: first_level_dependencies(target).count,
@@ -51,12 +52,12 @@ class Bundler::Stats::Tree
       return []
     end
 
-    top_level = @tree[target].dependencies
+    top_level = skiplist.filter(@tree[target].dependencies)
     all_level = top_level + top_level.inject([]) do |arr, dep|
       # turns out bundler refuses to include itself in the dependency tree,
       # which is sneaky
       next arr if dep.name == "bundler"
-      next arr if @skiplist.include? dep.name
+      next arr if skiplist.include? dep
 
       arr += transitive_dependencies(dep.name)
     end

data/lib/bundler/stats/version.rb

@@ -1,5 +1,5 @@
 module Bundler
   module Stats
-    VERSION = '2.0.1'
+    VERSION = '2.3.0'
   end
 end

data/lib/bundler/stats.rb

@@ -1,5 +1,6 @@
 require_relative 'stats/calculator'
 require_relative 'stats/printer'
 require_relative 'stats/remover'
+require_relative 'stats/skiplist'
 require_relative 'stats/tree'
 require_relative 'stats/version'

data/spec/lib/bundler/stats/calculator_spec.rb

@@ -1,5 +1,4 @@
-require 'bundler'
-require 'bundler/stats'
+require 'spec_helper'
 
 describe Bundler::Stats::Calculator do
   subject { described_class }
@@ -39,9 +38,9 @@ describe Bundler::Stats::Calculator do
       # this sucks. break this dependency further
       allow(Bundler::LockfileParser).to receive(:new) { "parser" }
 
-      target = subject.new(gemfile_path, lockfile_path, skiplist: "abc")
+      target = subject.new(gemfile_path, lockfile_path, "skiplist")
 
-      expect(Bundler::Stats::Tree).to have_received(:new).with("parser", skiplist: "abc")
+      expect(Bundler::Stats::Tree).to have_received(:new).with("parser", skiplist: "skiplist")
     end
   end
 
@@ -77,6 +76,16 @@ describe Bundler::Stats::Calculator do
   end
 
   context "#gem_stats" do
+    let(:partial_sorted_result) do
+      [
+        ["rolify", 0],
+        ["rubocop-rspec", 0],
+        ["spring", 0],
+        ["state_machine", 0],
+        ["will_paginate", 0]
+      ]
+    end
+
     it "includes entries for each gem" do
       calculator = subject.new(gemfile_path, lockfile_path)
 
@@ -85,6 +94,14 @@ describe Bundler::Stats::Calculator do
       expect(target).to be_a(Array)
       expect(target.length).to eq(calculator.gemfile.length)
     end
+
+    it "sorts entries by total dependencies descending and name ascending" do
+      calculator = subject.new(gemfile_path, lockfile_path)
+
+      target = calculator.gem_stats
+      tuple = target.map {|x| [x[:name], x[:total_dependencies]] }
+      expect(tuple).to end_with(*partial_sorted_result)
+    end
   end
 
   context "#summary" do

data/spec/lib/bundler/stats/printer_spec.rb

@@ -1,5 +1,4 @@
-require 'bundler'
-require 'bundler/stats'
+require 'spec_helper'
 
 describe Bundler::Stats::Printer do
   subject { described_class }
@@ -30,6 +29,17 @@ describe Bundler::Stats::Printer do
         expect(response).to eq(80)
       end
     end
+
+    context "tput returns an error" do
+      it "returns the default value" do
+        allow(Kernel).to receive(:send).and_return("tput: No value for $TERM and no -T specified")
+
+        printer   = subject.new
+        response  = printer.terminal_width
+
+        expect(response).to eq(80)
+      end
+    end
   end
 
   describe "#column_widths" do

data/spec/lib/bundler/stats/remover_spec.rb

@@ -1,5 +1,4 @@
-require 'bundler'
-require 'bundler/stats'
+require 'spec_helper'
 
 LazyLazySpec = Struct.new(:name, :dependencies)
 FakeLockfileParser = Struct.new(:specs)

data/spec/lib/bundler/stats/tree_spec.rb

@@ -1,5 +1,4 @@
-require 'bundler'
-require 'bundler/stats'
+require 'spec_helper'
 
 describe Bundler::Stats::Tree do
   subject { described_class }
@@ -141,20 +140,24 @@ describe Bundler::Stats::Tree do
   end
 
   context "skip lists" do
-    it "still includes the skipped entry" do
+    it "skips processing the requested entry, plus children of skipped entries" do
       tree = subject.new(parser, skiplist: ["depth-three"])
 
       target = tree.transitive_dependencies("depth-one")
 
-      expect(target.map(&:name)).to include("depth-three")
+      expect(target.map(&:name)).to include("depth-two")
+      expect(target.map(&:name)).not_to include("depth-three")
+      expect(target.map(&:name)).not_to include("depth-four")
     end
 
-    it "stops processing the children of skipped entries" do
-      tree = subject.new(parser, skiplist: ["depth-three"])
+    it "allows skiplist wildcards" do
+      tree = subject.new(parser, skiplist: ["depth-t*"])
 
       target = tree.transitive_dependencies("depth-one")
 
-      expect(target.map(&:name)).not_to include("depth-four")
+      expect(target.map(&:name)).not_to include("depth-two")
+      expect(target.map(&:name)).not_to include("depth-three")
+      expect(target.map(&:name)).not_to include("depth-four") # because transitive
     end
   end
 end

data/spec/spec_helper.rb

@@ -0,0 +1,11 @@
+require 'bundler'
+require 'bundler/stats'
+
+RSpec.configure do |config|
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+
+  config.expect_with :rspec do |expectations|
+    expectations.syntax = :expect
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bundler-stats
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.3.0
 platform: ruby
 authors:
 - Joseph Mastey
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-05-05 00:00:00.000000000 Z
+date: 2022-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -17,6 +17,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1.9'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -24,20 +27,29 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1.9'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.19.0
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.19.0
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0.19'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -70,16 +82,16 @@ dependencies:
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -98,16 +110,16 @@ dependencies:
   name: rb-readline
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1'
 description: Looks through your lockfile and tries to identify problematic use of
   dependencies
 email: jmmastey@gmail.com
@@ -120,9 +132,10 @@ extra_rdoc_files:
 - CODE_OF_CONDUCT.md
 - README.md
 files:
+- ".github/workflows/main.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
+- ".ruby-version"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
@@ -137,22 +150,24 @@ files:
 - lib/bundler/stats/cli.rb
 - lib/bundler/stats/printer.rb
 - lib/bundler/stats/remover.rb
+- lib/bundler/stats/skiplist.rb
 - lib/bundler/stats/tree.rb
 - lib/bundler/stats/version.rb
 - spec/lib/bundler/stats/calculator_spec.rb
 - spec/lib/bundler/stats/printer_spec.rb
 - spec/lib/bundler/stats/remover_spec.rb
 - spec/lib/bundler/stats/tree_spec.rb
+- spec/spec_helper.rb
 - spec/test_gemfile
 - spec/test_gemfile.lock
-homepage: ''
+homepage: http://github.com/jmmastey/bundler-stats
 licenses:
 - MIT
 metadata:
   homepage_uri: http://github.com/jmmastey/bundler-stats
   changelog_uri: https://github.com/jmmastey/bundler-stats/blob/master/CHANGELOG.md
   source_code_uri: http://github.com/jmmastey/bundler-stats
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -167,9 +182,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Dependency investigation for Bundler
 test_files: []

data/.travis.yml

@@ -1,6 +0,0 @@
-language: ruby
-script: bundle exec rspec
-rvm:
- - 2.3
- - 2.4
- - 2.5

data/bin/bundler-stats

@@ -1 +0,0 @@
-bin/bundle-stats