From d52340e10d32d2d85deea75a532526b42062d212 Mon Sep 17 00:00:00 2001 From: Angel Garbarino Date: Thu, 27 Feb 2025 13:56:34 -0700 Subject: [PATCH] update failing test (#29758) --- .../acceptance/secrets/backend/aws/aws-configuration-test.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ui/tests/acceptance/secrets/backend/aws/aws-configuration-test.js b/ui/tests/acceptance/secrets/backend/aws/aws-configuration-test.js index 26d8ee6afb..03db1d07fd 100644 --- a/ui/tests/acceptance/secrets/backend/aws/aws-configuration-test.js +++ b/ui/tests/acceptance/secrets/backend/aws/aws-configuration-test.js @@ -173,7 +173,6 @@ module('Acceptance | aws | configuration', function (hooks) { test('it should show identityTokenTtl or maxRetries even if they have not been set', async function (assert) { // documenting the intention that we show fields that have not been set but are returned by the api due to defaults - // this test also documents that maxRetries returns 0 while the API docs indicate -1 is the default value const path = `aws-${this.uid}`; await enablePage.enable('aws', path); @@ -187,13 +186,12 @@ module('Acceptance | aws | configuration', function (hooks) { await click(GENERAL.toggleGroup('Root config options')); await fillIn(GENERAL.inputByAttr('region'), 'eu-central-1'); await click(GENERAL.saveButton); - // the Serializer removes these two from the payload if the API returns their default value. assert .dom(GENERAL.infoRowValue('Identity token TTL')) .hasText('0', 'Identity token TTL shows default.'); assert .dom(GENERAL.infoRowValue('Max retries')) - .hasText('0', 'Max retries shows 0 indicating the default will be used.'); + .hasText('-1', 'Max retries shows -1 indicating the default will be used.'); // cleanup await runCmd(`delete sys/mounts/${path}`); });