diff options
author | The Bundler Bot <bot@bundler.io> | 2017-06-12 22:15:09 +0000 |
---|---|---|
committer | Samuel Giddins <segiddins@segiddins.me> | 2017-07-17 12:01:45 -0500 |
commit | 37e7f637d1bac901467bfd3c68202c1980f3fb77 (patch) | |
tree | b92cd6553548bc1f6a8ee5b6f2cc62fc8e781706 /spec | |
parent | fc943555bdbbd9926a01dbbc947522cb53cfdcfd (diff) | |
download | bundler-37e7f637d1bac901467bfd3c68202c1980f3fb77.tar.gz |
Auto merge of #5713 - bundler:seg-viz-other-gem, r=indirect
[Viz] Work correctly when another gem with a graphviz file is present
Closes #5706 by ensuring we always have the correct gem that contains a requirable `graphviz` file loaded (`ruby-graphviz` is the one we test against and suggest installing)
(cherry picked from commit f580c7e3e2fd0525871b25e69f4e7442f35b5660)
Diffstat (limited to 'spec')
-rw-r--r-- | spec/commands/viz_spec.rb | 47 | ||||
-rw-r--r-- | spec/support/helpers.rb | 2 |
2 files changed, 45 insertions, 4 deletions
diff --git a/spec/commands/viz_spec.rb b/spec/commands/viz_spec.rb index 123e7bef62..77112aace4 100644 --- a/spec/commands/viz_spec.rb +++ b/spec/commands/viz_spec.rb @@ -2,13 +2,13 @@ require "spec_helper" RSpec.describe "bundle viz", :ruby => "1.9.3", :if => Bundler.which("dot") do - let(:graphviz_lib) do - graphviz_glob = base_system_gems.join("gems/ruby-graphviz*/lib") - Dir[graphviz_glob].first + let(:ruby_graphviz) do + graphviz_glob = base_system_gems.join("cache/ruby-graphviz*") + Pathname.glob(graphviz_glob).first end before do - ENV["RUBYOPT"] = "-I #{graphviz_lib}" + system_gems ruby_graphviz end it "graphs gems from the Gemfile" do @@ -75,6 +75,45 @@ RSpec.describe "bundle viz", :ruby => "1.9.3", :if => Bundler.which("dot") do EOS end + context "with another gem that has a graphviz file" do + before do + build_repo4 do + build_gem "graphviz", "999" do |s| + s.write("lib/graphviz.rb", "abort 'wrong graphviz gem loaded'") + end + end + + system_gems ruby_graphviz, "graphviz-999", :gem_repo => gem_repo4 + end + + it "loads the correct ruby-graphviz gem" do + install_gemfile <<-G + source "file://#{gem_repo1}" + gem "rack" + gem "rack-obama" + G + + bundle! "viz", :format => "debug" + expect(out).to eq(strip_whitespace(<<-DOT).strip) + digraph Gemfile { + concentrate = "true"; + normalize = "true"; + nodesep = "0.55"; + edge[ weight = "2"]; + node[ fontname = "Arial, Helvetica, SansSerif"]; + edge[ fontname = "Arial, Helvetica, SansSerif" , fontsize = "12"]; + default [style = "filled", fillcolor = "#B9B9D5", shape = "box3d", fontsize = "16", label = "default"]; + rack [style = "filled", fillcolor = "#B9B9D5", label = "rack"]; + default -> rack [constraint = "false"]; + "rack-obama" [style = "filled", fillcolor = "#B9B9D5", label = "rack-obama"]; + default -> "rack-obama" [constraint = "false"]; + "rack-obama" -> rack; + } + debugging bundle viz... + DOT + end + end + context "--without option" do it "one group" do install_gemfile <<-G diff --git a/spec/support/helpers.rb b/spec/support/helpers.rb index 800ea82c03..a1b74d2726 100644 --- a/spec/support/helpers.rb +++ b/spec/support/helpers.rb @@ -250,6 +250,8 @@ module Spec path = if g == :bundler Dir.chdir(root) { gem_command! :build, "#{root}/bundler.gemspec" } bundler_path = root + "bundler-#{Bundler::VERSION}.gem" + elsif g.to_s =~ %r{\A/.*\.gem\z} + g else "#{gem_repo}/gems/#{g}.gem" end |