From 5ad6b025508fb933139007e33897c04707f8c86a Mon Sep 17 00:00:00 2001
From: Susmitha Girumala <susmitha@hashicorp.com>
Date: Wed, 19 Aug 2020 17:10:48 -0400
Subject: [PATCH] chore: Add Manage>Delete dropdown to an auth method

---
 ui/core/app/routes/scopes/scope/auth-methods.js | 17 +++++++++++++++++
 .../scope/auth-methods/auth-method/index.hbs    | 12 ++++++++++++
 ui/core/tests/acceptance/auth-methods-test.js   |  4 ++++
 ui/core/translations/en-us.yaml                 |  1 +
 4 files changed, 34 insertions(+)

diff --git a/ui/core/app/routes/scopes/scope/auth-methods.js b/ui/core/app/routes/scopes/scope/auth-methods.js
index 0e35b2c28..7527ac784 100644
--- a/ui/core/app/routes/scopes/scope/auth-methods.js
+++ b/ui/core/app/routes/scopes/scope/auth-methods.js
@@ -58,4 +58,21 @@ export default class ScopesScopeAuthMethodsRoute extends Route {
       this.notify.error(error.message, { closeAfter: null });
     }
   }
+
+  /**
+   * Delete an auth method in current scope and redirect to index
+   * @param {AuthMethodModel} authMethod
+   */
+  @action
+  async delete(authMethod) {
+    try {
+      await authMethod.destroyRecord();
+      this.refresh();
+      this.notify.success(this.intl.t('notify.delete-success'));
+      this.transitionTo('scopes.scope.auth-methods');
+    } catch (error) {
+      //TODO: replace with translated strings
+      this.notify.error(error.message, { closeAfter: null });
+    }
+  }
 }
diff --git a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/index.hbs b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/index.hbs
index c08c3fd4a..753e4fdf2 100644
--- a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/index.hbs
+++ b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/index.hbs
@@ -18,6 +18,18 @@
     </Rose::Nav::Tabs>
   </page.navigation>
 
+  <page.actions>
+    <Rose::Dropdown @text="{{t 'form.label.manage'}}" @dropdownRight={{true}} as |dropdown|>
+      <dropdown.button
+        @style="danger"
+        @disabled={{@model.canSave}}
+        {{on "click" (route-action "delete" @model)}}
+      >
+        {{t 'actions.delete-auth-method'}}
+      </dropdown.button>
+    </Rose::Dropdown>
+  </page.actions>
+
   <page.body>
     <Form::AuthMethod
       @model={{@model}}
diff --git a/ui/core/tests/acceptance/auth-methods-test.js b/ui/core/tests/acceptance/auth-methods-test.js
index 5b816f253..f91e2cf75 100644
--- a/ui/core/tests/acceptance/auth-methods-test.js
+++ b/ui/core/tests/acceptance/auth-methods-test.js
@@ -72,6 +72,10 @@ module('Acceptance | auth methods', function (hooks) {
     assert.expect(0);
   });
 
+  test('can delete an auth method', async function (assert) {
+    assert.expect(0);
+  });
+
   test('saving a new auth method with invalid fields displays error messages', async function (assert) {
     assert.expect(0);
   });
diff --git a/ui/core/translations/en-us.yaml b/ui/core/translations/en-us.yaml
index cd73f8bbb..08a4926eb 100644
--- a/ui/core/translations/en-us.yaml
+++ b/ui/core/translations/en-us.yaml
@@ -14,6 +14,7 @@ actions:
   delete-role: Delete role
   delete-project: Delete project
   delete-host-catalog: Delete host catalog
+  delete-auth-method: Delete auth method
 auth:
   titles:
     no-scopes: No Scopes Available