From 8e4fdfb5a554724c7dfc4af94d46ea8d2175c6c0 Mon Sep 17 00:00:00 2001
From: Grégoire Aubert <gregoire.aubert@sonarsource.com>
Date: Thu, 13 Apr 2017 15:36:08 +0200
Subject: SONAR-6911 Improve the display of default groups

---
 .../src/main/js/apps/groups/list-item-view.js         | 12 +++++++++---
 .../js/apps/groups/templates/groups-list-item.hbs     | 15 +++++++++++----
 .../components/OrganizationGroupCheckbox.js           | 13 +++++++++----
 .../__tests__/OrganizationGroupCheckbox-test.js       | 17 ++++++++++++++++-
 .../OrganizationGroupCheckbox-test.js.snap            | 19 +++++++++++++++++++
 5 files changed, 64 insertions(+), 12 deletions(-)

(limited to 'server/sonar-web/src/main/js/apps')

diff --git a/server/sonar-web/src/main/js/apps/groups/list-item-view.js b/server/sonar-web/src/main/js/apps/groups/list-item-view.js
index 58ac7ef50b1..ea6a0824950 100644
--- a/server/sonar-web/src/main/js/apps/groups/list-item-view.js
+++ b/server/sonar-web/src/main/js/apps/groups/list-item-view.js
@@ -46,18 +46,24 @@ export default Marionette.ItemView.extend({
 
   onUpdateClick(e) {
     e.preventDefault();
-    this.updateGroup();
+    if (!this.model.get('default')) {
+      this.updateGroup();
+    }
   },
 
   onDeleteClick(e) {
     e.preventDefault();
-    this.deleteGroup();
+    if (!this.model.get('default')) {
+      this.deleteGroup();
+    }
   },
 
   onUsersClick(e) {
     e.preventDefault();
     $('.tooltip').remove();
-    this.showUsers();
+    if (!this.model.get('default')) {
+      this.showUsers();
+    }
   },
 
   updateGroup() {
diff --git a/server/sonar-web/src/main/js/apps/groups/templates/groups-list-item.hbs b/server/sonar-web/src/main/js/apps/groups/templates/groups-list-item.hbs
index 809a41b624b..85306c7f827 100644
--- a/server/sonar-web/src/main/js/apps/groups/templates/groups-list-item.hbs
+++ b/server/sonar-web/src/main/js/apps/groups/templates/groups-list-item.hbs
@@ -1,19 +1,26 @@
 <div class="pull-right big-spacer-left nowrap">
-  <a class="js-group-update icon-edit little-spacer-right" title="Update Details" data-toggle="tooltip" href="#"></a>
-  <a class="js-group-delete icon-delete" title="Delete" data-toggle="tooltip" href="#"></a>
+  {{#unless default}}
+    <a class="js-group-update icon-edit little-spacer-right" title={{t 'users.update_details'}} data-toggle="tooltip" href="#"></a>
+    <a class="js-group-delete icon-delete" title={{t 'delete'}} data-toggle="tooltip" href="#"></a>
+  {{/unless}}
 </div>
 
 <div class="display-inline-block text-top width-20">
   <strong class="js-group-name">{{name}}</strong>
+  {{#if default}}
+    <span class="little-spacer-left">({{t 'default'}})</span>
+  {{/if}}
 </div>
 
 <div class="display-inline-block text-top big-spacer-left width-25">
   <div class="pull-left spacer-right">
-    <strong>Members</strong>
+    <strong>{{t 'members'}}</strong>
   </div>
   <div class="overflow-hidden bordered-left">
     <span class="spacer-left spacer-right">{{membersCount}}</span>
-    <a class="js-group-users icon-bullet-list" title="Update Users" data-toggle="tooltip" href="#"></a>
+    {{#unless default}}
+      <a class="js-group-users icon-bullet-list" title={{t 'users.update'}} data-toggle="tooltip" href="#"></a>
+    {{/unless}}
   </div>
 </div>
 
diff --git a/server/sonar-web/src/main/js/apps/organizations/components/OrganizationGroupCheckbox.js b/server/sonar-web/src/main/js/apps/organizations/components/OrganizationGroupCheckbox.js
index 3324cc4ec77..b8afa70b90d 100644
--- a/server/sonar-web/src/main/js/apps/organizations/components/OrganizationGroupCheckbox.js
+++ b/server/sonar-web/src/main/js/apps/organizations/components/OrganizationGroupCheckbox.js
@@ -32,22 +32,27 @@ export default class OrganizationGroupCheckbox extends React.PureComponent {
   props: Props;
 
   onCheck = (checked: boolean) => {
-    this.props.onCheck(this.props.group.name, checked);
+    const { group } = this.props;
+    if (!group.default) {
+      this.props.onCheck(group.name, checked);
+    }
   };
 
   toggleCheck = () => {
-    this.props.onCheck(this.props.group.name, !this.props.checked);
+    this.onCheck(!this.props.checked);
   };
 
   render() {
+    const { group } = this.props;
     return (
       <li
         className="capitalize list-item-checkable-link"
         onClick={this.toggleCheck}
         tabIndex={0}
-        role="listitem">
+        role="listitem"
+        disabled={group.default}>
         <Checkbox checked={this.props.checked} onCheck={this.onCheck} />
-        {' '}{this.props.group.name}
+        {' '}{group.name}
       </li>
     );
   }
diff --git a/server/sonar-web/src/main/js/apps/organizations/components/__tests__/OrganizationGroupCheckbox-test.js b/server/sonar-web/src/main/js/apps/organizations/components/__tests__/OrganizationGroupCheckbox-test.js
index 146e8aacba1..cb68211b8e4 100644
--- a/server/sonar-web/src/main/js/apps/organizations/components/__tests__/OrganizationGroupCheckbox-test.js
+++ b/server/sonar-web/src/main/js/apps/organizations/components/__tests__/OrganizationGroupCheckbox-test.js
@@ -25,7 +25,8 @@ const group = {
   id: '7',
   name: 'professionals',
   description: '',
-  membersCount: 12
+  membersCount: 12,
+  default: false
 };
 
 it('should render unchecked', () => {
@@ -45,3 +46,17 @@ it('should be able to toggle check', () => {
   expect(onCheck.mock.calls).toMatchSnapshot();
   expect(wrapper).toMatchSnapshot();
 });
+
+it('should disabled default groups', () => {
+  const onCheck = jest.fn((group, checked) => wrapper.setProps({ checked }));
+  const wrapper = shallow(
+    <OrganizationGroupCheckbox
+      group={{ ...group, default: true }}
+      checked={true}
+      onCheck={onCheck}
+    />
+  );
+  expect(wrapper).toMatchSnapshot();
+  wrapper.instance().toggleCheck();
+  expect(onCheck.mock.calls.length).toBe(0);
+});
diff --git a/server/sonar-web/src/main/js/apps/organizations/components/__tests__/__snapshots__/OrganizationGroupCheckbox-test.js.snap b/server/sonar-web/src/main/js/apps/organizations/components/__tests__/__snapshots__/OrganizationGroupCheckbox-test.js.snap
index 5ee947cf614..00ac21f63b3 100644
--- a/server/sonar-web/src/main/js/apps/organizations/components/__tests__/__snapshots__/OrganizationGroupCheckbox-test.js.snap
+++ b/server/sonar-web/src/main/js/apps/organizations/components/__tests__/__snapshots__/OrganizationGroupCheckbox-test.js.snap
@@ -1,6 +1,7 @@
 exports[`test should be able to toggle check 1`] = `
 <li
   className="capitalize list-item-checkable-link"
+  disabled={false}
   onClick={[Function]}
   role="listitem"
   tabIndex={0}>
@@ -25,6 +26,7 @@ Array [
 exports[`test should be able to toggle check 3`] = `
 <li
   className="capitalize list-item-checkable-link"
+  disabled={false}
   onClick={[Function]}
   role="listitem"
   tabIndex={0}>
@@ -37,9 +39,26 @@ exports[`test should be able to toggle check 3`] = `
 </li>
 `;
 
+exports[`test should disabled default groups 1`] = `
+<li
+  className="capitalize list-item-checkable-link"
+  disabled={true}
+  onClick={[Function]}
+  role="listitem"
+  tabIndex={0}>
+  <Checkbox
+    checked={true}
+    onCheck={[Function]}
+    thirdState={false} />
+   
+  professionals
+</li>
+`;
+
 exports[`test should render unchecked 1`] = `
 <li
   className="capitalize list-item-checkable-link"
+  disabled={false}
   onClick={[Function]}
   role="listitem"
   tabIndex={0}>
-- 
cgit v1.2.3