UI: MFA methods now display namespace_path instead of namespace_id (#29588)

* update mfa methods to use namespace_path instead of namespace_id

* add test for empty namespace path and add namespace path to factories

* add changelog entry

* fix name

* hide namespace label when path is empty and test updates

changelog/29588.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+ui: MFA methods now display the namespace path instead of the namespace id.
+```

ui/app/models/mfa-method.js

@@ -58,7 +58,7 @@ export default class MfaMethod extends Model {
   @attr('string', {
     label: 'Namespace',
   })
-  namespace_id;
+  namespace_path;
   @attr('string') mount_accessor;
 
   // PING ID

ui/app/templates/components/mfa/method-list-item.hbs

@@ -17,12 +17,14 @@
             {{@model.name}}
           </span>
           <Hds::Badge @text={{@model.id}} class="has-left-margin-xs" />
+          {{#if @model.namespace_path}}
             <div class="has-top-margin-xs">
               <code class="is-size-9">
                 Namespace:
-              {{@model.namespace_id}}
+                {{@model.namespace_path}}
               </code>
             </div>
+          {{/if}}
         </div>
       </div>
     </div>

ui/mirage/factories/mfa-duo-method.js

@@ -9,7 +9,7 @@ export default Factory.extend({
   api_hostname: 'api-foobar.duosecurity.com',
   mount_accessor: '',
   name: '', // returned but cannot be set at this time
-  namespace_id: 'root',
+  namespace_path: '', // the root namespace returns an empty string for the path
   pushinfo: '',
   type: 'duo',
   use_passcode: false,

ui/mirage/factories/mfa-okta-method.js

@@ -9,7 +9,7 @@ export default Factory.extend({
   base_url: 'okta.com',
   mount_accessor: '',
   name: '', // returned but cannot be set at this time
-  namespace_id: 'root',
+  namespace_path: 'admin/',
   org_name: 'dev-foobar',
   type: 'okta',
   username_template: '', // returned but cannot be set at this time

ui/mirage/factories/mfa-pingid-method.js

@@ -13,5 +13,5 @@ export default Factory.extend({
   org_alias: 'foobarbaz',
   type: 'pingid',
   username_template: '',
-  namespace_id: 'root',
+  namespace_path: 'admin/',
 });

ui/mirage/factories/mfa-totp-method.js

@@ -12,7 +12,7 @@ export default Factory.extend({
   key_size: 20,
   max_validation_attempts: 5,
   name: '', // returned but cannot be set at this time
-  namespace_id: 'root',
+  namespace_path: 'admin/',
   period: 30,
   qr_size: 200,
   skew: 1,

ui/tests/acceptance/mfa-login-enforcement-test.js

@@ -171,7 +171,7 @@ module('Acceptance | mfa-login-enforcement', function (hooks) {
     assert
       .dom(`[data-test-mfa-method-list-item="${method.id}"]`)
       .includesText(
-        `${method.name} ${method.id} Namespace: ${method.namespace_id}`,
+        `${method.name} ${method.id} Namespace: ${method.namespace_path}`,
         'Method list item renders'
       );
     await click('[data-test-popup-menu-trigger]');

ui/tests/acceptance/mfa-method-test.js

@@ -66,7 +66,7 @@ module('Acceptance | mfa-method', function (hooks) {
     assert
       .dom(`[data-test-mfa-method-list-item="${model.id}"]`)
       .includesText(
-        `${model.name} ${model.id} Namespace: ${model.namespace_id}`,
+        `${model.name} ${model.id} Namespace: ${model.namespace_path}`,
         'Copy renders for list item'
       );
 
@@ -87,6 +87,17 @@ module('Acceptance | mfa-method', function (hooks) {
     );
   });
 
+  test('it should not display for the root namespace', async function (assert) {
+    await visit('/vault/access/mfa');
+    const methods = this.getMethods();
+    const duoModel = this.store.peekRecord('mfa-method', methods[1].id);
+    assert.strictEqual(duoModel.namespace_path, '', 'Namespace path is unset');
+    assert
+      .dom(`[data-test-mfa-method-list-item="${duoModel.id}"]`)
+      .includesText(`${duoModel.name} ${duoModel.id}`, 'Copy renders for list item without namespace path')
+      .doesNotContainText('Namespace:', 'Does not include the namespace label');
+  });
+
   test('it should display method details', async function (assert) {
     // ensure methods are tied to an enforcement
     this.server.get('/identity/mfa/login-enforcement', () => {