Skip to content

Conversation

@yahonda
Copy link
Collaborator

@yahonda yahonda commented Mar 14, 2022

This pull request enables RSpec/RepeatedDescription cop.

These offenses are fixed manually.

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

Offenses:

spec/active_record/connection_adapters/oracle_enhanced/dbms_output_spec.rb:50:3: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
  it "should log dbms output lines to the rails log" do
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/dbms_output_spec.rb:60:3: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
  it "should log dbms output lines to the rails log" do
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:884:5: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
    it "should add lob column with non_default tablespace" do
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:892:5: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
    it "should add lob column with non_default tablespace" do
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:81:7: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
      it "should get primary key from database at first time" do
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:86:7: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
      it "should get primary key from database at first time" do
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

72 files inspected, 6 offenses detected
$

These offenses are fixed manually.

```ruby
$ bundle exec rubocop
Inspecting 72 files
...............................................C...C.C..................

Offenses:

spec/active_record/connection_adapters/oracle_enhanced/dbms_output_spec.rb:50:3: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
  it "should log dbms output lines to the rails log" do
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/dbms_output_spec.rb:60:3: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
  it "should log dbms output lines to the rails log" do
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:884:5: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
    it "should add lob column with non_default tablespace" do
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced/schema_statements_spec.rb:892:5: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
    it "should add lob column with non_default tablespace" do
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:81:7: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
      it "should get primary key from database at first time" do
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
spec/active_record/connection_adapters/oracle_enhanced_adapter_spec.rb:86:7: C: RSpec/RepeatedDescription: Don't repeat descriptions within an example group.
      it "should get primary key from database at first time" do
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

72 files inspected, 6 offenses detected
$
```
@yahonda yahonda requested a review from koic March 14, 2022 11:12
@koic koic merged commit 8201212 into rsim:master Mar 14, 2022
@koic
Copy link
Collaborator

koic commented Mar 14, 2022

Thanks!

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