Project

General

Profile

« Previous | Next » 

Revision 1a51088d

Added by Ohad Levy about 13 years ago

  • ID 1a51088d87c3420e435a3798358bec4b3b2b7de1

fixes #815 - Redirect to login page when editing multiple hosts

this patch converts the mulitple selections to be pure javascript,
removing the buggy AJAX implementation.

View differences:

test/functional/hosts_controller_test.rb
setup_multiple_environments
assert @host1.environment == environments(:production)
assert @host2.environment == environments(:production)
post :update_multiple_environment,
{:environment => { :id => environments(:global_puppetmaster).id}},
{:selected => [@host1.id, @host2.id], :user => User.first.id}
post :update_multiple_environment, { :host_ids => [@host1.id, @host2.id],
:environment => { :id => environments(:global_puppetmaster).id}},
{ :user => User.first.id}
assert Host.find(@host1.id).environment == environments(:global_puppetmaster)
assert Host.find(@host2.id).environment == environments(:global_puppetmaster)
end
......
@host1.host_parameters = [HostParameter.create(:name => "p1", :value => "yo")]
@host2.host_parameters = [HostParameter.create(:name => "p1", :value => "hi")]
post :update_multiple_parameters,
{:name => { "p1" => "hello"}},
{:selected => [@host1.id, @host2.id], :user => User.first.id}
{:name => { "p1" => "hello"},:host_ids => [@host1.id, @host2.id]},
{:user => User.first.id}
assert Host.find(@host1.id).host_parameters[0][:value] == "hello"
assert Host.find(@host2.id).host_parameters[0][:value] == "hello"
end
......
def test_submit_multiple_build
assert !hosts(:one).build
assert !hosts(:two).build
post :submit_multiple_build, {}, set_session_user.merge(:selected => [hosts(:one).id, hosts(:two).id])
post :submit_multiple_build, {:host_ids => [hosts(:one).id, hosts(:two).id]}, set_session_user
assert_redirected_to hosts_path
assert flash[:notice] == "The selected hosts will execute a build operation on next reboot"
assert Host.find(hosts(:one)).build

Also available in: Unified diff