Skip to content

Commit

Permalink
Merge pull request #4163 from Glutexo/remove_unused_ansible_repositor…
Browse files Browse the repository at this point in the history
…y_page_refresh

Remove unused Ansible Repository page refresh
  • Loading branch information
mzazrivec committed Jun 20, 2018
2 parents 098fc18 + fafd853 commit d422f08
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 15 deletions.
1 change: 0 additions & 1 deletion app/controllers/ansible_repository_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ def button
page.replace("gtl_div", :partial => "layouts/gtl")
end
when "ansible_repository_reload"
params[:display] = @display if @display
show
render :update do |page|
page << javascript_prologue
Expand Down
14 changes: 0 additions & 14 deletions spec/controllers/ansible_repository_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -50,14 +50,6 @@
expect(response.status).to eq(200)
expect(response.body).to include("Test Repository (All Playbooks)")
end

it 'reload button will stay on the nested list' do
get :show, :params => {:id => @repository.id, :display => 'playbooks'}
expect(response.status).to eq(200)
post :button, :params => { :id => @repository.id, :pressed => 'ansible_repository_reload' }
expect(response.status).to eq(200)
expect(response).to render_template(:partial => "layouts/_gtl")
end
end

describe '#button' do
Expand Down Expand Up @@ -104,17 +96,11 @@

context 'refreshing an actual repository' do
let(:params) { {:pressed => "ansible_repository_reload"} }
let(:display) { "main" }

before do
controller.instance_variable_set(:@display, display)
end

it 'calls show and render methods' do
expect(controller).to receive(:show)
expect(controller).to receive(:render)
controller.send(:button)
expect(controller.instance_variable_get(:@_params)[:display]).to eq(display)
end
end

Expand Down

0 comments on commit d422f08

Please sign in to comment.