bundler-stats 1.3.2 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66d53289a79dd7b950987a512f6876f60f415862ab169c2b3fe935a4199b25c3
-  data.tar.gz: 130b1e1a7d84163bea7fa95802639e1078bb53c1176f1d7482c0168ff752e37e
+  metadata.gz: 00521574e50038905fce9c0f58c700220922c38ff91ce987448e2372121333ff
+  data.tar.gz: 76632e66f4e00c8cc0a00e4ae755fddbd29361526067140c3610b5023042008d
 SHA512:
-  metadata.gz: 882690eb56bbbd2bd001703277cd499ddd2e13ce3376470ae5ff1f657a23425a86112acd6bb2d3fa54e9ad92d1c6ccb1adfcca1742ec0a7b821b519cf77c305f
-  data.tar.gz: 7f3dcd1216f8747c1f36bc4627017eebf94f7eb1f668eb8e923e56ed3975958861bf48ceb605004e7f368216de638e2a2065f100c8a52c422417ac6ef0c68063
+  metadata.gz: 88b9b171219491c7590c41758137200a2c07e85456f4d21aaa00ea786b8c6574622ca872e133cf39bfc05603cb2372bd9a6dda67841b114f0c080abb4b471898
+  data.tar.gz: 8fb71219134a7722f93811d61d29aef329a39cfd67a9d52a5a282de1dc708a3eada49321347e087fa8b4df73db16a2396cdf15acc9b34511ad69dde966a2b0f6

data/lib/bundler/stats/remover.rb

@@ -1,11 +1,12 @@
 class Bundler::Stats::Remover
-  ERR_MESSAGE = "Trying to check whether `%s` can be removed, but was unable " \
+  ERR_MESSAGE = "Trying to check whether dep can be removed, but was unable " \
     "to resolve whether it is used by `%s`. It may not be in your Gemfile.lock. " \
     "This often happens when a dependency isn't installed on your platform."
 
   def initialize(tree, top_level)
-    @tree       = tree
-    @top_level  = top_level
+    @tree           = tree
+    @top_level      = top_level
+    @trace_warnings = []
   end
 
   def potential_removals(target)
@@ -33,7 +34,7 @@ class Bundler::Stats::Remover
       return true if candidate == target
 
       if modified_tree[candidate].nil?
-        warn(ERR_MESSAGE % [target, candidate])
+        warn_of_bad_tracing(candidate)
       else
         deps_to_check += modified_tree[candidate].dependencies
       end
@@ -44,7 +45,10 @@ class Bundler::Stats::Remover
 
   private
 
-  def warn(str)
-    STDERR.puts(str)
+  def warn_of_bad_tracing(candidate)
+    return if @trace_warnings.include? candidate
+
+    STDERR.puts(ERR_MESSAGE % [candidate])
+    @trace_warnings << candidate
   end
 end

data/lib/bundler/stats/version.rb

@@ -1,5 +1,5 @@
 module Bundler
   module Stats
-    VERSION = '1.3.2'
+    VERSION = '1.3.3'
   end
 end

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

@@ -106,11 +106,11 @@ describe Bundler::Stats::Remover do
     it "raises an error if the top level dependency isn't in the lockfile" do
       top_level << dep("tzinfo")
       remover = subject.new(tree, top_level)
-      allow(remover).to receive(:warn)
+      allow(remover).to receive(:warn_of_bad_tracing)
 
       remover.still_used?("actionview", deleted: "rails")
 
-      expect(remover).to have_received(:warn)
+      expect(remover).to have_received(:warn_of_bad_tracing)
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bundler-stats
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.3
 platform: ruby
 authors:
 - Joseph Mastey