diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-05-04 06:09:07 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-05-04 06:09:07 +0300 |
commit | 44ca34ad66b8b7f54c0c8a8f549aafb7293c8a38 (patch) | |
tree | 2c53ff23c11272b1546d5901c95ce1357bceb5c7 /spec/frontend | |
parent | b301036d090d8a7365f98a0444d09f8a09664834 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/frontend')
90 files changed, 2 insertions, 517 deletions
diff --git a/spec/frontend/alert_management/components/alert_management_empty_state_spec.js b/spec/frontend/alert_management/components/alert_management_empty_state_spec.js index 0d6bc1b74fb..b2889d429b1 100644 --- a/spec/frontend/alert_management/components/alert_management_empty_state_spec.js +++ b/spec/frontend/alert_management/components/alert_management_empty_state_spec.js @@ -19,12 +19,6 @@ describe('AlertManagementEmptyState', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - const EmptyState = () => wrapper.findComponent(GlEmptyState); describe('Empty state', () => { diff --git a/spec/frontend/alert_management/components/alert_management_list_wrapper_spec.js b/spec/frontend/alert_management/components/alert_management_list_wrapper_spec.js index 3a5fb99fdf1..3cc2d59295c 100644 --- a/spec/frontend/alert_management/components/alert_management_list_wrapper_spec.js +++ b/spec/frontend/alert_management/components/alert_management_list_wrapper_spec.js @@ -20,12 +20,6 @@ describe('AlertManagementList', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('Alert List Wrapper', () => { it('should show the empty state when alerts are not enabled', () => { expect(wrapper.findComponent(AlertManagementEmptyState).exists()).toBe(true); diff --git a/spec/frontend/alert_management/components/alert_management_table_spec.js b/spec/frontend/alert_management/components/alert_management_table_spec.js index 3f709d8c9f5..afd88e1a6ac 100644 --- a/spec/frontend/alert_management/components/alert_management_table_spec.js +++ b/spec/frontend/alert_management/components/alert_management_table_spec.js @@ -79,9 +79,6 @@ describe('AlertManagementTable', () => { }); afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } mock.restore(); }); diff --git a/spec/frontend/alerts_settings/components/alert_mapping_builder_spec.js b/spec/frontend/alerts_settings/components/alert_mapping_builder_spec.js index 2b8479eab6d..04dc0fef5da 100644 --- a/spec/frontend/alerts_settings/components/alert_mapping_builder_spec.js +++ b/spec/frontend/alerts_settings/components/alert_mapping_builder_spec.js @@ -19,13 +19,6 @@ describe('AlertMappingBuilder', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - beforeEach(() => { mountComponent(); }); diff --git a/spec/frontend/alerts_settings/components/alerts_form_spec.js b/spec/frontend/alerts_settings/components/alerts_form_spec.js index 33098282bf8..c4e5598ed39 100644 --- a/spec/frontend/alerts_settings/components/alerts_form_spec.js +++ b/spec/frontend/alerts_settings/components/alerts_form_spec.js @@ -22,12 +22,6 @@ describe('Alert integration settings form', () => { }); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('default state', () => { it('should match the default snapshot', () => { expect(wrapper.element).toMatchSnapshot(); diff --git a/spec/frontend/alerts_settings/components/alerts_integrations_list_spec.js b/spec/frontend/alerts_settings/components/alerts_integrations_list_spec.js index 9983af873c2..76d0c12e434 100644 --- a/spec/frontend/alerts_settings/components/alerts_integrations_list_spec.js +++ b/spec/frontend/alerts_settings/components/alerts_integrations_list_spec.js @@ -42,13 +42,6 @@ describe('AlertIntegrationsList', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - beforeEach(() => { mountComponent(); }); diff --git a/spec/frontend/alerts_settings/components/alerts_settings_form_spec.js b/spec/frontend/alerts_settings/components/alerts_settings_form_spec.js index 5eb5ae2f783..4a0c7f65493 100644 --- a/spec/frontend/alerts_settings/components/alerts_settings_form_spec.js +++ b/spec/frontend/alerts_settings/components/alerts_settings_form_spec.js @@ -63,12 +63,6 @@ describe('AlertsSettingsForm', () => { const findActionBtn = () => wrapper.findByTestId('payload-action-btn'); const findTabs = () => wrapper.findAllComponents(GlTab); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - const selectOptionAtIndex = async (index) => { const options = findSelect().findAll('option'); await options.at(index).setSelected(); diff --git a/spec/frontend/boards/components/new_board_button_spec.js b/spec/frontend/boards/components/new_board_button_spec.js index 2bbd3797abf..7ec35d1b796 100644 --- a/spec/frontend/boards/components/new_board_button_spec.js +++ b/spec/frontend/boards/components/new_board_button_spec.js @@ -21,12 +21,6 @@ describe('NewBoardButton', () => { }), ); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('control variant', () => { beforeAll(() => { stubExperiments({ [FEATURE]: 'control' }); diff --git a/spec/frontend/branches/components/sort_dropdown_spec.js b/spec/frontend/branches/components/sort_dropdown_spec.js index bd41b0daaaa..64ef30bb8a8 100644 --- a/spec/frontend/branches/components/sort_dropdown_spec.js +++ b/spec/frontend/branches/components/sort_dropdown_spec.js @@ -29,12 +29,6 @@ describe('Branches Sort Dropdown', () => { const findSearchBox = () => wrapper.findComponent(GlSearchBoxByClick); const findBranchesDropdown = () => wrapper.findByTestId('branches-dropdown'); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('When in overview mode', () => { beforeEach(() => { wrapper = createWrapper(); diff --git a/spec/frontend/clusters_list/components/agent_empty_state_spec.js b/spec/frontend/clusters_list/components/agent_empty_state_spec.js index 22775aa6603..2e52d16c739 100644 --- a/spec/frontend/clusters_list/components/agent_empty_state_spec.js +++ b/spec/frontend/clusters_list/components/agent_empty_state_spec.js @@ -22,12 +22,6 @@ describe('AgentEmptyStateComponent', () => { }); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('renders the empty state', () => { expect(findEmptyState().exists()).toBe(true); }); diff --git a/spec/frontend/content_editor/components/toolbar_text_style_dropdown_spec.js b/spec/frontend/content_editor/components/toolbar_text_style_dropdown_spec.js index 31ed13541e6..97f6bdaf778 100644 --- a/spec/frontend/content_editor/components/toolbar_text_style_dropdown_spec.js +++ b/spec/frontend/content_editor/components/toolbar_text_style_dropdown_spec.js @@ -117,7 +117,6 @@ describe('content_editor/components/toolbar_text_style_dropdown', () => { }, ], ]); - wrapper.destroy(); }); }); }); diff --git a/spec/frontend/diffs/components/app_spec.js b/spec/frontend/diffs/components/app_spec.js index aa74721ee3c..42eec0af961 100644 --- a/spec/frontend/diffs/components/app_spec.js +++ b/spec/frontend/diffs/components/app_spec.js @@ -93,12 +93,6 @@ describe('diffs/components/app', () => { // reset globals window.mrTabs = oldMrTabs; - // reset component - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - mock.restore(); }); diff --git a/spec/frontend/diffs/components/diff_file_spec.js b/spec/frontend/diffs/components/diff_file_spec.js index 8552e69012b..389b192a515 100644 --- a/spec/frontend/diffs/components/diff_file_spec.js +++ b/spec/frontend/diffs/components/diff_file_spec.js @@ -511,8 +511,6 @@ describe('DiffFile', () => { }); it('loads collapsed file on mounted when single file mode is enabled', async () => { - wrapper.destroy(); - const file = { ...getReadableFile(), load_collapsed_diff_url: '/diff_for_path', @@ -531,10 +529,6 @@ describe('DiffFile', () => { }); describe('merge conflicts', () => { - beforeEach(() => { - wrapper.destroy(); - }); - it('does not render conflict alert', () => { const file = { ...getReadableFile(), diff --git a/spec/frontend/environments/canary_ingress_spec.js b/spec/frontend/environments/canary_ingress_spec.js index 17ecd93361f..e0247731b63 100644 --- a/spec/frontend/environments/canary_ingress_spec.js +++ b/spec/frontend/environments/canary_ingress_spec.js @@ -33,12 +33,6 @@ describe('/environments/components/canary_ingress.vue', () => { createComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('stable weight', () => { let stableWeightDropdown; diff --git a/spec/frontend/environments/canary_update_modal_spec.js b/spec/frontend/environments/canary_update_modal_spec.js index a101ed4e00a..4fa7b34d817 100644 --- a/spec/frontend/environments/canary_update_modal_spec.js +++ b/spec/frontend/environments/canary_update_modal_spec.js @@ -30,12 +30,6 @@ describe('/environments/components/canary_update_modal.vue', () => { modal = wrapper.findComponent(GlModal); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - beforeEach(() => { createComponent(); }); diff --git a/spec/frontend/environments/environment_item_spec.js b/spec/frontend/environments/environment_item_spec.js index 59e94dfd662..690db66efd1 100644 --- a/spec/frontend/environments/environment_item_spec.js +++ b/spec/frontend/environments/environment_item_spec.js @@ -19,10 +19,6 @@ describe('Environment item', () => { let tracking; const factory = (options = {}) => { - // This destroys any wrappers created before a nested call to factory reassigns it - if (wrapper && wrapper.destroy) { - wrapper.destroy(); - } wrapper = mount(EnvironmentItem, { ...options, }); diff --git a/spec/frontend/environments/environment_pin_spec.js b/spec/frontend/environments/environment_pin_spec.js index 2f38dea2833..ee195b41bc8 100644 --- a/spec/frontend/environments/environment_pin_spec.js +++ b/spec/frontend/environments/environment_pin_spec.js @@ -11,10 +11,6 @@ describe('Pin Component', () => { let wrapper; const factory = (options = {}) => { - // This destroys any wrappers created before a nested call to factory reassigns it - if (wrapper && wrapper.destroy) { - wrapper.destroy(); - } wrapper = shallowMount(PinComponent, { ...options, }); diff --git a/spec/frontend/environments/environment_table_spec.js b/spec/frontend/environments/environment_table_spec.js index 652b0f807fe..f41d1324b81 100644 --- a/spec/frontend/environments/environment_table_spec.js +++ b/spec/frontend/environments/environment_table_spec.js @@ -16,10 +16,6 @@ describe('Environment table', () => { let wrapper; const factory = (options = {}) => { - // This destroys any wrappers created before a nested call to factory reassigns it - if (wrapper && wrapper.destroy) { - wrapper.destroy(); - } wrapper = mount(EnvironmentTable, { ...options, }); diff --git a/spec/frontend/error_tracking/components/error_details_spec.js b/spec/frontend/error_tracking/components/error_details_spec.js index 4d77560ffe8..4e73db13a17 100644 --- a/spec/frontend/error_tracking/components/error_details_spec.js +++ b/spec/frontend/error_tracking/components/error_details_spec.js @@ -108,12 +108,6 @@ describe('ErrorDetails', () => { }; }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('loading', () => { beforeEach(() => { mountComponent(); diff --git a/spec/frontend/error_tracking/components/error_tracking_actions_spec.js b/spec/frontend/error_tracking/components/error_tracking_actions_spec.js index 5f6c9ddb4d7..d959d73c86b 100644 --- a/spec/frontend/error_tracking/components/error_tracking_actions_spec.js +++ b/spec/frontend/error_tracking/components/error_tracking_actions_spec.js @@ -29,12 +29,6 @@ describe('Error Tracking Actions', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - const findButtons = () => wrapper.findAllComponents(GlButton); describe('when error status is unresolved', () => { diff --git a/spec/frontend/error_tracking/components/stacktrace_entry_spec.js b/spec/frontend/error_tracking/components/stacktrace_entry_spec.js index 0de4277b08a..45fc1ad04ff 100644 --- a/spec/frontend/error_tracking/components/stacktrace_entry_spec.js +++ b/spec/frontend/error_tracking/components/stacktrace_entry_spec.js @@ -28,12 +28,6 @@ describe('Stacktrace Entry', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('should render stacktrace entry collapsed', () => { mountComponent({ lines }); expect(wrapper.findComponent(StackTraceEntry).exists()).toBe(true); diff --git a/spec/frontend/error_tracking/components/stacktrace_spec.js b/spec/frontend/error_tracking/components/stacktrace_spec.js index cd5a57f5683..29301c3e5ee 100644 --- a/spec/frontend/error_tracking/components/stacktrace_spec.js +++ b/spec/frontend/error_tracking/components/stacktrace_spec.js @@ -25,12 +25,6 @@ describe('ErrorDetails', () => { } describe('Stacktrace', () => { - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('should render single Stacktrace entry', () => { mountComponent([stackTraceEntry]); expect(wrapper.findAllComponents(StackTraceEntry).length).toBe(1); diff --git a/spec/frontend/error_tracking_settings/components/app_spec.js b/spec/frontend/error_tracking_settings/components/app_spec.js index dfe9652073e..9b7701d46bc 100644 --- a/spec/frontend/error_tracking_settings/components/app_spec.js +++ b/spec/frontend/error_tracking_settings/components/app_spec.js @@ -68,12 +68,6 @@ describe('error tracking settings app', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('section', () => { it('renders the form and dropdown', () => { expect(wrapper.findComponent(ErrorTrackingForm).exists()).toBe(true); diff --git a/spec/frontend/error_tracking_settings/components/error_tracking_form_spec.js b/spec/frontend/error_tracking_settings/components/error_tracking_form_spec.js index 69d684faec2..b1cf5d673f1 100644 --- a/spec/frontend/error_tracking_settings/components/error_tracking_form_spec.js +++ b/spec/frontend/error_tracking_settings/components/error_tracking_form_spec.js @@ -24,12 +24,6 @@ describe('error tracking settings form', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('an empty form', () => { it('is rendered', () => { expect(wrapper.findAllComponents(GlFormInput).length).toBe(2); diff --git a/spec/frontend/error_tracking_settings/components/project_dropdown_spec.js b/spec/frontend/error_tracking_settings/components/project_dropdown_spec.js index 8653ebac20d..03d090c5314 100644 --- a/spec/frontend/error_tracking_settings/components/project_dropdown_spec.js +++ b/spec/frontend/error_tracking_settings/components/project_dropdown_spec.js @@ -33,12 +33,6 @@ describe('error tracking settings project dropdown', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('empty project list', () => { it('renders the dropdown', () => { expect(wrapper.find('#project-dropdown').exists()).toBe(true); diff --git a/spec/frontend/feature_flags/components/edit_feature_flag_spec.js b/spec/frontend/feature_flags/components/edit_feature_flag_spec.js index c26fd80865d..b8d058e7bc5 100644 --- a/spec/frontend/feature_flags/components/edit_feature_flag_spec.js +++ b/spec/frontend/feature_flags/components/edit_feature_flag_spec.js @@ -24,10 +24,6 @@ describe('Edit feature flag form', () => { }); const factory = (provide = { searchPath: '/search' }) => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } wrapper = shallowMount(EditFeatureFlag, { store, provide, diff --git a/spec/frontend/feature_flags/components/empty_state_spec.js b/spec/frontend/feature_flags/components/empty_state_spec.js index d983332f7c1..4aa0b261e2a 100644 --- a/spec/frontend/feature_flags/components/empty_state_spec.js +++ b/spec/frontend/feature_flags/components/empty_state_spec.js @@ -44,12 +44,6 @@ describe('feature_flags/components/feature_flags_tab.vue', () => { }, ); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - describe('alerts', () => { let alerts; diff --git a/spec/frontend/feature_flags/components/new_environments_dropdown_spec.js b/spec/frontend/feature_flags/components/new_environments_dropdown_spec.js index 14e1f34bc59..6156addd63f 100644 --- a/spec/frontend/feature_flags/components/new_environments_dropdown_spec.js +++ b/spec/frontend/feature_flags/components/new_environments_dropdown_spec.js @@ -22,10 +22,6 @@ describe('New Environments Dropdown', () => { afterEach(() => { axiosMock.restore(); - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } }); describe('before results', () => { diff --git a/spec/frontend/feature_flags/components/new_feature_flag_spec.js b/spec/frontend/feature_flags/components/new_feature_flag_spec.js index 46c9118cbd9..c5418477661 100644 --- a/spec/frontend/feature_flags/components/new_feature_flag_spec.js +++ b/spec/frontend/feature_flags/components/new_feature_flag_spec.js @@ -22,10 +22,6 @@ describe('New feature flag form', () => { }); const factory = (opts = {}) => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } wrapper = shallowMount(NewFeatureFlag, { store, provide: { diff --git a/spec/frontend/feature_flags/components/strategies/flexible_rollout_spec.js b/spec/frontend/feature_flags/components/strategies/flexible_rollout_spec.js index 5feaf094701..a6eb81ef6f0 100644 --- a/spec/frontend/feature_flags/components/strategies/flexible_rollout_spec.js +++ b/spec/frontend/feature_flags/components/strategies/flexible_rollout_spec.js @@ -20,12 +20,6 @@ describe('feature_flags/components/strategies/flexible_rollout.vue', () => { const factory = (props = {}) => mount(FlexibleRollout, { propsData: { ...DEFAULT_PROPS, ...props } }); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - describe('with valid percentage', () => { beforeEach(() => { wrapper = factory(); diff --git a/spec/frontend/feature_flags/components/strategies/parameter_form_group_spec.js b/spec/frontend/feature_flags/components/strategies/parameter_form_group_spec.js index 365f1e534b5..8ad70466e90 100644 --- a/spec/frontend/feature_flags/components/strategies/parameter_form_group_spec.js +++ b/spec/frontend/feature_flags/components/strategies/parameter_form_group_spec.js @@ -24,12 +24,6 @@ describe('~/feature_flags/strategies/parameter_form_group.vue', () => { slot = wrapper.find('[data-testid="slot"]'); }); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - it('should display the default slot', () => { expect(slot.exists()).toBe(true); }); diff --git a/spec/frontend/feature_flags/components/strategies/percent_rollout_spec.js b/spec/frontend/feature_flags/components/strategies/percent_rollout_spec.js index b20061c12a2..e00869fdd09 100644 --- a/spec/frontend/feature_flags/components/strategies/percent_rollout_spec.js +++ b/spec/frontend/feature_flags/components/strategies/percent_rollout_spec.js @@ -18,12 +18,6 @@ describe('~/feature_flags/components/strategies/percent_rollout.vue', () => { const factory = (props = {}) => mount(PercentRollout, { propsData: { ...DEFAULT_PROPS, ...props } }); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - describe('with valid percentage', () => { beforeEach(() => { wrapper = factory(); diff --git a/spec/frontend/feature_flags/components/strategies/users_with_id_spec.js b/spec/frontend/feature_flags/components/strategies/users_with_id_spec.js index ae489f3a6e6..f3b8535a650 100644 --- a/spec/frontend/feature_flags/components/strategies/users_with_id_spec.js +++ b/spec/frontend/feature_flags/components/strategies/users_with_id_spec.js @@ -18,12 +18,6 @@ describe('~/feature_flags/components/users_with_id.vue', () => { textarea = wrapper.findComponent(GlFormTextarea); }); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - it('should display the current value of the parameters', () => { expect(textarea.element.value).toBe(usersWithIdStrategy.parameters.userIds); }); diff --git a/spec/frontend/feature_flags/components/strategy_parameters_spec.js b/spec/frontend/feature_flags/components/strategy_parameters_spec.js index cd8270f1801..bc34888d1c1 100644 --- a/spec/frontend/feature_flags/components/strategy_parameters_spec.js +++ b/spec/frontend/feature_flags/components/strategy_parameters_spec.js @@ -28,12 +28,6 @@ describe('~/feature_flags/components/strategy_parameters.vue', () => { }, }); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - describe.each` name | component ${ROLLOUT_STRATEGY_ALL_USERS} | ${Default} diff --git a/spec/frontend/feature_flags/components/strategy_spec.js b/spec/frontend/feature_flags/components/strategy_spec.js index 84d4180fe63..1428d99aa76 100644 --- a/spec/frontend/feature_flags/components/strategy_spec.js +++ b/spec/frontend/feature_flags/components/strategy_spec.js @@ -44,10 +44,6 @@ describe('Feature flags strategy', () => { provide, }, ) => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } wrapper = mount(Strategy, { store: createStore({ projectId: '1' }), ...opts }); }; @@ -55,13 +51,6 @@ describe('Feature flags strategy', () => { Api.searchFeatureFlagUserLists.mockResolvedValue({ data: [userList] }); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('helper links', () => { const propsData = { strategy: {}, index: 0, userLists: [userList] }; factory({ propsData, provide }); diff --git a/spec/frontend/grafana_integration/components/grafana_integration_spec.js b/spec/frontend/grafana_integration/components/grafana_integration_spec.js index 9cb27670c98..540fc597aa9 100644 --- a/spec/frontend/grafana_integration/components/grafana_integration_spec.js +++ b/spec/frontend/grafana_integration/components/grafana_integration_spec.js @@ -28,11 +28,8 @@ describe('grafana integration component', () => { }); afterEach(() => { - if (wrapper.destroy) { - wrapper.destroy(); - createAlert.mockReset(); - refreshCurrentPage.mockReset(); - } + createAlert.mockReset(); + refreshCurrentPage.mockReset(); }); describe('default state', () => { diff --git a/spec/frontend/groups/components/item_caret_spec.js b/spec/frontend/groups/components/item_caret_spec.js index 2333f04bb2e..ff273fcf6da 100644 --- a/spec/frontend/groups/components/item_caret_spec.js +++ b/spec/frontend/groups/components/item_caret_spec.js @@ -15,13 +15,6 @@ describe('ItemCaret', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findAllGlIcons = () => wrapper.findAllComponents(GlIcon); const findGlIcon = () => wrapper.findComponent(GlIcon); diff --git a/spec/frontend/groups/components/item_stats_spec.js b/spec/frontend/groups/components/item_stats_spec.js index 0c2912adc66..b98e60bed63 100644 --- a/spec/frontend/groups/components/item_stats_spec.js +++ b/spec/frontend/groups/components/item_stats_spec.js @@ -17,13 +17,6 @@ describe('ItemStats', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findItemStatsValue = () => wrapper.findComponent(ItemStatsValue); describe('template', () => { diff --git a/spec/frontend/groups/components/item_stats_value_spec.js b/spec/frontend/groups/components/item_stats_value_spec.js index b9db83c7dd7..e110004dbac 100644 --- a/spec/frontend/groups/components/item_stats_value_spec.js +++ b/spec/frontend/groups/components/item_stats_value_spec.js @@ -18,13 +18,6 @@ describe('ItemStatsValue', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findGlIcon = () => wrapper.findComponent(GlIcon); const findStatValue = () => wrapper.find('[data-testid="itemStatValue"]'); diff --git a/spec/frontend/groups/components/item_type_icon_spec.js b/spec/frontend/groups/components/item_type_icon_spec.js index aa00e82150b..c269dc98a45 100644 --- a/spec/frontend/groups/components/item_type_icon_spec.js +++ b/spec/frontend/groups/components/item_type_icon_spec.js @@ -16,13 +16,6 @@ describe('ItemTypeIcon', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findGlIcon = () => wrapper.findComponent(GlIcon); describe('template', () => { diff --git a/spec/frontend/ide/components/jobs/list_spec.js b/spec/frontend/ide/components/jobs/list_spec.js index b4c7eb51781..0ece42bce51 100644 --- a/spec/frontend/ide/components/jobs/list_spec.js +++ b/spec/frontend/ide/components/jobs/list_spec.js @@ -50,11 +50,6 @@ describe('IDE stages list', () => { Object.values(storeActions).forEach((actionMock) => actionMock.mockClear()); }); - afterAll(() => { - wrapper.destroy(); - wrapper = null; - }); - it('renders loading icon when no stages & loading', () => { createComponent({ loading: true, stages: [] }); diff --git a/spec/frontend/import_entities/import_projects/components/bitbucket_status_table_spec.js b/spec/frontend/import_entities/import_projects/components/bitbucket_status_table_spec.js index 9eae4ed974e..5773ab33e05 100644 --- a/spec/frontend/import_entities/import_projects/components/bitbucket_status_table_spec.js +++ b/spec/frontend/import_entities/import_projects/components/bitbucket_status_table_spec.js @@ -14,13 +14,6 @@ const ImportProjectsTableStub = { describe('BitbucketStatusTable', () => { let wrapper; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - function createComponent(propsData, importProjectsTableStub = true, slots) { wrapper = shallowMount(BitbucketStatusTable, { propsData, diff --git a/spec/frontend/import_entities/import_projects/components/import_projects_table_spec.js b/spec/frontend/import_entities/import_projects/components/import_projects_table_spec.js index 51f82dab381..f7bc0e4a0e8 100644 --- a/spec/frontend/import_entities/import_projects/components/import_projects_table_spec.js +++ b/spec/frontend/import_entities/import_projects/components/import_projects_table_spec.js @@ -81,13 +81,6 @@ describe('ImportProjectsTable', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - it('renders a loading icon while repos are loading', () => { createComponent({ state: { isLoadingRepos: true } }); diff --git a/spec/frontend/incidents/components/incidents_list_spec.js b/spec/frontend/incidents/components/incidents_list_spec.js index 6e64eeaf295..a0710ddb06c 100644 --- a/spec/frontend/incidents/components/incidents_list_spec.js +++ b/spec/frontend/incidents/components/incidents_list_spec.js @@ -97,13 +97,6 @@ describe('Incidents List', () => { ); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - it('shows the loading state', () => { mountComponent({ loading: true, diff --git a/spec/frontend/issuable/related_issues/components/issue_token_spec.js b/spec/frontend/issuable/related_issues/components/issue_token_spec.js index bacebbade7f..4f2a96306e3 100644 --- a/spec/frontend/issuable/related_issues/components/issue_token_spec.js +++ b/spec/frontend/issuable/related_issues/components/issue_token_spec.js @@ -24,13 +24,6 @@ describe('IssueToken', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findLink = () => wrapper.findComponent({ ref: 'link' }); const findReference = () => wrapper.findComponent({ ref: 'reference' }); const findReferenceIcon = () => wrapper.find('[data-testid="referenceIcon"]'); diff --git a/spec/frontend/issuable/related_issues/components/related_issues_block_spec.js b/spec/frontend/issuable/related_issues/components/related_issues_block_spec.js index 8807bc311f0..e97c0312181 100644 --- a/spec/frontend/issuable/related_issues/components/related_issues_block_spec.js +++ b/spec/frontend/issuable/related_issues/components/related_issues_block_spec.js @@ -57,13 +57,6 @@ describe('RelatedIssuesBlock', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('with defaults', () => { beforeEach(() => { createComponent(); diff --git a/spec/frontend/issuable/related_issues/components/related_issues_list_spec.js b/spec/frontend/issuable/related_issues/components/related_issues_list_spec.js index 0a6a0a90d44..592dc19f0ea 100644 --- a/spec/frontend/issuable/related_issues/components/related_issues_list_spec.js +++ b/spec/frontend/issuable/related_issues/components/related_issues_list_spec.js @@ -37,13 +37,6 @@ describe('RelatedIssuesList', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('with defaults', () => { const heading = 'Related to'; diff --git a/spec/frontend/issues/show/components/incidents/highlight_bar_spec.js b/spec/frontend/issues/show/components/incidents/highlight_bar_spec.js index 1cfb7d12a91..ad730fd69f7 100644 --- a/spec/frontend/issues/show/components/incidents/highlight_bar_spec.js +++ b/spec/frontend/issues/show/components/incidents/highlight_bar_spec.js @@ -34,13 +34,6 @@ describe('Highlight Bar', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findLink = () => wrapper.findComponent(GlLink); describe('empty state', () => { diff --git a/spec/frontend/issues/show/components/incidents/timeline_events_tab_spec.js b/spec/frontend/issues/show/components/incidents/timeline_events_tab_spec.js index 48c3f0984a0..41c103d5bcb 100644 --- a/spec/frontend/issues/show/components/incidents/timeline_events_tab_spec.js +++ b/spec/frontend/issues/show/components/incidents/timeline_events_tab_spec.js @@ -44,12 +44,6 @@ describe('TimelineEventsTab', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - const findLoadingSpinner = () => wrapper.findComponent(GlLoadingIcon); const findEmptyState = () => wrapper.findComponent(GlEmptyState); const findTimelineEventsList = () => wrapper.findComponent(IncidentTimelineEventsList); diff --git a/spec/frontend/issues/show/components/sentry_error_stack_trace_spec.js b/spec/frontend/issues/show/components/sentry_error_stack_trace_spec.js index d4202f4a6ab..02b20b9e7b7 100644 --- a/spec/frontend/issues/show/components/sentry_error_stack_trace_spec.js +++ b/spec/frontend/issues/show/components/sentry_error_stack_trace_spec.js @@ -53,12 +53,6 @@ describe('Sentry Error Stack Trace', () => { }); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('loading', () => { it('should show spinner while loading', () => { mountComponent(); diff --git a/spec/frontend/jobs/components/job/empty_state_spec.js b/spec/frontend/jobs/components/job/empty_state_spec.js index c6ab259bf46..970c2591795 100644 --- a/spec/frontend/jobs/components/job/empty_state_spec.js +++ b/spec/frontend/jobs/components/job/empty_state_spec.js @@ -35,13 +35,6 @@ describe('Empty State', () => { const findAction = () => wrapper.findByTestId('job-empty-state-action'); const findManualVarsForm = () => wrapper.findComponent(ManualVariablesForm); - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('renders image and title', () => { beforeEach(() => { createWrapper(); diff --git a/spec/frontend/jobs/components/job/job_log_controllers_spec.js b/spec/frontend/jobs/components/job/job_log_controllers_spec.js index db060c0d606..218096b9745 100644 --- a/spec/frontend/jobs/components/job/job_log_controllers_spec.js +++ b/spec/frontend/jobs/components/job/job_log_controllers_spec.js @@ -16,9 +16,6 @@ describe('Job log controllers', () => { }); afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } commonUtils.backOff.mockReset(); }); diff --git a/spec/frontend/jobs/components/job/job_retry_forward_deployment_modal_spec.js b/spec/frontend/jobs/components/job/job_retry_forward_deployment_modal_spec.js index 712269a1e83..a44a13259aa 100644 --- a/spec/frontend/jobs/components/job/job_retry_forward_deployment_modal_spec.js +++ b/spec/frontend/jobs/components/job/job_retry_forward_deployment_modal_spec.js @@ -27,13 +27,6 @@ describe('Job Retry Forward Deployment Modal', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - beforeEach(createWrapper); describe('Modal configuration', () => { diff --git a/spec/frontend/jobs/components/job/job_sidebar_details_container_spec.js b/spec/frontend/jobs/components/job/job_sidebar_details_container_spec.js index 4da17ed8366..c1028f3929d 100644 --- a/spec/frontend/jobs/components/job/job_sidebar_details_container_spec.js +++ b/spec/frontend/jobs/components/job/job_sidebar_details_container_spec.js @@ -26,13 +26,6 @@ describe('Job Sidebar Details Container', () => { ); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('when no details are available', () => { beforeEach(() => { createWrapper(); diff --git a/spec/frontend/jobs/components/job/job_sidebar_retry_button_spec.js b/spec/frontend/jobs/components/job/job_sidebar_retry_button_spec.js index 91821a38a78..8a63bfdc3d6 100644 --- a/spec/frontend/jobs/components/job/job_sidebar_retry_button_spec.js +++ b/spec/frontend/jobs/components/job/job_sidebar_retry_button_spec.js @@ -24,12 +24,6 @@ describe('Job Sidebar Retry Button', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - beforeEach(createWrapper); it.each([ diff --git a/spec/frontend/jobs/components/job/stuck_block_spec.js b/spec/frontend/jobs/components/job/stuck_block_spec.js index 8dc570cce27..0f014a9222b 100644 --- a/spec/frontend/jobs/components/job/stuck_block_spec.js +++ b/spec/frontend/jobs/components/job/stuck_block_spec.js @@ -5,13 +5,6 @@ import StuckBlock from '~/jobs/components/job/stuck_block.vue'; describe('Stuck Block Job component', () => { let wrapper; - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - wrapper = null; - } - }); - const createWrapper = (props) => { wrapper = shallowMount(StuckBlock, { propsData: { diff --git a/spec/frontend/monitoring/components/dashboard_url_time_spec.js b/spec/frontend/monitoring/components/dashboard_url_time_spec.js index 98791906700..c43f6446b99 100644 --- a/spec/frontend/monitoring/components/dashboard_url_time_spec.js +++ b/spec/frontend/monitoring/components/dashboard_url_time_spec.js @@ -46,9 +46,6 @@ describe('dashboard invalid url parameters', () => { }); afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } mock.restore(); queryToObject.mockReset(); }); diff --git a/spec/frontend/monitoring/components/embeds/embed_group_spec.js b/spec/frontend/monitoring/components/embeds/embed_group_spec.js index 6695353bdb5..beb698c838f 100644 --- a/spec/frontend/monitoring/components/embeds/embed_group_spec.js +++ b/spec/frontend/monitoring/components/embeds/embed_group_spec.js @@ -48,9 +48,6 @@ describe('Embed Group', () => { afterEach(() => { metricsWithDataGetter.mockReset(); - if (wrapper) { - wrapper.destroy(); - } }); describe('interactivity', () => { diff --git a/spec/frontend/monitoring/components/embeds/metric_embed_spec.js b/spec/frontend/monitoring/components/embeds/metric_embed_spec.js index beff3da2baf..db25d524592 100644 --- a/spec/frontend/monitoring/components/embeds/metric_embed_spec.js +++ b/spec/frontend/monitoring/components/embeds/metric_embed_spec.js @@ -52,9 +52,6 @@ describe('MetricEmbed', () => { afterEach(() => { metricsWithDataGetter.mockClear(); - if (wrapper) { - wrapper.destroy(); - } }); describe('no metrics are available yet', () => { diff --git a/spec/frontend/monitoring/pages/dashboard_page_spec.js b/spec/frontend/monitoring/pages/dashboard_page_spec.js index 3de99673e71..7fcb7607772 100644 --- a/spec/frontend/monitoring/pages/dashboard_page_spec.js +++ b/spec/frontend/monitoring/pages/dashboard_page_spec.js @@ -38,13 +38,6 @@ describe('monitoring/pages/dashboard_page', () => { jest.spyOn(store, 'dispatch').mockResolvedValue(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - it('throws errors if dashboard props are not passed', () => { expect(() => assertProps(DashboardPage, {})).toThrow('Missing required prop: "dashboardProps"'); }); diff --git a/spec/frontend/notes/components/discussion_navigator_spec.js b/spec/frontend/notes/components/discussion_navigator_spec.js index 14181287381..885a7e2802e 100644 --- a/spec/frontend/notes/components/discussion_navigator_spec.js +++ b/spec/frontend/notes/components/discussion_navigator_spec.js @@ -32,12 +32,6 @@ describe('notes/components/discussion_navigator', () => { jumpToPreviousDiscussion = jest.fn(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('on create', () => { let onSpy; let vm; diff --git a/spec/frontend/notes/components/timeline_toggle_spec.js b/spec/frontend/notes/components/timeline_toggle_spec.js index cf79416d300..caa6f95d5da 100644 --- a/spec/frontend/notes/components/timeline_toggle_spec.js +++ b/spec/frontend/notes/components/timeline_toggle_spec.js @@ -35,10 +35,6 @@ describe('Timeline toggle', () => { }); afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } store.dispatch.mockReset(); mockEvent.currentTarget.blur.mockReset(); Tracking.event.mockReset(); diff --git a/spec/frontend/operation_settings/components/metrics_settings_spec.js b/spec/frontend/operation_settings/components/metrics_settings_spec.js index 6ea08d4a9a5..5bccf4943ae 100644 --- a/spec/frontend/operation_settings/components/metrics_settings_spec.js +++ b/spec/frontend/operation_settings/components/metrics_settings_spec.js @@ -47,9 +47,6 @@ describe('operation settings external dashboard component', () => { }); afterEach(() => { - if (wrapper.destroy) { - wrapper.destroy(); - } axios.patch.mockReset(); refreshCurrentPage.mockReset(); createAlert.mockReset(); diff --git a/spec/frontend/packages_and_registries/shared/components/package_tags_spec.js b/spec/frontend/packages_and_registries/shared/components/package_tags_spec.js index 33e96c0775e..b025517ae47 100644 --- a/spec/frontend/packages_and_registries/shared/components/package_tags_spec.js +++ b/spec/frontend/packages_and_registries/shared/components/package_tags_spec.js @@ -20,10 +20,6 @@ describe('PackageTags', () => { const tagBadges = () => wrapper.findAll('[data-testid="tagBadge"]'); const moreBadge = () => wrapper.find('[data-testid="moreBadge"]'); - afterEach(() => { - if (wrapper) wrapper.destroy(); - }); - describe('tag label', () => { it('shows the tag label by default', () => { createComponent(); diff --git a/spec/frontend/pages/import/bitbucket_server/components/bitbucket_server_status_table_spec.js b/spec/frontend/pages/import/bitbucket_server/components/bitbucket_server_status_table_spec.js index 8eab5061e97..40d5dff9d06 100644 --- a/spec/frontend/pages/import/bitbucket_server/components/bitbucket_server_status_table_spec.js +++ b/spec/frontend/pages/import/bitbucket_server/components/bitbucket_server_status_table_spec.js @@ -18,13 +18,6 @@ describe('BitbucketServerStatusTable', () => { .filter((w) => w.props().variant === 'info') .at(0); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - function createComponent(bitbucketStatusTableStub = true) { wrapper = shallowMount(BitbucketServerStatusTable, { propsData: { providerTitle: 'Test', reconfigurePath: '/reconfigure' }, diff --git a/spec/frontend/pipeline_wizard/components/step_spec.js b/spec/frontend/pipeline_wizard/components/step_spec.js index 4d5f563228c..99a7eff7acc 100644 --- a/spec/frontend/pipeline_wizard/components/step_spec.js +++ b/spec/frontend/pipeline_wizard/components/step_spec.js @@ -56,10 +56,6 @@ describe('Pipeline Wizard - Step Page', () => { }); }; - afterEach(async () => { - await wrapper.destroy(); - }); - describe('input children', () => { beforeEach(() => { createComponent(); diff --git a/spec/frontend/pipeline_wizard/components/widgets/text_spec.js b/spec/frontend/pipeline_wizard/components/widgets/text_spec.js index abfb4a33c0f..041ca05fd2c 100644 --- a/spec/frontend/pipeline_wizard/components/widgets/text_spec.js +++ b/spec/frontend/pipeline_wizard/components/widgets/text_spec.js @@ -27,12 +27,6 @@ describe('Pipeline Wizard - Text Widget', () => { }); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('creates an input element with the correct label', () => { createComponent(); diff --git a/spec/frontend/pipeline_wizard/components/wrapper_spec.js b/spec/frontend/pipeline_wizard/components/wrapper_spec.js index 2808fd0c7a5..b8d84572873 100644 --- a/spec/frontend/pipeline_wizard/components/wrapper_spec.js +++ b/spec/frontend/pipeline_wizard/components/wrapper_spec.js @@ -211,10 +211,6 @@ describe('Pipeline Wizard - wrapper.vue', () => { createComponent(); }); - afterAll(() => { - wrapper.destroy(); - }); - it('editor reflects changes', async () => { const newCompiledDoc = new Document({ faa: 'bur' }); await getStepWrapper().vm.$emit('update:compiled', newCompiledDoc); diff --git a/spec/frontend/pipelines/components/dag/dag_annotations_spec.js b/spec/frontend/pipelines/components/dag/dag_annotations_spec.js index aecaa640266..124f02bcec7 100644 --- a/spec/frontend/pipelines/components/dag/dag_annotations_spec.js +++ b/spec/frontend/pipelines/components/dag/dag_annotations_spec.js @@ -14,10 +14,6 @@ describe('The DAG annotations', () => { const getToggleButton = () => wrapper.findComponent(GlButton); const createComponent = (propsData = {}, method = shallowMount) => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - wrapper = method(DagAnnotations, { propsData, data() { diff --git a/spec/frontend/pipelines/components/dag/dag_spec.js b/spec/frontend/pipelines/components/dag/dag_spec.js index 5483c1c7b99..53719065611 100644 --- a/spec/frontend/pipelines/components/dag/dag_spec.js +++ b/spec/frontend/pipelines/components/dag/dag_spec.js @@ -30,10 +30,6 @@ describe('Pipeline DAG graph wrapper', () => { provideOverride = {}, method = shallowMount, } = {}) => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - wrapper = method(Dag, { provide: { pipelineProjectPath: 'root/abc-dag', diff --git a/spec/frontend/projects/pipelines/charts/components/ci_cd_analytics_charts_spec.js b/spec/frontend/projects/pipelines/charts/components/ci_cd_analytics_charts_spec.js index cf28eda5349..38760a724ff 100644 --- a/spec/frontend/projects/pipelines/charts/components/ci_cd_analytics_charts_spec.js +++ b/spec/frontend/projects/pipelines/charts/components/ci_cd_analytics_charts_spec.js @@ -47,13 +47,6 @@ describe('~/vue_shared/components/ci_cd_analytics/ci_cd_analytics_charts.vue', ( }, }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findMetricsSlot = () => wrapper.findByTestId('metrics-slot'); const findSegmentedControl = () => wrapper.findComponent(SegmentedControlButtonGroup); diff --git a/spec/frontend/projects/settings_service_desk/components/service_desk_setting_spec.js b/spec/frontend/projects/settings_service_desk/components/service_desk_setting_spec.js index f9762491507..84eafc3d0f3 100644 --- a/spec/frontend/projects/settings_service_desk/components/service_desk_setting_spec.js +++ b/spec/frontend/projects/settings_service_desk/components/service_desk_setting_spec.js @@ -27,12 +27,6 @@ describe('ServiceDeskSetting', () => { }), ); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('when isEnabled=true', () => { describe('only isEnabled', () => { describe('as project admin', () => { diff --git a/spec/frontend/projects/settings_service_desk/components/service_desk_template_dropdown_spec.js b/spec/frontend/projects/settings_service_desk/components/service_desk_template_dropdown_spec.js index 6adcfbe8157..7090db5cad7 100644 --- a/spec/frontend/projects/settings_service_desk/components/service_desk_template_dropdown_spec.js +++ b/spec/frontend/projects/settings_service_desk/components/service_desk_template_dropdown_spec.js @@ -19,12 +19,6 @@ describe('ServiceDeskTemplateDropdown', () => { }), ); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('templates dropdown', () => { it('renders a dropdown to choose a template', () => { wrapper = createComponent(); diff --git a/spec/frontend/sidebar/components/severity/severity_spec.js b/spec/frontend/sidebar/components/severity/severity_spec.js index 99d33e840d5..939d86917bb 100644 --- a/spec/frontend/sidebar/components/severity/severity_spec.js +++ b/spec/frontend/sidebar/components/severity/severity_spec.js @@ -14,13 +14,6 @@ describe('SeverityToken', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findIcon = () => wrapper.findComponent(GlIcon); it('renders severity token for each severity type', () => { diff --git a/spec/frontend/tags/components/sort_dropdown_spec.js b/spec/frontend/tags/components/sort_dropdown_spec.js index b0fd98ec68e..e0ff370d313 100644 --- a/spec/frontend/tags/components/sort_dropdown_spec.js +++ b/spec/frontend/tags/components/sort_dropdown_spec.js @@ -26,12 +26,6 @@ describe('Tags sort dropdown', () => { const findSearchBox = () => wrapper.findComponent(GlSearchBoxByClick); const findTagsDropdown = () => wrapper.findByTestId('tags-dropdown'); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('default state', () => { beforeEach(() => { wrapper = createWrapper(); diff --git a/spec/frontend/vue_merge_request_widget/components/mr_widget_pipeline_spec.js b/spec/frontend/vue_merge_request_widget/components/mr_widget_pipeline_spec.js index 6a899c00b98..820e486c13f 100644 --- a/spec/frontend/vue_merge_request_widget/components/mr_widget_pipeline_spec.js +++ b/spec/frontend/vue_merge_request_widget/components/mr_widget_pipeline_spec.js @@ -53,13 +53,6 @@ describe('MRWidgetPipeline', () => { ); }; - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - wrapper = null; - } - }); - it('should render CI error if there is a pipeline, but no status', () => { createWrapper({ ciStatus: null }, mount); expect(findCIErrorMessage().text()).toBe(ciErrorMessage); diff --git a/spec/frontend/vue_merge_request_widget/components/states/mr_widget_closed_spec.js b/spec/frontend/vue_merge_request_widget/components/states/mr_widget_closed_spec.js index 270a37f87e7..04cc396af40 100644 --- a/spec/frontend/vue_merge_request_widget/components/states/mr_widget_closed_spec.js +++ b/spec/frontend/vue_merge_request_widget/components/states/mr_widget_closed_spec.js @@ -67,12 +67,6 @@ describe('MRWidgetClosed', () => { wrapper = createComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('renders closed icon', () => { expect(wrapper.findComponent(StateContainer).exists()).toBe(true); expect(wrapper.findComponent(StateContainer).props().status).toBe('closed'); diff --git a/spec/frontend/vue_merge_request_widget/deployment/deployment_actions_spec.js b/spec/frontend/vue_merge_request_widget/deployment/deployment_actions_spec.js index 1fdbbadf8b0..f2b78dedf3a 100644 --- a/spec/frontend/vue_merge_request_widget/deployment/deployment_actions_spec.js +++ b/spec/frontend/vue_merge_request_widget/deployment/deployment_actions_spec.js @@ -30,11 +30,6 @@ describe('DeploymentAction component', () => { let executeActionSpy; const factory = (options = {}) => { - // This destroys any wrappers created before a nested call to factory reassigns it - if (wrapper && wrapper.destroy) { - wrapper.destroy(); - } - wrapper = mount(DeploymentActions, options); }; diff --git a/spec/frontend/vue_merge_request_widget/deployment/deployment_spec.js b/spec/frontend/vue_merge_request_widget/deployment/deployment_spec.js index 74122f47ad3..234491c531a 100644 --- a/spec/frontend/vue_merge_request_widget/deployment/deployment_spec.js +++ b/spec/frontend/vue_merge_request_widget/deployment/deployment_spec.js @@ -16,10 +16,6 @@ describe('Deployment component', () => { let wrapper; const factory = (options = {}) => { - // This destroys any wrappers created before a nested call to factory reassigns it - if (wrapper && wrapper.destroy) { - wrapper.destroy(); - } wrapper = mount(DeploymentComponent, options); }; diff --git a/spec/frontend/vue_shared/alert_details/alert_status_spec.js b/spec/frontend/vue_shared/alert_details/alert_status_spec.js index 2a37ff2b784..98cb2f5cb0b 100644 --- a/spec/frontend/vue_shared/alert_details/alert_status_spec.js +++ b/spec/frontend/vue_shared/alert_details/alert_status_spec.js @@ -45,12 +45,6 @@ describe('AlertManagementStatus', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('sidebar', () => { it('displays the dropdown status header', () => { mountComponent({ props: { isSidebar: true } }); diff --git a/spec/frontend/vue_shared/alert_details/alert_summary_row_spec.js b/spec/frontend/vue_shared/alert_details/alert_summary_row_spec.js index a2981478954..0ecca0a69b9 100644 --- a/spec/frontend/vue_shared/alert_details/alert_summary_row_spec.js +++ b/spec/frontend/vue_shared/alert_details/alert_summary_row_spec.js @@ -16,13 +16,6 @@ describe('AlertSummaryRow', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('Alert Summary Row', () => { beforeEach(() => { mountComponent({ diff --git a/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_spec.js b/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_spec.js index 3b38349622f..89d02cc9de8 100644 --- a/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_spec.js +++ b/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_spec.js @@ -44,9 +44,6 @@ describe('Alert Details Sidebar', () => { } afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } mock.restore(); }); diff --git a/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_status_spec.js b/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_status_spec.js index a3adbcf8d3a..7df744cd11d 100644 --- a/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_status_spec.js +++ b/spec/frontend/vue_shared/alert_details/sidebar/alert_sidebar_status_spec.js @@ -45,12 +45,6 @@ describe('Alert Details Sidebar Status', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - describe('sidebar expanded', () => { beforeEach(() => { mountComponent({ diff --git a/spec/frontend/vue_shared/alert_details/system_notes/alert_management_system_note_spec.js b/spec/frontend/vue_shared/alert_details/system_notes/alert_management_system_note_spec.js index 6a750bb99c0..72c16e8ff22 100644 --- a/spec/frontend/vue_shared/alert_details/system_notes/alert_management_system_note_spec.js +++ b/spec/frontend/vue_shared/alert_details/system_notes/alert_management_system_note_spec.js @@ -17,13 +17,6 @@ describe('Alert Details System Note', () => { }); } - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('System notes', () => { beforeEach(() => { mountComponent({}); diff --git a/spec/frontend/vue_shared/components/metric_images/metric_images_tab_spec.js b/spec/frontend/vue_shared/components/metric_images/metric_images_tab_spec.js index 1789610dba9..4b0b89fe1e7 100644 --- a/spec/frontend/vue_shared/components/metric_images/metric_images_tab_spec.js +++ b/spec/frontend/vue_shared/components/metric_images/metric_images_tab_spec.js @@ -45,13 +45,6 @@ describe('Metric images tab', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findUploadDropzone = () => wrapper.findComponent(UploadDropzone); const findImages = () => wrapper.findAllComponents(MetricImagesTable); const findModal = () => wrapper.findComponent(GlModal); diff --git a/spec/frontend/vue_shared/components/metric_images/metric_images_table_spec.js b/spec/frontend/vue_shared/components/metric_images/metric_images_table_spec.js index 9c91dc9b5fc..12dca95e9ba 100644 --- a/spec/frontend/vue_shared/components/metric_images/metric_images_table_spec.js +++ b/spec/frontend/vue_shared/components/metric_images/metric_images_table_spec.js @@ -39,13 +39,6 @@ describe('Metrics upload item', () => { ); }; - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - const findImageLink = () => wrapper.findComponent(GlLink); const findLabelTextSpan = () => wrapper.find('[data-testid="metric-image-label-span"]'); const findCollapseButton = () => wrapper.find('[data-testid="collapse-button"]'); diff --git a/spec/frontend/vue_shared/components/notes/noteable_warning_spec.js b/spec/frontend/vue_shared/components/notes/noteable_warning_spec.js index f87674246d1..d7fcb9a25d4 100644 --- a/spec/frontend/vue_shared/components/notes/noteable_warning_spec.js +++ b/spec/frontend/vue_shared/components/notes/noteable_warning_spec.js @@ -22,13 +22,6 @@ describe('Issue Warning Component', () => { }, }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - wrapper = null; - } - }); - describe('when issue is locked but not confidential', () => { beforeEach(() => { wrapper = createComponent({ diff --git a/spec/frontend/vue_shared/components/paginated_table_with_search_and_tabs/paginated_table_with_search_and_tabs_spec.js b/spec/frontend/vue_shared/components/paginated_table_with_search_and_tabs/paginated_table_with_search_and_tabs_spec.js index 25bfa688e5b..9b6f5ae3e38 100644 --- a/spec/frontend/vue_shared/components/paginated_table_with_search_and_tabs/paginated_table_with_search_and_tabs_spec.js +++ b/spec/frontend/vue_shared/components/paginated_table_with_search_and_tabs/paginated_table_with_search_and_tabs_spec.js @@ -90,12 +90,6 @@ describe('AlertManagementEmptyState', () => { mountComponent(); }); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - const EmptyState = () => wrapper.find('.empty-state'); const ItemsTable = () => wrapper.find('.gl-table'); const ErrorAlert = () => wrapper.findComponent(GlAlert); diff --git a/spec/frontend/vue_shared/components/resizable_chart/skeleton_loader_spec.js b/spec/frontend/vue_shared/components/resizable_chart/skeleton_loader_spec.js index bfc3aeb0303..043552baf0c 100644 --- a/spec/frontend/vue_shared/components/resizable_chart/skeleton_loader_spec.js +++ b/spec/frontend/vue_shared/components/resizable_chart/skeleton_loader_spec.js @@ -19,12 +19,6 @@ describe('Resizable Skeleton Loader', () => { expect(labelItems.length).toBe(8); }; - afterEach(() => { - if (wrapper?.destroy) { - wrapper.destroy(); - } - }); - describe('default setup', () => { beforeEach(() => { createComponent({ uniqueKey: null }); diff --git a/spec/frontend/vue_shared/components/slot_switch_spec.js b/spec/frontend/vue_shared/components/slot_switch_spec.js index daca4977817..3a2147c6c89 100644 --- a/spec/frontend/vue_shared/components/slot_switch_spec.js +++ b/spec/frontend/vue_shared/components/slot_switch_spec.js @@ -20,12 +20,6 @@ describe('SlotSwitch', () => { const getChildrenHtml = () => wrapper.findAll('* *').wrappers.map((c) => c.html()); - afterEach(() => { - if (wrapper) { - wrapper.destroy(); - } - }); - it('throws an error if activeSlotNames is missing', () => { expect(() => assertProps(SlotSwitch, {})).toThrow( '[Vue warn]: Missing required prop: "activeSlotNames"', |