Fix #360 - Enum cases using wrong separator in switch statements

dangmai · Apr 26, 2021 · 041bc4a · 041bc4a
1 parent 038a74f
commit 041bc4a
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+# 1.9.1
+
+- Fix Enum cases using wrong separator in `switch` statements ([issue](https://github.com/dangmai/prettier-plugin-apex/issues/360)).
+
 # 1.9.0
 
 ## Dependency Changes

diff --git a/src/printer.js b/src/printer.js
@@ -908,7 +908,7 @@ function handleTypeWhen(path, print) {
 }
 
 function handleEnumCase(path, print) {
-  return join(",", path.map(print, "identifiers"));
+  return join(".", path.map(print, "identifiers"));
 }
 
 function handleRunAsBlock(path, print) {

diff --git a/tests/switch/SwitchClass.cls b/tests/switch/SwitchClass.cls
@@ -31,6 +31,12 @@ class SwitchClass {
     }
 
     switch on randomObject {
+      when EmumVariable.CaseOne {
+        System.debug('Object');
+      }
+      when EmumVariable.CaseTwo, EnumVariable.CaseTwo {
+        System.debug('Object');
+      }
       when Account a {
         System.debug('Account ' + a);
       }

diff --git a/tests/switch/__snapshots__/jsfmt.spec.js.snap b/tests/switch/__snapshots__/jsfmt.spec.js.snap
@@ -34,6 +34,12 @@ class SwitchClass {
     }
 
     switch on randomObject {
+      when EmumVariable.CaseOne {
+        System.debug('Object');
+      }
+      when EmumVariable.CaseTwo, EnumVariable.CaseTwo {
+        System.debug('Object');
+      }
       when Account a {
         System.debug('Account ' + a);
       }
@@ -139,6 +145,12 @@ class SwitchClass {
     }
 
     switch on randomObject {
+      when EmumVariable.CaseOne {
+        System.debug('Object');
+      }
+      when EmumVariable.CaseTwo, EnumVariable.CaseTwo {
+        System.debug('Object');
+      }
       when Account a {
         System.debug('Account ' + a);
       }