Remove default case in switch case list

.ci/validation/test/fixtures/invalid/switch-missing-when.yaml

@@ -0,0 +1,45 @@
+document:
+  dsl: 1.0.0-alpha2
+  namespace: test
+  name: switch-missing-when
+  version: 0.1.0
+do:
+  - processOrder:
+      switch:
+        - case1:
+            when: .orderType == "electronic"
+            then: processElectronicOrder
+        - case2:
+            when: .orderType == "physical"
+            then: processPhysicalOrder
+        - default:
+            then: handleUnknownOrderType
+  - processElectronicOrder:
+      do:
+        - validatePayment:
+            set:
+              validate: true
+        - fulfillOrder:
+            set:
+              status: fulfilled
+      then: exit
+  - processPhysicalOrder:
+      do:
+        - checkInventory:
+            set:
+              inventory: clear
+        - packItems:
+            set:
+              items: 1
+        - scheduleShipping:
+            set:
+              address: Elmer St
+      then: exit
+  - handleUnknownOrderType:
+      do:
+        - logWarning:
+            set:
+              log: warn
+        - notifyAdmin:
+            set:
+              message: something's wrong

ctk/features/switch.feature

@@ -108,8 +108,7 @@ Feature: Switch Task
             - blue:
                 when: '.color == "blue"'
                 then: setBlue
-            - anyOtherColor:
-                then: setCustomColor
+          then: setCustomColor
       - setRed:
           set:
             colors: '${ .colors + [ "red" ] }'

dsl-reference.md

@@ -673,8 +673,7 @@ do:
         - lowPriority:
             when: .ticket.priority == "low"
             then: resolveTicket
-        - default:
-            then: raiseUndefinedPriorityError
+      then: raiseUndefinedPriorityError
   - raiseUndefinedPriorityError:
       raise:
         error:
@@ -926,8 +925,7 @@ do:
         - case2:
             when: .orderType == "physical"
             then: processPhysicalOrder
-        - default:
-            then: handleUnknownOrderType
+      then: handleUnknownOrderType
   - processElectronicOrder:
       do:
         - validatePayment:
@@ -979,7 +977,7 @@ Defines a switch case, encompassing a condition for matching and an associated a
 
 | Name | Type | Required | Description |
 |:--|:---:|:---:|:---|
-| when | `string` | `no` | A runtime expression used to determine whether or not the case matches.<br>*If not set, the case will be matched by default if no other case match.*<br>*Note that there can be only one default case, all others **MUST** set a condition.*
+| when | `string` | `yes` | A runtime expression used to determine whether or not the case matches.<br>*To define a default case, use the task level `then` property, which will be used when no task matches.* |
 | then | [`flowDirective`](#flow-directive) | `yes` | The flow directive to execute when the case matches. |
 
 #### Try

examples/switch-then-string.yaml

@@ -12,8 +12,7 @@ do:
         - case2:
             when: .orderType == "physical"
             then: processPhysicalOrder
-        - default:
-            then: handleUnknownOrderType
+      then: handleUnknownOrderType
   - processElectronicOrder:
       do:
         - validatePayment:

schema/workflow.yaml

@@ -702,7 +702,7 @@ $defs:
             title: SwitchCase
             description: The definition of a case within a switch task, defining a condition and corresponding tasks to execute if the condition is met.
             unevaluatedProperties: false
-            required: [ then ]
+            required: [ when, then ]
             properties:
               when:
                 type: string

use-cases/managing-ev-charging-stations/README.md

@@ -119,8 +119,7 @@ do:
         - sessionInProgress:
             when: .session != null
             then: endSession
-        - noActiveSession:
-            then: tryAquireSlot
+      then: tryAquireSlot
 
   - tryAquireSlot:
       try:

use-cases/managing-github-issues/README.md

@@ -114,8 +114,7 @@ do:
         - requestDetails:
             when: $context.issue.action == "requestDetails"
             then: awaitDetailsFromQA
-        - default:
-            then: raiseUnsupportedActionError
+      then: raiseUnsupportedActionError
 
   - awaitDetailsFromQA:
       do:
@@ -174,8 +173,7 @@ do:
          - reviewerIsNotAssignedDev:
               when: $context.issue.reviewer != $context.issue.dev
               then: evaluateReview
-         - reviewerIsAssignedDev:
-              then: raiseAssignedDevCannotBeReviewer
+        then: raiseAssignedDevCannotBeReviewer
 
   - evaluateReview:
       do:
@@ -206,8 +204,7 @@ do:
               - closeIssue:
                   when: $context.issue.action == "close"
                   then: closeIssue
-              - default:
-                  then: exit
+            then: exit
         - closeIssue:
             do:
               - initialize: