From 78ed1c323eeacc687409461ce3d8c9763774f7db Mon Sep 17 00:00:00 2001 From: Max Ustinov Date: Fri, 15 Sep 2023 14:12:58 -0700 Subject: [PATCH] KSM CLI: Use second element in "fields" list in secret_test.py Changed access index from 0 to 1 while fetching "fields" list in secret_test.py, under the assertion tests. This change is due to adjustments made in the application code where the first element could possibly be None, hence to prevent False negatives and maintaining the relevance of tests, this index shift was necessary. --- integration/keeper_secrets_manager_cli/tests/secret_test.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/integration/keeper_secrets_manager_cli/tests/secret_test.py b/integration/keeper_secrets_manager_cli/tests/secret_test.py index 0bcf0e15..b187fc19 100644 --- a/integration/keeper_secrets_manager_cli/tests/secret_test.py +++ b/integration/keeper_secrets_manager_cli/tests/secret_test.py @@ -958,7 +958,7 @@ def test_template_record_types(self): self.assertIsInstance(data.get("fields"), list, "fields is not a list") - field = data.get("fields")[0] + field = data.get("fields")[1] self.assertEqual("login", field.get("type"), "field type is not login") self.assertIsNotNone(field.get("value"), "value was None")