Merge pull request #122 from publify/update-rubocop-configuration

Update RuboCop configuration and autocorrect new offenses

Author: mvz

.rubocop.yml

@@ -4,7 +4,7 @@ inherit_mode:
   merge:
     - Exclude
 
-require:
+plugins:
   - rubocop-capybara
   - rubocop-factory_bot
   - rubocop-performance
@@ -23,6 +23,14 @@ AllCops:
 Rails:
   Enabled: true
 
+# Require tests to specify whether an element is a link or a button
+Capybara/ClickLinkOrButtonStyle:
+  EnforcedStyle: strict
+
+# Match style of regular expectations
+Capybara/NegationMatcher:
+  EnforcedStyle: not_to
+
 # Put development dependencies in the gemspec so rubygems.org knows about them
 Gemspec/DevelopmentDependencies:
   EnforcedStyle: gemspec
@@ -103,6 +111,10 @@ Rails/SkipsModelValidations:
   Exclude:
     - 'spec/dummy/db/migrate/*'
 
+# Spec type inference is disabled, so this cop must be disabled too.
+RSpecRails/InferredSpecType:
+  Enabled: false
+
 # Allow the use of 'and' 'or' in control structures.
 Style/AndOr:
   EnforcedStyle: conditionals
@@ -163,14 +175,6 @@ Style/StringLiteralsInInterpolation:
 Style/SymbolArray:
   EnforcedStyle: brackets
 
-# Prefer consistent lines for multi-line arrays
-Style/TrailingCommaInArrayLiteral:
-  EnforcedStyleForMultiline: comma
-
-# Prefer consistent lines for multi-line hashes
-Style/TrailingCommaInHashLiteral:
-  EnforcedStyleForMultiline: comma
-
 # Allow small arrays of words with quotes
 Style/WordArray:
   MinSize: 4

spec/dummy/config/environments/development.rb

@@ -21,7 +21,7 @@
 
     config.cache_store = :memory_store
     config.public_file_server.headers = {
-      "Cache-Control" => "public, max-age=#{2.days.to_i}",
+      "Cache-Control" => "public, max-age=#{2.days.to_i}"
     }
   else
     config.action_controller.perform_caching = false

spec/dummy/config/environments/test.rb

@@ -17,7 +17,7 @@
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
   config.public_file_server.headers = {
-    "Cache-Control" => "public, max-age=#{1.hour.to_i}",
+    "Cache-Control" => "public, max-age=#{1.hour.to_i}"
   }
 
   # Show full error reports and disable caching.