From 663e9fe9f6f826c11e361ceb9b454d3e4ea2a8a0 Mon Sep 17 00:00:00 2001 From: Kushagar Garg Date: Mon, 22 Jun 2026 19:58:16 +0530 Subject: [PATCH] Fix configure get to support printing section values --- awscli/customizations/configure/get.py | 10 +++------- tests/unit/customizations/configure/test_get.py | 11 +++-------- 2 files changed, 6 insertions(+), 15 deletions(-) diff --git a/awscli/customizations/configure/get.py b/awscli/customizations/configure/get.py index 1d48e06fa46d..c41f8385f2d6 100644 --- a/awscli/customizations/configure/get.py +++ b/awscli/customizations/configure/get.py @@ -60,13 +60,9 @@ def _run_main(self, args, parsed_globals): self._stream.write('\n') return 0 elif isinstance(value, dict): - # TODO: add support for this. We would need to print it off in - # the same format as the config file. - self._error_stream.write( - 'varname (%s) must reference a value, not a section or ' - 'sub-section.' % varname - ) - return 1 + for key, val in value.items(): + self._stream.write('%s = %s\n' % (key, val)) + return 0 else: return 1 diff --git a/tests/unit/customizations/configure/test_get.py b/tests/unit/customizations/configure/test_get.py index 71c239e6ba73..ff45a26a9b7f 100644 --- a/tests/unit/customizations/configure/test_get.py +++ b/tests/unit/customizations/configure/test_get.py @@ -120,14 +120,9 @@ def test_get_section_returns_error(self): session.config = {'s3': {'signature_version': 's3v4'}} stream, error_stream, config_get = self.create_command(session) rc = config_get(args=['s3'], parsed_globals=None) - self.assertEqual(rc, 1) - - error_message = error_stream.getvalue() - expected_message = ( - 'varname (s3) must reference a value, not a section or ' - 'sub-section.') - self.assertEqual(error_message, expected_message) - self.assertEqual(stream.getvalue(), '') + self.assertEqual(rc, 0) + self.assertEqual(stream.getvalue(), 'signature_version = s3v4\n') + self.assertEqual(error_stream.getvalue(), '') def test_get_non_string_returns_error(self): # This should never happen, but we handle this case so we should