decko-commons/decko

View on GitHub

Showing 661 of 695 total issues

Use let instead of an instance variable.
Open

          expect(@email_card.db_content).to eq("quire@example.com")

Use let instead of an instance variable.
Open

      body = @mail.parts[0].body.raw_source

Empty example group detected.
Open

RSpec.describe Card::Set::All::Comment do

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

  context "a welcome email card exists" do

Use let instead of an instance variable.
Open

        expect(@signup.account).to be_active

Use let instead of an instance variable.
Open

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

Use let instead of an instance variable.
Open

        @email_card.update content: "boop"

Use let instead of an instance variable.
Open

        expect(@email_card.errors[:content].first)

Empty example group detected.
Open

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

Use let instead of an instance variable.
Open

      @account.send_verification_email

Use let instead of an instance variable.
Open

      expect(@signup.refresh(true).type_id).to eq(Card::UserID)

Use let instead of an instance variable.
Open

        @signup.update! trigger: :approve_without_verification

Use let instead of an instance variable.
Open

      @signup.update! trigger: :approve_without_verification

Use let instead of an instance variable.
Open

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

Use let instead of an instance variable.
Open

      @card.update! "+*email" => "schmoe@admin.com",

Use let instead of an instance variable.
Open

      expect(@card.errors[:signin].first).to match(/Unrecognized email/)

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

  context "updates" do

Use let instead of an instance variable.
Open

        expect(@parties)

Empty example group detected.
Open

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

Use let instead of an instance variable.
Open

      ["/update/#{@account.name.url_key}",
Severity
Category
Status
Source
Language