diff --git a/lib/vagrant/action/builtin/box_add.rb b/lib/vagrant/action/builtin/box_add.rb index 7dd09e0da..deed8b9ef 100644 --- a/lib/vagrant/action/builtin/box_add.rb +++ b/lib/vagrant/action/builtin/box_add.rb @@ -57,7 +57,7 @@ module Vagrant raise Errors::BoxServerNotSet if !server expanded = true - url[i] = "#{server}/api/v1/vagrant/#{url[i]}" + url[i] = "#{server}/#{url[i]}" end end diff --git a/test/unit/vagrant/action/builtin/box_add_test.rb b/test/unit/vagrant/action/builtin/box_add_test.rb index 82b476f5d..ad3a56dfc 100644 --- a/test/unit/vagrant/action/builtin/box_add_test.rb +++ b/test/unit/vagrant/action/builtin/box_add_test.rb @@ -48,7 +48,7 @@ describe Vagrant::Action::Builtin::BoxAdd do AccessLog: [], Logger: WEBrick::Log.new(tf.path, 7), Port: port, - DocumentRoot: path.to_s, + DocumentRoot: path.dirname.to_s, MimeTypes: mime_types) thr = Thread.new { server.start } yield port @@ -308,7 +308,7 @@ describe Vagrant::Action::Builtin::BoxAdd do it "adds from shorthand path" do box_path = iso_env.box2_file(:virtualbox) td = Pathname.new(Dir.mktmpdir) - tf = td.join("api", "v1", "vagrant", "mitchellh", "precise64.json") + tf = td.join("mitchellh", "precise64.json") tf.dirname.mkpath tf.open("w") do |f| f.write(<<-RAW) @@ -332,7 +332,7 @@ describe Vagrant::Action::Builtin::BoxAdd do RAW end - with_web_server(td) do |port| + with_web_server(tf.dirname) do |port| url = "http://127.0.0.1:#{port}" env[:box_url] = "mitchellh/precise64.json" @@ -341,7 +341,7 @@ describe Vagrant::Action::Builtin::BoxAdd do expect(version).to eq("0.7") expect(checksum(path)).to eq(checksum(box_path)) expect(opts[:metadata_url]).to eq( - "#{url}/api/v1/vagrant/#{env[:box_url]}") + "#{url}/#{env[:box_url]}") true }.and_return(box) @@ -356,7 +356,7 @@ describe Vagrant::Action::Builtin::BoxAdd do it "add from shorthand path with configured server url" do box_path = iso_env.box2_file(:virtualbox) td = Pathname.new(Dir.mktmpdir) - tf = td.join("api", "v1", "vagrant", "mitchellh", "precise64.json") + tf = td.join("mitchellh", "precise64.json") tf.dirname.mkpath tf.open("w") do |f| f.write(<<-RAW) @@ -380,7 +380,7 @@ describe Vagrant::Action::Builtin::BoxAdd do RAW end - with_web_server(td) do |port| + with_web_server(tf.dirname) do |port| url = "http://127.0.0.1:#{port}" env[:box_url] = "mitchellh/precise64.json" env[:box_server_url] = url @@ -390,7 +390,7 @@ describe Vagrant::Action::Builtin::BoxAdd do expect(version).to eq("0.7") expect(checksum(path)).to eq(checksum(box_path)) expect(opts[:metadata_url]).to eq( - "#{url}/api/v1/vagrant/#{env[:box_url]}") + "#{url}/#{env[:box_url]}") true }.and_return(box) @@ -558,11 +558,10 @@ describe Vagrant::Action::Builtin::BoxAdd do end it "raises an error if shorthand is invalid" do - td = Pathname.new(Dir.mktmpdir) - tf = td.join("api", "v1", "vagrant", "foo") - tf.dirname.mkpath + tf = Tempfile.new("foo") + tf.close - with_web_server(td) do |port| + with_web_server(Pathname.new(tf.path)) do |port| env[:box_url] = "mitchellh/precise64.json" expect(box_collection).to receive(:add).never