This MR removes direct data manipulation

As part of Vue 3 migration overriding method behavior and direct data manipulation should be avoided

This MR removes direct data manipulation
As part of Vue 3 migration overriding method behavior and direct data manipulation should be avoided
e8e9e5e7 · Artur Fedorov · 48b9dae0 · e8e9e5e7
--- a/spec/frontend/ci/runner/group_runners/group_runners_app_spec.js
+++ b/spec/frontend/ci/runner/group_runners/group_runners_app_spec.js
@@ -82,6 +82,7 @@ jest.mock('~/lib/utils/url_utility', () => ({

 describe('GroupRunnersApp', () => {
  let wrapper;
+  const showToast = jest.fn();

  const findRunnerStats = () => wrapper.findComponent(RunnerStats);
  const findRunnerActionsCell = () => wrapper.findComponent(RunnerActionsCell);
@@ -123,6 +124,11 @@ describe('GroupRunnersApp', () => {
        staleTimeoutSecs,
        ...provide,
      },
+      mocks: {
+        $toast: {
+          show: showToast,
+        },
+      },
      ...options,
    });

@@ -250,8 +256,6 @@ describe('GroupRunnersApp', () => {
  });

  describe('Single runner row', () => {
-    let showToast;
-
    const { webUrl, editUrl, node } = mockGroupRunnersEdges[0];
    const { id: graphqlId, shortSha, jobExecutionStatus } = node;
    const id = getIdFromGraphQLId(graphqlId);
@@ -260,7 +264,6 @@ describe('GroupRunnersApp', () => {

    beforeEach(async () => {
      await createComponent({ mountFn: mountExtended });
-      showToast = jest.spyOn(wrapper.vm.$root.$toast, 'show');
    });

    it('Shows job status and links to jobs', () => {