Rename User#ci_authorized_runners -> ci_owned_runners

app/controllers/projects/settings/ci_cd_controller.rb

@@ -69,7 +69,7 @@ module Projects
         @project_runners = @project.runners.ordered
         @assignable_runners = current_user
-          .ci_authorized_runners
+          .ci_owned_runners
           .assignable_for(project)
           .ordered
           .page(params[:page]).per(20)

app/models/user.rb

@@ -999,8 +999,8 @@ class User < ActiveRecord::Base
     !solo_owned_groups.present?
   end
-  def ci_authorized_runners
-    @ci_authorized_runners ||= begin
+  def ci_owned_runners
+    @ci_owned_runners ||= begin
       project_runner_ids = Ci::RunnerProject
         .where(project: authorized_projects(Gitlab::Access::MASTER))
         .select(:runner_id)

app/policies/ci/runner_policy.rb

@@ -3,14 +3,14 @@ module Ci
     with_options scope: :subject, score: 0
     condition(:locked, scope: :subject) { @subject.locked? }
-    condition(:authorized_runner) { @user.ci_authorized_runners.exists?(@subject.id) }
+    condition(:owned_runner) { @user.ci_owned_runners.exists?(@subject.id) }
     rule { anonymous }.prevent_all
-    rule { admin | authorized_runner }.enable :assign_runner
-    rule { admin | authorized_runner }.enable :read_runner
-    rule { admin | authorized_runner }.enable :update_runner
-    rule { admin | authorized_runner }.enable :delete_runner
-    rule { admin | authorized_runner }.enable :list_runner_jobs
+    rule { admin | owned_runner }.enable :assign_runner
+    rule { admin | owned_runner }.enable :read_runner
+    rule { admin | owned_runner }.enable :update_runner
+    rule { admin | owned_runner }.enable :delete_runner
+    rule { admin | owned_runner  }.enable :list_runner_jobs
     rule { ~admin & locked }.prevent :assign_runner
   end
 end

lib/api/runners.rb

@@ -14,7 +14,7 @@ module API
         use :pagination
       end
       get do
-        runners = filter_runners(current_user.ci_authorized_runners, params[:scope], without: %w(specific shared))
+        runners = filter_runners(current_user.ci_owned_runners, params[:scope], without: %w(specific shared))
         present paginate(runners), with: Entities::Runner
       end

lib/api/v3/runners.rb

@@ -58,7 +58,7 @@ module API
         end
         def user_can_access_runner?(runner)
-          current_user.ci_authorized_runners.exists?(runner.id)
+          current_user.ci_owned_runners.exists?(runner.id)
         end
       end
     end

spec/models/user_spec.rb

@@ -1786,7 +1786,7 @@ describe User do
     end
   end
-  describe '#ci_authorized_runners' do
+  describe '#ci_owned_runners' do
     let(:user) { create(:user) }
     let(:runner_1) { create(:ci_runner) }
     let(:runner_2) { create(:ci_runner) }
@@ -1796,7 +1796,7 @@ describe User do
       let!(:group) { create(:group) }
       it 'does not load' do
-        expect(user.ci_authorized_runners).to be_empty
+        expect(user.ci_owned_runners).to be_empty
       end
     end
@@ -1805,7 +1805,7 @@ describe User do
       let!(:project) { create(:project, namespace: namespace, runners: [runner_1]) }
       it 'loads' do
-        expect(user.ci_authorized_runners).to contain_exactly(runner_1)
+        expect(user.ci_owned_runners).to contain_exactly(runner_1)
       end
     end
@@ -1818,7 +1818,7 @@ describe User do
       end
       it 'loads' do
-        expect(user.ci_authorized_runners).to contain_exactly(runner_2)
+        expect(user.ci_owned_runners).to contain_exactly(runner_2)
       end
     end
@@ -1833,7 +1833,7 @@ describe User do
       end
       it 'loads' do
-        expect(user.ci_authorized_runners).to contain_exactly(runner_1, runner_2)
+        expect(user.ci_owned_runners).to contain_exactly(runner_1, runner_2)
       end
     end
@@ -1844,7 +1844,7 @@ describe User do
         end
         it 'loads' do
-          expect(user.ci_authorized_runners).to contain_exactly(runner_1)
+          expect(user.ci_owned_runners).to contain_exactly(runner_1)
         end
       end
@@ -1854,7 +1854,7 @@ describe User do
         end
         it 'does not load' do
-          expect(user.ci_authorized_runners).to be_empty
+          expect(user.ci_owned_runners).to be_empty
         end
       end
     end