Merge pull request #324 from testdouble/change-case-indentation

Change case indentation to match if/else
standardrb · Mar 23, 2022 · 91d40ed · 91d40ed
2 parents 350e08f + 437cba5
commit 91d40ed
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 3 deletions.
diff --git a/config/base.yml b/config/base.yml
@@ -83,9 +83,8 @@ Layout/BlockEndNewline:
   Enabled: true
 
 Layout/CaseIndentation:
-  # Disabled because IndentOneStep can't be configured for one-liner cases. See:
-  # https://github.com/rubocop-hq/rubocop/issues/6447
-  Enabled: false
+  Enabled: true
+  EnforcedStyle: end
 
 Layout/ClassStructure:
   Enabled: false

diff --git a/test/fixture/cli/autocorrectable-bad.rb b/test/fixture/cli/autocorrectable-bad.rb
@@ -107,3 +107,13 @@ def bad_function(test: true, a:, b:)
     b
   end
 end
+
+def count_carbs(food)
+  carbs = case food
+          when :pancakes
+            23
+          when :mushrooms
+            4
+          end
+  carbs + 1
+end
diff --git a/test/fixture/cli/autocorrectable-good.rb b/test/fixture/cli/autocorrectable-good.rb
@@ -97,3 +97,13 @@ def bad_function(a:, b:, test: true)
     b
   end
 end
+
+def count_carbs(food)
+  carbs = case food
+  when :pancakes
+    23
+  when :mushrooms
+    4
+  end
+  carbs + 1
+end