Enable RSpec/DescribeClass rubocop rspec cop & fix issues

Manually fixes all issues & "stashed" false positives at .rubocop_todo.yml

Read about the cop at http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/DescribeClass
This commit is contained in:
Bertocq
2018-01-07 00:09:05 +01:00
parent 61be1015dd
commit 6ec6f64675
24 changed files with 44 additions and 27 deletions

View File

@@ -51,3 +51,6 @@ Capybara/FeatureMethods:
RSpec/ContextWording: RSpec/ContextWording:
Enabled: false Enabled: false
RSpec/DescribeClass:
Enabled: true

View File

@@ -783,3 +783,17 @@ Style/SafeNavigation:
Style/WordArray: Style/WordArray:
EnforcedStyle: percent EnforcedStyle: percent
MinSize: 8 MinSize: 8
# Offense count: 9
RSpec/DescribeClass:
Exclude:
- 'spec/customization_engine_spec.rb'
- 'spec/i18n_spec.rb'
- 'spec/lib/acts_as_paranoid_aliases_spec.rb'
- 'spec/lib/cache_spec.rb'
- 'spec/lib/graphql_spec.rb'
- 'spec/lib/tasks/communities_spec.rb'
- 'spec/lib/tasks/dev_seed_spec.rb'
- 'spec/lib/tasks/settings_spec.rb'
- 'spec/models/abilities/organization_spec.rb'
- 'spec/views/welcome/index.html.erb_spec.rb'

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe 'HasFilters' do describe HasFilters do
class FakeController < ActionController::Base; end class FakeController < ActionController::Base; end

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe 'HasOrders' do describe HasOrders do
class FakeController < ActionController::Base; end class FakeController < ActionController::Base; end

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe 'ActsAsTaggableOn' do describe ActsAsTaggableOn do
describe 'Tagging' do describe 'Tagging' do
describe "when tagging debates or proposals" do describe "when tagging debates or proposals" do

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Administrator" do describe Abilities::Administrator do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:user) { administrator.user } let(:user) { administrator.user }
let(:administrator) { create(:administrator) } let(:administrator) { create(:administrator) }

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Common" do describe Abilities::Common do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:geozone) { create(:geozone) } let(:geozone) { create(:geozone) }

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Everyone" do describe Abilities::Everyone do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:user) { nil } let(:user) { nil }

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Moderator" do describe Abilities::Moderator do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:user) { moderator.user } let(:user) { moderator.user }
let(:moderator) { create(:moderator) } let(:moderator) { create(:moderator) }

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Organization" do describe 'Abilities::Organization' do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:user) { organization.user } let(:user) { organization.user }
let(:organization) { create(:organization) } let(:organization) { create(:organization) }

View File

@@ -1,7 +1,7 @@
require 'rails_helper' require 'rails_helper'
require 'cancan/matchers' require 'cancan/matchers'
describe "Abilities::Valuator" do describe Abilities::Valuator do
subject(:ability) { Ability.new(user) } subject(:ability) { Ability.new(user) }
let(:user) { valuator.user } let(:user) { valuator.user }
let(:valuator) { create(:valuator) } let(:valuator) { create(:valuator) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe "Budget::Ballot::Line" do describe Budget::Ballot::Line do
let(:budget){ create(:budget) } let(:budget){ create(:budget) }
let(:group){ create(:budget_group, budget: budget) } let(:group){ create(:budget_group, budget: budget) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe "Budget::Investment::Milestone" do describe Budget::Investment::Milestone do
describe "Validations" do describe "Validations" do
let(:milestone) { build(:budget_investment_milestone) } let(:milestone) { build(:budget_investment_milestone) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe 'Verification::Letter' do describe Verification::Letter do
let(:user) { create(:user) } let(:user) { create(:user) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :booth_assignment do describe Poll::BoothAssignment do
let(:poll){create(:poll)} let(:poll){create(:poll)}
let(:booth){create(:poll_booth)} let(:booth){create(:poll_booth)}
let(:booth1){create(:poll_booth)} let(:booth1){create(:poll_booth)}

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :booth do describe Poll::Booth do
let(:booth) { build(:poll_booth) } let(:booth) { build(:poll_booth) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :officer_assignment do describe Poll::OfficerAssignment do
it "should log user data on creation" do it "should log user data on creation" do
user = create(:user, username: "Larry Bird", email: "larry@lege.nd") user = create(:user, username: "Larry Bird", email: "larry@lege.nd")
officer = create(:poll_officer, user: user) officer = create(:poll_officer, user: user)

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :officer do describe Poll::Officer do
describe "#voting_days_assigned_polls" do describe "#voting_days_assigned_polls" do
it "should return all polls with this officer assigned during voting days" do it "should return all polls with this officer assigned during voting days" do

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :shift do describe Poll::Shift do
let(:poll) { create(:poll) } let(:poll) { create(:poll) }
let(:booth) { create(:poll_booth) } let(:booth) { create(:poll_booth) }
let(:user) { create(:user, username: "Ana", email: "ana@example.com") } let(:user) { create(:user, username: "Ana", email: "ana@example.com") }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe :voter do describe Poll::Voter do
let(:poll) { create(:poll) } let(:poll) { create(:poll) }
let(:booth) { create(:poll_booth) } let(:booth) { create(:poll_booth) }

View File

@@ -1,6 +1,6 @@
require 'rails_helper' require 'rails_helper'
describe 'Vote' do describe Vote do
describe '#for_debates' do describe '#for_debates' do
it 'does not returns votes for other votables' do it 'does not returns votes for other votables' do

View File

@@ -1,6 +1,6 @@
require "rails_helper" require "rails_helper"
RSpec.describe "welcome/index" do RSpec.describe "welcome#index" do
it 'Display images on orbit carrousel when we have defined image_default' do it 'Display images on orbit carrousel when we have defined image_default' do
debate = create(:debate) debate = create(:debate)