From 3030d87abc0739f961b7be3ea1875ed830ff7340 Mon Sep 17 00:00:00 2001
From: Susmitha Girumala <susmitha@hashicorp.com>
Date: Mon, 14 Sep 2020 17:43:07 -0400
Subject: [PATCH] feat: Create an account

---
 ui/core/app/components/form/account/index.hbs | 17 ++++
 .../form/account/password/index.hbs           | 88 +++++++++++++++++++
 ui/core/app/components/form/host/index.hbs    |  1 -
 ui/core/app/router.js                         |  1 +
 .../auth-methods/auth-method/accounts.js      |  5 +-
 .../auth-methods/auth-method/accounts/new.js  | 43 +++++++++
 .../auth-methods/auth-method/-actions.hbs     |  6 ++
 .../auth-methods/auth-method/accounts/new.hbs |  5 ++
 .../auth-method/accounts/new/-header.hbs      |  1 +
 .../components/form/account/password-test.js  | 24 +++++
 .../auth-method/accounts/new-test.js          | 11 +++
 ui/core/translations/en-us.yaml               |  1 +
 12 files changed, 201 insertions(+), 2 deletions(-)
 create mode 100644 ui/core/app/components/form/account/password/index.hbs
 create mode 100644 ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts/new.js
 create mode 100644 ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new.hbs
 create mode 100644 ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new/-header.hbs
 create mode 100644 ui/core/tests/integration/components/form/account/password-test.js
 create mode 100644 ui/core/tests/unit/routes/scopes/scope/auth-methods/auth-method/accounts/new-test.js

diff --git a/ui/core/app/components/form/account/index.hbs b/ui/core/app/components/form/account/index.hbs
index eb41e9b5d..b7f418d63 100644
--- a/ui/core/app/components/form/account/index.hbs
+++ b/ui/core/app/components/form/account/index.hbs
@@ -46,6 +46,23 @@
     {{/if}}
   </form.textarea>
 
