spec/rubocop/cop/rspec/be_eql_spec.rb in rubocop-rspec-1.32.0 vs spec/rubocop/cop/rspec/be_eql_spec.rb in rubocop-rspec-1.33.0

- old
+ new

@@ -1,47 +1,72 @@ +# frozen_string_literal: true + RSpec.describe RuboCop::Cop::RSpec::BeEql do subject(:cop) { described_class.new } it 'registers an offense for `eql` when argument is a boolean' do expect_offense(<<-RUBY) it { expect(foo).to eql(true) } ^^^ Prefer `be` over `eql`. it { expect(foo).to eql(false) } ^^^ Prefer `be` over `eql`. RUBY + + expect_correction(<<-RUBY) + it { expect(foo).to be(true) } + it { expect(foo).to be(false) } + RUBY end it 'registers an offense for `eql` when argument is an integer' do expect_offense(<<-RUBY) it { expect(foo).to eql(0) } ^^^ Prefer `be` over `eql`. it { expect(foo).to eql(123) } ^^^ Prefer `be` over `eql`. RUBY + + expect_correction(<<-RUBY) + it { expect(foo).to be(0) } + it { expect(foo).to be(123) } + RUBY end it 'registers an offense for `eql` when argument is a float' do expect_offense(<<-RUBY) it { expect(foo).to eql(1.0) } ^^^ Prefer `be` over `eql`. it { expect(foo).to eql(1.23) } ^^^ Prefer `be` over `eql`. RUBY + + expect_correction(<<-RUBY) + it { expect(foo).to be(1.0) } + it { expect(foo).to be(1.23) } + RUBY end it 'registers an offense for `eql` when argument is a symbol' do expect_offense(<<-RUBY) it { expect(foo).to eql(:foo) } ^^^ Prefer `be` over `eql`. RUBY + + expect_correction(<<-RUBY) + it { expect(foo).to be(:foo) } + RUBY end it 'registers an offense for `eql` when argument is nil' do expect_offense(<<-RUBY) it { expect(foo).to eql(nil) } ^^^ Prefer `be` over `eql`. RUBY + + expect_correction(<<-RUBY) + it { expect(foo).to be(nil) } + RUBY end it 'does not register an offense for `eql` when argument is a string' do expect_no_offenses(<<-RUBY) it { expect(foo).to eql('foo') } @@ -51,10 +76,6 @@ it 'does not register an offense for `eql` when expectation is negated' do expect_no_offenses(<<-RUBY) it { expect(foo).to_not eql(1) } RUBY end - - include_examples 'autocorrect', - 'it { expect(foo).to eql(1) }', - 'it { expect(foo).to be(1) }' end