Merge branch '498945_02-add-restore-list-action-and-api' into 'master'

List Actions - Add support for Restore

See merge request !173467



Merged-by: Alex Pennells <apennells@gitlab.com>
Approved-by: James Rushford <jrushford@gitlab.com>
Approved-by: Alex Pennells <apennells@gitlab.com>
Reviewed-by: Zack Cuddy <zcuddy@gitlab.com>
Co-authored-by: Zachary Cuddy <zcuddy@gitlab.com>

app/assets/javascripts/vue_shared/components/list_actions/constants.js

 import { __ } from '~/locale';
 
 export const ACTION_EDIT = 'edit';
+export const ACTION_RESTORE = 'restore';
 export const ACTION_DELETE = 'delete';
 
 export const BASE_ACTIONS = {
   [ACTION_EDIT]: {
     text: __('Edit'),
+    order: 1,
+  },
+  [ACTION_RESTORE]: {
+    text: __('Restore'),
+    order: 2,
   },
   [ACTION_DELETE]: {
     text: __('Delete'),
     extraAttrs: {
       class: '!gl-text-red-500',
     },
+    order: 3,
   },
 };

ee/app/assets/javascripts/api/projects_api.js

+import axios from '~/lib/utils/axios_utils';
+import { buildApiUrl } from '~/api/api_utils';
+
+const PROJECT_RESTORE_PATH = '/api/:version/projects/:id/restore';
+
+export function restoreProject(projectId) {
+  const url = buildApiUrl(PROJECT_RESTORE_PATH).replace(':id', projectId);
+
+  return axios.post(url);
+}

ee/app/assets/javascripts/rest_api.js

@@ -2,3 +2,4 @@ export * from './api/groups_api';
 export * from './api/subscriptions_api';
 export * from './api/dora_api';
 export * from './api/status_check_api';
+export * from './api/projects_api';

ee/spec/frontend/api/projects_api_spec.js

+import MockAdapter from 'axios-mock-adapter';
+import * as projectsApi from 'ee/api/projects_api';
+import axios from '~/lib/utils/axios_utils';
+import { HTTP_STATUS_OK } from '~/lib/utils/http_status';
+
+describe('ee/api/projects_api.js', () => {
+  let mock;
+
+  const projectId = 1;
+
+  beforeEach(() => {
+    mock = new MockAdapter(axios);
+
+    window.gon = { api_version: 'v7' };
+  });
+
+  afterEach(() => {
+    mock.restore();
+  });
+
+  describe('restoreProject', () => {
+    beforeEach(() => {
+      jest.spyOn(axios, 'post');
+    });
+
+    it('calls POST to the correct URL', () => {
+      const expectedUrl = `/api/v7/projects/${projectId}/restore`;
+
+      mock.onPost(expectedUrl).replyOnce(HTTP_STATUS_OK);
+
+      return projectsApi.restoreProject(projectId).then(() => {
+        expect(axios.post).toHaveBeenCalledWith(expectedUrl);
+      });
+    });
+  });
+});

spec/frontend/vue_shared/components/list_actions/list_actions_spec.js

@@ -37,6 +37,7 @@ describe('ListActions', () => {
       {
         text: 'Edit',
         href: '/-/edit',
+        order: 1,
       },
       {
         text: 'Delete',
@@ -44,6 +45,7 @@ describe('ListActions', () => {
           class: '!gl-text-red-500',
         },
         action: expect.any(Function),
+        order: 3,
       },
     ]);
   });
@@ -68,6 +70,7 @@ describe('ListActions', () => {
       {
         text: 'Edit',
         href: '/-/edit',
+        order: 1,
       },
       {
         text: 'Leave project',
@@ -79,6 +82,7 @@ describe('ListActions', () => {
           class: '!gl-text-red-500',
         },
         action: expect.any(Function),
+        order: 3,
       },
     ]);
   });
@@ -94,6 +98,7 @@ describe('ListActions', () => {
       {
         text: 'Edit',
         href: '/-/edit',
+        order: 1,
       },
     ]);
   });
@@ -112,10 +117,12 @@ describe('ListActions', () => {
           class: '!gl-text-red-500',
         },
         action: expect.any(Function),
+        order: 3,
       },
       {
         text: 'Edit',
         href: '/-/edit',
+        order: 1,
       },
     ]);
   });