Use yes_no trait to simplify creating answers

We lose some variety in our answers, though, but on the plus side, we
reduce the risk of trademark issues :P.
This commit is contained in:
Javi Martín
2019-09-24 21:14:56 +02:00
parent 19a41eb651
commit 4edab79910
4 changed files with 58 additions and 95 deletions

View File

@@ -239,9 +239,7 @@ describe "Admin booths assignments" do
booth_assignment = create(:poll_booth_assignment, poll: poll)
other_booth_assignment = create(:poll_booth_assignment, poll: poll)
question_1 = create(:poll_question, poll: poll)
create(:poll_question_answer, title: "Yes", question: question_1)
create(:poll_question_answer, title: "No", question: question_1)
question_1 = create(:poll_question, :yes_no, poll: poll)
question_2 = create(:poll_question, poll: poll)
create(:poll_question_answer, title: "Today", question: question_2)

View File

@@ -133,9 +133,7 @@ describe "Admin polls" do
scenario "Can destroy poll with questions and answers", :js do
poll = create(:poll)
question = create(:poll_question, poll: poll)
create(:poll_question_answer, question: question, title: "Yes")
create(:poll_question_answer, question: question, title: "No")
question = create(:poll_question, :yes_no, poll: poll)
visit admin_polls_path
@@ -438,9 +436,7 @@ describe "Admin polls" do
booth_assignment_2 = create(:poll_booth_assignment, poll: poll)
booth_assignment_3 = create(:poll_booth_assignment, poll: poll)
question_1 = create(:poll_question, poll: poll)
create(:poll_question_answer, title: "Yes", question: question_1)
create(:poll_question_answer, title: "No", question: question_1)
question_1 = create(:poll_question, :yes_no, poll: poll)
question_2 = create(:poll_question, poll: poll)
create(:poll_question_answer, title: "Today", question: question_2)
@@ -494,9 +490,7 @@ describe "Admin polls" do
booth_assignment1 = create(:poll_booth_assignment, poll: poll)
booth_assignment2 = create(:poll_booth_assignment, poll: poll)
question = create(:poll_question, poll: poll)
create(:poll_question_answer, title: "Yes", question: question)
create(:poll_question_answer, title: "No", question: question)
question = create(:poll_question, :yes_no, poll: poll)
create(:poll_partial_result,
booth_assignment: booth_assignment1,

View File

@@ -129,9 +129,7 @@ describe "Polls" do
scenario "Edit poll should allow to remove answers", :js do
poll = create(:poll, related: proposal, starts_at: 1.week.from_now)
question = create(:poll_question, poll: poll)
create(:poll_question_answer, question: question)
create(:poll_question_answer, question: question)
create(:poll_question, :yes_no, poll: poll)
visit proposal_dashboard_polls_path(proposal)
within "div#poll_#{poll.id}" do
click_link "Edit survey"

View File

@@ -98,9 +98,7 @@ describe "Polls" do
expect(page).to have_content("This poll is not available on your geozone")
poll_with_question = create(:poll)
question = create(:poll_question, poll: poll_with_question)
answer1 = create(:poll_question_answer, question: question, title: "Yes")
answer2 = create(:poll_question_answer, question: question, title: "No")
question = create(:poll_question, :yes_no, poll: poll_with_question)
vote_for_poll_via_web(poll_with_question, question, "Yes")
visit polls_path
@@ -188,15 +186,13 @@ describe "Polls" do
end
scenario "Non-logged in users" do
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
create(:poll_question, :yes_no, poll: poll)
visit poll_path(poll)
expect(page).to have_content("You must Sign in or Sign up to participate")
expect(page).to have_link("Han Solo", href: new_user_session_path)
expect(page).to have_link("Chewbacca", href: new_user_session_path)
expect(page).to have_link("Yes", href: new_user_session_path)
expect(page).to have_link("No", href: new_user_session_path)
end
scenario "Level 1 users" do
@@ -206,36 +202,32 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << geozone
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
create(:poll_question, :yes_no, poll: poll)
login_as(create(:user, geozone: geozone))
visit poll_path(poll)
expect(page).to have_content("You must verify your account in order to answer")
expect(page).to have_link("Han Solo", href: verification_path)
expect(page).to have_link("Chewbacca", href: verification_path)
expect(page).to have_link("Yes", href: verification_path)
expect(page).to have_link("No", href: verification_path)
end
scenario "Level 2 users in an expired poll" do
expired_poll = create(:poll, :expired, geozone_restricted: true)
expired_poll.geozones << geozone
question = create(:poll_question, poll: expired_poll)
answer1 = create(:poll_question_answer, question: question, title: "Luke")
answer2 = create(:poll_question_answer, question: question, title: "Leia")
question = create(:poll_question, :yes_no, poll: expired_poll)
login_as(create(:user, :level_two, geozone: geozone))
visit poll_path(expired_poll)
within("#poll_question_#{question.id}_answers") do
expect(page).to have_content("Luke")
expect(page).to have_content("Leia")
expect(page).not_to have_link("Luke")
expect(page).not_to have_link("Leia")
expect(page).to have_content("Yes")
expect(page).to have_content("No")
expect(page).not_to have_link("Yes")
expect(page).not_to have_link("No")
end
expect(page).to have_content("This poll has finished")
end
@@ -244,19 +236,17 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << create(:geozone)
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Vader")
answer2 = create(:poll_question_answer, question: question, title: "Palpatine")
question = create(:poll_question, :yes_no, poll: poll)
login_as(create(:user, :level_two))
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
expect(page).to have_content("Vader")
expect(page).to have_content("Palpatine")
expect(page).not_to have_link("Vader")
expect(page).not_to have_link("Palpatine")
expect(page).to have_content("Yes")
expect(page).to have_content("No")
expect(page).not_to have_link("Yes")
expect(page).not_to have_link("No")
end
end
@@ -264,46 +254,40 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << geozone
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
login_as(create(:user, :level_two, geozone: geozone))
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
expect(page).to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).to have_link("Yes")
expect(page).to have_link("No")
end
end
scenario "Level 2 users reading a all-geozones poll" do
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
login_as(create(:user, :level_two))
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
expect(page).to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).to have_link("Yes")
expect(page).to have_link("No")
end
end
scenario "Level 2 users who have already answered" do
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
user = create(:user, :level_two)
create(:poll_answer, question: question, author: user, answer: "Chewbacca")
create(:poll_answer, question: question, author: user, answer: "No")
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
expect(page).to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).to have_link("Yes")
expect(page).to have_link("No")
end
end
@@ -311,20 +295,17 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << geozone
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
user = create(:user, :level_two, geozone: geozone)
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
click_link "Han Solo"
click_link "Yes"
expect(page).not_to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).not_to have_link("Yes")
expect(page).to have_link("No")
end
end
@@ -332,25 +313,22 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << geozone
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
user = create(:user, :level_two, geozone: geozone)
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
click_link "Han Solo"
click_link "Yes"
expect(page).not_to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).not_to have_link("Yes")
expect(page).to have_link("No")
click_link "Chewbacca"
click_link "No"
expect(page).not_to have_link("Chewbacca")
expect(page).to have_link("Han Solo")
expect(page).not_to have_link("No")
expect(page).to have_link("Yes")
end
end
@@ -358,40 +336,37 @@ describe "Polls" do
poll.update(geozone_restricted: true)
poll.geozones << geozone
question = create(:poll_question, poll: poll)
answer1 = create(:poll_question_answer, question: question, title: "Han Solo")
answer2 = create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
user = create(:user, :level_two, geozone: geozone)
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
click_link "Han Solo"
click_link "Yes"
expect(page).not_to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).not_to have_link("Yes")
expect(page).to have_link("No")
end
click_link "Sign out"
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
click_link "Han Solo"
click_link "Yes"
expect(page).not_to have_link("Han Solo")
expect(page).to have_link("Chewbacca")
expect(page).not_to have_link("Yes")
expect(page).to have_link("No")
end
click_link "Sign out"
login_as user
visit poll_path(poll)
within("#poll_question_#{question.id}_answers") do
click_link "Chewbacca"
click_link "No"
expect(page).not_to have_link("Chewbacca")
expect(page).to have_link("Han Solo")
expect(page).not_to have_link("No")
expect(page).to have_link("Yes")
end
end
end
@@ -407,9 +382,7 @@ describe "Polls" do
create(:poll_shift, officer: officer, booth: booth, date: Date.current, task: :vote_collection)
booth_assignment = create(:poll_booth_assignment, poll: poll, booth: booth)
create(:poll_officer_assignment, officer: officer, booth_assignment: booth_assignment, date: Date.current)
question = create(:poll_question, poll: poll)
create(:poll_question_answer, question: question, title: "Han Solo")
create(:poll_question_answer, question: question, title: "Chewbacca")
question = create(:poll_question, :yes_no, poll: poll)
user = create(:user, :level_two, :in_census)
login_as(officer.user)
@@ -427,11 +400,11 @@ describe "Polls" do
expect(page).to have_content "You have already participated in a physical booth. You can not participate again."
within("#poll_question_#{question.id}_answers") do
expect(page).to have_content("Han Solo")
expect(page).to have_content("Chewbacca")
expect(page).to have_content("Yes")
expect(page).to have_content("No")
expect(page).not_to have_link("Han Solo")
expect(page).not_to have_link("Chewbacca")
expect(page).not_to have_link("Yes")
expect(page).not_to have_link("No")
end
end