Warn users of impacted escalation policies on leave/delete

When a user is attempting to leave a namespace or an admin
is attempting to delete a user, we want to let them know
if there are any escalation policies which notify the
target user directly, so they can mitigate any missed
notifications.

Changelog: added
EE: true

app/assets/javascripts/admin/users/components/actions/delete.vue

@@ -14,7 +14,7 @@ export default {
       type: Object,
       required: true,
     },
-    oncallSchedules: {
+    userDeletionObstacles: {
       type: Array,
       required: false,
       default: () => [],
@@ -29,7 +29,7 @@ export default {
     :username="username"
     :paths="paths"
     :delete-path="paths.delete"
-    :oncall-schedules="oncallSchedules"
+    :user-deletion-obstacles="userDeletionObstacles"
   >
     <slot></slot>
   </shared-delete-action>

app/assets/javascripts/admin/users/components/actions/delete_with_contributions.vue

@@ -14,7 +14,7 @@ export default {
       type: Object,
       required: true,
     },
-    oncallSchedules: {
+    userDeletionObstacles: {
       type: Array,
       required: false,
       default: () => [],
@@ -29,7 +29,7 @@ export default {
     :username="username"
     :paths="paths"
     :delete-path="paths.deleteWithContributions"
-    :oncall-schedules="oncallSchedules"
+    :user-deletion-obstacles="userDeletionObstacles"
   >
     <slot></slot>
   </shared-delete-action>

app/assets/javascripts/admin/users/components/actions/shared/shared_delete_action.vue

@@ -22,7 +22,7 @@ export default {
       type: String,
       required: true,
     },
-    oncallSchedules: {
+    userDeletionObstacles: {
       type: Array,
       required: true,
     },
@@ -34,7 +34,7 @@ export default {
         'data-delete-user-url': this.deletePath,
         'data-gl-modal-action': this.modalType,
         'data-username': this.username,
-        'data-oncall-schedules': JSON.stringify(this.oncallSchedules),
+        'data-user-deletion-obstacles': JSON.stringify(this.userDeletionObstacles),
       };
     },
   },

app/assets/javascripts/admin/users/components/modals/delete_user_modal.vue

@@ -2,7 +2,7 @@
 import { GlModal, GlButton, GlFormInput, GlSprintf } from '@gitlab/ui';
 import * as Sentry from '@sentry/browser';
 import { s__, sprintf } from '~/locale';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
 
 export default {
   components: {
@@ -10,7 +10,7 @@ export default {
     GlButton,
     GlFormInput,
     GlSprintf,
-    OncallSchedulesList,
+    UserDeletionObstaclesList,
   },
   props: {
     title: {
@@ -45,7 +45,7 @@ export default {
       type: String,
       required: true,
     },
-    oncallSchedules: {
+    userDeletionObstacles: {
       type: String,
       required: false,
       default: '[]',
@@ -66,9 +66,9 @@ export default {
     canSubmit() {
       return this.enteredUsername === this.username;
     },
-    schedules() {
+    obstacles() {
       try {
-        return JSON.parse(this.oncallSchedules);
+        return JSON.parse(this.userDeletionObstacles);
       } catch (e) {
         Sentry.captureException(e);
       }
@@ -112,7 +112,11 @@ export default {
       </gl-sprintf>
     </p>
 
-    <oncall-schedules-list v-if="schedules.length" :schedules="schedules" :user-name="username" />
+    <user-deletion-obstacles-list
+      v-if="obstacles.length"
+      :obstacles="obstacles"
+      :user-name="username"
+    />
 
     <p>
       <gl-sprintf :message="s__('AdminUsers|To confirm, type %{username}')">

app/assets/javascripts/admin/users/components/user_actions.vue

@@ -9,6 +9,7 @@ import {
 } from '@gitlab/ui';
 import { convertArrayToCamelCase } from '~/lib/utils/common_utils';
 import { capitalizeFirstCharacter } from '~/lib/utils/text_utility';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
 import { I18N_USER_ACTIONS } from '../constants';
 import { generateUserPaths } from '../utils';
 import Actions from './actions';
@@ -72,6 +73,9 @@ export default {
         href: this.userPaths.edit,
       };
     },
+    obstaclesForUserDeletion() {
+      return parseUserDeletionObstacles(this.user);
+    },
   },
   methods: {
     isLdapAction(action) {
@@ -141,7 +145,7 @@ export default {
             :key="action"
             :paths="userPaths"
             :username="user.name"
-            :oncall-schedules="user.oncallSchedules"
+            :user-deletion-obstacles="obstaclesForUserDeletion"
             :data-testid="`delete-${action}`"
           >
             {{ $options.i18n[action] }}

app/assets/javascripts/members/components/action_buttons/remove_member_button.vue

@@ -42,7 +42,7 @@ export default {
       required: false,
       default: false,
     },
-    oncallSchedules: {
+    userDeletionObstacles: {
       type: Object,
       required: false,
       default: () => ({}),
@@ -61,7 +61,7 @@ export default {
         memberPath: this.memberPath.replace(':id', this.memberId),
         memberType: this.memberType,
         message: this.message,
-        oncallSchedules: this.oncallSchedules,
+        userDeletionObstacles: this.userDeletionObstacles,
       };
     },
   },

app/assets/javascripts/members/components/action_buttons/user_action_buttons.vue

 <script>
 import { s__, sprintf } from '~/locale';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
 import ActionButtonGroup from './action_button_group.vue';
 import LeaveButton from './leave_button.vue';
 import RemoveMemberButton from './remove_member_button.vue';
@@ -49,9 +50,11 @@ export default {
         },
       );
     },
-    oncallScheduleUserData() {
-      const { user: { name, oncallSchedules: schedules } = {} } = this.member;
-      return { name, schedules };
+    userDeletionObstaclesUserData() {
+      return {
+        name: this.member.user?.name,
+        obstacles: parseUserDeletionObstacles(this.member.user),
+      };
     },
   },
 };
@@ -65,7 +68,7 @@ export default {
         v-else
         :member-id="member.id"
         :member-type="member.type"
-        :oncall-schedules="oncallScheduleUserData"
+        :user-deletion-obstacles="userDeletionObstaclesUserData"
         :message="message"
         :title="s__('Member|Remove member')"
       />

app/assets/javascripts/members/components/modals/leave_modal.vue

@@ -3,7 +3,8 @@ import { GlModal, GlForm, GlSprintf, GlTooltipDirective } from '@gitlab/ui';
 import { mapState } from 'vuex';
 import csrf from '~/lib/utils/csrf';
 import { __, s__, sprintf } from '~/locale';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
 import { LEAVE_MODAL_ID } from '../../constants';
 
 export default {
@@ -20,7 +21,7 @@ export default {
   csrf,
   modalId: LEAVE_MODAL_ID,
   modalContent: s__('Members|Are you sure you want to leave "%{source}"?'),
-  components: { GlModal, GlForm, GlSprintf, OncallSchedulesList },
+  components: { GlModal, GlForm, GlSprintf, UserDeletionObstaclesList },
   directives: {
     GlTooltip: GlTooltipDirective,
   },
@@ -43,11 +44,11 @@ export default {
     modalTitle() {
       return sprintf(s__('Members|Leave "%{source}"'), { source: this.member.source.fullName });
     },
-    schedules() {
-      return this.member.user?.oncallSchedules;
+    obstacles() {
+      return parseUserDeletionObstacles(this.member.user);
     },
-    isPartOfOnCallSchedules() {
-      return this.schedules?.length;
+    hasObstaclesToUserDeletion() {
+      return this.obstacles?.length;
     },
   },
   methods: {
@@ -74,9 +75,9 @@ export default {
         </gl-sprintf>
       </p>
 
-      <oncall-schedules-list
-        v-if="isPartOfOnCallSchedules"
-        :schedules="schedules"
+      <user-deletion-obstacles-list
+        v-if="hasObstaclesToUserDeletion"
+        :obstacles="obstacles"
         :is-current-user="true"
       />
 

app/assets/javascripts/members/components/modals/remove_member_modal.vue

@@ -3,7 +3,7 @@ import { GlFormCheckbox, GlModal } from '@gitlab/ui';
 import { mapActions, mapState } from 'vuex';
 import csrf from '~/lib/utils/csrf';
 import { s__, __ } from '~/locale';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
 
 export default {
   actionCancel: {
@@ -13,7 +13,7 @@ export default {
   components: {
     GlFormCheckbox,
     GlModal,
-    OncallSchedulesList,
+    UserDeletionObstaclesList,
   },
   inject: ['namespace'],
   computed: {
@@ -33,8 +33,8 @@ export default {
       message(state) {
         return state[this.namespace].removeMemberModalData.message;
       },
-      oncallSchedules(state) {
-        return state[this.namespace].removeMemberModalData.oncallSchedules ?? {};
+      userDeletionObstacles(state) {
+        return state[this.namespace].removeMemberModalData.userDeletionObstacles ?? {};
       },
       removeMemberModalVisible(state) {
         return state[this.namespace].removeMemberModalVisible;
@@ -60,11 +60,11 @@ export default {
         },
       };
     },
-    showUnassignIssuablesCheckbox() {
+    hasWorkspaceAccess() {
       return !this.isAccessRequest && !this.isInvite;
     },
-    isPartOfOncallSchedules() {
-      return !this.isAccessRequest && this.oncallSchedules.schedules?.length;
+    hasObstaclesToUserDeletion() {
+      return this.hasWorkspaceAccess && this.userDeletionObstacles.obstacles?.length;
     },
   },
   methods: {
@@ -95,10 +95,10 @@ export default {
     <form ref="form" :action="memberPath" method="post">
       <p>{{ message }}</p>
 
-      <oncall-schedules-list
-        v-if="isPartOfOncallSchedules"
-        :schedules="oncallSchedules.schedules"
-        :user-name="oncallSchedules.name"
+      <user-deletion-obstacles-list
+        v-if="hasObstaclesToUserDeletion"
+        :obstacles="userDeletionObstacles.obstacles"
+        :user-name="userDeletionObstacles.name"
       />
 
       <input ref="method" type="hidden" name="_method" value="delete" />
@@ -106,7 +106,7 @@ export default {
       <gl-form-checkbox v-if="isGroupMember" name="remove_sub_memberships">
         {{ __('Also remove direct user membership from subgroups and projects') }}
       </gl-form-checkbox>
-      <gl-form-checkbox v-if="showUnassignIssuablesCheckbox" name="unassign_issuables">
+      <gl-form-checkbox v-if="hasWorkspaceAccess" name="unassign_issuables">
         {{ __('Also unassign this user from related issues and merge requests') }}
       </gl-form-checkbox>
     </form>

app/assets/javascripts/vue_shared/components/user_deletion_obstacles/constants.js

+// Types of obstacles to user deletion
+export const OBSTACLE_TYPES = Object.freeze({
+  oncallSchedules: 'ONCALL_SCHEDULE',
+  escalationPolicies: 'ESCALATION_POLICY',
+});

app/assets/javascripts/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.stories.js

+/* eslint-disable @gitlab/require-i18n-strings */
+
+import { OBSTACLE_TYPES } from './constants';
+import UserDeletionObstaclesList from './user_deletion_obstacles_list.vue';
+
+export default {
+  component: UserDeletionObstaclesList,
+  title: 'vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list',
+};
+
+const Template = (args, { argTypes }) => ({
+  components: { UserDeletionObstaclesList },
+  props: Object.keys(argTypes),
+  template: '<user-deletion-obstacles-list v-bind="$props" v-on="$props" />',
+});
+
+export const Default = Template.bind({});
+Default.args = {
+  obstacles: [
+    {
+      type: OBSTACLE_TYPES.oncallSchedules,
+      name: 'APAC',
+      url: 'https://domain.com/group/main-application/oncall_schedules',
+      projectName: 'main-application',
+      projectUrl: 'https://domain.com/group/main-application',
+    },
+    {
+      type: OBSTACLE_TYPES.escalationPolicies,
+      name: 'Engineering On-call',
+      url: 'https://domain.com/group/microservice-backend/escalation_policies',
+      projectName: 'Microservice Backend',
+      projectUrl: 'https://domain.com/group/microservice-backend',
+    },
+  ],
+  userName: 'Thomspon Smith',
+  isCurrentUser: false,
+};

app/assets/javascripts/vue_shared/components/oncall_schedules_list.vue → app/assets/javascripts/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue

 <script>
 import { GlSprintf, GlLink } from '@gitlab/ui';
 import { sprintf, s__ } from '~/locale';
+import { OBSTACLE_TYPES } from './constants';
+
+const OBSTACLE_TEXT = {
+  [OBSTACLE_TYPES.oncallSchedules]: s__(
+    'OnCallSchedules|On-call schedule %{obstacle} in Project %{project}',
+  ),
+  [OBSTACLE_TYPES.escalationPolicies]: s__(
+    'EscalationPolicies|Escalation policy %{obstacle} in Project %{project}',
+  ),
+};
 
 export default {
   components: {
@@ -8,7 +18,7 @@ export default {
     GlLink,
   },
   props: {
-    schedules: {
+    obstacles: {
       type: Array,
       required: true,
     },
@@ -45,6 +55,15 @@ export default {
           );
     },
   },
+  methods: {
+    textForObstacle(obstacle) {
+      return OBSTACLE_TEXT[obstacle.type];
+    },
+    urlForObstacle(obstacle) {
+      // Fallback to scheduleUrl for backwards compatibility
+      return obstacle.url || obstacle.scheduleUrl;
+    },
+  },
 };
 </script>
 
@@ -52,17 +71,15 @@ export default {
   <div>
     <p data-testid="title">{{ title }}</p>
 
-    <ul data-testid="schedules-list">
-      <li v-for="(schedule, index) in schedules" :key="`${schedule.name}-${index}`">
-        <gl-sprintf
-          :message="s__('OnCallSchedules|On-call schedule %{schedule} in Project %{project}')"
-        >
-          <template #schedule>
-            <gl-link :href="schedule.scheduleUrl" target="_blank">{{ schedule.name }}</gl-link>
+    <ul data-testid="obstacles-list">
+      <li v-for="(obstacle, index) in obstacles" :key="`${obstacle.name}-${index}`">
+        <gl-sprintf :message="textForObstacle(obstacle)">
+          <template #obstacle>
+            <gl-link :href="urlForObstacle(obstacle)" target="_blank">{{ obstacle.name }}</gl-link>
           </template>
           <template #project>
-            <gl-link :href="schedule.projectUrl" target="_blank">{{
-              schedule.projectName
+            <gl-link :href="obstacle.projectUrl" target="_blank">{{
+              obstacle.projectName
             }}</gl-link>
           </template>
         </gl-sprintf>

app/assets/javascripts/vue_shared/components/user_deletion_obstacles/utils.js

+import { OBSTACLE_TYPES } from './constants';
+
+const addTypeToObstacles = (obstacles, type) => {
+  if (!obstacles) return [];
+
+  return obstacles?.map((obstacle) => ({ type, ...obstacle }));
+};
+
+// For use with user objects formatted via internal REST API.
+// If the removal/deletion of a user could cause critical
+// problems, return a single array containing all affected
+// associations including their type.
+export const parseUserDeletionObstacles = (user) => {
+  if (!user) return [];
+
+  return Object.keys(OBSTACLE_TYPES).flatMap((type) => {
+    return addTypeToObstacles(user[type], OBSTACLE_TYPES[type]);
+  });
+};

ee/app/controllers/ee/admin/users_controller.rb

@@ -21,7 +21,7 @@ module EE
 
       override :users_with_included_associations
       def users_with_included_associations(users)
-        super.includes(:oncall_schedules) # rubocop: disable CodeReuse/ActiveRecord
+        super.includes(:oncall_schedules, :escalation_policies) # rubocop: disable CodeReuse/ActiveRecord
       end
 
       override :log_impersonation_event

ee/app/models/ee/members_preloader.rb

@@ -9,8 +9,8 @@ module EE
       super
 
       ActiveRecord::Associations::Preloader.new.preload(members, user: { group_saml_identities: :saml_provider })
-      ActiveRecord::Associations::Preloader.new.preload(members, user: { oncall_participants: { rotation: :schedule } })
       ActiveRecord::Associations::Preloader.new.preload(members, user: :oncall_schedules)
+      ActiveRecord::Associations::Preloader.new.preload(members, user: :escalation_policies)
       ActiveRecord::Associations::Preloader.new.preload(members, user: :user_detail)
     end
   end

ee/app/models/ee/user.rb

@@ -74,9 +74,11 @@ module EE
 
       has_many :user_permission_export_uploads
 
-      has_many :oncall_participants, class_name: 'IncidentManagement::OncallParticipant', inverse_of: :user
+      has_many :oncall_participants, -> { not_removed }, class_name: 'IncidentManagement::OncallParticipant', inverse_of: :user
       has_many :oncall_rotations, class_name: 'IncidentManagement::OncallRotation', through: :oncall_participants, source: :rotation
-      has_many :oncall_schedules, class_name: 'IncidentManagement::OncallSchedule', through: :oncall_rotations, source: :schedule
+      has_many :oncall_schedules, -> { distinct }, class_name: 'IncidentManagement::OncallSchedule', through: :oncall_rotations, source: :schedule
+      has_many :escalation_rules, -> { not_removed }, class_name: 'IncidentManagement::EscalationRule', inverse_of: :user
+      has_many :escalation_policies, -> { distinct }, class_name: 'IncidentManagement::EscalationPolicy', through: :escalation_rules, source: :policy
 
       scope :not_managed, ->(group: nil) {
         scope = where(managing_group_id: nil)

ee/app/models/incident_management/escalation_policy.rb

@@ -12,6 +12,10 @@ module IncidentManagement
     validates :name, presence: true, uniqueness: { scope: [:project_id] }, length: { maximum: 72 }
     validates :description, length: { maximum: 160 }
 
+    scope :for_project, -> (project) { where(project: project) }
+
     accepts_nested_attributes_for :rules
+
+    delegate :name, to: :project, prefix: true
   end
 end

ee/app/serializers/ee/admin/user_entity.rb

@@ -7,10 +7,7 @@ module EE
 
       prepended do
         expose :oncall_schedules, with: ::IncidentManagement::OncallScheduleEntity
-
-        def oncall_schedules
-          object.oncall_schedules.uniq
-        end
+        expose :escalation_policies, with: ::IncidentManagement::EscalationPolicyEntity
       end
     end
   end

ee/app/serializers/ee/member_user_entity.rb

@@ -3,20 +3,32 @@
 module EE
   module MemberUserEntity
     extend ActiveSupport::Concern
+    include ::Gitlab::Utils::StrongMemoize
 
     prepended do
       unexpose :gitlab_employee
       unexpose :email
       expose :oncall_schedules, with: ::IncidentManagement::OncallScheduleEntity
+      expose :escalation_policies, with: ::IncidentManagement::EscalationPolicyEntity
 
-      # options[:source] is required to scope the schedules
-      # It should be either a Group or Project
       def oncall_schedules
-        return [] unless options[:source].present?
+        object.oncall_schedules.for_project(project_ids)
+      end
+
+      def escalation_policies
+        object.escalation_policies.for_project(project_ids)
+      end
+    end
+
+    private
 
-        project_ids = options[:source].is_a?(Group) ? options[:source].project_ids : [options[:source].id]
+    # options[:source] is required to scope oncall schedules or policies
+    # It should be either a Group or Project
+    def project_ids
+      strong_memoize(:project_ids) do
+        next [] unless options[:source].present?
 
-        object.oncall_schedules.select { |schedule| project_ids.include?(schedule.project_id) }
+        options[:source].is_a?(Group) ? options[:source].project_ids : [options[:source].id]
       end
     end
   end

ee/app/serializers/incident_management/escalation_policy_entity.rb

+# frozen_string_literal: true
+
+module IncidentManagement
+  class EscalationPolicyEntity < Grape::Entity
+    include ::Gitlab::Routing
+
+    expose :name
+    expose :url do |policy|
+      project_incident_management_escalation_policies_url(policy.project)
+    end
+
+    expose :project_name
+    expose :project_url do |policy|
+      project_url(policy.project)
+    end
+  end
+end

ee/app/serializers/incident_management/oncall_schedule_entity.rb

@@ -5,11 +5,14 @@ module IncidentManagement
     include Gitlab::Routing
 
     expose :name
-    expose :project_name
-    expose :schedule_url do |schedule|
+    expose :schedule_url do |schedule| # for backwards compatibility
+      project_incident_management_oncall_schedules_url(schedule.project)
+    end
+    expose :url do |schedule|
       project_incident_management_oncall_schedules_url(schedule.project)
     end
 
+    expose :project_name
     expose :project_url do |schedule|
       project_url(schedule.project)
     end

ee/spec/controllers/admin/users_controller_spec.rb

@@ -10,6 +10,15 @@ RSpec.describe Admin::UsersController do
     sign_in(admin)
   end
 
+  describe 'GET #index' do
+    it 'eager loads obstacles to user deletion' do
+      get :index
+
+      expect(assigns(:users).first.association(:oncall_schedules)).to be_loaded
+      expect(assigns(:users).first.association(:escalation_policies)).to be_loaded
+    end
+  end
+
   describe 'POST update' do
     context 'updating name' do
       shared_examples_for 'admin can update the name of a user' do

ee/spec/models/ee/members_preloader_spec.rb

@@ -3,25 +3,44 @@
 require 'spec_helper'
 
 RSpec.describe EE::MembersPreloader do
+  include OncallHelpers
+
   describe '#preload_all' do
-    let(:group) { create(:group) }
-    let(:saml_provider) { create(:saml_provider, group: group) }
+    let_it_be(:group) { create(:group) }
+    let_it_be(:project) { create(:project, group: group) }
+    let_it_be(:saml_provider) { create(:saml_provider, group: group) }
+    let_it_be(:escalation_policy) { create(:incident_management_escalation_policy, project: project, rule_count: 0) }
+
+    it 'preloads associations to avoid N+1 queries' do
+      member = create(:group_member, group: group)
+      create_member_associations(member)
+
+      control = ActiveRecord::QueryRecorder.new { access_group_with_preload([member]) }
 
-    def group_sso_with_preload(members)
+      members = create_list(:group_member, 3, group: group)
+      create_member_associations(members.first)
+      create_member_associations(members.last)
+
+      expect { access_group_with_preload(members) }.not_to exceed_query_limit(control)
+    end
+
+    def access_group_with_preload(members)
       MembersPreloader.new(members).preload_all
       MembersPresenter.new(members, current_user: nil).map(&:group_sso?)
+
+      members.each do |member|
+        member.user.oncall_schedules.any?
+        member.user.escalation_policies.any?
+        member.user.user_detail
+      end
     end
 
-    it 'preloads SAML identities to avoid N+1 queries in MembersPresenter' do
-      member = create(:group_member, group: group)
+    def create_member_associations(member)
       create(:group_saml_identity, user: member.user, saml_provider: saml_provider)
-      control = ActiveRecord::QueryRecorder.new { group_sso_with_preload([member]) }
-
-      members = create_list(:group_member, 3, group: group)
-      create(:group_saml_identity, user: members.first.user, saml_provider: saml_provider)
-      create(:group_saml_identity, user: members.last.user, saml_provider: saml_provider)
-
-      expect { group_sso_with_preload(members) }.not_to exceed_query_limit(control)
+      create_schedule_with_user(project, member.user)
+      create(:incident_management_escalation_rule, :with_user, policy: escalation_policy, user: member.user)
+      member.user.user_detail.save!
+      member.reload
     end
   end
 end

ee/spec/models/ee/user_spec.rb

@@ -31,6 +31,8 @@ RSpec.describe User do
     it { is_expected.to have_many(:oncall_participants).class_name('IncidentManagement::OncallParticipant') }
     it { is_expected.to have_many(:oncall_rotations).class_name('IncidentManagement::OncallRotation').through(:oncall_participants) }
     it { is_expected.to have_many(:oncall_schedules).class_name('IncidentManagement::OncallSchedule').through(:oncall_rotations) }
+    it { is_expected.to have_many(:escalation_rules).class_name('IncidentManagement::EscalationRule') }
+    it { is_expected.to have_many(:escalation_policies).class_name('IncidentManagement::EscalationPolicy').through(:escalation_rules) }
     it { is_expected.to have_many(:epic_board_recent_visits).inverse_of(:user) }
   end
 
@@ -1841,4 +1843,42 @@ RSpec.describe User do
       is_expected.to be(false)
     end
   end
+
+  describe '.oncall_schedules' do
+    let_it_be(:user) { create(:user) }
+    let_it_be(:participant, reload: true) { create(:incident_management_oncall_participant, user: user) }
+    let_it_be(:schedule, reload: true) { participant.rotation.schedule }
+
+    it 'excludes removed participants' do
+      participant.update!(is_removed: true)
+
+      expect(user.oncall_schedules).to be_empty
+    end
+
+    it 'excludes duplicates' do
+      create(:incident_management_oncall_rotation, schedule: schedule) do |rotation|
+        create(:incident_management_oncall_participant, user: user, rotation: rotation)
+      end
+
+      expect(user.oncall_schedules).to contain_exactly(schedule)
+    end
+  end
+
+  describe '.escalation_policies' do
+    let_it_be(:rule, reload: true) { create(:incident_management_escalation_rule, :with_user) }
+    let_it_be(:policy, reload: true) { rule.policy }
+    let_it_be(:user) { rule.user }
+
+    it 'excludes removed rules' do
+      rule.update!(is_removed: true)
+
+      expect(user.escalation_policies).to be_empty
+    end
+
+    it 'excludes duplicates' do
+      create(:incident_management_escalation_rule, :with_user, :resolved, policy: policy, user: user)
+
+      expect(user.escalation_policies).to contain_exactly(policy)
+    end
+  end
 end

ee/spec/models/incident_management/escalation_policy_spec.rb

@@ -30,4 +30,20 @@ RSpec.describe IncidentManagement::EscalationPolicy do
     it { is_expected.to validate_length_of(:name).is_at_most(72) }
     it { is_expected.to validate_length_of(:description).is_at_most(160) }
   end
+
+  describe 'delegations' do
+    it { is_expected.to delegate_method(:name).to(:project).with_prefix }
+  end
+
+  describe 'scopes' do
+    let_it_be(:project) { create(:project) }
+    let_it_be(:policy) { create(:incident_management_escalation_policy, project: project) }
+    let_it_be(:other_policy) { create(:incident_management_escalation_policy) }
+
+    describe '.for_project' do
+      subject { described_class.for_project(project) }
+
+      it { is_expected.to contain_exactly(policy) }
+    end
+  end
 end

ee/spec/serializers/ee/admin/user_entity_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe Admin::UserEntity do
     subject { entity.as_json&.keys }
 
     it 'exposes correct attributes' do
-      is_expected.to include(:oncall_schedules)
+      is_expected.to include(:oncall_schedules, :escalation_policies)
     end
   end
 end

ee/spec/serializers/incident_management/escalation_policy_entity_spec.rb

+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe IncidentManagement::EscalationPolicyEntity do
+  let(:policy) { create(:incident_management_escalation_policy) }
+  let(:url) { Gitlab::Routing.url_helpers.project_incident_management_escalation_policies_url(policy.project) }
+  let(:project_url) { Gitlab::Routing.url_helpers.project_url(policy.project) }
+
+  subject { described_class.new(policy) }
+
+  describe '.as_json' do
+    it 'includes escalation policy attributes' do
+      attributes = subject.as_json
+
+      expect(attributes[:name]).to eq(policy.name)
+      expect(attributes[:url]).to eq(url)
+      expect(attributes[:project_name]).to eq(policy.project.name)
+      expect(attributes[:project_url]).to eq(project_url)
+    end
+  end
+end

ee/spec/serializers/incident_management/oncall_schedule_entity_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe IncidentManagement::OncallScheduleEntity do
 
       expect(attributes[:name]).to eq(schedule.name)
       expect(attributes[:project_name]).to eq(schedule.project.name)
-      expect(attributes[:schedule_url]).to eq(schedule_url)
+      expect(attributes[:url]).to eq(schedule_url)
       expect(attributes[:project_url]).to eq(project_url)
     end
   end

ee/spec/serializers/member_user_entity_spec.rb

@@ -15,44 +15,84 @@ RSpec.describe MemberUserEntity do
     expect(entity.to_json).to match_schema('entities/member_user')
   end
 
-  context 'with oncall schedules' do
+  context 'when using on-call management' do
     let_it_be(:group) { create(:group) }
     let_it_be(:project_1) { create(:project, group: group )}
     let_it_be(:project_2) { create(:project, group: group )}
 
-    let_it_be(:oncall_schedule_1) { create_schedule_with_user(project_1, user) }
-    let_it_be(:oncall_schedule_2) { create_schedule_with_user(project_2, user) }
+    context 'with oncall schedules' do
+      let_it_be(:oncall_schedule_1) { create_schedule_with_user(project_1, user) }
+      let_it_be(:oncall_schedule_2) { create_schedule_with_user(project_2, user) }
 
-    it 'returns an empty array if no source option is given' do
-      expect(entity_hash[:oncall_schedules]).to eq []
-    end
+      subject { entity_hash[:oncall_schedules] }
+
+      context 'with no source given' do
+        it { is_expected.to eq [] }
+      end
+
+      context 'source is project' do
+        let(:source) { project_1 }
+
+        it { is_expected.to contain_exactly(expected_hash(oncall_schedule_1)) }
+      end
 
-    context 'source is project' do
-      let(:source) { project_1 }
+      context 'source is group' do
+        let(:source) { group }
+
+        it { is_expected.to contain_exactly(expected_hash(oncall_schedule_1), expected_hash(oncall_schedule_2)) }
+      end
+
+      private
+
+      def get_url(schedule)
+        Gitlab::Routing.url_helpers.project_incident_management_oncall_schedules_url(schedule.project)
+      end
 
-      it 'correctly exposes `oncall_schedules`' do
-        expect(entity_hash[:oncall_schedules]).to contain_exactly(schedule_hash(oncall_schedule_1))
+      def expected_hash(schedule)
+        # for backwards compatibility
+        super.merge(schedule_url: get_url(schedule))
       end
     end
 
-    context 'source is group' do
-      let(:source) { group }
+    context 'with escalation policies' do
+      let_it_be(:policy_1) { create(:incident_management_escalation_policy, project: project_1, rule_count: 0) }
+      let_it_be(:rule_1) { create(:incident_management_escalation_rule, :with_user, policy: policy_1, user: user) }
+      let_it_be(:policy_2) { create(:incident_management_escalation_policy, project: project_2, rule_count: 0) }
+      let_it_be(:rule_2) { create(:incident_management_escalation_rule, :with_user, policy: policy_2, user: user) }
+
+      subject { entity_hash[:escalation_policies] }
+
+      context 'with no source given' do
+        it { is_expected.to eq [] }
+      end
+
+      context 'source is project' do
+        let(:source) { project_1 }
+
+        it { is_expected.to contain_exactly(expected_hash(policy_1)) }
+      end
+
+      context 'source is group' do
+        let(:source) { group }
+
+        it { is_expected.to contain_exactly(expected_hash(policy_1), expected_hash(policy_2)) }
+      end
+
+      private
 
-      it 'correctly exposes `oncall_schedules`' do
-        expect(entity_hash[:oncall_schedules]).to contain_exactly(schedule_hash(oncall_schedule_1), schedule_hash(oncall_schedule_2))
+      def get_url(policy)
+        Gitlab::Routing.url_helpers.project_incident_management_escalation_policies_url(policy.project)
       end
     end
 
     private
 
-    def schedule_hash(schedule)
-      schedule_url = Gitlab::Routing.url_helpers.project_incident_management_oncall_schedules_url(schedule.project)
-      project_url = Gitlab::Routing.url_helpers.project_url(schedule.project)
+    def expected_hash(oncall_object)
       {
-        name: schedule.name,
-        project_name: schedule.project.name,
-        schedule_url: schedule_url,
-        project_url: project_url
+        name: oncall_object.name,
+        url: get_url(oncall_object),
+        project_name: oncall_object.project.name,
+        project_url: Gitlab::Routing.url_helpers.project_url(oncall_object.project)
       }
     end
   end

locale/gitlab.pot

@@ -13531,6 +13531,9 @@ msgstr ""
 msgid "EscalationPolicies|Escalation policies"
 msgstr ""
 
+msgid "EscalationPolicies|Escalation policy %{obstacle} in Project %{project}"
+msgstr ""
+
 msgid "EscalationPolicies|Escalation rules"
 msgstr ""
 
@@ -23534,7 +23537,7 @@ msgstr ""
 msgid "OnCallSchedules|For this rotation, on-call will be:"
 msgstr ""
 
-msgid "OnCallSchedules|On-call schedule %{schedule} in Project %{project}"
+msgid "OnCallSchedules|On-call schedule %{obstacle} in Project %{project}"
 msgstr ""
 
 msgid "OnCallSchedules|On-call schedules"

spec/frontend/admin/users/components/actions/actions_spec.js

@@ -5,6 +5,7 @@ import { nextTick } from 'vue';
 import Actions from '~/admin/users/components/actions';
 import SharedDeleteAction from '~/admin/users/components/actions/shared/shared_delete_action.vue';
 import { capitalizeFirstCharacter } from '~/lib/utils/text_utility';
+import { OBSTACLE_TYPES } from '~/vue_shared/components/user_deletion_obstacles/constants';
 import { CONFIRMATION_ACTIONS, DELETE_ACTIONS } from '../../constants';
 import { paths } from '../../mock_data';
 
@@ -46,7 +47,10 @@ describe('Action components', () => {
   });
 
   describe('DELETE_ACTION_COMPONENTS', () => {
-    const oncallSchedules = [{ name: 'schedule1' }, { name: 'schedule2' }];
+    const userDeletionObstacles = [
+      { name: 'schedule1', type: OBSTACLE_TYPES.oncallSchedules },
+      { name: 'policy1', type: OBSTACLE_TYPES.escalationPolicies },
+    ];
 
     it.each(DELETE_ACTIONS.map((action) => [action, paths[action]]))(
       'renders a dropdown item for "%s"',
@@ -56,7 +60,7 @@ describe('Action components', () => {
           props: {
             username: 'John Doe',
             paths,
-            oncallSchedules,
+            userDeletionObstacles,
           },
           stubs: { SharedDeleteAction },
         });
@@ -69,8 +73,8 @@ describe('Action components', () => {
         expect(sharedAction.attributes('data-delete-user-url')).toBe(expectedPath);
         expect(sharedAction.attributes('data-gl-modal-action')).toBe(kebabCase(action));
         expect(sharedAction.attributes('data-username')).toBe('John Doe');
-        expect(sharedAction.attributes('data-oncall-schedules')).toBe(
-          JSON.stringify(oncallSchedules),
+        expect(sharedAction.attributes('data-user-deletion-obstacles')).toBe(
+          JSON.stringify(userDeletionObstacles),
         );
         expect(findDropdownItem().exists()).toBe(true);
       },

spec/frontend/admin/users/components/modals/__snapshots__/delete_user_modal_spec.js.snap

@@ -8,8 +8,8 @@ exports[`User Operation confirmation modal renders modal with form included 1`]
     />
   </p>
    
-  <oncall-schedules-list-stub
-    schedules="schedule1,schedule2"
+  <user-deletion-obstacles-list-stub
+    obstacles="schedule1,policy1"
     username="username"
   />
    

spec/frontend/admin/users/components/modals/delete_user_modal_spec.js

 import { GlButton, GlFormInput } from '@gitlab/ui';
 import { shallowMount } from '@vue/test-utils';
 import DeleteUserModal from '~/admin/users/components/modals/delete_user_modal.vue';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
 import ModalStub from './stubs/modal_stub';
 
 const TEST_DELETE_USER_URL = 'delete-url';
@@ -25,7 +25,7 @@ describe('User Operation confirmation modal', () => {
   const getUsername = () => findUsernameInput().attributes('value');
   const getMethodParam = () => new FormData(findForm().element).get('_method');
   const getFormAction = () => findForm().attributes('action');
-  const findOnCallSchedulesList = () => wrapper.findComponent(OncallSchedulesList);
+  const findUserDeletionObstaclesList = () => wrapper.findComponent(UserDeletionObstaclesList);
 
   const setUsername = (username) => {
     findUsernameInput().vm.$emit('input', username);
@@ -33,7 +33,7 @@ describe('User Operation confirmation modal', () => {
 
   const username = 'username';
   const badUsername = 'bad_username';
-  const oncallSchedules = '["schedule1", "schedule2"]';
+  const userDeletionObstacles = '["schedule1", "policy1"]';
 
   const createComponent = (props = {}) => {
     wrapper = shallowMount(DeleteUserModal, {
@@ -46,7 +46,7 @@ describe('User Operation confirmation modal', () => {
         deleteUserUrl: TEST_DELETE_USER_URL,
         blockUserUrl: TEST_BLOCK_USER_URL,
         csrfToken: TEST_CSRF,
-        oncallSchedules,
+        userDeletionObstacles,
         ...props,
       },
       stubs: {
@@ -150,18 +150,18 @@ describe('User Operation confirmation modal', () => {
     });
   });
 
-  describe('Related oncall-schedules list', () => {
-    it('does NOT render the list when user has no related schedules', () => {
-      createComponent({ oncallSchedules: '[]' });
-      expect(findOnCallSchedulesList().exists()).toBe(false);
+  describe('Related user-deletion-obstacles list', () => {
+    it('does NOT render the list when user has no related obstacles', () => {
+      createComponent({ userDeletionObstacles: '[]' });
+      expect(findUserDeletionObstaclesList().exists()).toBe(false);
     });
 
-    it('renders the list when user has related schedules', () => {
+    it('renders the list when user has related obstalces', () => {
       createComponent();
 
-      const schedules = findOnCallSchedulesList();
-      expect(schedules.exists()).toBe(true);
-      expect(schedules.props('schedules')).toEqual(JSON.parse(oncallSchedules));
+      const obstacles = findUserDeletionObstaclesList();
+      expect(obstacles.exists()).toBe(true);
+      expect(obstacles.props('obstacles')).toEqual(JSON.parse(userDeletionObstacles));
     });
   });
 });

spec/frontend/members/components/action_buttons/remove_member_button_spec.js

@@ -45,7 +45,7 @@ describe('RemoveMemberButton', () => {
         title: 'Remove member',
         isAccessRequest: true,
         isInvite: true,
-        oncallSchedules: { name: 'user', schedules: [] },
+        userDeletionObstacles: { name: 'user', obstacles: [] },
         ...propsData,
       },
       directives: {

spec/frontend/members/components/action_buttons/user_action_buttons_spec.js

@@ -2,6 +2,7 @@ import { shallowMount } from '@vue/test-utils';
 import LeaveButton from '~/members/components/action_buttons/leave_button.vue';
 import RemoveMemberButton from '~/members/components/action_buttons/remove_member_button.vue';
 import UserActionButtons from '~/members/components/action_buttons/user_action_buttons.vue';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
 import { member, orphanedMember } from '../../mock_data';
 
 describe('UserActionButtons', () => {
@@ -45,9 +46,9 @@ describe('UserActionButtons', () => {
         isAccessRequest: false,
         isInvite: false,
         icon: 'remove',
-        oncallSchedules: {
+        userDeletionObstacles: {
           name: member.user.name,
-          schedules: member.user.oncallSchedules,
+          obstacles: parseUserDeletionObstacles(member.user),
         },
       });
     });

spec/frontend/members/components/modals/leave_modal_spec.js

@@ -6,7 +6,8 @@ import { nextTick } from 'vue';
 import Vuex from 'vuex';
 import LeaveModal from '~/members/components/modals/leave_modal.vue';
 import { LEAVE_MODAL_ID, MEMBER_TYPES } from '~/members/constants';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
 import { member } from '../../mock_data';
 
 jest.mock('~/lib/utils/csrf', () => ({ token: 'mock-csrf-token' }));
@@ -51,7 +52,7 @@ describe('LeaveModal', () => {
 
   const findModal = () => wrapper.findComponent(GlModal);
   const findForm = () => findModal().findComponent(GlForm);
-  const findOncallSchedulesList = () => findModal().findComponent(OncallSchedulesList);
+  const findUserDeletionObstaclesList = () => findModal().findComponent(UserDeletionObstaclesList);
 
   const getByText = (text, options) =>
     createWrapper(within(findModal().element).getByText(text, options));
@@ -89,25 +90,27 @@ describe('LeaveModal', () => {
     );
   });
 
-  describe('On-call schedules list', () => {
-    it("displays oncall schedules list when member's user is part of on-call schedules ", () => {
-      const schedulesList = findOncallSchedulesList();
-      expect(schedulesList.exists()).toBe(true);
-      expect(schedulesList.props()).toMatchObject({
+  describe('User deletion obstacles list', () => {
+    it("displays obstacles list when member's user is part of on-call management", () => {
+      const obstaclesList = findUserDeletionObstaclesList();
+      expect(obstaclesList.exists()).toBe(true);
+      expect(obstaclesList.props()).toMatchObject({
         isCurrentUser: true,
-        schedules: member.user.oncallSchedules,
+        obstacles: parseUserDeletionObstacles(member.user),
       });
     });
 
-    it("does NOT display oncall schedules list when member's user is NOT a part of on-call schedules ", async () => {
+    it("does NOT display obstacles list when member's user is NOT a part of on-call management", async () => {
       wrapper.destroy();
 
-      const memberWithoutOncallSchedules = cloneDeep(member);
-      delete memberWithoutOncallSchedules.user.oncallSchedules;
-      createComponent({ member: memberWithoutOncallSchedules });
+      const memberWithoutOncall = cloneDeep(member);
+      delete memberWithoutOncall.user.oncallSchedules;
+      delete memberWithoutOncall.user.escalationPolicies;
+
+      createComponent({ member: memberWithoutOncall });
       await nextTick();
 
-      expect(findOncallSchedulesList().exists()).toBe(false);
+      expect(findUserDeletionObstaclesList().exists()).toBe(false);
     });
   });
 

spec/frontend/members/components/modals/remove_member_modal_spec.js

@@ -4,15 +4,19 @@ import Vue from 'vue';
 import Vuex from 'vuex';
 import RemoveMemberModal from '~/members/components/modals/remove_member_modal.vue';
 import { MEMBER_TYPES } from '~/members/constants';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import { OBSTACLE_TYPES } from '~/vue_shared/components/user_deletion_obstacles/constants';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
 
 Vue.use(Vuex);
 
 describe('RemoveMemberModal', () => {
   const memberPath = '/gitlab-org/gitlab-test/-/project_members/90';
-  const mockSchedules = {
+  const mockObstacles = {
     name: 'User1',
-    schedules: [{ id: 1, name: 'Schedule 1' }],
+    obstacles: [
+      { name: 'Schedule 1', type: OBSTACLE_TYPES.oncallSchedules },
+      { name: 'Policy 1', type: OBSTACLE_TYPES.escalationPolicies },
+    ],
   };
   let wrapper;
 
@@ -44,18 +48,18 @@ describe('RemoveMemberModal', () => {
 
   const findForm = () => wrapper.find({ ref: 'form' });
   const findGlModal = () => wrapper.findComponent(GlModal);
-  const findOnCallSchedulesList = () => wrapper.findComponent(OncallSchedulesList);
+  const findUserDeletionObstaclesList = () => wrapper.findComponent(UserDeletionObstaclesList);
 
   afterEach(() => {
     wrapper.destroy();
   });
 
   describe.each`
-    state                          | memberType         | isAccessRequest | isInvite | actionText               | removeSubMembershipsCheckboxExpected | unassignIssuablesCheckboxExpected | message                                                                                                           | onCallSchedules
-    ${'removing a group member'}   | ${'GroupMember'}   | ${false}        | ${false} | ${'Remove member'}       | ${true}                              | ${true}                           | ${'Are you sure you want to remove Jane Doe from the Gitlab Org / Gitlab Test project?'}                          | ${{}}
-    ${'removing a project member'} | ${'ProjectMember'} | ${false}        | ${false} | ${'Remove member'}       | ${false}                             | ${true}                           | ${'Are you sure you want to remove Jane Doe from the Gitlab Org / Gitlab Test project?'}                          | ${mockSchedules}
-    ${'denying an access request'} | ${'ProjectMember'} | ${true}         | ${false} | ${'Deny access request'} | ${false}                             | ${false}                          | ${"Are you sure you want to deny Jane Doe's request to join the Gitlab Org / Gitlab Test project?"}               | ${{}}
-    ${'revoking invite'}           | ${'ProjectMember'} | ${false}        | ${true}  | ${'Revoke invite'}       | ${false}                             | ${false}                          | ${'Are you sure you want to revoke the invitation for foo@bar.com to join the Gitlab Org / Gitlab Test project?'} | ${mockSchedules}
+    state                          | memberType         | isAccessRequest | isInvite | actionText               | removeSubMembershipsCheckboxExpected | unassignIssuablesCheckboxExpected | message                                                                                                           | userDeletionObstacles | isPartOfOncall
+    ${'removing a group member'}   | ${'GroupMember'}   | ${false}        | ${false} | ${'Remove member'}       | ${true}                              | ${true}                           | ${'Are you sure you want to remove Jane Doe from the Gitlab Org / Gitlab Test project?'}                          | ${{}}                 | ${false}
+    ${'removing a project member'} | ${'ProjectMember'} | ${false}        | ${false} | ${'Remove member'}       | ${false}                             | ${true}                           | ${'Are you sure you want to remove Jane Doe from the Gitlab Org / Gitlab Test project?'}                          | ${mockObstacles}      | ${true}
+    ${'denying an access request'} | ${'ProjectMember'} | ${true}         | ${false} | ${'Deny access request'} | ${false}                             | ${false}                          | ${"Are you sure you want to deny Jane Doe's request to join the Gitlab Org / Gitlab Test project?"}               | ${{}}                 | ${false}
+    ${'revoking invite'}           | ${'ProjectMember'} | ${false}        | ${true}  | ${'Revoke invite'}       | ${false}                             | ${false}                          | ${'Are you sure you want to revoke the invitation for foo@bar.com to join the Gitlab Org / Gitlab Test project?'} | ${mockObstacles}      | ${false}
   `(
     'when $state',
     ({
@@ -66,7 +70,8 @@ describe('RemoveMemberModal', () => {
       message,
       removeSubMembershipsCheckboxExpected,
       unassignIssuablesCheckboxExpected,
-      onCallSchedules,
+      userDeletionObstacles,
+      isPartOfOncall,
     }) => {
       beforeEach(() => {
         createComponent({
@@ -75,12 +80,10 @@ describe('RemoveMemberModal', () => {
           message,
           memberPath,
           memberType,
-          onCallSchedules,
+          userDeletionObstacles,
         });
       });
 
-      const isPartOfOncallSchedules = Boolean(isAccessRequest && onCallSchedules.schedules?.length);
-
       it(`has the title ${actionText}`, () => {
         expect(findGlModal().attributes('title')).toBe(actionText);
       });
@@ -109,8 +112,8 @@ describe('RemoveMemberModal', () => {
         );
       });
 
-      it(`shows ${isPartOfOncallSchedules ? 'all' : 'no'} related on-call schedules`, () => {
-        expect(findOnCallSchedulesList().exists()).toBe(isPartOfOncallSchedules);
+      it(`shows ${isPartOfOncall ? 'all' : 'no'} related on-call schedules or policies`, () => {
+        expect(findUserDeletionObstaclesList().exists()).toBe(isPartOfOncall);
       });
 
       it('submits the form when the modal is submitted', () => {

spec/frontend/members/mock_data.js

@@ -23,6 +23,7 @@ export const member = {
     blocked: false,
     twoFactorEnabled: false,
     oncallSchedules: [{ name: 'schedule 1' }],
+    escalationPolicies: [{ name: 'policy 1' }],
   },
   id: 238,
   createdAt: '2020-07-17T16:22:46.923Z',
@@ -63,7 +64,7 @@ export const modalData = {
   memberPath: '/groups/foo-bar/-/group_members/1',
   memberType: 'GroupMember',
   message: 'Are you sure you want to remove John Smith?',
-  oncallSchedules: { name: 'user', schedules: [] },
+  userDeletionObstacles: { name: 'user', obstacles: [] },
 };
 
 const { user, ...memberNoUser } = member;

spec/frontend/vue_shared/oncall_schedules_list_spec.js → spec/frontend/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list_spec.js

 import { GlLink, GlSprintf } from '@gitlab/ui';
 import { shallowMount } from '@vue/test-utils';
 import { extendedWrapper } from 'helpers/vue_test_utils_helper';
-import OncallSchedulesList from '~/vue_shared/components/oncall_schedules_list.vue';
+import { OBSTACLE_TYPES } from '~/vue_shared/components/user_deletion_obstacles/constants';
+import UserDeletionObstaclesList from '~/vue_shared/components/user_deletion_obstacles/user_deletion_obstacles_list.vue';
 
 const mockSchedules = [
   {
+    type: OBSTACLE_TYPES.oncallSchedules,
     name: 'Schedule 1',
-    scheduleUrl: 'http://gitlab.com/gitlab-org/gitlab-shell/-/oncall_schedules',
+    url: 'http://gitlab.com/gitlab-org/gitlab-shell/-/oncall_schedules',
     projectName: 'Shell',
     projectUrl: 'http://gitlab.com/gitlab-org/gitlab-shell/',
   },
   {
+    type: OBSTACLE_TYPES.oncallSchedules,
     name: 'Schedule 2',
-    scheduleUrl: 'http://gitlab.com/gitlab-org/gitlab-ui/-/oncall_schedules',
+    url: 'http://gitlab.com/gitlab-org/gitlab-ui/-/oncall_schedules',
     projectName: 'UI',
     projectUrl: 'http://gitlab.com/gitlab-org/gitlab-ui/',
   },
 ];
+const mockPolicies = [
+  {
+    type: OBSTACLE_TYPES.escalationPolicies,
+    name: 'Policy 1',
+    url: 'http://gitlab.com/gitlab-org/gitlab-ui/-/escalation-policies',
+    projectName: 'UI',
+    projectUrl: 'http://gitlab.com/gitlab-org/gitlab-ui/',
+  },
+];
+const mockObstacles = mockSchedules.concat(mockPolicies);
 
 const userName = "O'User";
 
-describe('On-call schedules list', () => {
+describe('User deletion obstacles list', () => {
   let wrapper;
 
   function createComponent(props) {
     wrapper = extendedWrapper(
-      shallowMount(OncallSchedulesList, {
+      shallowMount(UserDeletionObstaclesList, {
         propsData: {
-          schedules: mockSchedules,
+          obstacles: mockObstacles,
           userName,
           ...props,
         },
@@ -45,14 +58,14 @@ describe('On-call schedules list', () => {
   const findLinks = () => wrapper.findAllComponents(GlLink);
   const findTitle = () => wrapper.findByTestId('title');
   const findFooter = () => wrapper.findByTestId('footer');
-  const findSchedules = () => wrapper.findByTestId('schedules-list');
+  const findObstacles = () => wrapper.findByTestId('obstacles-list');
 
   describe.each`
     isCurrentUser | titleText                                   | footerText
     ${true}       | ${'You are currently a part of:'}           | ${'Removing yourself may put your on-call team at risk of missing a notification.'}
     ${false}      | ${`User ${userName} is currently part of:`} | ${'Removing this user may put their on-call team at risk of missing a notification.'}
-  `('when current user ', ({ isCurrentUser, titleText, footerText }) => {
-    it(`${isCurrentUser ? 'is' : 'is not'} a part of on-call schedule`, async () => {
+  `('when current user', ({ isCurrentUser, titleText, footerText }) => {
+    it(`${isCurrentUser ? 'is' : 'is not'} a part of on-call management`, async () => {
       createComponent({
         isCurrentUser,
       });
@@ -62,25 +75,41 @@ describe('On-call schedules list', () => {
     });
   });
 
-  describe.each(mockSchedules)(
-    'renders each on-call schedule data',
-    ({ name, scheduleUrl, projectName, projectUrl }) => {
-      beforeEach(() => {
-        createComponent({ schedules: [{ name, scheduleUrl, projectName, projectUrl }] });
+  describe.each(mockObstacles)(
+    'renders all obstacles',
+    ({ type, name, url, projectName, projectUrl }) => {
+      it(`includes the project name and link for ${name}`, () => {
+        createComponent({ obstacles: [{ type, name, url, projectName, projectUrl }] });
+        const msg = findObstacles().text();
+
+        expect(msg).toContain(`in Project ${projectName}`);
+        expect(findLinks().at(1).attributes('href')).toBe(projectUrl);
       });
+    },
+  );
 
-      it(`renders schedule ${name}'s name and link`, () => {
-        const msg = findSchedules().text();
+  describe.each(mockSchedules)(
+    'renders on-call schedules',
+    ({ type, name, url, projectName, projectUrl }) => {
+      it(`includes the schedule name and link for ${name}`, () => {
+        createComponent({ obstacles: [{ type, name, url, projectName, projectUrl }] });
+        const msg = findObstacles().text();
 
         expect(msg).toContain(`On-call schedule ${name}`);
-        expect(findLinks().at(0).attributes('href')).toBe(scheduleUrl);
+        expect(findLinks().at(0).attributes('href')).toBe(url);
       });
+    },
+  );
 
-      it(`renders project ${projectName}'s name and link`, () => {
-        const msg = findSchedules().text();
+  describe.each(mockPolicies)(
+    'renders escalation policies',
+    ({ type, name, url, projectName, projectUrl }) => {
+      it(`includes the policy name and link for ${name}`, () => {
+        createComponent({ obstacles: [{ type, name, url, projectName, projectUrl }] });
+        const msg = findObstacles().text();
 
-        expect(msg).toContain(`in Project ${projectName}`);
-        expect(findLinks().at(1).attributes('href')).toBe(projectUrl);
+        expect(msg).toContain(`Escalation policy ${name}`);
+        expect(findLinks().at(0).attributes('href')).toBe(url);
       });
     },
   );

spec/frontend/vue_shared/components/user_deletion_obstacles/utils_spec.js

+import { OBSTACLE_TYPES } from '~/vue_shared/components/user_deletion_obstacles/constants';
+import { parseUserDeletionObstacles } from '~/vue_shared/components/user_deletion_obstacles/utils';
+
+describe('parseUserDeletionObstacles', () => {
+  const mockObstacles = [{ name: 'Obstacle' }];
+  const expectedSchedule = { name: 'Obstacle', type: OBSTACLE_TYPES.oncallSchedules };
+  const expectedPolicy = { name: 'Obstacle', type: OBSTACLE_TYPES.escalationPolicies };
+
+  it('is undefined when user is not available', () => {
+    expect(parseUserDeletionObstacles()).toHaveLength(0);
+  });
+
+  it('is empty when obstacles are not available for user', () => {
+    expect(parseUserDeletionObstacles({})).toHaveLength(0);
+  });
+
+  it('is empty when user has no obstacles to deletion', () => {
+    const input = { oncallSchedules: [], escalationPolicies: [] };
+
+    expect(parseUserDeletionObstacles(input)).toHaveLength(0);
+  });
+
+  it('returns obstacles with type when user is part of on-call schedules', () => {
+    const input = { oncallSchedules: mockObstacles, escalationPolicies: [] };
+    const expectedOutput = [expectedSchedule];
+
+    expect(parseUserDeletionObstacles(input)).toEqual(expectedOutput);
+  });
+
+  it('returns obstacles with type when user is part of escalation policies', () => {
+    const input = { oncallSchedules: [], escalationPolicies: mockObstacles };
+    const expectedOutput = [expectedPolicy];
+
+    expect(parseUserDeletionObstacles(input)).toEqual(expectedOutput);
+  });
+
+  it('returns obstacles with type when user have every obstacle type', () => {
+    const input = { oncallSchedules: mockObstacles, escalationPolicies: mockObstacles };
+    const expectedOutput = [expectedSchedule, expectedPolicy];
+
+    expect(parseUserDeletionObstacles(input)).toEqual(expectedOutput);
+  });
+});