Added routes for project workflow executions and associated tests

phac-nml · Jan 9, 2025 · 86ca610 · 86ca610
1 parent f5d7650
commit 86ca610
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 1 deletion.
diff --git a/app/controllers/projects/workflow_executions_controller.rb b/app/controllers/projects/workflow_executions_controller.rb
@@ -19,6 +19,10 @@ def workflow_execution
       @workflow_execution = WorkflowExecution.find_by!(id: params[:id], submitter: @project.namespace.automation_bot)
     end
 
+    def workflow_execution_update_params
+      params.require(:workflow_execution).permit(:name)
+    end
+
     def load_workflows
       authorized_scope(WorkflowExecution, type: :relation, as: :automated, scope_options: { project: @project })
     end

diff --git a/config/routes/project.rb b/config/routes/project.rb
@@ -35,7 +35,7 @@
       if Irida::Pipelines.instance.available_pipelines.any?
         resources :automated_workflow_executions
 
-        resources :workflow_executions, only: %i[index destroy show] do
+        resources :workflow_executions, only: %i[index destroy show edit update] do
           member do
             put :cancel
           end

diff --git a/test/controllers/projects/workflow_executions_controller_test.rb b/test/controllers/projects/workflow_executions_controller_test.rb
@@ -214,5 +214,25 @@ class WorkflowExecutionsControllerTest < ActionDispatch::IntegrationTest
 
       assert_redirected_to namespace_project_workflow_executions_path(@namespace, @project)
     end
+
+    test 'analyst or higher access level can update workflow execution name post launch' do
+      update_params = { workflow_execution: { name: 'New Name' } }
+
+      put namespace_project_workflow_execution_path(@namespace, @project, @workflow_execution, format: :turbo_stream),
+                                                    params: update_params
+
+      assert_response :success
+    end
+
+    test 'access level less than analyst cannot update workflow execution name post launch for an automated workflow' do
+      sign_in users(:ryan_doe)
+
+      update_params = { workflow_execution: { name: 'New Name' } }
+
+      put namespace_project_workflow_execution_path(@namespace, @project, @workflow_execution, format: :turbo_stream),
+                                                    params: update_params
+
+      assert_response :unauthorized
+    end
   end
 end