Migrate project settings to use GlToggle

It was using the `vue_shared` toggle component
which is now deprecated in favour of GlToggle

app/assets/javascripts/pages/projects/shared/permissions/components/project_feature_setting.vue

 <script>
-import { GlIcon } from '@gitlab/ui';
-import projectFeatureToggle from '~/vue_shared/components/toggle_button.vue';
+import { GlIcon, GlToggle } from '@gitlab/ui';
 import { featureAccessLevelNone } from '../constants';
 
 export default {
   components: {
     GlIcon,
-    projectFeatureToggle,
+    GlToggle,
   },
   model: {
     prop: 'value',
@@ -78,11 +77,11 @@ export default {
     class="project-feature-controls gl-display-flex gl-align-items-center gl-my-3 gl-mx-0"
   >
     <input v-if="name" :name="name" :value="value" type="hidden" />
-    <project-feature-toggle
+    <gl-toggle
       v-if="showToggle"
-      class="gl-flex-grow-0 gl-mr-3"
+      class="gl-mr-3"
       :value="featureEnabled"
-      :disabled-input="disabledInput"
+      :disabled="disabledInput"
       @change="toggleFeature"
     />
     <div class="select-wrapper gl-flex-fill-1">

app/assets/javascripts/pages/projects/shared/permissions/components/settings_panel.vue

 <script>
-import { GlIcon, GlSprintf, GlLink, GlFormCheckbox } from '@gitlab/ui';
+import { GlIcon, GlSprintf, GlLink, GlFormCheckbox, GlToggle } from '@gitlab/ui';
 
 import settingsMixin from 'ee_else_ce/pages/projects/shared/permissions/mixins/settings_pannel_mixin';
 import { s__ } from '~/locale';
-import projectFeatureToggle from '~/vue_shared/components/toggle_button.vue';
 import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
 import {
   visibilityOptions,
@@ -22,12 +21,12 @@ const PAGE_FEATURE_ACCESS_LEVEL = s__('ProjectSettings|Everyone');
 export default {
   components: {
     projectFeatureSetting,
-    projectFeatureToggle,
     projectSettingRow,
     GlIcon,
     GlSprintf,
     GlLink,
     GlFormCheckbox,
+    GlToggle,
   },
   mixins: [settingsMixin, glFeatureFlagsMixin()],
 
@@ -483,9 +482,10 @@ export default {
               )
             }}
           </div>
-          <project-feature-toggle
+          <gl-toggle
             v-model="containerRegistryEnabled"
-            :disabled-input="!repositoryEnabled"
+            class="gl-my-2"
+            :disabled="!repositoryEnabled"
             name="project[container_registry_enabled]"
           />
         </project-setting-row>
@@ -498,9 +498,10 @@ export default {
             s__('ProjectSettings|Manages large files such as audio, video, and graphics files.')
           "
         >
-          <project-feature-toggle
+          <gl-toggle
             v-model="lfsEnabled"
-            :disabled-input="!repositoryEnabled"
+            class="gl-my-2"
+            :disabled="!repositoryEnabled"
             name="project[lfs_enabled]"
           />
           <p v-if="!lfsEnabled && lfsObjectsExist">
@@ -530,9 +531,10 @@ export default {
             s__('ProjectSettings|Every project can have its own space to store its packages.')
           "
         >
-          <project-feature-toggle
+          <gl-toggle
             v-model="packagesEnabled"
-            :disabled-input="!repositoryEnabled"
+            class="gl-my-2"
+            :disabled="!repositoryEnabled"
             name="project[packages_enabled]"
           />
         </project-setting-row>

changelogs/unreleased/cngo-migrate-project-settings-to-use-gl-toggle.yml

+---
+title: Migrate project settings to use GlToggle
+merge_request: 52729
+author:
+type: changed

ee/spec/features/projects/settings/issues_settings_spec.rb

@@ -22,7 +22,7 @@ RSpec.describe 'Project settings > Issues', :js do
         expect(page).to have_content('Set a default description template to be used for new issues.')
 
         within('.sharing-permissions-form') do
-          find('.project-feature-controls[data-for="project[project_feature_attributes][issues_access_level]"] .project-feature-toggle').click
+          find('.project-feature-controls[data-for="project[project_feature_attributes][issues_access_level]"] .gl-toggle').click
           click_on('Save changes')
         end
 
@@ -41,7 +41,7 @@ RSpec.describe 'Project settings > Issues', :js do
       expect(page).not_to have_content('Set a default description template to be used for new issues.')
 
       within('.sharing-permissions-form') do
-        find('.project-feature-controls[data-for="project[project_feature_attributes][issues_access_level]"] .project-feature-toggle').click
+        find('.project-feature-controls[data-for="project[project_feature_attributes][issues_access_level]"] .gl-toggle').click
         click_on('Save changes')
       end
 

spec/features/projects/features_visibility_spec.rb

@@ -187,7 +187,7 @@ RSpec.describe 'Edit Project Settings' do
         click_button "Save changes"
       end
 
-      expect(find(".sharing-permissions")).to have_selector(".project-feature-toggle.is-disabled", count: 4)
+      expect(find(".sharing-permissions")).to have_selector(".gl-toggle.is-disabled", minimum: 4)
     end
 
     it "shows empty features project homepage" do
@@ -282,10 +282,10 @@ RSpec.describe 'Edit Project Settings' do
   end
 
   def toggle_feature_off(feature_name)
-    find(".project-feature-controls[data-for=\"#{feature_name}\"] .project-feature-toggle.is-checked").click
+    find(".project-feature-controls[data-for=\"#{feature_name}\"] .gl-toggle.is-checked").click
   end
 
   def toggle_feature_on(feature_name)
-    find(".project-feature-controls[data-for=\"#{feature_name}\"] .project-feature-toggle:not(.is-checked)").click
+    find(".project-feature-controls[data-for=\"#{feature_name}\"] .gl-toggle:not(.is-checked)").click
   end
 end

spec/features/projects/settings/project_settings_spec.rb

@@ -39,7 +39,7 @@ RSpec.describe 'Projects settings' do
       visit edit_project_path(project)
 
       forking_enabled_input = find('input[name="project[project_feature_attributes][forking_access_level]"]', visible: :hidden)
-      forking_enabled_button = find('input[name="project[project_feature_attributes][forking_access_level]"] + label > button')
+      forking_enabled_button = find('[data-for="project[project_feature_attributes][forking_access_level]"] .gl-toggle')
 
       expect(forking_enabled_input.value).to eq('20')
 

spec/features/projects/settings/user_manages_merge_requests_settings_spec.rb

@@ -51,7 +51,7 @@ RSpec.describe 'Projects > Settings > User manages merge request settings' do
         expect(page).to have_content 'All discussions must be resolved'
 
         within('.sharing-permissions-form') do
-          find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click
+          find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .gl-toggle').click
           find('input[value="Save changes"]').send_keys(:return)
         end
 
@@ -71,7 +71,7 @@ RSpec.describe 'Projects > Settings > User manages merge request settings' do
         expect(page).to have_content 'All discussions must be resolved'
 
         within('.sharing-permissions-form') do
-          find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .project-feature-toggle').click
+          find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .gl-toggle').click
           find('input[value="Save changes"]').send_keys(:return)
         end
 
@@ -92,7 +92,7 @@ RSpec.describe 'Projects > Settings > User manages merge request settings' do
       expect(page).not_to have_content 'All discussions must be resolved'
 
       within('.sharing-permissions-form') do
-        find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click
+        find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .gl-toggle').click
         find('input[value="Save changes"]').send_keys(:return)
       end
 

spec/features/projects/settings/visibility_settings_spec.rb

@@ -30,7 +30,7 @@ RSpec.describe 'Projects > Settings > Visibility settings', :js do
 
     context 'merge requests select' do
       it 'hides merge requests section' do
-        find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .project-feature-toggle').click
+        find('.project-feature-controls[data-for="project[project_feature_attributes][merge_requests_access_level]"] .gl-toggle').click
 
         expect(page).to have_selector('.merge-requests-feature', visible: false)
       end
@@ -46,7 +46,7 @@ RSpec.describe 'Projects > Settings > Visibility settings', :js do
 
     context 'builds select' do
       it 'hides builds select section' do
-        find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .project-feature-toggle').click
+        find('.project-feature-controls[data-for="project[project_feature_attributes][builds_access_level]"] .gl-toggle').click
 
         expect(page).to have_selector('.builds-feature', visible: false)
       end

spec/frontend/pages/projects/shared/permissions/components/project_feature_settings_spec.js

-import { mount, shallowMount } from '@vue/test-utils';
-
-import projectFeatureSetting from '~/pages/projects/shared/permissions/components/project_feature_setting.vue';
-import projectFeatureToggle from '~/vue_shared/components/toggle_button.vue';
+import { GlToggle } from '@gitlab/ui';
+import { shallowMount } from '@vue/test-utils';
+import ProjectFeatureSetting from '~/pages/projects/shared/permissions/components/project_feature_setting.vue';
 
 describe('Project Feature Settings', () => {
   const defaultProps = {
@@ -19,81 +18,76 @@ describe('Project Feature Settings', () => {
   };
   let wrapper;
 
-  const mountComponent = (customProps) => {
-    const propsData = { ...defaultProps, ...customProps };
-    return shallowMount(projectFeatureSetting, { propsData });
-  };
+  const findHiddenInput = () => wrapper.find(`input[name=${defaultProps.name}]`);
+  const findToggle = () => wrapper.findComponent(GlToggle);
 
-  beforeEach(() => {
-    wrapper = mountComponent();
-  });
+  const mountComponent = (customProps = {}) =>
+    shallowMount(ProjectFeatureSetting, {
+      propsData: {
+        ...defaultProps,
+        ...customProps,
+      },
+    });
 
   afterEach(() => {
     wrapper.destroy();
+    wrapper = null;
   });
 
   describe('Hidden name input', () => {
     it('should set the hidden name input if the name exists', () => {
-      expect(wrapper.find(`input[name=${defaultProps.name}]`).attributes().value).toBe('1');
+      wrapper = mountComponent();
+
+      expect(findHiddenInput().attributes('value')).toBe('1');
     });
 
     it('should not set the hidden name input if the name does not exist', () => {
-      wrapper.setProps({ name: null });
+      wrapper = mountComponent({ name: null });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find(`input[name=${defaultProps.name}]`).exists()).toBe(false);
-      });
+      expect(findHiddenInput().exists()).toBe(false);
     });
   });
 
   describe('Feature toggle', () => {
-    it('should be hidden if "showToggle" is passed false', async () => {
-      wrapper.setProps({ showToggle: false });
-
-      await wrapper.vm.$nextTick();
+    it('should be hidden if "showToggle" is passed false', () => {
+      wrapper = mountComponent({ showToggle: false });
 
-      expect(wrapper.find(projectFeatureToggle).element).toBeUndefined();
+      expect(findToggle().exists()).toBe(false);
     });
 
     it('should enable the feature toggle if the value is not 0', () => {
-      expect(wrapper.find(projectFeatureToggle).props().value).toBe(true);
+      wrapper = mountComponent();
+
+      expect(findToggle().props('value')).toBe(true);
     });
 
     it('should enable the feature toggle if the value is less than 0', () => {
-      wrapper.setProps({ value: -1 });
+      wrapper = mountComponent({ value: -1 });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find(projectFeatureToggle).props().value).toBe(true);
-      });
+      expect(findToggle().props('value')).toBe(true);
     });
 
     it('should disable the feature toggle if the value is 0', () => {
-      wrapper.setProps({ value: 0 });
+      wrapper = mountComponent({ value: 0 });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find(projectFeatureToggle).props().value).toBe(false);
-      });
+      expect(findToggle().props('value')).toBe(false);
     });
 
     it('should disable the feature toggle if disabledInput is set', () => {
-      wrapper.setProps({ disabledInput: true });
+      wrapper = mountComponent({ disabledInput: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find(projectFeatureToggle).props().disabledInput).toBe(true);
-      });
+      expect(findToggle().props('disabled')).toBe(true);
     });
 
     it('should emit a change event when the feature toggle changes', () => {
-      // Needs to be fully mounted to be able to trigger the click event on the internal button
-      wrapper = mount(projectFeatureSetting, { propsData: defaultProps });
+      wrapper = mountComponent({ propsData: defaultProps });
+
+      expect(wrapper.emitted('change')).toBeUndefined();
 
-      expect(wrapper.emitted().change).toBeUndefined();
-      wrapper.find(projectFeatureToggle).find('button').trigger('click');
+      findToggle().vm.$emit('change', false);
 
-      return wrapper.vm.$nextTick().then(() => {
-        expect(wrapper.emitted().change.length).toBe(1);
-        expect(wrapper.emitted().change[0]).toEqual([0]);
-      });
+      expect(wrapper.emitted('change')).toHaveLength(1);
+      expect(wrapper.emitted('change')[0]).toEqual([0]);
     });
   });
 
@@ -108,26 +102,23 @@ describe('Project Feature Settings', () => {
     `(
       'should set disabled to $isDisabled when disabledInput is $disabledInput, the value is $value and options are $options',
       ({ disabledInput, value, options, isDisabled }) => {
-        wrapper.setProps({ disabledInput, value, options });
-
-        return wrapper.vm.$nextTick(() => {
-          if (isDisabled) {
-            expect(wrapper.find('select').attributes().disabled).toEqual('disabled');
-          } else {
-            expect(wrapper.find('select').attributes().disabled).toBeUndefined();
-          }
-        });
+        wrapper = mountComponent({ disabledInput, value, options });
+
+        const expected = isDisabled ? 'disabled' : undefined;
+
+        expect(wrapper.find('select').attributes('disabled')).toBe(expected);
       },
     );
 
     it('should emit the change when a new option is selected', () => {
-      expect(wrapper.emitted().change).toBeUndefined();
+      wrapper = mountComponent();
+
+      expect(wrapper.emitted('change')).toBeUndefined();
+
       wrapper.findAll('option').at(1).trigger('change');
 
-      return wrapper.vm.$nextTick().then(() => {
-        expect(wrapper.emitted().change.length).toBe(1);
-        expect(wrapper.emitted().change[0]).toEqual([2]);
-      });
+      expect(wrapper.emitted('change')).toHaveLength(1);
+      expect(wrapper.emitted('change')[0]).toEqual([2]);
     });
   });
 });

spec/frontend/pages/projects/shared/permissions/components/settings_panel_spec.js

+import { GlToggle } from '@gitlab/ui';
 import { shallowMount, mount } from '@vue/test-utils';
-
 import settingsPanel from '~/pages/projects/shared/permissions/components/settings_panel.vue';
 import {
   featureAccessLevel,
@@ -7,7 +7,6 @@ import {
   visibilityOptions,
 } from '~/pages/projects/shared/permissions/constants';
 import projectFeatureSetting from '~/pages/projects/shared/permissions/components/project_feature_setting.vue';
-import projectFeatureToggle from '~/vue_shared/components/toggle_button.vue';
 
 const defaultProps = {
   currentSettings: {
@@ -69,57 +68,61 @@ describe('Settings Panel', () => {
     });
   };
 
-  const overrideCurrentSettings = (
-    currentSettingsProps,
-    extraProps = {},
-    mountFn = shallowMount,
-  ) => {
-    return mountComponent({ ...extraProps, currentSettings: currentSettingsProps }, mountFn);
-  };
-
   const findLFSSettingsRow = () => wrapper.find({ ref: 'git-lfs-settings' });
   const findLFSSettingsMessage = () => findLFSSettingsRow().find('p');
-  const findLFSFeatureToggle = () => findLFSSettingsRow().find(projectFeatureToggle);
-
+  const findLFSFeatureToggle = () => findLFSSettingsRow().find(GlToggle);
   const findRepositoryFeatureProjectRow = () => wrapper.find({ ref: 'repository-settings' });
   const findRepositoryFeatureSetting = () =>
     findRepositoryFeatureProjectRow().find(projectFeatureSetting);
-
+  const findProjectVisibilitySettings = () => wrapper.find({ ref: 'project-visibility-settings' });
   const findAnalyticsRow = () => wrapper.find({ ref: 'analytics-settings' });
-
-  beforeEach(() => {
-    wrapper = mountComponent();
-  });
+  const findProjectVisibilityLevelInput = () => wrapper.find('[name="project[visibility_level]"]');
+  const findRequestAccessEnabledInput = () =>
+    wrapper.find('[name="project[request_access_enabled]"]');
+  const findMergeRequestsAccessLevelInput = () =>
+    wrapper.find('[name="project[project_feature_attributes][merge_requests_access_level]"]');
+  const findForkingAccessLevelInput = () =>
+    wrapper.find('[name="project[project_feature_attributes][forking_access_level]"]');
+  const findBuildsAccessLevelInput = () =>
+    wrapper.find('[name="project[project_feature_attributes][builds_access_level]"]');
+  const findContainerRegistrySettings = () => wrapper.find({ ref: 'container-registry-settings' });
+  const findContainerRegistryEnabledInput = () =>
+    wrapper.find('[name="project[container_registry_enabled]"]');
+  const findPackageSettings = () => wrapper.find({ ref: 'package-settings' });
+  const findPackagesEnabledInput = () => wrapper.find('[name="project[packages_enabled]"]');
+  const findPagesSettings = () => wrapper.find({ ref: 'pages-settings' });
+  const findEmailSettings = () => wrapper.find({ ref: 'email-settings' });
+  const findShowDefaultAwardEmojis = () =>
+    wrapper.find('input[name="project[project_setting_attributes][show_default_award_emojis]"]');
+  const findMetricsVisibilitySettings = () => wrapper.find({ ref: 'metrics-visibility-settings' });
+  const findAllowEditingCommitMessages = () =>
+    wrapper.find({ ref: 'allow-editing-commit-messages' }).exists();
+  const findOperationsSettings = () => wrapper.find({ ref: 'operations-settings' });
 
   afterEach(() => {
     wrapper.destroy();
+    wrapper = null;
   });
 
   describe('Project Visibility', () => {
     it('should set the project visibility help path', () => {
-      expect(wrapper.find({ ref: 'project-visibility-settings' }).props().helpPath).toBe(
+      wrapper = mountComponent();
+
+      expect(findProjectVisibilitySettings().props('helpPath')).toBe(
         defaultProps.visibilityHelpPath,
       );
     });
 
     it('should not disable the visibility level dropdown', () => {
-      wrapper.setProps({ canChangeVisibilityLevel: true });
+      wrapper = mountComponent({ canChangeVisibilityLevel: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(
-          wrapper.find('[name="project[visibility_level]"]').attributes().disabled,
-        ).toBeUndefined();
-      });
+      expect(findProjectVisibilityLevelInput().attributes('disabled')).toBeUndefined();
     });
 
     it('should disable the visibility level dropdown', () => {
-      wrapper.setProps({ canChangeVisibilityLevel: false });
+      wrapper = mountComponent({ canChangeVisibilityLevel: false });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find('[name="project[visibility_level]"]').attributes().disabled).toBe(
-          'disabled',
-        );
-      });
+      expect(findProjectVisibilityLevelInput().attributes('disabled')).toBe('disabled');
     });
 
     it.each`
@@ -133,56 +136,56 @@ describe('Settings Panel', () => {
     `(
       'sets disabled to $disabled for the visibility option $option when given $allowedOptions',
       ({ option, allowedOptions, disabled }) => {
-        wrapper.setProps({ allowedVisibilityOptions: allowedOptions });
-
-        return wrapper.vm.$nextTick(() => {
-          const attributeValue = wrapper
-            .find(`[name="project[visibility_level]"] option[value="${option}"]`)
-            .attributes().disabled;
-
-          if (disabled) {
-            expect(attributeValue).toBe('disabled');
-          } else {
-            expect(attributeValue).toBeUndefined();
-          }
-        });
+        wrapper = mountComponent({ allowedVisibilityOptions: allowedOptions });
+
+        const attributeValue = findProjectVisibilityLevelInput()
+          .find(`option[value="${option}"]`)
+          .attributes('disabled');
+
+        const expected = disabled ? 'disabled' : undefined;
+
+        expect(attributeValue).toBe(expected);
       },
     );
 
     it('should set the visibility level description based upon the selected visibility level', () => {
-      wrapper.find('[name="project[visibility_level]"]').setValue(visibilityOptions.INTERNAL);
+      wrapper = mountComponent();
+
+      findProjectVisibilityLevelInput().setValue(visibilityOptions.INTERNAL);
 
-      expect(wrapper.find({ ref: 'project-visibility-settings' }).text()).toContain(
+      expect(findProjectVisibilitySettings().text()).toContain(
         visibilityLevelDescriptions[visibilityOptions.INTERNAL],
       );
     });
 
     it('should show the request access checkbox if the visibility level is not private', () => {
-      wrapper = overrideCurrentSettings({ visibilityLevel: visibilityOptions.INTERNAL });
+      wrapper = mountComponent({
+        currentSettings: { visibilityLevel: visibilityOptions.INTERNAL },
+      });
 
-      expect(wrapper.find('[name="project[request_access_enabled]"]').exists()).toBe(true);
+      expect(findRequestAccessEnabledInput().exists()).toBe(true);
     });
 
     it('should not show the request access checkbox if the visibility level is private', () => {
-      wrapper = overrideCurrentSettings({ visibilityLevel: visibilityOptions.PRIVATE });
+      wrapper = mountComponent({ currentSettings: { visibilityLevel: visibilityOptions.PRIVATE } });
 
-      expect(wrapper.find('[name="project[request_access_enabled]"]').exists()).toBe(false);
+      expect(findRequestAccessEnabledInput().exists()).toBe(false);
     });
   });
 
   describe('Repository', () => {
     it('should set the repository help text when the visibility level is set to private', () => {
-      wrapper = overrideCurrentSettings({ visibilityLevel: visibilityOptions.PRIVATE });
+      wrapper = mountComponent({ currentSettings: { visibilityLevel: visibilityOptions.PRIVATE } });
 
-      expect(findRepositoryFeatureProjectRow().props().helpText).toBe(
+      expect(findRepositoryFeatureProjectRow().props('helpText')).toBe(
         'View and edit files in this project.',
       );
     });
 
     it('should set the repository help text with a read access warning when the visibility level is set to non-private', () => {
-      wrapper = overrideCurrentSettings({ visibilityLevel: visibilityOptions.PUBLIC });
+      wrapper = mountComponent({ currentSettings: { visibilityLevel: visibilityOptions.PUBLIC } });
 
-      expect(findRepositoryFeatureProjectRow().props().helpText).toBe(
+      expect(findRepositoryFeatureProjectRow().props('helpText')).toBe(
         'View and edit files in this project. Non-project members will only have read access.',
       );
     });
@@ -190,175 +193,152 @@ describe('Settings Panel', () => {
 
   describe('Merge requests', () => {
     it('should enable the merge requests access level input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.EVERYONE });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+      });
 
-      expect(
-        wrapper
-          .find('[name="project[project_feature_attributes][merge_requests_access_level]"]')
-          .props().disabledInput,
-      ).toBe(false);
+      expect(findMergeRequestsAccessLevelInput().props('disabledInput')).toBe(false);
     });
 
     it('should disable the merge requests access level input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.NOT_ENABLED });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+      });
 
-      expect(
-        wrapper
-          .find('[name="project[project_feature_attributes][merge_requests_access_level]"]')
-          .props().disabledInput,
-      ).toBe(true);
+      expect(findMergeRequestsAccessLevelInput().props('disabledInput')).toBe(true);
     });
   });
 
   describe('Forks', () => {
     it('should enable the forking access level input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.EVERYONE });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+      });
 
-      expect(
-        wrapper.find('[name="project[project_feature_attributes][forking_access_level]"]').props()
-          .disabledInput,
-      ).toBe(false);
+      expect(findForkingAccessLevelInput().props('disabledInput')).toBe(false);
     });
 
     it('should disable the forking access level input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.NOT_ENABLED });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+      });
 
-      expect(
-        wrapper.find('[name="project[project_feature_attributes][forking_access_level]"]').props()
-          .disabledInput,
-      ).toBe(true);
+      expect(findForkingAccessLevelInput().props('disabledInput')).toBe(true);
     });
   });
 
   describe('Pipelines', () => {
     it('should enable the builds access level input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.EVERYONE });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+      });
 
-      expect(
-        wrapper.find('[name="project[project_feature_attributes][builds_access_level]"]').props()
-          .disabledInput,
-      ).toBe(false);
+      expect(findBuildsAccessLevelInput().props('disabledInput')).toBe(false);
     });
 
     it('should disable the builds access level input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings({ repositoryAccessLevel: featureAccessLevel.NOT_ENABLED });
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+      });
 
-      expect(
-        wrapper.find('[name="project[project_feature_attributes][builds_access_level]"]').props()
-          .disabledInput,
-      ).toBe(true);
+      expect(findBuildsAccessLevelInput().props('disabledInput')).toBe(true);
     });
   });
 
   describe('Container registry', () => {
     it('should show the container registry settings if the registry is available', () => {
-      wrapper.setProps({ registryAvailable: true });
+      wrapper = mountComponent({ registryAvailable: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'container-registry-settings' }).exists()).toBe(true);
-      });
+      expect(findContainerRegistrySettings().exists()).toBe(true);
     });
 
     it('should hide the container registry settings if the registry is not available', () => {
-      wrapper.setProps({ registryAvailable: false });
+      wrapper = mountComponent({ registryAvailable: false });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'container-registry-settings' }).exists()).toBe(false);
-      });
+      expect(findContainerRegistrySettings().exists()).toBe(false);
     });
 
     it('should set the container registry settings help path', () => {
-      wrapper.setProps({ registryAvailable: true });
+      wrapper = mountComponent({ registryAvailable: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'container-registry-settings' }).props().helpPath).toBe(
-          defaultProps.registryHelpPath,
-        );
-      });
+      expect(findContainerRegistrySettings().props('helpPath')).toBe(defaultProps.registryHelpPath);
     });
 
     it('should show the container registry public note if the visibility level is public and the registry is available', () => {
-      wrapper = overrideCurrentSettings(
-        { visibilityLevel: visibilityOptions.PUBLIC },
-        { registryAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { visibilityLevel: visibilityOptions.PUBLIC },
+        registryAvailable: true,
+      });
 
-      expect(wrapper.find({ ref: 'container-registry-settings' }).text()).toContain(
+      expect(findContainerRegistrySettings().text()).toContain(
         'Note: the container registry is always visible when a project is public',
       );
     });
 
     it('should hide the container registry public note if the visibility level is private and the registry is available', () => {
-      wrapper = overrideCurrentSettings(
-        { visibilityLevel: visibilityOptions.PRIVATE },
-        { registryAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { visibilityLevel: visibilityOptions.PRIVATE },
+        registryAvailable: true,
+      });
 
-      expect(wrapper.find({ ref: 'container-registry-settings' }).text()).not.toContain(
+      expect(findContainerRegistrySettings().text()).not.toContain(
         'Note: the container registry is always visible when a project is public',
       );
     });
 
     it('should enable the container registry input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.EVERYONE },
-        { registryAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+        registryAvailable: true,
+      });
 
-      expect(
-        wrapper.find('[name="project[container_registry_enabled]"]').props().disabledInput,
-      ).toBe(false);
+      expect(findContainerRegistryEnabledInput().props('disabled')).toBe(false);
     });
 
     it('should disable the container registry input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
-        { registryAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+        registryAvailable: true,
+      });
 
-      expect(
-        wrapper.find('[name="project[container_registry_enabled]"]').props().disabledInput,
-      ).toBe(true);
+      expect(findContainerRegistryEnabledInput().props('disabled')).toBe(true);
     });
   });
 
   describe('Git Large File Storage', () => {
     it('should show the LFS settings if LFS is available', () => {
-      wrapper.setProps({ lfsAvailable: true });
+      wrapper = mountComponent({ lfsAvailable: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(findLFSSettingsRow().exists()).toBe(true);
-      });
+      expect(findLFSSettingsRow().exists()).toBe(true);
     });
 
     it('should hide the LFS settings if LFS is not available', () => {
-      wrapper.setProps({ lfsAvailable: false });
+      wrapper = mountComponent({ lfsAvailable: false });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(findLFSSettingsRow().exists()).toBe(false);
-      });
+      expect(findLFSSettingsRow().exists()).toBe(false);
     });
 
     it('should set the LFS settings help path', () => {
-      expect(findLFSSettingsRow().props().helpPath).toBe(defaultProps.lfsHelpPath);
+      wrapper = mountComponent();
+      expect(findLFSSettingsRow().props('helpPath')).toBe(defaultProps.lfsHelpPath);
     });
 
     it('should enable the LFS input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.EVERYONE },
-        { lfsAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+        lfsAvailable: true,
+      });
 
-      expect(findLFSFeatureToggle().props().disabledInput).toBe(false);
+      expect(findLFSFeatureToggle().props('disabled')).toBe(false);
     });
 
     it('should disable the LFS input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
-        { lfsAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+        lfsAvailable: true,
+      });
 
-      expect(findLFSFeatureToggle().props().disabledInput).toBe(true);
+      expect(findLFSFeatureToggle().props('disabled')).toBe(true);
     });
 
     it('should not change lfsEnabled when disabling the repository', async () => {
@@ -373,8 +353,7 @@ describe('Settings Panel', () => {
       expect(isToggleButtonChecked(lfsFeatureToggleButton)).toBe(true);
       expect(isToggleButtonChecked(repositoryFeatureToggleButton)).toBe(true);
 
-      repositoryFeatureToggleButton.trigger('click');
-      await wrapper.vm.$nextTick();
+      await repositoryFeatureToggleButton.trigger('click');
 
       expect(isToggleButtonChecked(repositoryFeatureToggleButton)).toBe(false);
       // LFS toggle should still be checked
@@ -418,47 +397,39 @@ describe('Settings Panel', () => {
 
   describe('Packages', () => {
     it('should show the packages settings if packages are available', () => {
-      wrapper.setProps({ packagesAvailable: true });
+      wrapper = mountComponent({ packagesAvailable: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'package-settings' }).exists()).toBe(true);
-      });
+      expect(findPackageSettings().exists()).toBe(true);
     });
 
     it('should hide the packages settings if packages are not available', () => {
-      wrapper.setProps({ packagesAvailable: false });
+      wrapper = mountComponent({ packagesAvailable: false });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'package-settings' }).exists()).toBe(false);
-      });
+      expect(findPackageSettings().exists()).toBe(false);
     });
 
     it('should set the package settings help path', () => {
-      wrapper.setProps({ packagesAvailable: true });
+      wrapper = mountComponent({ packagesAvailable: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'package-settings' }).props().helpPath).toBe(
-          defaultProps.packagesHelpPath,
-        );
-      });
+      expect(findPackageSettings().props('helpPath')).toBe(defaultProps.packagesHelpPath);
     });
 
     it('should enable the packages input when the repository is enabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.EVERYONE },
-        { packagesAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.EVERYONE },
+        packagesAvailable: true,
+      });
 
-      expect(wrapper.find('[name="project[packages_enabled]"]').props().disabledInput).toBe(false);
+      expect(findPackagesEnabledInput().props('disabled')).toBe(false);
     });
 
     it('should disable the packages input when the repository is disabled', () => {
-      wrapper = overrideCurrentSettings(
-        { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
-        { packagesAvailable: true },
-      );
+      wrapper = mountComponent({
+        currentSettings: { repositoryAccessLevel: featureAccessLevel.NOT_ENABLED },
+        packagesAvailable: true,
+      });
 
-      expect(wrapper.find('[name="project[packages_enabled]"]').props().disabledInput).toBe(true);
+      expect(findPackagesEnabledInput().props('disabled')).toBe(true);
     });
   });
 
@@ -472,78 +443,66 @@ describe('Settings Panel', () => {
     `(
       'should $visibility the page settings if pagesAvailable is $pagesAvailable and pagesAccessControlEnabled is $pagesAccessControlEnabled',
       ({ pagesAvailable, pagesAccessControlEnabled, visibility }) => {
-        wrapper.setProps({ pagesAvailable, pagesAccessControlEnabled });
+        wrapper = mountComponent({ pagesAvailable, pagesAccessControlEnabled });
 
-        return wrapper.vm.$nextTick(() => {
-          expect(wrapper.find({ ref: 'pages-settings' }).exists()).toBe(visibility === 'show');
-        });
+        expect(findPagesSettings().exists()).toBe(visibility === 'show');
       },
     );
 
     it('should set the pages settings help path', () => {
-      wrapper.setProps({ pagesAvailable: true, pagesAccessControlEnabled: true });
+      wrapper = mountComponent({ pagesAvailable: true, pagesAccessControlEnabled: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'pages-settings' }).props().helpPath).toBe(
-          defaultProps.pagesHelpPath,
-        );
-      });
+      expect(findPagesSettings().props('helpPath')).toBe(defaultProps.pagesHelpPath);
     });
   });
 
   describe('Email notifications', () => {
     it('should show the disable email notifications input if emails an be disabled', () => {
-      wrapper.setProps({ canDisableEmails: true });
+      wrapper = mountComponent({ canDisableEmails: true });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'email-settings' }).exists()).toBe(true);
-      });
+      expect(findEmailSettings().exists()).toBe(true);
     });
 
     it('should hide the disable email notifications input if emails cannot be disabled', () => {
-      wrapper.setProps({ canDisableEmails: false });
+      wrapper = mountComponent({ canDisableEmails: false });
 
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'email-settings' }).exists()).toBe(false);
-      });
+      expect(findEmailSettings().exists()).toBe(false);
     });
   });
 
   describe('Default award emojis', () => {
     it('should show the "Show default award emojis" input', () => {
-      return wrapper.vm.$nextTick(() => {
-        expect(
-          wrapper
-            .find('input[name="project[project_setting_attributes][show_default_award_emojis]"]')
-            .exists(),
-        ).toBe(true);
-      });
+      wrapper = mountComponent();
+
+      expect(findShowDefaultAwardEmojis().exists()).toBe(true);
     });
   });
 
   describe('Metrics dashboard', () => {
     it('should show the metrics dashboard access toggle', () => {
-      return wrapper.vm.$nextTick(() => {
-        expect(wrapper.find({ ref: 'metrics-visibility-settings' }).exists()).toBe(true);
-      });
+      wrapper = mountComponent();
+
+      expect(findMetricsVisibilitySettings().exists()).toBe(true);
     });
 
     it('should contain help text', () => {
-      expect(wrapper.find({ ref: 'metrics-visibility-settings' }).props().helpText).toBe(
+      wrapper = mountComponent();
+
+      expect(findMetricsVisibilitySettings().props('helpText')).toBe(
         "Visualize the project's performance metrics.",
       );
     });
 
     it.each`
-      scenario                                                     | selectedOption                                | selectedOptionLabel
-      ${{ visibilityLevel: visibilityOptions.PRIVATE }}            | ${String(featureAccessLevel.PROJECT_MEMBERS)} | ${'Only Project Members'}
-      ${{ operationsAccessLevel: featureAccessLevel.NOT_ENABLED }} | ${String(featureAccessLevel.NOT_ENABLED)}     | ${'Enable feature to choose access level'}
+      scenario                                                                          | selectedOption                                | selectedOptionLabel
+      ${{ currentSettings: { visibilityLevel: visibilityOptions.PRIVATE } }}            | ${String(featureAccessLevel.PROJECT_MEMBERS)} | ${'Only Project Members'}
+      ${{ currentSettings: { operationsAccessLevel: featureAccessLevel.NOT_ENABLED } }} | ${String(featureAccessLevel.NOT_ENABLED)}     | ${'Enable feature to choose access level'}
     `(
       'should disable the metrics visibility dropdown when #scenario',
       ({ scenario, selectedOption, selectedOptionLabel }) => {
-        wrapper = overrideCurrentSettings(scenario, {}, mount);
+        wrapper = mountComponent(scenario, mount);
 
-        const select = wrapper.find({ ref: 'metrics-visibility-settings' }).find('select');
+        const select = findMetricsVisibilitySettings().find('select');
         const option = select.find('option');
 
         expect(select.attributes('disabled')).toBe('disabled');
@@ -556,31 +515,29 @@ describe('Settings Panel', () => {
 
   describe('Settings panel with feature flags', () => {
     describe('Allow edit of commit message', () => {
-      it('should show the allow editing of commit messages checkbox', async () => {
+      it('should show the allow editing of commit messages checkbox', () => {
         wrapper = mountComponent({
           glFeatures: { allowEditingCommitMessages: true },
         });
 
-        await wrapper.vm.$nextTick();
-
-        expect(wrapper.find({ ref: 'allow-editing-commit-messages' }).exists()).toBe(true);
+        expect(findAllowEditingCommitMessages()).toBe(true);
       });
     });
   });
 
   describe('Analytics', () => {
-    it('should show the analytics toggle', async () => {
-      await wrapper.vm.$nextTick();
+    it('should show the analytics toggle', () => {
+      wrapper = mountComponent();
 
       expect(findAnalyticsRow().exists()).toBe(true);
     });
   });
 
   describe('Operations', () => {
-    it('should show the operations toggle', async () => {
-      await wrapper.vm.$nextTick();
+    it('should show the operations toggle', () => {
+      wrapper = mountComponent();
 
-      expect(wrapper.find({ ref: 'operations-settings' }).exists()).toBe(true);
+      expect(findOperationsSettings().exists()).toBe(true);
     });
   });
 });