Added PB126

.changelog/4349.yml

@@ -0,0 +1,5 @@
+changes:
+- description: Added DS105 to the new validation format. Ensure that conditional tasks have more than path which is not
+    the default one
+  type: internal
+pr_number: 4349

demisto_sdk/commands/validate/sdk_validation_config.toml

@@ -29,7 +29,7 @@ select = [
     "IN100", "IN101", "IN102", "IN104", "IN106", "IN107", "IN108", "IN109", "IN110", "IN112", "IN113", "IN114", "IN115", "IN117", "IN118", "IN121", "IN122", "IN123", "IN124", "IN125", "IN126", "IN127", "IN130",
     "IN131", "IN134", "IN135", "IN139", "IN141", "IN142", "IN144", "IN145", "IN146", "IN149", "IN150", "IN151", "IN152", "IN153", "IN154", "IN156", "IN158", "IN159", "IN160",
     "IN161", "IN162",
-    "PB100", "PB101","PB104","PB118", "PB123",
+    "PB100", "PB101","PB104","PB118", "PB123", "PB126",
     "DO100", "DO101", "DO102", "DO103", "DO104", "DO105", "DO106",
     "DS100", "DS107",
     "SC100", "SC105", "SC106", "SC109",
@@ -52,7 +52,7 @@ select = [
     "IN131", "IN134", "IN135", "IN139", "IN141", "IN142", "IN144", "IN145", "IN146", "IN149", "IN150", "IN151", "IN152", "IN153", "IN154", "IN156", "IN158", "IN159", "IN160",
     "IN161", "IN162",
     "LO107",
-    "PB100", "PB101","PB104", "PB123",
+    "PB100", "PB101","PB104", "PB123", "PB126",
     "BA100", "BA101", "BA105", "BA106", "BA110", "BA111", "BA113", "BA116", "BA118", "BA119", "BA126",
     "DS100",
     "PA100", "PA101", "PA102", "PA103", "PA104", "PA105", "PA107", "PA108", "PA109", "PA111", "PA113", "PA115", "PA117", "PA118", "PA119", "PA120",

demisto_sdk/commands/validate/tests/PB_validators_test.py

@@ -17,6 +17,9 @@
 from demisto_sdk.commands.validate.validators.PB_validators.PB123_is_conditional_task_has_unhandled_reply_options import (
     IsAskConditionHasUnhandledReplyOptionsValidator,
 )
+from demisto_sdk.commands.validate.validators.PB_validators.PB126_is_default_not_only_condition import (
+    IsDefaultNotOnlyConditionValidator,
+)
 
 
 @pytest.mark.parametrize(
@@ -187,7 +190,6 @@ def test_is_deprecated_with_invalid_description(content_item, expected_result):
 
 
 def test_IsAskConditionHasUnreachableConditionValidator():
-
     playbook = create_playbook_object()
     assert not IsAskConditionHasUnreachableConditionValidator().is_valid([playbook])
     playbook.tasks = {
@@ -206,7 +208,6 @@ def test_IsAskConditionHasUnreachableConditionValidator():
 
 
 def test_IsAskConditionHasUnhandledReplyOptionsValidator():
-
     playbook = create_playbook_object()
     assert not IsAskConditionHasUnhandledReplyOptionsValidator().is_valid([playbook])
     playbook.tasks = {
@@ -222,3 +223,33 @@ def test_IsAskConditionHasUnhandledReplyOptionsValidator():
         )
     }
     assert IsAskConditionHasUnhandledReplyOptionsValidator().is_valid([playbook])
+
+
+def test_IsDefaultNotOnlyConditionValidator():
+    playbook = create_playbook_object()
+    assert not IsDefaultNotOnlyConditionValidator().is_valid([playbook])
+    playbook.tasks = {
+        "0": TaskConfig(
+            **{
+                "id": "0",
+                "type": "condition",
+                "taskid": "27b9c747-b883-4878-8b60-7f352098a631",
+                "message": {"replyOptions": ["yes", "no"]},
+                "task": {"id": "27b9c747-b883-4878-8b60-7f352098a63c"},
+            }
+        )
+    }
+
+    assert not IsDefaultNotOnlyConditionValidator().is_valid([playbook])
+    playbook.tasks = {
+        "0": TaskConfig(
+            **{
+                "id": "0",
+                "type": "condition",
+                "taskid": "27b9c747-b883-4878-8b60-7f352098a631",
+                "message": {"replyOptions": ["#default#"]},
+                "task": {"id": "27b9c747-b883-4878-8b60-7f352098a63c"},
+            }
+        )
+    }
+    assert IsDefaultNotOnlyConditionValidator().is_valid([playbook])

demisto_sdk/commands/validate/validators/PB_validators/PB126_is_default_not_only_condition.py

@@ -0,0 +1,51 @@
+from __future__ import annotations
+
+from typing import Iterable, List
+
+from demisto_sdk.commands.common.constants import PlaybookTaskType
+from demisto_sdk.commands.content_graph.objects.playbook import Playbook
+from demisto_sdk.commands.validate.validators.base_validator import (
+    BaseValidator,
+    ValidationResult,
+)
+
+ContentTypes = Playbook
+
+
+class IsDefaultNotOnlyConditionValidator(BaseValidator[ContentTypes]):
+    error_code = "PB126"
+    description = (
+        "ensure that conditional tasks have more than path which is not the default one"
+    )
+    rationale = (
+        "We want to ensure that conditional tasks have more than path which is not the default one o/w it "
+        "make no sense to have such"
+    )
+    error_message = (
+        "Playbook task with id:{} only has a default option. Please remove this task or add another "
+        "non-default option to the task"
+    )
+    related_field = "conditions"
+    is_auto_fixable = False
+
+    def is_valid(self, content_items: Iterable[ContentTypes]) -> List[ValidationResult]:
+        validation_results = []
+        for content_item in content_items:
+            for task in content_item.tasks.values():
+                if (
+                    task.type == PlaybookTaskType.CONDITION
+                    and task.message
+                    and task.message.get("replyOptions")
+                ):
+                    reply_options = set(
+                        map(str.upper, task.message.get("replyOptions", []))
+                    )
+                    if len(reply_options) == 1 and "#default#".upper() in reply_options:
+                        validation_results.append(
+                            ValidationResult(
+                                validator=self,
+                                message=self.error_message.format(task.id),
+                                content_object=content_item,
+                            )
+                        )
+        return validation_results