diff --git a/addons/api/mirage/config.js b/addons/api/mirage/config.js index 9036f6653..0fe8fb6b1 100644 --- a/addons/api/mirage/config.js +++ b/addons/api/mirage/config.js @@ -77,8 +77,17 @@ export default function () { // Auth Method Accounts this.get( '/accounts', - ({ accounts }, { queryParams: { auth_method_id: authMethodId } }) => { - return accounts.where({ authMethodId }); + ( + { accounts }, + { queryParams: { auth_method_id: authMethodId, filter } } + ) => { + let resultSet; + if (authMethodId) { + resultSet = accounts.where({ authMethodId }); + } else { + resultSet = accounts.all(); + } + return resultSet.filter(makeBooleanFilter(filter)); } ); this.post('/accounts'); diff --git a/addons/api/mirage/factories/auth-method.js b/addons/api/mirage/factories/auth-method.js index 865c10b3d..2196a13a3 100644 --- a/addons/api/mirage/factories/auth-method.js +++ b/addons/api/mirage/factories/auth-method.js @@ -36,7 +36,17 @@ export default factory.extend({ }); if (type === 'oidc') { - server.createList('managed-group', 2, { scope, authMethod }); + server + .createList('managed-group', 2, { scope, authMethod }) + .map((managedGroup) => { + const accounts = server.createList('account', 2, { + scope, + type, + authMethod, + }); + const accountIds = accounts.map((account) => account.id); + managedGroup.update({ memberIds: accountIds }); + }); } }, }), diff --git a/addons/core/translations/resources/en-us.yaml b/addons/core/translations/resources/en-us.yaml index 4d7f4842f..f91ea6c4c 100644 --- a/addons/core/translations/resources/en-us.yaml +++ b/addons/core/translations/resources/en-us.yaml @@ -362,3 +362,6 @@ managed-group: welcome: title: Welcome to Managed Groups description: Create and manage managed groups for an auth method. + accounts: + title: Welcome to Accounts + description: No accounts in this managed group. diff --git a/ui/admin/app/router.js b/ui/admin/app/router.js index edc70b62d..18a09b6c5 100644 --- a/ui/admin/app/router.js +++ b/ui/admin/app/router.js @@ -59,7 +59,9 @@ Router.map(function () { this.route( 'managed-group', { path: ':managed_group_id' }, - function () {} + function () { + this.route('accounts'); + } ); this.route('new'); }); diff --git a/ui/admin/app/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.js b/ui/admin/app/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.js new file mode 100644 index 000000000..e3bdc5a91 --- /dev/null +++ b/ui/admin/app/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.js @@ -0,0 +1,26 @@ +import Route from '@ember/routing/route'; +import { inject as service } from '@ember/service'; + +export default class ScopesScopeAuthMethodsAuthMethodManagedGroupsManagedGroupAccountsRoute extends Route { + // =services + + @service resourceFilterStore; + + // =methods + + /** + * Returns the previously loaded managed group instances accounts. + * @return {Promise{[AccountModel]}} + */ + model() { + const { auth_method_id, member_ids } = this.modelFor( + 'scopes.scope.auth-methods.auth-method.managed-groups.managed-group' + ); + + return this.resourceFilterStore.queryBy( + 'account', + { id: member_ids }, + { auth_method_id } + ); + } +} diff --git a/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group.hbs b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group.hbs index a6696c3b6..f6557cda4 100644 --- a/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group.hbs +++ b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group.hbs @@ -1,2 +1,2 @@ -{{page-title 'ManagedGroup'}} +{{page-title @model.displayName}} {{outlet}} \ No newline at end of file diff --git a/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/-navigation.hbs b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/-navigation.hbs index 3a2ce54b7..18ece151c 100644 --- a/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/-navigation.hbs +++ b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/-navigation.hbs @@ -4,4 +4,9 @@ > {{t 'titles.details'}} + + {{t 'resources.account.title_plural'}} + \ No newline at end of file diff --git a/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.hbs b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.hbs new file mode 100644 index 000000000..2a1b1f286 --- /dev/null +++ b/ui/admin/app/templates/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts.hbs @@ -0,0 +1,62 @@ +{{page-title (t 'resources.account.title_plural')}} + +{{#if @model}} + + + + {{t 'form.name.label'}} + {{t 'form.issuer.label'}} + {{t 'form.subject.label'}} + {{t 'form.email.label'}} + {{t 'form.full_name.label'}} + {{t 'form.type.label'}} + {{t 'form.id.label'}} + + + + {{#each @model as |account|}} + + + {{account.accountName}} +

{{account.description}}

+ + + {{account.attributes.issuer}} + + + {{account.attributes.subject}} + + + {{account.attributes.email}} + + + {{account.attributes.full_name}} + + + {{account.type}} + + + + {{account.id}} + + + + {{/each}} + + +{{else}} + + + + {{t 'resources.managed-group.messages.accounts.description'}} + + + +{{/if}} \ No newline at end of file diff --git a/ui/admin/tests/acceptance/auth-methods/oidc-test.js b/ui/admin/tests/acceptance/auth-methods/oidc-test.js index ab37b1d62..2ab3e56bd 100644 --- a/ui/admin/tests/acceptance/auth-methods/oidc-test.js +++ b/ui/admin/tests/acceptance/auth-methods/oidc-test.js @@ -123,7 +123,6 @@ module('Acceptance | auth methods | oidc', function (hooks) { await visit(urls.authMethod); await click('.rose-layout-page-actions .rose-dropdown-trigger'); await click(`.rose-dropdown[open] input[value="${newState}"]`); - // debugger; const authMethod = this.server.db.authMethods.find(instances.authMethod.id); assert.notEqual( newState, diff --git a/ui/admin/tests/acceptance/managed-groups/accounts-test.js b/ui/admin/tests/acceptance/managed-groups/accounts-test.js new file mode 100644 index 000000000..3a1691d4a --- /dev/null +++ b/ui/admin/tests/acceptance/managed-groups/accounts-test.js @@ -0,0 +1,69 @@ +import { module, test } from 'qunit'; +import { visit, currentURL, findAll } from '@ember/test-helpers'; +import { setupApplicationTest } from 'ember-qunit'; +import setupMirage from 'ember-cli-mirage/test-support/setup-mirage'; +import a11yAudit from 'ember-a11y-testing/test-support/audit'; +import { + authenticateSession, + // These are left here intentionally for future reference. + //currentSession, + //invalidateSession, +} from 'ember-simple-auth/test-support'; + +module('Acceptance | managed-groups | accounts', function (hooks) { + setupApplicationTest(hooks); + setupMirage(hooks); + + const instances = { + scopes: { + global: null, + org: null, + }, + authMethod: null, + managedGroup: null, + accounts: null, + }; + const urls = { + orgScope: null, + authMethods: null, + managedGroups: null, + managedGroup: null, + managedGroupAccounts: null, + }; + + hooks.beforeEach(function () { + authenticateSession({}); + instances.scopes.global = this.server.create('scope', { id: 'global' }); + instances.scopes.org = this.server.create('scope', { + type: 'org', + scope: { id: 'global', type: 'global' }, + }); + instances.authMethod = this.server.create( + 'auth-method', + { + scope: instances.scopes.org, + type: 'oidc', + }, + 'withAccountsAndUsersAndManagedGroups' + ); + + instances.managedGroup = this.server.db.managedGroups[0]; + // Generate route URLs for resources + urls.orgScope = `/scopes/${instances.scopes.org.id}`; + urls.authMethods = `${urls.orgScope}/auth-methods`; + urls.authMethod = `${urls.authMethods}/${instances.authMethod.id}`; + urls.managedGroups = `${urls.authMethod}/managed-groups`; + urls.managedGroup = `${urls.managedGroups}/${instances.managedGroup.id}`; + urls.managedGroupAccounts = `${urls.managedGroup}/accounts`; + }); + + test('User can navigate to index', async function (assert) { + assert.expect(3); + const accountsCount = instances.managedGroup.memberIds.length; + await visit(urls.managedGroupAccounts); + await a11yAudit(); + assert.equal(currentURL(), urls.managedGroupAccounts); + assert.ok(accountsCount); + assert.equal(findAll('tbody tr').length, accountsCount); + }); +}); diff --git a/ui/admin/tests/unit/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts-test.js b/ui/admin/tests/unit/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts-test.js new file mode 100644 index 000000000..5713dd02c --- /dev/null +++ b/ui/admin/tests/unit/routes/scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts-test.js @@ -0,0 +1,16 @@ +import { module, test } from 'qunit'; +import { setupTest } from 'ember-qunit'; + +module( + 'Unit | Route | scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts', + function (hooks) { + setupTest(hooks); + + test('it exists', function (assert) { + let route = this.owner.lookup( + 'route:scopes/scope/auth-methods/auth-method/managed-groups/managed-group/accounts' + ); + assert.ok(route); + }); + } +);