From 8d99f79de446e2776307448d2df19f82e88913c5 Mon Sep 17 00:00:00 2001
From: nixocio <nixocio@gmail.com>
Date: Mon, 14 Jun 2021 13:38:08 -0400
Subject: [PATCH] Hide Edit button for non system admin

Hide Edit button for non system admin

See: https://github.com/ansible/tower/issues/5032
---
 .../SubscriptionDetail/SubscriptionDetail.jsx | 22 ++++++++++---------
 .../SubscriptionDetail.test.jsx               |  8 +++++++
 2 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.jsx b/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.jsx
index 3508f32138..a8fed74240 100644
--- a/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.jsx
@@ -23,7 +23,7 @@ import {
 } from '../../../../util/dates';
 
 function SubscriptionDetail() {
-  const { license_info, version } = useConfig();
+  const { me = {}, license_info, version } = useConfig();
   const baseURL = '/settings/subscription';
   const tabsArray = [
     {
@@ -164,15 +164,17 @@ function SubscriptionDetail() {
             contact us.
           </Button>
         </Trans>
-        <CardActionsRow>
-          <Button
-            aria-label={t`edit`}
-            component={Link}
-            to="/settings/subscription/edit"
-          >
-            <Trans>Edit</Trans>
-          </Button>
-        </CardActionsRow>
+        {me.is_superuser && (
+          <CardActionsRow>
+            <Button
+              aria-label={t`edit`}
+              component={Link}
+              to="/settings/subscription/edit"
+            >
+              <Trans>Edit</Trans>
+            </Button>
+          </CardActionsRow>
+        )}
       </CardBody>
     </>
   );
diff --git a/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.test.jsx b/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.test.jsx
index 3b79c35db8..fffea57a4f 100644
--- a/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.test.jsx
+++ b/awx/ui_next/src/screens/Setting/Subscription/SubscriptionDetail/SubscriptionDetail.test.jsx
@@ -71,6 +71,14 @@ describe('<SubscriptionDetail />', () => {
     assertDetail('Hosts remaining', '1000');
     assertDetail('Hosts automated', '12 since 3/2/2021, 7:43:48 PM');
 
+    expect(wrapper.find('Button[aria-label="edit"]').length).toBe(0);
+  });
+
+  test('should render edit button for system admin', () => {
+    wrapper = mountWithContexts(<SubscriptionDetail />, {
+      context: { ...config, me: { is_superuser: true } },
+    });
+
     expect(wrapper.find('Button[aria-label="edit"]').length).toBe(1);
   });
 });