chore: pass correct disabled args and reconsider cancel flow on set password
This commit is contained in:
2 files changed
+11
-8
No files matched your search
@@ -44,7 +44,7 @@
|
||||
</form.input>
|
||||
|
||||
<form.actions
|
||||
@disabled={{this.cannotSave}}
|
||||
@submitDisabled={{this.cannotSave}}
|
||||
@submitText={{t "actions.save"}}
|
||||
@cancelText={{t "actions.cancel"}} />
|
||||
|
||||
|
||||
@@ -71,13 +71,16 @@ module('Acceptance | accounts | set password', function (hooks) {
|
||||
await a11yAudit();
|
||||
});
|
||||
|
||||
test('can cancel setting new password', async function (assert) {
|
||||
assert.expect(1);
|
||||
await visit(urls.setPassword);
|
||||
await fillIn('[name="password"]', 'update password');
|
||||
await click('form button:not([type="submit"])');
|
||||
assert.notOk(find('[name="password"]').textContent.trim());
|
||||
});
|
||||
// NOTE: since set password is a tabbed route with easy navigation away
|
||||
// from the form, and the form does not directly operate on a model instance,
|
||||
// a "cancel" flow doesn't make much sense. Disabling for now.
|
||||
// test('can cancel setting new password', async function (assert) {
|
||||
// assert.expect(1);
|
||||
// await visit(urls.setPassword);
|
||||
// await fillIn('[name="password"]', 'update password');
|
||||
// await click('form button:not([type="submit"])');
|
||||
// assert.notOk(find('[name="password"]').textContent.trim());
|
||||
// });
|
||||
|
||||
test('errors are displayed when setting password fails', async function (assert) {
|
||||
assert.expect(1);
|
||||
|
||||
Reference in new issue
Block a user