CDJ11/CDJ

File polls_spec.rb has 384 lines of code (exceeds 250 allowed). Consider refactoring.

Open

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

feature 'Commenting polls' 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 2 locations. Consider refactoring.

Open

  feature "Administrators" do
    scenario "can create comment as an administrator", :js do
      skip "Feature not implemented yet, review soon"

      admin = create(:administrator)

• Read up
  Read up

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

Open

  feature "Moderators" do

    scenario "can create comment as a moderator", :js do
      skip "Feature not implemented yet, review soon"

• 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: poll)
    child_comment  = create(:comment, body: "First subcomment", commentable: poll, parent: parent_comment)
    grandchild_comment = create(:comment, body: "Last subcomment", commentable: poll, parent: child_comment)

• 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: poll, cached_votes_up: 100,
                                                  cached_votes_total: 120, created_at: Time.current - 2)
    c2 = create(:comment, :with_confidence_score, commentable: poll, cached_votes_up: 10,
                                                  cached_votes_total: 12, created_at: Time.current - 1)

• 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: poll, created_at: Time.current - 10)
   new_root = create(:comment, commentable: poll, created_at: Time.current)
   old_child = create(:comment, commentable: poll, parent_id: new_root.id, created_at: Time.current - 10)
   new_child = create(:comment, commentable: poll, 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 'Reply', :js do
    citizen = create(:user, username: 'Ana')
    manuela = create(:user, username: 'Manuela')
    comment = create(:comment, commentable: poll, 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: poll)}

    visit poll_path(poll)

• Read up
  Read up

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

Open

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

    visit poll_path(poll)

• 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: poll, body: 'Built with http://rubyonrails.org/'

    visit poll_path(poll)

• Read up
  Read up

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

Open

  scenario "Erasing a comment's author" do
    poll = create(:poll)
    comment = create(:comment, commentable: poll, 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: poll,
                     body: "<script>alert('hola')</script> <a href=\"javascript:alert('sorpresa!')\">click me<a/> http://www.url.com"

    visit poll_path(poll)

• Read up
  Read up

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

Open

  scenario 'Create', :js do
    login_as(user)
    visit poll_path(poll)

    fill_in "comment-body-poll_#{poll.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: poll)
      visit poll_path(poll)

• Read up
  Read up

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

Open

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

    7.times do
      create(:comment, commentable: poll, 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: poll, user: user)

    login_as(user)
    visit poll_path(poll)

• Read up
  Read up

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

Open

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

• 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

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

Open

   old_child = create(:comment, commentable: poll, 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

      visit poll_path(@poll)

• 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

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

      visit poll_path(@poll)

• 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

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

• 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

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

Open

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

• 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 poll_path(@poll)

• 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 poll_path(@poll)

• 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

      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

      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

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

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. [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