From 27e0c9780d56a3d57b9657e53a8d812d73b263a1 Mon Sep 17 00:00:00 2001
From: 7PH <benjamin.raymond@sonarsource.com>
Date: Thu, 16 Mar 2023 15:25:36 +0100
Subject: [PATCH] SONAR-18757 Create identity providers E2Es UI tests

---
 .../quality-profiles/details/ChangeProjectsForm.tsx  | 10 ++--------
 .../details/__tests__/ChangeProjectsForm-test.tsx    | 10 +---------
 .../__snapshots__/ChangeProjectsForm-test.tsx.snap   |  7 +++----
 .../src/main/js/apps/users/components/GroupsForm.tsx | 10 ++--------
 .../users/components/__tests__/GroupsForm-test.tsx   | 12 +-----------
 .../__tests__/__snapshots__/GroupsForm-test.tsx.snap |  6 ++----
 6 files changed, 11 insertions(+), 44 deletions(-)

diff --git a/server/sonar-web/src/main/js/apps/quality-profiles/details/ChangeProjectsForm.tsx b/server/sonar-web/src/main/js/apps/quality-profiles/details/ChangeProjectsForm.tsx
index 5a31e29c5d8..9c9d83303af 100644
--- a/server/sonar-web/src/main/js/apps/quality-profiles/details/ChangeProjectsForm.tsx
+++ b/server/sonar-web/src/main/js/apps/quality-profiles/details/ChangeProjectsForm.tsx
@@ -25,6 +25,7 @@ import {
   getProfileProjects,
   ProfileProject,
 } from '../../../api/quality-profiles';
+import { ResetButtonLink } from '../../../components/controls/buttons';
 import Modal from '../../../components/controls/Modal';
 import SelectList, {
   SelectListFilter,
@@ -118,11 +119,6 @@ export default class ChangeProjectsForm extends React.PureComponent<Props, State
       }
     });
 
