CDJ11/CDJ

File budget_investments_spec.rb has 386 lines of code (exceeds 250 allowed). Consider refactoring.

Open

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

feature 'Commenting Budget::Investments' do

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 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 2 locations. Consider refactoring.

Open

  scenario 'Comment order' do
    c1 = create(:comment, :with_confidence_score, commentable: investment, cached_votes_up: 100,
                                                  cached_votes_total: 120, created_at: Time.current - 2)
    c2 = create(:comment, :with_confidence_score, commentable: investment, 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 'Creation date works differently in roots and in child comments, when sorting by confidence_score' do
   old_root = create(:comment, commentable: investment, created_at: Time.current - 10)
   new_root = create(:comment, commentable: investment, created_at: Time.current)
   old_child = create(:comment, commentable: investment, parent_id: new_root.id, created_at: Time.current - 10)
   new_child = create(:comment, commentable: investment, parent_id: new_root.id, created_at: Time.current)

• Read up
  Read up

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

Open

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

• Read up
  Read up

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

Open

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

    visit budget_investment_path(investment.budget, investment)

• 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: investment)
    Flag.flag(user, comment)

    login_as(user)

• 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: investment)

    login_as(user)
    visit budget_investment_path(investment.budget, investment)

• Read up
  Read up

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

Open

    scenario 'Update', :js do
      visit budget_investment_path(@budget, @investment)

      within("#comment_#{@comment.id}_votes") do
        find('.in_favor a').click

• Read up
  Read up

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

Open

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

    visit budget_investment_path(investment.budget, investment)

• Read up
  Read up

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

Open

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

    visit budget_investment_path(investment.budget, investment)

• Read up
  Read up

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

Open

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

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

• Read up
  Read up

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

Open

  feature 'Not logged user' do
    scenario 'can not see comments forms' do
      create(:comment, commentable: investment)
      visit budget_investment_path(investment.budget, investment)

• Read up
  Read up

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

Open

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

    login_as(user)
    visit budget_investment_path(investment.budget, investment)

• Read up
  Read up

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

Open

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

• Read up
  Read up

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 budget_investment_path(@budget, @investment)

• 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

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

• 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

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

Open

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

• 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. [111/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

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

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

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    grandchild_comment = create(:comment, body: "Last subcomment", commentable: investment, 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

      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

      visit budget_investment_path(@budget, @investment)

• 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 budget_investment_path(@budget, @investment)

• 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 budget_investment_path(@budget, @investment)

• 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. [112/100] (https://github.com/bbatsov/ruby-style-guide#80-character-limits)

Open

    child_comment  = create(:comment, body: "First subcomment", commentable: investment, 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

      visit budget_investment_path(@budget, @investment)

• 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 budget_investment_path(@budget, @budget, @investment)

• 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 budget_investment_path(@budget, @budget, @investment)

• 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 budget_investment_path(@budget, @investment)

• 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

      @budget = @investment.budget

• 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 budget_investment_path(@budget, @budget, @investment)

• Exclude checks
  • Disable engine
  • Disable check

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

Open

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

• 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

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

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