From 3498753b1a2b27a10fa36e6fecef9670e166376f Mon Sep 17 00:00:00 2001
From: Julia Miocene <jmiocene@gitlab.com>
Date: Tue, 16 Jul 2024 21:18:31 +0200
Subject: [PATCH] Migrate merge-request-open icon to merge-request

---
 app/assets/javascripts/contribution_events/constants.js     | 2 +-
 app/assets/javascripts/issuable/components/status_badge.vue | 4 ++--
 .../javascripts/super_sidebar/components/user_bar.vue       | 2 +-
 .../javascripts/vue_shared/components/notes/system_note.vue | 2 +-
 app/helpers/branches_helper.rb                              | 2 +-
 .../contribution_event/contribution_event_reopened_spec.js  | 2 +-
 spec/frontend/issuable/components/status_badge_spec.js      | 2 +-
 spec/helpers/branches_helper_spec.rb                        | 6 +++---
 8 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/app/assets/javascripts/contribution_events/constants.js b/app/assets/javascripts/contribution_events/constants.js
index c0224ce94ffd1d..30bb32b4c48067 100644
--- a/app/assets/javascripts/contribution_events/constants.js
+++ b/app/assets/javascripts/contribution_events/constants.js
@@ -153,7 +153,7 @@ export const EVENT_CLOSED_ICONS = Object.freeze({
 });
 
 export const EVENT_REOPENED_ICONS = Object.freeze({
-  [TARGET_TYPE_MERGE_REQUEST]: 'merge-request-open',
+  [TARGET_TYPE_MERGE_REQUEST]: 'merge-request',
   [TYPE_FALLBACK]: 'status_open',
 });
 
