Skip to content

Conversation

@yahonda
Copy link
Collaborator

@yahonda yahonda commented Mar 14, 2022

This pull request enables RSpec/BeNil and RSpec/BeEq cops.

Introduced via RuboCop RSpec v2.9.0
https://github.com/rubocop/rubocop-rspec/releases/tag/v2.9.0

$ bundle exec rubocop -a
Inspecting 72 files
............................................C......C.C..C........C.C....

Offenses:

spec/active_record/connection_adapters/oracle_enhanced/connection_spec.rb:47:89: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(ActiveRecord::Base.connection.raw_connection.getImplicitCachingEnabled).to eq(false)
                                                                                        ^^
spec/active_record/connection_adapters/oracle_enhanced/connection_spec.rb:55:89: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(ActiveRecord::Base.connection.raw_connection.getImplicitCachingEnabled).to eq(true)
                                                                                        ^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:999:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1004:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil # not whatever is in DATA_DEFAULT column
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1017:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1021:27: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.rem).to be nil
                          ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1034:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1040:34: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.expression).to be nil
                                 ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1054:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1061:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1074:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1081:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:475:42: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(serialized_column.save!).to eq(true)
                                         ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:480:42: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(serialized_column.save!).to eq(true)
                                         ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:519:45: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(binary_column_object.save!).to eq(true)
                                            ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:627:52: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.table_exists?("TEST_POSTS")).to eq true
                                                   ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:628:54: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.table_exists?("NOT_EXISTING")).to eq false
                                                     ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:659:55: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.temporary_table?("TEST_POSTS")).to eq false
                                                      ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:154:59: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
                                                          ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:154:59: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(@employee3.test_boolean_before_type_cast).to be(nil)
                                                          ^^^^^^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:157:59: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
                                                          ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:157:59: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(@employee3.test_boolean_before_type_cast).to be(nil)
                                                          ^^^^^^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:82:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:93:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:98:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:81:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:92:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:97:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^

72 files inspected, 28 offenses detected, 28 offenses corrected
$

@yahonda yahonda requested a review from koic March 14, 2022 08:50
Enabled: true

RSpec/BeNil:
Enabled: true
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about arranging cop names in ASCII order in the department?

 # RuboCop RSpec cops
+RSpec/BeEq:
+  Enabled: true
+
+RSpec/BeNil:
+  Enabled: true
+
 RSpec/EmptyLineAfterExampleGroup:
   Enabled: true

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review. Updated.

Introduced via RuboCop RSpec v2.9.0
https://github.com/rubocop/rubocop-rspec/releases/tag/v2.9.0

```
$ bundle exec rubocop -a
Inspecting 72 files
............................................C......C.C..C........C.C....

Offenses:

spec/active_record/connection_adapters/oracle_enhanced/connection_spec.rb:47:89: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(ActiveRecord::Base.connection.raw_connection.getImplicitCachingEnabled).to eq(false)
                                                                                        ^^
spec/active_record/connection_adapters/oracle_enhanced/connection_spec.rb:55:89: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(ActiveRecord::Base.connection.raw_connection.getImplicitCachingEnabled).to eq(true)
                                                                                        ^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:999:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1004:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil # not whatever is in DATA_DEFAULT column
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1017:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1021:27: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.rem).to be nil
                          ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1034:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1040:34: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.expression).to be nil
                                 ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1054:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1061:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1074:25: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(tf).not_to be nil
                        ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:1081:31: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
        expect(tf.percent).to be nil
                              ^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:475:42: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(serialized_column.save!).to eq(true)
                                         ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:480:42: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(serialized_column.save!).to eq(true)
                                         ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:519:45: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(binary_column_object.save!).to eq(true)
                                            ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:627:52: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.table_exists?("TEST_POSTS")).to eq true
                                                   ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:628:54: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.table_exists?("NOT_EXISTING")).to eq false
                                                     ^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:659:55: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@conn.temporary_table?("TEST_POSTS")).to eq false
                                                      ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:154:59: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
                                                          ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:154:59: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(@employee3.test_boolean_before_type_cast).to be(nil)
                                                          ^^^^^^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:157:59: C: [Corrected] RSpec/BeEq: Prefer be over eq.
      expect(@employee3.test_boolean_before_type_cast).to eq(nil)
                                                          ^^
spec/active_record/oracle_enhanced/type/boolean_spec.rb:157:59: C: [Corrected] RSpec/BeNil: Prefer be_nil over be(nil).
      expect(@employee3.test_boolean_before_type_cast).to be(nil)
                                                          ^^^^^^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:82:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:93:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/national_character_text_spec.rb:98:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:81:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:92:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^
spec/active_record/oracle_enhanced/type/text_spec.rb:97:31: C: [Corrected] RSpec/BeEq: Prefer be over eq.
    expect(@employee.save).to eq(true)
                              ^^

72 files inspected, 28 offenses detected, 28 offenses corrected
$
```
@yahonda yahonda force-pushed the other_rubocop_rspec_cops branch from 9f926ca to 800cd1f Compare March 14, 2022 08:59
@koic koic merged commit 4f4dd16 into rsim:master Mar 14, 2022
@koic
Copy link
Collaborator

koic commented Mar 14, 2022

Looks good to me! Thanks!

@yahonda yahonda deleted the other_rubocop_rspec_cops branch March 14, 2022 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants