Showing 668 of 700 total issues
Missing top-level class documentation comment. Open
Open
class Object
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
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
Open
Card::Env.params[:token] = Card::Auth::Token.encode @signup.id, anonymous: true
- Create a ticketCreate a ticket
- Exclude checks
Method Cardname#key
is defined at both cardname/lib/cardname.rb:42 and cardname/lib/cardname.rb:74. Open
Open
def key
- Read upRead up
- Create a ticketCreate a ticket
- Exclude checks
This cop checks for duplicated instance (or singleton) method definitions.
Example:
# bad
def duplicated
1
end
def duplicated
2
end
Example:
# bad
def duplicated
1
end
alias duplicated other_duplicated
Example:
# good
def duplicated
1
end
def other_duplicated
2
end
Use let
instead of an instance variable. Open
Open
@card.delete
- Create a ticketCreate a ticket
- Exclude checks
Start context description with 'when', 'with', or 'without'. Open
Open
context "stabilize" do
- Create a ticketCreate a ticket
- Exclude checks
Start context description with 'when', 'with', or 'without'. Open
Open
context "for Joe User" do
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
body = @mail.parts[0].body.raw_source
- Create a ticketCreate a ticket
- Exclude checks
Start context description with 'when', 'with', or 'without'. Open
Open
context "+*account+*email" do
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
expect(@signup.refresh(true).type_id).to eq(Card::UserID)
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
expect(@account.salt).not_to eq("")
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
expect(@format.render_raw).to eq("u1@user.com")
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
@email_card.update content: "joe@user.com"
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
expect(@email_card.errors[:content].first).to match(/must be unique/)
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
["/update/#{@account.name.url_key}",
- Create a ticketCreate a ticket
- Exclude checks
Start context description with 'when', 'with', or 'without'. Open
Open
context "a welcome email card exists" do
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
expect(@signup.account).to be_active
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
rps_view = @card.format.render_reset_password_success
- Create a ticketCreate a ticket
- Exclude checks
Empty example group detected. Open
Open
RSpec.describe Card::Set::All::Comment do
- Create a ticketCreate a ticket
- Exclude checks
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
Open
before(:all) do
- Create a ticketCreate a ticket
- Exclude checks
Use let
instead of an instance variable. Open
Open
account = @signup.account
- Create a ticketCreate a ticket
- Exclude checks