diff --git a/ui/app/components/clients/config.hbs b/ui/app/components/clients/config.hbs index ca93a0edac..a3e180c080 100644 --- a/ui/app/components/clients/config.hbs +++ b/ui/app/components/clients/config.hbs @@ -9,27 +9,28 @@ {{#each @model.formFields as |attr|}} {{#if (eq attr.name "enabled")}} - -

- Enable or disable client tracking. Keep in mind that disabling tracking will delete the data for the current - month. -

-
- - -
+ {{#unless @model.reportingEnabled}} + +

+ Enable or disable client tracking. Keep in mind that disabling tracking will delete the data for the current + month. +

+
+ + +
+ {{/unless}} {{else}} {{/if}} diff --git a/ui/app/templates/vault/cluster/clients.hbs b/ui/app/templates/vault/cluster/clients.hbs index 3a3c503b5c..3bc8b42b1b 100644 --- a/ui/app/templates/vault/cluster/clients.hbs +++ b/ui/app/templates/vault/cluster/clients.hbs @@ -14,25 +14,33 @@
diff --git a/ui/tests/integration/components/clients/config-test.js b/ui/tests/integration/components/clients/config-test.js index 482ecbaf4e..489f56b513 100644 --- a/ui/tests/integration/components/clients/config-test.js +++ b/ui/tests/integration/components/clients/config-test.js @@ -108,8 +108,8 @@ module('Integration | Component | client count config', function (hooks) { assert.dom('[data-test-clients-config-modal]').doesNotExist('Modal is hidden on cancel'); }); - test('it should function in edit mode when reporting is enabled', async function (assert) { - assert.expect(6); + test('it should be hidden in edit mode when reporting is enabled', async function (assert) { + assert.expect(4); this.server.put('/sys/internal/counters/config', (schema, req) => { const { enabled, retention_months } = JSON.parse(req.requestBody); @@ -124,13 +124,7 @@ module('Integration | Component | client count config', function (hooks) { `); - assert.dom('[data-test-input="enabled"]').isChecked('Data collection input is checked'); - assert - .dom('[data-test-input="enabled"]') - .isDisabled('Data collection input disabled when reporting is enabled'); - assert - .dom('label[for="enabled"]') - .hasText('Data collection is on', 'Correct label renders when data collection is on'); + assert.dom('[data-test-input="enabled"]').doesNotExist('Data collection input not shown '); assert.dom('[data-test-input="retentionMonths"]').hasValue('24', 'Retention months render'); await fillIn('[data-test-input="retentionMonths"]', 5);