bundler-stats 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7cd8533d8297a0407a8129eaf08f5a9e7f603e388851b9baff7a85d0e3e6141b
-  data.tar.gz: ec911653587fbc4ed603a6f7ceb034a6c8bf659b149bd0432d8cf95a3cf5a2d6
+  metadata.gz: 3c6946fc4c79ee76c0201d258b8cc9e79357e9b404361011ec920d647bc40b1b
+  data.tar.gz: 2f70cf9458ca731f1582c0c95849a3279ed93aa8ae8af335e22b767479bdca59
 SHA512:
-  metadata.gz: 0edb13c6c5a1bf249498ea5c06d6edf56b5134835d0ee4b453b71cc97dc2bc16dace85e7313d22c4a15d2634ee885679586ac307df7a29acfa5951642bf8668e
-  data.tar.gz: 98008d13c6fd82f3e64431e7fdd88ccb7a45e34fe2a9e573345d0ea3aaa54a11da1d9d93c87333a43a360c6e0e66953b6c07386510d02da20b268a6b83bcd6a3
+  metadata.gz: 1743d02925583fd087a31a36788a2e2dd26c479944f9cd7062a53a24b93c704e3ee82d6335e50b0a0848ccacdcc78e78914406602d497a064a8bc070825cd2f0
+  data.tar.gz: 5a5b1d33d6f26dd6db125619a0b1715bb6d265f4477589e088b79f0e444d2d6c37632b23c3b4aad77b251cf7b5954dbd6692c9a59fd509e0127c0503b943d109

data/lib/bundler/stats/tree.rb

@@ -1,6 +1,10 @@
 class Bundler::Stats::Tree
   attr_accessor :tree
 
+  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: [])
     raise ArgumentError unless parser.respond_to?(:specs)
 
@@ -30,12 +34,19 @@ class Bundler::Stats::Tree
   end
 
   def first_level_dependencies(target)
-    raise ArgumentError, "Unknown gem #{target}" unless @tree.has_key? target
+    unless @tree.has_key? target
+      warn(ERR_MESSAGE % [target])
+      return []
+    end
+
     @tree[target].dependencies
   end
 
   def transitive_dependencies(target)
-    raise ArgumentError, "Unknown gem #{target}" unless @tree.has_key? target
+    unless @tree.has_key? target
+      warn(ERR_MESSAGE % [target])
+      return []
+    end
 
     top_level = @tree[target].dependencies
     all_level = top_level + top_level.inject([]) do |arr, dep|
@@ -78,6 +89,10 @@ class Bundler::Stats::Tree
 
   private
 
+  def warn(str)
+    STDERR.puts(str)
+  end
+
   def specs_as_tree(specs)
     specs.each_with_object({}) do |spec, hash|
       hash[spec.name] = spec

data/lib/bundler/stats/version.rb

@@ -1,6 +1,6 @@
 module Bundler
   module Stats
     # bundler-stats version
-    VERSION = '1.2.0'
+    VERSION = '1.2.1'
   end
 end

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

@@ -18,10 +18,13 @@ describe Bundler::Stats::Tree do
   end
 
   context "#first_level_dependencies" do
-    it "errors for non-existent dependencies" do
+    it "warns you of missing dependencies from other platforms" do
       tree = subject.new(parser)
+      allow(tree).to receive(:warn)
 
-      expect { tree.first_level_dependencies("none") }.to raise_error(ArgumentError)
+      tree.first_level_dependencies("none")
+
+      expect(tree).to have_received(:warn)
     end
 
     it "returns empty array for bottom-level dependencies" do
@@ -50,10 +53,13 @@ describe Bundler::Stats::Tree do
   end
 
   context "#transitive_dependencies" do
-    it "errors for non-existent dependencies" do
+    it "warns you of missing dependencies from other platforms" do
       tree = subject.new(parser)
+      allow(tree).to receive(:warn)
+
+      tree.transitive_dependencies("none")
 
-      expect { tree.transitive_dependencies("none") }.to raise_error(ArgumentError)
+      expect(tree).to have_received(:warn)
     end
 
     it "returns empty array for bottom-level dependencies" do

metadata

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