consul/consul

View on GitHub
spec/system/comments/legislation_questions_spec.rb

Summary

Maintainability
F
3 days
Test Coverage
require "rails_helper"

describe "Commenting legislation questions" do
  let(:user) { create(:user, :level_two) }
  let(:process) { create(:legislation_process, :in_debate_phase) }
  let(:question) { create(:legislation_question, process: process) }

  context "Concerns" do
    it_behaves_like "notifiable in-app", :legislation_question
    it_behaves_like "flaggable", :legislation_question_comment
  end

  scenario "Index" do
    3.times { create(:comment, commentable: question) }
    comment = Comment.includes(:user).last

    visit legislation_process_question_path(question.process, question)

    expect(page).to have_css(".comment", count: 3)

    within first(".comment") do
      expect(page).to have_content comment.user.name
      expect(page).to have_content I18n.l(comment.created_at, format: :datetime)
      expect(page).to have_content comment.body
    end
  end

  scenario "Show" do
    href           = legislation_process_question_path(question.process, question)
    parent_comment = create(:comment, commentable: question, body: "Parent")
    create(:comment, commentable: question, parent: parent_comment, body: "First subcomment")
    create(:comment, commentable: question, parent: parent_comment, body: "Last subcomment")

    visit comment_path(parent_comment)

    expect(page).to have_css(".comment", count: 3)
    expect(page).to have_content "Parent"
    expect(page).to have_content "First subcomment"
    expect(page).to have_content "Last subcomment"

    expect(page).to have_link "Go back to #{question.title}", href: href

    within ".comment", text: "Parent" do
      expect(page).to have_css ".comment", count: 2
    end
  end

  scenario "Link to comment show" do
    comment = create(:comment, commentable: question, user: user)

    visit legislation_process_question_path(question.process, question)

    within "#comment_#{comment.id}" do
      expect(page).to have_link comment.created_at.strftime("%Y-%m-%d %T")
    end

    click_link comment.created_at.strftime("%Y-%m-%d %T")

    expect(page).to have_link "Go back to #{question.title}"
    expect(page).to have_current_path(comment_path(comment))
  end

  scenario "Collapsable comments" do
    parent_comment = create(:comment, body: "Main comment", commentable: question)
    child_comment  = create(:comment, body: "First subcomment", commentable: question, parent: parent_comment)
    grandchild_comment = create(:comment,
                                body: "Last subcomment",
                                commentable: question,
                                parent: child_comment)

    visit legislation_process_question_path(question.process, question)

    expect(page).to have_css(".comment", count: 3)
    expect(page).to have_content("1 response (collapse)", count: 2)

    within ".comment .comment", text: "First subcomment" do
      click_link text: "1 response (collapse)"
    end

    expect(page).to have_css(".comment", count: 2)
    expect(page).to have_content("1 response (collapse)")
    expect(page).to have_content("1 response (show)")
    expect(page).not_to have_content grandchild_comment.body

    within ".comment .comment", text: "First subcomment" do
      click_link text: "1 response (show)"
    end

    expect(page).to have_css(".comment", count: 3)
    expect(page).to have_content("1 response (collapse)", count: 2)
    expect(page).to have_content grandchild_comment.body

    within ".comment", text: "Main comment" do
      click_link text: "1 response (collapse)", match: :first
    end

    expect(page).to have_css(".comment", count: 1)
    expect(page).to have_content("1 response (show)")
    expect(page).not_to have_content child_comment.body
    expect(page).not_to have_content grandchild_comment.body
  end

  scenario "Comment order" do
    c1 = create(:comment, :with_confidence_score, commentable: question, cached_votes_up: 100,
                                                  cached_votes_total: 120, created_at: Time.current - 2)
    c2 = create(:comment, :with_confidence_score, commentable: question, cached_votes_up: 10,
                                                  cached_votes_total: 12, created_at: Time.current - 1)
    c3 = create(:comment, :with_confidence_score, commentable: question, cached_votes_up: 1,
                                                  cached_votes_total: 2, created_at: Time.current)

    visit legislation_process_question_path(question.process, question, order: :most_voted)

    expect(c1.body).to appear_before(c2.body)
    expect(c2.body).to appear_before(c3.body)

    click_link "Newest first"

    expect(page).to have_link "Newest first", class: "is-active"
    expect(page).to have_current_path(/#comments/, url: true)
    expect(c3.body).to appear_before(c2.body)
    expect(c2.body).to appear_before(c1.body)

    click_link "Oldest first"

    expect(page).to have_link "Oldest first", class: "is-active"
    expect(page).to have_current_path(/#comments/, url: true)
    expect(c1.body).to appear_before(c2.body)
    expect(c2.body).to appear_before(c3.body)
  end

  scenario "Creation date works differently in roots and child comments when sorting by confidence_score" do
    old_root = create(:comment, commentable: question, created_at: Time.current - 10)
    new_root = create(:comment, commentable: question, created_at: Time.current)
    old_child = create(:comment, commentable: question, parent_id: new_root.id, created_at: Time.current - 10)
    new_child = create(:comment, commentable: question, parent_id: new_root.id, created_at: Time.current)

    visit legislation_process_question_path(question.process, question, order: :most_voted)

    expect(new_root.body).to appear_before(old_root.body)
    expect(old_child.body).to appear_before(new_child.body)

    visit legislation_process_question_path(question.process, question, order: :newest)

    expect(new_root.body).to appear_before(old_root.body)
    expect(new_child.body).to appear_before(old_child.body)

    visit legislation_process_question_path(question.process, question, order: :oldest)

    expect(old_root.body).to appear_before(new_root.body)
    expect(old_child.body).to appear_before(new_child.body)
  end

  scenario "Turns links into html links" do
    create(:comment, commentable: question, body: "Built with http://rubyonrails.org/")

    visit legislation_process_question_path(question.process, question)

    within first(".comment") do
      expect(page).to have_content "Built with http://rubyonrails.org/"
      expect(page).to have_link("http://rubyonrails.org/", href: "http://rubyonrails.org/")
      expect(find_link("http://rubyonrails.org/")[:rel]).to eq("nofollow")
      expect(find_link("http://rubyonrails.org/")[:target]).to be_blank
    end
  end

  scenario "Sanitizes comment body for security" do
    create(:comment, commentable: question,
                     body: "<script>alert('hola')</script> " \
                           "<a href=\"javascript:alert('sorpresa!')\">click me<a/> " \
                           "http://www.url.com")

    visit legislation_process_question_path(question.process, question)

    within first(".comment") do
      expect(page).to have_content "click me http://www.url.com"
      expect(page).to have_link("http://www.url.com", href: "http://www.url.com")
      expect(page).not_to have_link("click me")
    end
  end

  scenario "Paginated comments" do
    per_page = 10
    (per_page + 2).times { create(:comment, commentable: question) }

    visit legislation_process_question_path(question.process, question)

    expect(page).to have_css(".comment", count: per_page)
    within("ul.pagination") do
      expect(page).to have_content("1")
      expect(page).to have_content("2")
      expect(page).not_to have_content("3")
      click_link "Next", exact: false
    end

    expect(page).to have_css(".comment", count: 2)
    expect(page).to have_current_path(/#comments/, url: true)
  end

  describe "Not logged user" do
    scenario "can not see comments forms" do
      create(:comment, commentable: question)
      visit legislation_process_question_path(question.process, question)

      expect(page).to have_content "You must sign in or sign up to leave a comment"
      within("#comments") do
        expect(page).not_to have_content "Write a comment"
        expect(page).not_to have_content "Reply"
      end
    end
  end

  scenario "Create" do
    login_as(user)
    visit legislation_process_question_path(question.process, question)

    fill_in "Leave your answer", with: "Have you thought about...?"
    click_button "Publish answer"

    within "#comments" do
      expect(page).to have_content "Have you thought about...?"
      expect(page).to have_content "(1)"
    end
  end

  scenario "Errors on create" do
    login_as(user)
    visit legislation_process_question_path(question.process, question)

    click_button "Publish answer"

    expect(page).to have_content "Can't be blank"
  end

  scenario "Unverified user can't create comments" do
    unverified_user = create(:user)
    login_as(unverified_user)

    visit legislation_process_question_path(question.process, question)

    expect(page).to have_content "To participate verify your account"
  end

  scenario "Can't create comments if debate phase is not open" do
    process.update!(debate_start_date: Date.current - 2.days, debate_end_date: Date.current - 1.day)
    login_as(user)

    visit legislation_process_question_path(question.process, question)

    expect(page).to have_content "Closed phase"
  end

  scenario "Reply" do
    citizen = create(:user, username: "Ana")
    manuela = create(:user, :level_two, username: "Manuela")
    comment = create(:comment, commentable: question, user: citizen)

    login_as(manuela)
    visit legislation_process_question_path(question.process, question)

    click_link "Reply"

    within "#js-comment-form-comment_#{comment.id}" do
      fill_in "Leave your answer", with: "It will be done next week."
      click_button "Publish reply"
    end

    within "#comment_#{comment.id}" do
      expect(page).to have_content "It will be done next week."
    end

    expect(page).not_to have_css "#js-comment-form-comment_#{comment.id}"
  end

  scenario "Reply update parent comment responses count" do
    manuela = create(:user, :level_two, username: "Manuela")
    comment = create(:comment, commentable: question)

    login_as(manuela)
    visit legislation_process_question_path(question.process, question)

    within ".comment", text: comment.body do
      click_link "Reply"
      fill_in "Leave your answer", with: "It will be done next week."
      click_button "Publish reply"

      expect(page).to have_content("1 response (collapse)")
    end
  end

  scenario "Reply show parent comments responses when hidden" do
    manuela = create(:user, :level_two, username: "Manuela")
    comment = create(:comment, commentable: question)
    create(:comment, commentable: question, parent: comment)

    login_as(manuela)
    visit legislation_process_question_path(question.process, question)

    within ".comment", text: comment.body do
      click_link text: "1 response (collapse)"
      click_link "Reply"
      fill_in "Leave your answer", with: "It will be done next week."
      click_button "Publish reply"

      expect(page).to have_content("It will be done next week.")
    end
  end

  scenario "Errors on reply" do
    comment = create(:comment, commentable: question, user: user)

    login_as(user)
    visit legislation_process_question_path(question.process, question)

    click_link "Reply"

    within "#js-comment-form-comment_#{comment.id}" do
      click_button "Publish reply"
      expect(page).to have_content "Can't be blank"
    end
  end

  scenario "N replies" do
    parent = create(:comment, commentable: question)

    7.times do
      create(:comment, commentable: question, parent: parent)
      parent = parent.children.first
    end

    visit legislation_process_question_path(question.process, question)
    expect(page).to have_css(".comment.comment.comment.comment.comment.comment.comment.comment")
  end

  scenario "Erasing a comment's author" do
    comment = create(:comment, commentable: question, body: "this should be visible")
    comment.user.erase

    visit legislation_process_question_path(question.process, question)
    within "#comment_#{comment.id}" do
      expect(page).to have_content("User deleted")
      expect(page).to have_content("this should be visible")
    end
  end

  scenario "Submit button is disabled after clicking" do
    login_as(user)
    visit legislation_process_question_path(question.process, question)

    fill_in "Leave your answer", with: "Testing submit button!"
    click_button "Publish answer"

    expect(page).to have_button "Publish answer", disabled: true
    expect(page).to have_content "Testing submit button!"
    expect(page).to have_button "Publish answer", disabled: false
  end

  describe "Moderators" do
    scenario "can create comment as a moderator" do
      moderator = create(:moderator)

      login_as(moderator.user)
      visit legislation_process_question_path(question.process, question)

      fill_in "Leave your answer", with: "I am moderating!"
      check "comment-as-moderator-legislation_question_#{question.id}"
      click_button "Publish answer"

      within "#comments" do
        expect(page).to have_content "I am moderating!"
        expect(page).to have_content "Moderator ##{moderator.id}"
        expect(page).to have_css "div.is-moderator"
        expect(page).to have_css "img.moderator-avatar"
      end
    end

    scenario "can create reply as a moderator" do
      citizen = create(:user, username: "Ana")
      manuela = create(:user, username: "Manuela")
      moderator = create(:moderator, user: manuela)
      comment = create(:comment, commentable: question, user: citizen)

      login_as(manuela)
      visit legislation_process_question_path(question.process, question)

      click_link "Reply"

      within "#js-comment-form-comment_#{comment.id}" do
        fill_in "Leave your answer", with: "I am moderating!"
        check "comment-as-moderator-comment_#{comment.id}"
        click_button "Publish reply"
      end

      within "#comment_#{comment.id}" do
        expect(page).to have_content "I am moderating!"
        expect(page).to have_content "Moderator ##{moderator.id}"
        expect(page).to have_css "div.is-moderator"
        expect(page).to have_css "img.moderator-avatar"
      end

      expect(page).not_to have_css "#js-comment-form-comment_#{comment.id}"
    end

    scenario "can not comment as an administrator" do
      moderator = create(:moderator)

      login_as(moderator.user)
      visit legislation_process_question_path(question.process, question)

      expect(page).not_to have_content "Comment as administrator"
    end
  end

  describe "Administrators" do
    scenario "can create comment as an administrator" do
      admin = create(:administrator)

      login_as(admin.user)
      visit legislation_process_question_path(question.process, question)

      fill_in "Leave your answer", with: "I am your Admin!"
      check "comment-as-administrator-legislation_question_#{question.id}"
      click_button "Publish answer"

      within "#comments" do
        expect(page).to have_content "I am your Admin!"
        expect(page).to have_content "Administrator ##{admin.id}"
        expect(page).to have_css "div.is-admin"
        expect(page).to have_css "img.admin-avatar"
      end
    end

    scenario "can create reply as an administrator" do
      citizen = create(:user, username: "Ana")
      manuela = create(:user, username: "Manuela")
      admin   = create(:administrator, user: manuela)
      comment = create(:comment, commentable: question, user: citizen)

      login_as(manuela)
      visit legislation_process_question_path(question.process, question)

      click_link "Reply"

      within "#js-comment-form-comment_#{comment.id}" do
        fill_in "Leave your answer", with: "Top of the world!"
        check "comment-as-administrator-comment_#{comment.id}"
        click_button "Publish reply"
      end

      within "#comment_#{comment.id}" do
        expect(page).to have_content "Top of the world!"
        expect(page).to have_content "Administrator ##{admin.id}"
        expect(page).to have_css "div.is-admin"
        expect(page).to have_css "img.admin-avatar"
      end

      expect(page).not_to have_css "#js-comment-form-comment_#{comment.id}"
    end

    scenario "can not comment as a moderator", :admin do
      visit legislation_process_question_path(question.process, question)

      expect(page).not_to have_content "Comment as moderator"
    end
  end

  describe "Voting comments" do
    let(:verified)   { create(:user, verified_at: Time.current) }
    let(:unverified) { create(:user) }
    let(:question)   { create(:legislation_question) }
    let!(:comment)   { create(:comment, commentable: question) }

    before do
      login_as(verified)
    end

    scenario "Show" do
      create(:vote, voter: verified, votable: comment, vote_flag: true)
      create(:vote, voter: unverified, votable: comment, vote_flag: false)

      visit legislation_process_question_path(question.process, question)

      within("#comment_#{comment.id}_votes") do
        within(".in-favor") do
          expect(page).to have_content "1"
        end

        within(".against") do
          expect(page).to have_content "1"
        end

        expect(page).to have_content "2 votes"
      end
    end

    scenario "Create" do
      visit legislation_process_question_path(question.process, question)

      within("#comment_#{comment.id}_votes") do
        click_button "I agree"

        within(".in-favor") do
          expect(page).to have_content "1"
        end

        within(".against") do
          expect(page).to have_content "0"
        end

        expect(page).to have_content "1 vote"
      end
    end

    scenario "Update" do
      visit legislation_process_question_path(question.process, question)

      within("#comment_#{comment.id}_votes") do
        click_button "I agree"

        within(".in-favor") do
          expect(page).to have_content "1"
        end

        click_button "I disagree"

        within(".in-favor") do
          expect(page).to have_content "0"
        end

        within(".against") do
          expect(page).to have_content "1"
        end

        expect(page).to have_content "1 vote"
      end
    end

    scenario "Allow undoing votes" do
      visit legislation_process_question_path(question.process, question)

      within("#comment_#{comment.id}_votes") do
        click_button "I agree"
        within(".in-favor") do
          expect(page).to have_content "1"
        end

        click_button "I agree"
        within(".in-favor") do
          expect(page).not_to have_content "2"
          expect(page).to have_content "0"
        end

        within(".against") do
          expect(page).to have_content "0"
        end

        expect(page).to have_content "No votes"
      end
    end
  end
end