-  handleCloseClick = (event: React.SyntheticEvent<HTMLElement>) => {
-    event.preventDefault();
-    this.props.onClose();
-  };
-
   renderElement = (key: string): React.ReactNode => {
     const project = find(this.state.projects, { key });
     return (
@@ -172,9 +168,7 @@ export default class ChangeProjectsForm extends React.PureComponent<Props, State
         </div>
 
         <div className="modal-foot">
-          <a href="#" onClick={this.handleCloseClick}>
-            {translate('close')}
-          </a>
+          <ResetButtonLink onClick={this.props.onClose}>{translate('close')}</ResetButtonLink>
         </div>
       </Modal>
     );
diff --git a/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/ChangeProjectsForm-test.tsx b/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/ChangeProjectsForm-test.tsx
index 351b57857f0..d48a59bc77f 100644
--- a/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/ChangeProjectsForm-test.tsx
+++ b/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/ChangeProjectsForm-test.tsx
@@ -25,7 +25,7 @@ import {
   getProfileProjects,
 } from '../../../../api/quality-profiles';
 import SelectList, { SelectListFilter } from '../../../../components/controls/SelectList';
-import { click, waitAndUpdate } from '../../../../helpers/testUtils';
+import { waitAndUpdate } from '../../../../helpers/testUtils';
 import ChangeProjectsForm from '../ChangeProjectsForm';
 
 const profile: any = { key: 'profFile_key' };
@@ -95,14 +95,6 @@ it('should handle deselection properly', async () => {
   expect(wrapper.state().needToReload).toBe(true);
 });
 
-it('should close modal properly', () => {
-  const spy = jest.fn();
-  const wrapper = shallowRender({ onClose: spy });
-  click(wrapper.find('a'));
-
-  expect(spy).toHaveBeenCalled();
-});
-
 function shallowRender(props: Partial<ChangeProjectsForm['props']> = {}) {
   return shallow<ChangeProjectsForm>(
     <ChangeProjectsForm onClose={jest.fn()} profile={profile} {...props} />
diff --git a/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/__snapshots__/ChangeProjectsForm-test.tsx.snap b/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/__snapshots__/ChangeProjectsForm-test.tsx.snap
index 255176e76b5..68a5559cfca 100644
--- a/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/__snapshots__/ChangeProjectsForm-test.tsx.snap
+++ b/server/sonar-web/src/main/js/apps/quality-profiles/details/__tests__/__snapshots__/ChangeProjectsForm-test.tsx.snap
@@ -45,12 +45,11 @@ exports[`should render correctly 1`] = `
   <div
     className="modal-foot"
   >
-    <a
-      href="#"
-      onClick={[Function]}
+    <ResetButtonLink
+      onClick={[MockFunction]}
     >
       close
-    </a>
+    </ResetButtonLink>
   </div>
 </Modal>
 `;
diff --git a/server/sonar-web/src/main/js/apps/users/components/GroupsForm.tsx b/server/sonar-web/src/main/js/apps/users/components/GroupsForm.tsx
index 50338c76e9c..60d14160b74 100644
--- a/server/sonar-web/src/main/js/apps/users/components/GroupsForm.tsx
+++ b/server/sonar-web/src/main/js/apps/users/components/GroupsForm.tsx
@@ -21,6 +21,7 @@ import { find, without } from 'lodash';
 import * as React from 'react';
 import { getUserGroups, UserGroup } from '../../../api/users';
 import { addUserToGroup, removeUserFromGroup } from '../../../api/user_groups';
+import { ResetButtonLink } from '../../../components/controls/buttons';
 import Modal from '../../../components/controls/Modal';
 import SelectList, {
   SelectListFilter,
@@ -119,11 +120,6 @@ export default class GroupsForm extends React.PureComponent<Props, State> {
       }
     });
 
-  handleCloseClick = (event: React.SyntheticEvent<HTMLElement>) => {
-    event.preventDefault();
-    this.handleClose();
-  };
-
   handleClose = () => {
     this.props.onUpdateUsers();
     this.props.onClose();
@@ -174,9 +170,7 @@ export default class GroupsForm extends React.PureComponent<Props, State> {
         </div>
 
         <footer className="modal-foot">
-          <a className="js-modal-close" href="#" onClick={this.handleCloseClick}>
-            {translate('done')}
-          </a>
+          <ResetButtonLink onClick={this.handleClose}>{translate('done')}</ResetButtonLink>
         </footer>
       </Modal>
     );
diff --git a/server/sonar-web/src/main/js/apps/users/components/__tests__/GroupsForm-test.tsx b/server/sonar-web/src/main/js/apps/users/components/__tests__/GroupsForm-test.tsx
index b6b5a2f83a1..d793476b43c 100644
--- a/server/sonar-web/src/main/js/apps/users/components/__tests__/GroupsForm-test.tsx
+++ b/server/sonar-web/src/main/js/apps/users/components/__tests__/GroupsForm-test.tsx
@@ -23,7 +23,7 @@ import { getUserGroups } from '../../../../api/users';
 import { addUserToGroup, removeUserFromGroup } from '../../../../api/user_groups';
 import SelectList, { SelectListFilter } from '../../../../components/controls/SelectList';
 import { mockUser } from '../../../../helpers/testMocks';
-import { click, waitAndUpdate } from '../../../../helpers/testUtils';
+import { waitAndUpdate } from '../../../../helpers/testUtils';
 import GroupsForm from '../GroupsForm';
 
 const user = mockUser();
@@ -121,16 +121,6 @@ it('should handle deselection properly', async () => {
   expect(wrapper.state().needToReload).toBe(true);
 });
 
-it('should close modal properly', () => {
-  const spyOnClose = jest.fn();
-  const spyOnUpdateUsers = jest.fn();
-  const wrapper = shallowRender({ onClose: spyOnClose, onUpdateUsers: spyOnUpdateUsers });
-  click(wrapper.find('.js-modal-close'));
-
-  expect(spyOnClose).toHaveBeenCalled();
-  expect(spyOnUpdateUsers).toHaveBeenCalled();
-});
-
 function shallowRender(props: Partial<GroupsForm['props']> = {}) {
   return shallow<GroupsForm>(
     <GroupsForm onClose={jest.fn()} onUpdateUsers={jest.fn()} user={user} {...props} />
diff --git a/server/sonar-web/src/main/js/apps/users/components/__tests__/__snapshots__/GroupsForm-test.tsx.snap b/server/sonar-web/src/main/js/apps/users/components/__tests__/__snapshots__/GroupsForm-test.tsx.snap
index 5bceb55bb99..cbdda1a7dac 100644
--- a/server/sonar-web/src/main/js/apps/users/components/__tests__/__snapshots__/GroupsForm-test.tsx.snap
+++ b/server/sonar-web/src/main/js/apps/users/components/__tests__/__snapshots__/GroupsForm-test.tsx.snap
@@ -41,13 +41,11 @@ exports[`should render correctly 1`] = `
   <footer
     className="modal-foot"
   >
-    <a
-      className="js-modal-close"
-      href="#"
+    <ResetButtonLink
       onClick={[Function]}
     >
       done
-    </a>
+    </ResetButtonLink>
   </footer>
 </Modal>
 `;
-- 
2.39.5