consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #3483 from consul/backport-fix_respond_with_bip

Browse Source 
Fix obsolete `respond_with_bip` usage
This commit is contained in:
Javier Martín
2019-05-10 15:08:41 +02:00
committed by GitHub
parent a1d6885132 55c8fa3297
commit 21ec1af094
2 changed files with 15 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/controllers/dashboard/polls_controller.rb
View File

@@ -35,10 +35,10 @@ class Dashboard::PollsController < Dashboard::BaseController
if poll.update(poll_params)
format.html { redirect_to proposal_dashboard_polls_path(proposal),
notice: t("flash.actions.update.poll") }
format.json { respond_with_bip(poll) }
format.json { head :no_content }
else
format.html { render :edit }
format.json { respond_with_bip(poll) }
format.json { render json: poll.errors.full_messages, status: :unprocessable_entity }
end
end
end

13
spec/features/dashboard/polls_spec.rb
View File

@@ -197,6 +197,19 @@ feature "Polls" do
end
end
scenario "Enable and disable results", :js do
create(:poll, related: proposal)
visit proposal_dashboard_polls_path(proposal)
check "Show results"
expect(find_field("Show results")).to be_checked
uncheck "Show results"
expect(find_field("Show results")).not_to be_checked
end
scenario "Poll card" do
poll = create(:poll, :expired, related: proposal)