CDJ11/CDJ

File proposals_spec.rb has 374 lines of code (exceeds 250 allowed). Consider refactoring.

Open

require 'rails_helper'
include ActionView::Helpers::DateHelper

feature 'Commenting proposals' do
  let(:user) { create :user }

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  feature 'Voting comments' do

    background do
      @manuela = create(:user, verified_at: Time.current)
      @pablo = create(:user)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

  feature "Moderators" do
    scenario "can create comment as a moderator", :js do
      moderator = create(:moderator)

      login_as(moderator.user)

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

  feature "Administrators" do
    scenario "can create comment as an administrator", :js do
      admin = create(:administrator)

      login_as(admin.user)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  scenario 'Show' do
    parent_comment = create(:comment, commentable: proposal)
    first_child    = create(:comment, commentable: proposal, parent: parent_comment)
    second_child   = create(:comment, commentable: proposal, parent: parent_comment)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  scenario 'Reply', :js do
    citizen = create(:user, username: 'Ana')
    manuela = create(:user, username: 'Manuela')
    comment = create(:comment, commentable: proposal, user: citizen)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

    visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  scenario "Undoing flagging as inappropriate", :js do
    comment = create(:comment, commentable: proposal)
    Flag.flag(user, comment)

    login_as(user)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  scenario 'Index' do
    3.times { create(:comment, commentable: proposal) }

    visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  scenario "Flagging as inappropriate", :js do
    comment = create(:comment, commentable: proposal)

    login_as(user)
    visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  scenario 'Turns links into html links' do
    create :comment, commentable: proposal, body: 'Built with http://rubyonrails.org/'

    visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  scenario "Flagging turbolinks sanity check", :js do
    proposal = create(:proposal, title: "Should we change the world?")
    comment = create(:comment, commentable: proposal)

    login_as(user)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

    visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  scenario 'Create', :js do
    login_as(user)
    visit proposal_path(proposal)

    fill_in "comment-body-proposal_#{proposal.id}", with: 'Have you thought about...?'

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  feature 'Not logged user' do
    scenario 'can not see comments forms' do
      create(:comment, commentable: proposal)
      visit proposal_path(proposal)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

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

    7.times do
      create(:comment, commentable: proposal, parent: parent)

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  scenario 'Errors on reply', :js do
    comment = create(:comment, commentable: proposal, user: user)

    login_as(user)
    visit proposal_path(proposal)

• Read up
  Read up

Line is too long. [112/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

  scenario 'Creation date works differently in roots and in child comments, when sorting by confidence_score' do

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      login_as(@manuela)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      create(:vote, voter: @pablo, votable: @comment, vote_flag: false)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      visit proposal_path(@proposal)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      within("#comment_#{@comment.id}_votes") do

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [104/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

                                                  cached_votes_total: 120, created_at: Time.current - 2)

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [133/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

                     body: "<script>alert('hola')</script> <a href=\"javascript:alert('sorpresa!')\">click me<a/> http://www.url.com"

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      create(:vote, voter: @manuela, votable: @comment, vote_flag: true)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      within("#comment_#{@comment.id}_votes") do

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [109/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

   old_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.current - 10)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      within("#comment_#{@comment.id}_votes") do

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [110/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

    child_comment  = create(:comment, body: "First subcomment", commentable: proposal, parent: parent_comment)

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [112/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

    grandchild_comment = create(:comment, body: "Last subcomment", commentable: proposal, parent: child_comment)

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [104/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

   new_child = create(:comment, commentable: proposal, parent_id: new_root.id, created_at: Time.current)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      visit proposal_path(@proposal)

• Exclude checks
  • Disable engine
  • Disable check

Line is too long. [103/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

                                                  cached_votes_total: 12, created_at: Time.current - 1)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      create(:vote, voter: @pablo, votable: @comment, vote_flag: false)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      within("#comment_#{@comment.id}_votes") do

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      visit proposal_path(@proposal)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      @comment = create(:comment, commentable: @proposal)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      create(:vote, voter: @manuela, votable: @comment, vote_flag: true)

• Exclude checks
  • Disable engine
  • Disable check

Use let instead of an instance variable. (http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InstanceVariable)

Open

      visit proposal_path(@proposal)

• Exclude checks
  • Disable engine
  • Disable check