Merge branch 'jl-fix-compliance-frameworks-edit-loading' into 'master'

Fix compliance framework edit button loading on delete

See merge request gitlab-org/gitlab!56222

ee/app/assets/javascripts/groups/settings/compliance_frameworks/components/list_item.vue

@@ -50,7 +50,6 @@ export default {
     <div class="gl-display-flex">
       <gl-button
         v-gl-tooltip="$options.i18n.editFramework"
-        :loading="loading"
         :disabled="loading"
         :aria-label="$options.i18n.editFramework"
         :href="framework.editPath"

ee/spec/frontend/groups/settings/compliance_frameworks/components/list_item_spec.js

@@ -50,11 +50,6 @@ describe('ListItem', () => {
     expect(button.attributes('aria-label')).toBe(ariaLabel);
   };
 
-  const disablesTheButton = (button) => {
-    expect(button.props('disabled')).toBe(true);
-    expect(button.props('loading')).toBe(true);
-  };
-
   it('displays the description defined by the framework', () => {
     createComponent();
 
@@ -115,11 +110,17 @@ describe('ListItem', () => {
     });
 
     it('disables the delete button and shows loading', () => {
-      disablesTheButton(findDeleteButton());
+      const button = findDeleteButton();
+
+      expect(button.props('disabled')).toBe(true);
+      expect(button.props('loading')).toBe(true);
     });
 
-    it('disables the edit button and shows loading', () => {
-      disablesTheButton(findEditButton());
+    it('disables the edit button and does not show loading', () => {
+      const button = findEditButton();
+
+      expect(button.props('disabled')).toBe(true);
+      expect(button.props('loading')).toBe(false);
     });
   });
 });