Prefer attributes_for_keys so that it ignores nils

Also add a test for setting locked.

1 files changed, 3 insertions, 7 deletions

diff --git a/lib/ci/api/runners.rb b/lib/ci/api/runners.rb
index b4b7261fa3b..bcc82969eb3 100644
--- a/lib/ci/api/runners.rb
+++ b/lib/ci/api/runners.rb
@@ -28,13 +28,9 @@ module Ci
         post "register" do
           required_attributes! [:token]
 
-          attributes = { description: params[:description],
-                         tag_list: params[:tag_list],
-                         locked: !!params[:locked] }
-
-          unless params[:run_untagged].nil?
-            attributes[:run_untagged] = params[:run_untagged]
-          end
+          attributes = attributes_for_keys(
+            [:description, :tag_list, :run_untagged, :locked]
+          )
 
           runner =
             if runner_registration_token_valid?