Merge branch 'pedropombeiro/348299/2-graphql-new-mutation' into 'master'

GraphQL: Add maintainer_note to UpdateRunner mutation

See merge request gitlab-org/gitlab!87576

app/graphql/mutations/ci/runner/update.rb

@@ -18,6 +18,10 @@ class Update < BaseMutation
                  required: false,
                  description: 'Description of the runner.'
+        argument :maintenance_note, GraphQL::Types::String,
+                 required: false,
+                 description: 'Runner\'s maintenance notes.'
+
         argument :maximum_timeout, GraphQL::Types::Int,
                  required: false,
                  description: 'Maximum timeout (in seconds) for jobs processed by the runner.'

doc/api/graphql/reference/index.md

@@ -4283,6 +4283,7 @@ Input type: `RunnerUpdateInput`
 | <a id="mutationrunnerupdatedescription"></a>`description` | [`String`](#string) | Description of the runner. |
 | <a id="mutationrunnerupdateid"></a>`id` | [`CiRunnerID!`](#cirunnerid) | ID of the runner to update. |
 | <a id="mutationrunnerupdatelocked"></a>`locked` | [`Boolean`](#boolean) | Indicates the runner is locked. |
+| <a id="mutationrunnerupdatemaintenancenote"></a>`maintenanceNote` | [`String`](#string) | Runner's maintenance notes. |
 | <a id="mutationrunnerupdatemaximumtimeout"></a>`maximumTimeout` | [`Int`](#int) | Maximum timeout (in seconds) for jobs processed by the runner. |
 | <a id="mutationrunnerupdatepaused"></a>`paused` | [`Boolean`](#boolean) | Indicates the runner is not allowed to receive jobs. |
 | <a id="mutationrunnerupdateprivateprojectsminutescostfactor"></a>`privateProjectsMinutesCostFactor` | [`Float`](#float) | Private projects' "minutes cost factor" associated with the runner (GitLab.com only). |

spec/graphql/mutations/ci/runner/update_spec.rb

@@ -49,6 +49,7 @@
         {
           id: runner.to_global_id,
           description: 'updated description',
+          maintenance_note: 'updated maintenance note',
           maximum_timeout: 900,
           access_level: 'ref_protected',
           active: false,
@@ -84,6 +85,16 @@
           )
         end
       end
+
+      context 'with too long maintenance note' do
+        it 'returns a descriptive error' do
+          mutation_params[:maintenance_note] = '1' * 1025
+
+          expect(subject[:errors]).to contain_exactly(
+            'Maintenance note is too long (maximum is 1024 characters)'
+          )
+        end
+      end
     end
   end
 end