haskal
/
vagrant
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Box collection reloads after downloading during CheckBox. [closes GH-229]

This commit is contained in:
Mitchell Hashimoto 2010-11-30 20:19:24 -08:00
parent 0c5231e7b7
commit 331e0841e5
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -10,6 +10,7 @@
- Multiple NFS folders now work on linux. [GH-215] - Multiple NFS folders now work on linux. [GH-215]
- Add translation for state "stuck" which is very rare. [GH-218] - Add translation for state "stuck" which is very rare. [GH-218]
- virtualbox gem dependency minimum raised to 0.7.6 to verify FFI < 1.0.0 is used. - virtualbox gem dependency minimum raised to 0.7.6 to verify FFI < 1.0.0 is used.
- Fix issue where box downloading from `vagrant up` didn't reload the box collection. [GH-229]
## 0.6.7 (November 3, 2010) ## 0.6.7 (November 3, 2010)

1
lib/vagrant/action/vm/check_box.rb
View File

@ -16,6 +16,7 @@ module Vagrant
env.ui.info I18n.t("vagrant.actions.vm.check_box.not_found", :name => box_name) env.ui.info I18n.t("vagrant.actions.vm.check_box.not_found", :name => box_name)
Vagrant::Box.add(env.env, box_name, box_url) Vagrant::Box.add(env.env, box_name, box_url)
env["boxes"].reload!
end end
@app.call(env) @app.call(env)

4
test/vagrant/action/vm/check_box_test.rb
View File

@ -6,9 +6,6 @@ class CheckBoxVMActionTest < Test::Unit::TestCase
end end
context "calling" do context "calling" do
setup do
end
should "raise error if box not specified" do should "raise error if box not specified" do
app, env = action_env(vagrant_env(vagrantfile(<<-vf))) app, env = action_env(vagrant_env(vagrantfile(<<-vf)))
config.vm.box = nil config.vm.box = nil
@ -47,6 +44,7 @@ class CheckBoxVMActionTest < Test::Unit::TestCase
seq = sequence("seq") seq = sequence("seq")
env.env.boxes.expects(:find).returns(nil) env.env.boxes.expects(:find).returns(nil)
Vagrant::Box.expects(:add).with(env.env, env["config"].vm.box, env["config"].vm.box_url).in_sequence(seq) Vagrant::Box.expects(:add).with(env.env, env["config"].vm.box, env["config"].vm.box_url).in_sequence(seq)
env.env.boxes.expects(:reload!).in_sequence(seq)
app.expects(:call).with(env).once.in_sequence(seq) app.expects(:call).with(env).once.in_sequence(seq)
assert_nothing_raised { assert_nothing_raised {