+  <form.input
+    @name="login_name"
+    @type="text"
+    @value={{@model.attributes.login_name}}
+    @label={{t "form.label.identification"}}
+    @error={{@model.errors.name}}
+    readonly={{false}}
+  as |field|>
+    {{#if @model.errors.name}}
+      <field.errors as |errors|>
+        {{#each @model.errors.name as |error|}}
+          <errors.message>{{error.message}}</errors.message>
+        {{/each}}
+      </field.errors>
+    {{/if}}
+  </form.input>
+
   <form.actions
     @disabled={{if @model.cannotSave @model.cannotSave}}
     @submitText={{t "actions.save"}}
diff --git a/ui/core/app/components/form/account/password/index.hbs b/ui/core/app/components/form/account/password/index.hbs
new file mode 100644
index 000000000..4f167188f
--- /dev/null
+++ b/ui/core/app/components/form/account/password/index.hbs
@@ -0,0 +1,88 @@
+<Rose::Form
+  @onSubmit={{@submit}}
+  @cancel={{@cancel}}
+  @disabled={{@model.isSaving}}
+  @error={{@model.errors.base}}
+  as |form|
+>
+  <form.input
+  @value={{t (concat 'account.types.' @model.type)}}
+  @label={{t "form.label.type"}}
+  readonly={{true}}
+  @disabled={{true}} />
+
+  <form.input
+    @name="name"
+    @type="name"
+    @value={{@model.name}}
+    @label={{t "form.label.name"}}
+    @error={{@model.errors.name}}
+    readonly={{false}}
+  as |field|>
+    {{#if @model.errors.name}}
+      <field.errors as |errors|>
+        {{#each @model.errors.name as |error|}}
+          <errors.message>{{error.message}}</errors.message>
+        {{/each}}
+      </field.errors>
+    {{/if}}
+  </form.input>
+
+  <form.textarea
+    @name="description"
+    @type="description"
+    @value={{@model.description}}
+    @label={{t "form.label.description"}}
+    @error={{@model.errors.description}}
+    disabled={{@model.isSaving}}
+    readonly={{false}}
+  as |field|>
+    {{#if @model.errors.description}}
+      <field.errors as |errors|>
+        {{#each @model.errors.description as |error|}}
+          <errors.message>{{error.message}}</errors.message>
+        {{/each}}
+      </field.errors>
+    {{/if}}
+  </form.textarea>
+
+  <form.input
+    @name="login_name"
+    @type="text"
+    @value={{@model.attributes.login_name}}
+    @label={{t "form.label.identification"}}
+    @error={{@model.errors.name}}
+    readonly={{false}}
+  as |field|>
+    {{#if @model.errors.name}}
+      <field.errors as |errors|>
+        {{#each @model.errors.name as |error|}}
+          <errors.message>{{error.message}}</errors.message>
+        {{/each}}
+      </field.errors>
+    {{/if}}
+  </form.input>
+
+  <form.input
+    @name="password"
+    @type="password"
+    @value={{@model.attributes.password}}
+    @label={{t "form.label.password"}}
+    @error={{@model.errors.name}}
+    readonly={{false}}
+    autocomplete="new-password"
+  as |field|>
+    {{#if @model.errors.name}}
+      <field.errors as |errors|>
+        {{#each @model.errors.name as |error|}}
+          <errors.message>{{error.message}}</errors.message>
+        {{/each}}
+      </field.errors>
+    {{/if}}
+  </form.input>
+
+  <form.actions
+    @disabled={{if @model.cannotSave @model.cannotSave}}
+    @submitText={{t "actions.save"}}
+    @cancelText={{t "actions.cancel"}} />
+</Rose::Form>
\ No newline at end of file
diff --git a/ui/core/app/components/form/host/index.hbs b/ui/core/app/components/form/host/index.hbs
index 2168b7285..72951a2a4 100644
--- a/ui/core/app/components/form/host/index.hbs
+++ b/ui/core/app/components/form/host/index.hbs
@@ -64,7 +64,6 @@
     {{/if}}
   </form.input>
 
-
   <form.actions
     @disabled={{if @model.cannotSave @model.cannotSave}}
     @submitText={{t "actions.save"}}
diff --git a/ui/core/app/router.js b/ui/core/app/router.js
index fcd91a2f2..d0b1980f6 100644
--- a/ui/core/app/router.js
+++ b/ui/core/app/router.js
@@ -75,6 +75,7 @@ Router.map(function () {
         this.route('auth-method', { path: ':auth_method_id' }, function() {
           this.route('accounts', function() {
             this.route('account', { path: ':account_id' }, function() {});
+            this.route('new');
           });
         });
         this.route('new');
diff --git a/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts.js b/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts.js
index 2de991ee2..a33f74879 100644
--- a/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts.js
+++ b/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts.js
@@ -27,7 +27,9 @@ export default class ScopesScopeAuthMethodsAuthMethodAccountsRoute extends Route
    */
   @action
   cancel(account) {
+    const { isNew } = account;
     account.rollbackAttributes();
+    if (isNew) this.transitionTo('scopes.scope.auth-methods.auth-method.accounts');
   }
 
   /**
@@ -36,11 +38,12 @@ export default class ScopesScopeAuthMethodsAuthMethodAccountsRoute extends Route
    */
   @action
   async save(account) {
+    const { isNew } = account;
     try {
       await account.save();
       this.refresh();
       this.notify.success(
-        this.intl.t('notify.save-success')
+        this.intl.t(isNew ? 'notify.create-success' : 'notify.save-success')
       );
       this.transitionTo('scopes.scope.auth-methods.auth-method.accounts.account', account);
     } catch (error) {
diff --git a/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts/new.js b/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts/new.js
new file mode 100644
index 000000000..37dbf4933
--- /dev/null
+++ b/ui/core/app/routes/scopes/scope/auth-methods/auth-method/accounts/new.js
@@ -0,0 +1,43 @@
+import Route from '@ember/routing/route';
+
+export default class ScopesScopeAuthMethodsAuthMethodAccountsNewRoute extends Route {
+
+  // =methods
+
+  /**
+   * Creates a new unsaved account in current scope.
+   * @return {AccountModel}
+   */
+  model() {
+    const { id: auth_method_id } =
+      this.modelFor('scopes.scope.auth-methods.auth-method');
+    return this.store.createRecord('account', {
+      type: 'password',
+      auth_method_id
+    });
+  }
+
+  /**
+   * Renders new account templates for header, navigation, and action page sections.
+   * @override
+   */
+  renderTemplate() {
+    super.renderTemplate(...arguments);
+
+    this.render('scopes/scope/auth-methods/auth-method/accounts/new/-header', {
+      into: 'scopes/scope/auth-methods/auth-method',
+      outlet: 'header'
+    });
+
+    this.render('_empty', {
+      into: 'scopes/scope/auth-methods/auth-method',
+      outlet: 'actions'
+    });
+
+    this.render('_empty', {
+      into: 'scopes/scope/auth-methods/auth-method',
+      outlet: 'navigation'
+    });
+  }
+
+}
diff --git a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/-actions.hbs b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/-actions.hbs
index 16b9c9813..71a762a46 100644
--- a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/-actions.hbs
+++ b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/-actions.hbs
@@ -1,4 +1,10 @@
 <Rose::Dropdown @text="{{t 'actions.manage'}}" @dropdownRight={{true}} as |dropdown|>
+  <dropdown.link
+    @route="scopes.scope.auth-methods.auth-method.accounts.new"
+    @model={{@model}}
+  >
+    {{t "actions.add-account"}}
+  </dropdown.link>
   <dropdown.button
     @style="danger"
     @disabled={{@model.canSave}}
diff --git a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new.hbs b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new.hbs
new file mode 100644
index 000000000..41e9ca5c6
--- /dev/null
+++ b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new.hbs
@@ -0,0 +1,5 @@
+<Form::Account::Password
+  @model={{@model}}
+  @submit={{route-action "save" @model}}
+  @cancel={{route-action "cancel" @model}}
+/>
diff --git a/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new/-header.hbs b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new/-header.hbs
new file mode 100644
index 000000000..ace8ccb08
--- /dev/null
+++ b/ui/core/app/templates/scopes/scope/auth-methods/auth-method/accounts/new/-header.hbs
@@ -0,0 +1 @@
+<h2>{{t "titles.new"}}</h2>
\ No newline at end of file
diff --git a/ui/core/tests/integration/components/form/account/password-test.js b/ui/core/tests/integration/components/form/account/password-test.js
new file mode 100644
index 000000000..0b25dc826
--- /dev/null
+++ b/ui/core/tests/integration/components/form/account/password-test.js
@@ -0,0 +1,24 @@
+import { module, test } from 'qunit';
+import { setupRenderingTest } from 'ember-qunit';
+import { render, find } from '@ember/test-helpers';
+import { hbs } from 'ember-cli-htmlbars';
+
+module('Integration | Component | form/account/password', function(hooks) {
+  setupRenderingTest(hooks);
+
+  test('it renders', async function (assert) {
+    this.submit = () => {};
+    this.cancel = () => {};
+    this.model = { type: 'password' };
+
+    await render(hbs`
+      <Form::Account::Password
+        @model={{this.model}}
+        @submit={{this.submit}}
+        @cancel={{this.cancel}}
+      />`);
+
+    assert.ok(find('form'));
+  });
+
+});
diff --git a/ui/core/tests/unit/routes/scopes/scope/auth-methods/auth-method/accounts/new-test.js b/ui/core/tests/unit/routes/scopes/scope/auth-methods/auth-method/accounts/new-test.js
new file mode 100644
index 000000000..389c7b7a1
--- /dev/null
+++ b/ui/core/tests/unit/routes/scopes/scope/auth-methods/auth-method/accounts/new-test.js
@@ -0,0 +1,11 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'ember-qunit';
+
+module('Unit | Route | scopes/scope/auth-methods/auth-method/accounts/new', function(hooks) {
+  setupTest(hooks);
+
+  test('it exists', function(assert) {
+    let route = this.owner.lookup('route:scopes/scope/auth-methods/auth-method/accounts/new');
+    assert.ok(route);
+  });
+});
diff --git a/ui/core/translations/en-us.yaml b/ui/core/translations/en-us.yaml
index 563e5d898..f1ef3e9cd 100644
--- a/ui/core/translations/en-us.yaml
+++ b/ui/core/translations/en-us.yaml
@@ -28,6 +28,7 @@ actions:
   add-principals: Add Principals
   add-members: Add Members
   add-hosts: Add Hosts
+  add-account: Create Account
 auth:
   titles:
     no-scopes: No Scopes Available