Issues - decko-commons/decko

Missing top-level module documentation comment.
Open

module ClassMethods

Found in mod/account/set/all/account.rb by rubocop

Read up
Read up
Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

This cop checks for missing top-level documentation of classes and modules. Classes with no body are exempt from the check and so are namespace modules - modules that have nothing in their bodies except classes, other modules, or constant definitions.

The documentation requirement is annulled if the class or module has a "#:nodoc:" comment next to it. Likewise, "#:nodoc: all" does the same for all its children.

Example:

# bad
class Person
  # ...
end

# good
# Description/Explanation of Person class
class Person
  # ...
end

Use `let` instead of an instance variable.
Open

      expect(@mail.body.raw_source).to include("Welcome Big Bad Sheep")

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      expect(@signup.type_id).to eq(Card::SignupID)

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      expect(@signup.format.invitation?).to be(true)

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Beware of using `before(:all)` as it may cause state to leak between tests. If you are using `rspec-rails`, and `use_transactional_fixtures` is enabled, then records created in `before(:all)` are not automatically rolled back.
Open

    before(:all) do

Found in mod/account/spec/set/all/account_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Empty example group detected.
Open

describe Card::Set::Right::Status do

Found in mod/account/spec/set/right/status_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Start context description with 'when', 'with', or 'without'.
Open

    context "for Joe User" do

Found in mod/account/spec/set/all/account_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Start context description with 'when', 'with', or 'without'.
Open

  context "<User>+*email" do

Found in mod/account/spec/set/right/email_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      @format = @card.format

Found in mod/account/spec/set/right/email_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

          @email_card.update! content: "QuIrE@example.com"

Found in mod/account/spec/set/right/email_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

        expect(@signup.account.status).to eq("unverified")

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

        expect(@parties)

Found in mod/account/spec/set/all/account_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

        Card::Auth.as_bot { r1_members.items = [@joe_user_card.id] }

Found in mod/account/spec/set/all/account_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Start context description with 'when', 'with', or 'without'.
Open

    context "blank password" do

Found in mod/account/spec/set/right/password_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Empty example group detected.
Open

describe Card::Set::Right::Salt do

Found in mod/account/spec/set/right/salt_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Freeze mutable objects assigned to constants.
Open

  OK4KEY_RE = '\p{Word}\*'

Found in cardname/lib/cardname.rb by rubocop

Read up
Read up
Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

This cop checks whether some constant value isn't a mutable literal (e.g. array or hash).

Example:

# bad
CONST = [1, 2, 3]

# good
CONST = [1, 2, 3].freeze

Start context description with 'when', 'with', or 'without'.
Open

    context "do not stabilize" do

Found in cardname/spec/lib/cardname_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      @account = @signup.account

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      expect(@signup.type_id).to eq(Card::SignupID)

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

Use `let` instead of an instance variable.
Open

      expect(@account.email).to eq("wolf@decko.org")

Found in mod/account/spec/set/type/signup_spec.rb by rubocop

Create a ticket
Create a ticket
Exclude checks
- Disable engine
- Disable check

decko-commons/decko

Showing 666 of 699 total issues

Missing top-level module documentation comment.
Open

Example:

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Beware of using `before(:all)` as it may cause state to leak between tests. If you are using `rspec-rails`, and `use_transactional_fixtures` is enabled, then records created in `before(:all)` are not automatically rolled back.
Open

Empty example group detected.
Open

Start context description with 'when', 'with', or 'without'.
Open

Start context description with 'when', 'with', or 'without'.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Start context description with 'when', 'with', or 'without'.
Open

Empty example group detected.
Open

Freeze mutable objects assigned to constants.
Open

Example:

Start context description with 'when', 'with', or 'without'.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Severity

Category

Status

Source

Language

decko-commons/decko

Showing 666 of 699 total issues

Missing top-level module documentation comment. Open

Example:

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Beware of using before(:all) as it may cause state to leak between tests. If you are using rspec-rails, and use_transactional_fixtures is enabled, then records created in before(:all) are not automatically rolled back. Open

Empty example group detected. Open

Start context description with 'when', 'with', or 'without'. Open

Start context description with 'when', 'with', or 'without'. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Start context description with 'when', 'with', or 'without'. Open

Empty example group detected. Open

Freeze mutable objects assigned to constants. Open

Example:

Start context description with 'when', 'with', or 'without'. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Use let instead of an instance variable. Open

Severity

Category

Status

Source

Language

Missing top-level module documentation comment.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Beware of using `before(:all)` as it may cause state to leak between tests. If you are using `rspec-rails`, and `use_transactional_fixtures` is enabled, then records created in `before(:all)` are not automatically rolled back.
Open

Empty example group detected.
Open

Start context description with 'when', 'with', or 'without'.
Open

Start context description with 'when', 'with', or 'without'.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Start context description with 'when', 'with', or 'without'.
Open

Empty example group detected.
Open

Freeze mutable objects assigned to constants.
Open

Start context description with 'when', 'with', or 'without'.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open

Use `let` instead of an instance variable.
Open