[Logic] logic workflow update: --definition is not to be mandatory while updating

.github/CODEOWNERS

@@ -104,7 +104,7 @@
 
 /src/storage-or-preview/ @Juliehzl
 
-/src/logic/ @bquantump
+/src/logic/ @jsntcy @msyyc @kairu
 
 /src/hardware-security-modules/ @bquantump
 

src/logic/HISTORY.rst

@@ -3,6 +3,10 @@
 Release History
 ===============
 
+0.1.2
+++++++
+* `logic workflow update`: --definition is not to be mandatory while updating
+
 0.1.1
 ++++++
 * Fix parsing argument --integration-service-environment.

src/logic/azext_logic/azext_metadata.json

@@ -1,4 +1,4 @@
 {
     "azext.isExperimental": true,
-    "azext.minCliCoreVersion": "2.3.1"
+    "azext.minCliCoreVersion": "2.15.0"
 }

src/logic/azext_logic/manual/custom.py

@@ -0,0 +1,37 @@
+# --------------------------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for license information.
+# --------------------------------------------------------------------------------------------
+
+from knack.util import CLIError
+
+
+def logic_workflow_update(cmd,
+                          client,
+                          resource_group_name,
+                          name,
+                          definition=None,
+                          tags=None,
+                          state=None):
+
+    from azure.cli.core.azclierror import ValidationError
+
+    if definition is not None and "definition" not in definition:
+        raise ValidationError(str(definition) + " does not contain a 'definition' key")
+
+    workflow = client.get(resource_group_name=resource_group_name, workflow_name=name)
+
+    from ..generated.custom import logic_workflow_create
+    return logic_workflow_create(
+        cmd,
+        client,
+        resource_group_name,
+        name,
+        definition=definition if definition else {"definition": workflow.definition},
+        location=workflow.location,
+        tags=tags if tags else workflow.tags,
+        state=state if state else workflow.state,
+        endpoints_configuration=workflow.endpoints_configuration,
+        integration_account=workflow.integration_account,
+        integration_service_environment=workflow.integration_service_environment,
+    )

src/logic/azext_logic/tests/latest/test_logic_scenario.py

@@ -96,6 +96,13 @@ def test_logic(self, resource_group):
                  checks=[JMESPathCheck('tags.atag', 123),
                          JMESPathCheck('definition.triggers.When_a_feed_item_is_published.recurrence.interval', 2)])
 
+        self.cmd('az logic workflow update '
+                 '--resource-group "{rg}" '
+                 '--tag atag=foo '
+                 '--name "{testWorkflow}"',
+                 checks=[JMESPathCheck('tags.atag', 'foo'),
+                         JMESPathCheck('definition.triggers.When_a_feed_item_is_published.recurrence.interval', 2)])
+
         self.cmd('az logic workflow delete '
                  '--resource-group "{rg}" '
                  '--name "{testWorkflow}" '

src/logic/setup.py

@@ -16,7 +16,7 @@
 
 # TODO: Confirm this is the right version number you want and it matches your
 # HISTORY.rst entry.
-VERSION = '0.1.1'
+VERSION = '0.1.2'
 
 # The full list of classifiers is available at
 # https://pypi.python.org/pypi?%3Aaction=list_classifiers