Set the Workflow description to ASL Comment

ManageIQ · Apr 5, 2024 · 9c008e4 · 9c008e4
1 parent 4c83d36
commit 9c008e4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/app/models/manageiq/providers/workflows/automation_manager/configuration_script_source.rb b/app/models/manageiq/providers/workflows/automation_manager/configuration_script_source.rb
@@ -76,22 +76,24 @@ def sync_from_content(to_delete)
   end
 
   def create_workflow_from_payload(name, payload)
-    payload_valid, payload_error =
+    floe_workflow, payload_error =
       begin
         Floe::Workflow.new(payload)
-        true
       rescue Floe::InvalidWorkflowError => err
-        [false, err.message]
+        [nil, err.message]
       end
 
+    description = floe_workflow&.comment
+
     configuration_script_payloads.find_or_initialize_by(:name => name).tap do |wf|
       wf.update!(
         :name          => name,
+        :description   => description,
         :manager_id    => manager_id,
         :type          => self.class.module_parent::Workflow.name,
         :payload       => payload,
         :payload_type  => "json",
-        :payload_valid => payload_valid,
+        :payload_valid => !!floe_workflow,
         :payload_error => payload_error
       )
     end

diff --git a/...odels/manageiq/providers/workflows/automation_manager/configuration_script_source_spec.rb b/...odels/manageiq/providers/workflows/automation_manager/configuration_script_source_spec.rb
@@ -195,6 +195,7 @@ def files_in_repository(git_repo_dir)
 
         expect(record.configuration_script_payloads.first).to have_attributes(
           :name          => "hello_world.asl",
+          :description   => "hello world",
           :payload       => a_string_including("\"Comment\": \"hello world\""),
           :payload_type  => "json",
           :payload_valid => true,