chore: full coverage for orgs route
This commit is contained in:
2 files changed
+10
-5
No files matched your search
@@ -38,9 +38,8 @@ export default class ScopesScopeOrgsRoute extends Route {
|
||||
*/
|
||||
@action
|
||||
cancel(org) {
|
||||
const { isNew } = org;
|
||||
org.rollbackAttributes();
|
||||
if (isNew) this.transitionTo('scopes.scope.orgs');
|
||||
this.transitionTo('scopes.scope.orgs');
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -51,13 +50,12 @@ export default class ScopesScopeOrgsRoute extends Route {
|
||||
@action
|
||||
@loading
|
||||
async save(org) {
|
||||
const { isNew } = org;
|
||||
try {
|
||||
await org.save();
|
||||
await this.transitionTo('scopes.scope', org);
|
||||
await this.transitionTo('scopes.scope.projects', org);
|
||||
this.refresh();
|
||||
this.notify.success(
|
||||
this.intl.t(isNew ? 'notifications.create-success' : 'notifications.save-success')
|
||||
this.intl.t('notifications.create-success')
|
||||
);
|
||||
} catch (error) {
|
||||
// TODO: replace with translated strings
|
||||
|
||||
@@ -105,6 +105,13 @@ module('Acceptance | authentication', function (hooks) {
|
||||
assert.notOk(currentSession().isAuthenticated);
|
||||
});
|
||||
|
||||
test('visiting orgs while unauthenticated redirects to first global authenticate method', async function (assert) {
|
||||
assert.expect(2);
|
||||
await visit(orgsURL);
|
||||
assert.equal(currentURL(), authMethodGlobalAuthenticateURL);
|
||||
assert.notOk(currentSession().isAuthenticated);
|
||||
});
|
||||
|
||||
test('visiting projects while unauthenticated redirects to first global authenticate method', async function (assert) {
|
||||
assert.expect(2);
|
||||
await visit(projectsURL);
|
||||
|
||||
Reference in new issue
Block a user