From 6be0a2a17d4d3f772c6e7cffd1002c377045406d Mon Sep 17 00:00:00 2001
From: Randall Morey <randall@randallmorey.com>
Date: Thu, 2 Sep 2021 10:58:06 -0400
Subject: [PATCH] chore: add scope delete capability

---
 ui/admin/app/templates/scopes/scope/edit.hbs  | 38 ++++++++++---------
 .../tests/acceptance/scopes/delete-test.js    |  9 +++++
 2 files changed, 29 insertions(+), 18 deletions(-)

diff --git a/ui/admin/app/templates/scopes/scope/edit.hbs b/ui/admin/app/templates/scopes/scope/edit.hbs
index 9e3898e0f..648a03111 100644
--- a/ui/admin/app/templates/scopes/scope/edit.hbs
+++ b/ui/admin/app/templates/scopes/scope/edit.hbs
@@ -36,25 +36,27 @@
     </Rose::Nav::Tabs>
   </page.navigation>
 
-  <page.actions>
-    <Rose::Dropdown
-      @text='{{t 'actions.manage'}}'
-      @dropdownRight={{true}}
-      as |dropdown|
-    >
-      <dropdown.button
-        @style='danger'
-        @disabled={{@model.canSave}}
-        {{on 'click' (route-action 'delete' @model)}}
+  {{#if (can 'delete model' @model)}}
+    <page.actions>
+      <Rose::Dropdown
+        @text='{{t 'actions.manage'}}'
+        @dropdownRight={{true}}
+        as |dropdown|
       >
-        {{#if @model.isOrg}}
-          {{t 'resources.org.actions.delete'}}
-        {{else}}
-          {{t 'resources.project.actions.delete'}}
-        {{/if}}
-      </dropdown.button>
-    </Rose::Dropdown>
-  </page.actions>
+        <dropdown.button
+          @style='danger'
+          @disabled={{@model.canSave}}
+          {{on 'click' (route-action 'delete' @model)}}
+        >
+          {{#if @model.isOrg}}
+            {{t 'resources.org.actions.delete'}}
+          {{else}}
+            {{t 'resources.project.actions.delete'}}
+          {{/if}}
+        </dropdown.button>
+      </Rose::Dropdown>
+    </page.actions>
+  {{/if}}
 
   <page.body>
     <Form::Scope
diff --git a/ui/admin/tests/acceptance/scopes/delete-test.js b/ui/admin/tests/acceptance/scopes/delete-test.js
index 1651dd0b6..6b35ae24d 100644
--- a/ui/admin/tests/acceptance/scopes/delete-test.js
+++ b/ui/admin/tests/acceptance/scopes/delete-test.js
@@ -83,6 +83,15 @@ module('Acceptance | scopes | delete', function (hooks) {
     assert.equal(getScopeCount('org'), orgScopeCount - 1);
   });
 
+  test('cannot delete scope without proper authorization', async function (assert) {
+    assert.expect(1);
+    instances.scopes.org.update({ authorized_actions: [] });
+    await visit(urls.orgScopeEdit);
+    assert.notOk(
+      find('.rose-layout-page-actions .rose-dropdown-button-danger')
+    );
+  });
+
   test('can accept delete scope via dialog', async function (assert) {
     assert.expect(2);
     const confirmService = this.owner.lookup('service:confirm');