Skip to content

Commit 2b65337

Browse files
author
Krish
committed
fix test
1 parent 009d613 commit 2b65337

File tree

2 files changed

+6
-9
lines changed

2 files changed

+6
-9
lines changed

source/credentials_provider_profile.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -353,6 +353,7 @@ static struct aws_credentials_provider *s_create_sts_based_provider(
353353
.profile_collection_cached = merged_profiles,
354354
.tls_ctx = options->tls_ctx,
355355
.function_table = options->function_table,
356+
.proxy_ev_settings = options->proxy_ev_settings,
356357
};
357358
sts_options.creds_provider =
358359
s_credentials_provider_new_profile_internal(allocator, &profile_provider_options, source_profiles_table);

tests/credentials_provider_sts_tests.c

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1280,15 +1280,17 @@ AWS_TEST_CASE(
12801280
credentials_provider_sts_from_profile_config_with_chain,
12811281
s_credentials_provider_sts_from_profile_config_with_chain_fn)
12821282

1283-
static int s_credentials_provider_sts_from_profile_config_with_proxy_settings(struct aws_allocator *allocator, void *ctx) {
1283+
static int s_credentials_provider_sts_from_profile_config_with_proxy_settings(
1284+
struct aws_allocator *allocator,
1285+
void *ctx) {
12841286
(void)ctx;
12851287

12861288
aws_unset_environment_value(s_default_profile_env_variable_name);
12871289
aws_unset_environment_value(s_default_config_path_env_variable_name);
12881290
aws_unset_environment_value(s_default_credentials_path_env_variable_name);
12891291

12901292
s_aws_sts_tester_init(allocator);
1291-
s_tester.expected_connection_manager_shutdown_callback_count = 3;
1293+
12921294
struct aws_string *config_contents = aws_string_new_from_c_str(allocator, s_source_profile_chain_config_file);
12931295

12941296
struct aws_string *config_file_str = aws_create_process_unique_file_name(allocator);
@@ -1304,7 +1306,6 @@ static int s_credentials_provider_sts_from_profile_config_with_proxy_settings(st
13041306
s_tester.proxy_config = &proxy_config;
13051307
s_tester.fail_connection = true;
13061308

1307-
13081309
struct aws_credentials_provider_profile_options options = {
13091310
.config_file_name_override = aws_byte_cursor_from_string(config_file_str),
13101311
.credentials_file_name_override = aws_byte_cursor_from_string(creds_file_str),
@@ -1317,11 +1318,6 @@ static int s_credentials_provider_sts_from_profile_config_with_proxy_settings(st
13171318
},
13181319
.proxy_ev_settings = &proxy_config,
13191320
};
1320-
int expected_num_requests = 3;
1321-
for (int i = 0; i < expected_num_requests; i++) {
1322-
aws_array_list_push_back(&s_tester.response_data_callbacks, &s_success_creds_doc);
1323-
}
1324-
s_tester.mock_response_code = 200;
13251321

13261322
struct aws_credentials_provider *provider = aws_credentials_provider_new_profile(allocator, &options);
13271323
ASSERT_NOT_NULL(provider);
@@ -1333,7 +1329,7 @@ static int s_credentials_provider_sts_from_profile_config_with_proxy_settings(st
13331329

13341330
s_aws_wait_for_credentials_result();
13351331

1336-
ASSERT_SUCCESS(s_verify_credentials(s_tester.credentials));
1332+
ASSERT_TRUE(s_tester.credentials == NULL);
13371333

13381334
aws_credentials_provider_release(provider);
13391335
s_aws_wait_for_connection_manager_shutdown_callback();

0 commit comments

Comments
 (0)