diff --git a/app/assets/javascripts/issuable/components/status_badge.vue b/app/assets/javascripts/issuable/components/status_badge.vue
index c08153df839853..3ce768298c8295 100644
--- a/app/assets/javascripts/issuable/components/status_badge.vue
+++ b/app/assets/javascripts/issuable/components/status_badge.vue
@@ -43,7 +43,7 @@ const badgePropertiesMap = {
   },
   [TYPE_MERGE_REQUEST]: {
     [STATUS_OPEN]: {
-      icon: 'merge-request-open',
+      icon: 'merge-request',
       text: __('Open'),
       variant: 'success',
     },
@@ -58,7 +58,7 @@ const badgePropertiesMap = {
       variant: 'info',
     },
     [STATUS_LOCKED]: {
-      icon: 'merge-request-open',
+      icon: 'merge-request',
       text: __('Open'),
       variant: 'success',
     },
diff --git a/app/assets/javascripts/super_sidebar/components/user_bar.vue b/app/assets/javascripts/super_sidebar/components/user_bar.vue
index 4dd227745aca09..cc8d26e39080c8 100644
--- a/app/assets/javascripts/super_sidebar/components/user_bar.vue
+++ b/app/assets/javascripts/super_sidebar/components/user_bar.vue
@@ -180,7 +180,7 @@ export default {
         <counter
           v-gl-tooltip:super-sidebar.bottom="mrMenuShown ? '' : $options.i18n.mergeRequests"
           class="gl-w-full"
-          icon="merge-request-open"
+          icon="merge-request"
           :href="sidebarData.merge_request_dashboard_path"
           :count="mergeRequestTotalCount"
           :label="$options.i18n.mergeRequests"
diff --git a/app/assets/javascripts/vue_shared/components/notes/system_note.vue b/app/assets/javascripts/vue_shared/components/notes/system_note.vue
index 5a5d255fc59181..2553fe6501bd5f 100644
--- a/app/assets/javascripts/vue_shared/components/notes/system_note.vue
+++ b/app/assets/javascripts/vue_shared/components/notes/system_note.vue
@@ -108,7 +108,7 @@ export default {
         if (this.note.noteable_type === 'Issue') {
           icon = 'issue-open-m';
         } else if (this.note.noteable_type === 'MergeRequest') {
-          icon = 'merge-request-open';
+          icon = 'merge-request';
         }
       }
       return icon;
diff --git a/app/helpers/branches_helper.rb b/app/helpers/branches_helper.rb
index ecc253ca967e96..91616afb5e0b40 100644
--- a/app/helpers/branches_helper.rb
+++ b/app/helpers/branches_helper.rb
@@ -30,7 +30,7 @@ def merge_request_status(merge_request)
       variant = :success
       variant = :warning if merge_request.draft?
 
-      mr_icon = 'merge-request-open'
+      mr_icon = 'merge-request'
       mr_status = _('Open')
     elsif merge_request.merged?
       variant = :info
diff --git a/spec/frontend/contribution_events/components/contribution_event/contribution_event_reopened_spec.js b/spec/frontend/contribution_events/components/contribution_event/contribution_event_reopened_spec.js
index 9001bb284c7c4c..e3660acce27069 100644
--- a/spec/frontend/contribution_events/components/contribution_event/contribution_event_reopened_spec.js
+++ b/spec/frontend/contribution_events/components/contribution_event/contribution_event_reopened_spec.js
@@ -23,7 +23,7 @@ describe('ContributionEventReopened', () => {
     event                                       | expectedMessage                                                     | iconName
     ${eventMilestoneReopened()}                 | ${'Reopened milestone %{targetLink} in %{resourceParentLink}.'}     | ${'status_open'}
     ${eventIssueReopened()}                     | ${'Reopened issue %{targetLink} in %{resourceParentLink}.'}         | ${'status_open'}
-    ${eventMergeRequestReopened()}              | ${'Reopened merge request %{targetLink} in %{resourceParentLink}.'} | ${'merge-request-open'}
+    ${eventMergeRequestReopened()}              | ${'Reopened merge request %{targetLink} in %{resourceParentLink}.'} | ${'merge-request'}
     ${{ target: { type: 'unsupported type' } }} | ${'Reopened resource.'}                                             | ${'status_open'}
   `('when event target type is $event.target.type', ({ event, expectedMessage, iconName }) => {
     it('renders `ContributionEventBase` with correct props', () => {
diff --git a/spec/frontend/issuable/components/status_badge_spec.js b/spec/frontend/issuable/components/status_badge_spec.js
index 9ab5b4f714933b..21fa5fbc208ae5 100644
--- a/spec/frontend/issuable/components/status_badge_spec.js
+++ b/spec/frontend/issuable/components/status_badge_spec.js
@@ -13,7 +13,7 @@ describe('StatusBadge component', () => {
 
   describe.each`
     issuableType       | badgeText   | state       | badgeVariant | badgeIcon
-    ${'merge_request'} | ${'Open'}   | ${'opened'} | ${'success'} | ${'merge-request-open'}
+    ${'merge_request'} | ${'Open'}   | ${'opened'} | ${'success'} | ${'merge-request'}
     ${'merge_request'} | ${'Closed'} | ${'closed'} | ${'danger'}  | ${'merge-request-close'}
     ${'merge_request'} | ${'Merged'} | ${'merged'} | ${'info'}    | ${'merge'}
     ${'issue'}         | ${'Open'}   | ${'opened'} | ${'success'} | ${'issue-open-m'}
diff --git a/spec/helpers/branches_helper_spec.rb b/spec/helpers/branches_helper_spec.rb
index 3375686765382e..c9e82c03863e8f 100644
--- a/spec/helpers/branches_helper_spec.rb
+++ b/spec/helpers/branches_helper_spec.rb
@@ -69,19 +69,19 @@
     end
 
     context 'when merge request is open' do
-      it { is_expected.to eq(icon: 'merge-request-open', title: "Open - #{title}", variant: :success) }
+      it { is_expected.to eq(icon: 'merge-request', title: "Open - #{title}", variant: :success) }
     end
 
     context 'when merge request is locked' do
       let(:merge_request) { build(:merge_request, :locked, title: title) }
 
-      it { is_expected.to eq(icon: 'merge-request-open', title: "Open - #{title}", variant: :success) }
+      it { is_expected.to eq(icon: 'merge-request', title: "Open - #{title}", variant: :success) }
     end
 
     context 'when merge request is draft' do
       let(:title) { 'Draft: Test MR' }
 
-      it { is_expected.to eq(icon: 'merge-request-open', title: "Open - #{title}", variant: :warning) }
+      it { is_expected.to eq(icon: 'merge-request', title: "Open - #{title}", variant: :warning) }
     end
 
     context 'when merge request is merged' do
-- 
GitLab