added more tests

Author: kai-ion

src/aws-cpp-sdk-core/source/config/AWSConfigFileProfileConfigLoader.cpp

@@ -42,6 +42,7 @@ namespace Aws
         static const char SSO_SESSION_SECTION[]              = "sso-session";
         static const char SERVICES_SECTION[]                 = "services";
         static const char ENDPOINT_URL_KEY[]                 = "endpoint_url";
+        static const char IGNORE_CONFIGURED_ENDPOINT_URLS_KEY[] = "ignore_configured_endpoint_urls";
         static const char DEFAULTS_MODE_KEY[]                = "defaults_mode";
         static const char EQ                                 = '=';
         static const char LEFT_BRACKET                       = '[';

tests/aws-cpp-sdk-core-tests/aws/config/ServiceEndpointsConfigFileLoaderTest.cpp

@@ -207,3 +207,76 @@ TEST_F(ServiceEndpointsConfigFileLoaderTest, TestIgnoreGlobalEndpointInServicesS
     auto s3Endpoint = loader.GetServiceEndpointUrl("testing", "s3");
     ASSERT_EQ(nullptr, s3Endpoint);
 }
+
+TEST_F(ServiceEndpointsConfigFileLoaderTest, TestSourceProfileEndpointIsolation)
+{
+    TempFile configFile(std::ios_base::out | std::ios_base::trunc);
+    ASSERT_TRUE(configFile.good());
+
+    configFile << "[profile A]\n";
+    configFile << "credential_source = Ec2InstanceMetadata\n";
+    configFile << "endpoint_url = https://profile-a-endpoint.aws/\n";
+    configFile << "\n[profile B]\n";
+    configFile << "source_profile = A\n";
+    configFile << "role_arn = arn:aws:iam::123456789012:role/roleB\n";
+    configFile << "services = profileB\n";
+    configFile << "\n[services profileB]\n";
+    configFile << "ec2 =\n";
+    configFile << "  endpoint_url = https://profile-b-ec2-endpoint.aws\n";
+    configFile.flush();
+
+    AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
+    ASSERT_TRUE(loader.Load());
+    
+    // Test that profile B gets EC2 endpoint from its own services definition
+    auto ec2Endpoint = loader.GetServiceEndpointUrl("B", "ec2");
+    ASSERT_NE(nullptr, ec2Endpoint);
+    ASSERT_STREQ("https://profile-b-ec2-endpoint.aws", ec2Endpoint->c_str());
+    
+    // Test that profile B has no global endpoint (doesn't inherit from profile A)
+    auto globalEndpointB = loader.GetGlobalEndpointUrl("B");
+    ASSERT_EQ(nullptr, globalEndpointB);
+    
+    // Test that profile A still has its own global endpoint
+    auto globalEndpointA = loader.GetGlobalEndpointUrl("A");
+    ASSERT_NE(nullptr, globalEndpointA);
+    ASSERT_STREQ("https://profile-a-endpoint.aws/", globalEndpointA->c_str());
+    
+    // Test that other services in profile B return null (no chaining to profile A)
+    auto s3Endpoint = loader.GetServiceEndpointUrl("B", "s3");
+    ASSERT_EQ(nullptr, s3Endpoint);
+}
+TEST_F(ServiceEndpointsConfigFileLoaderTest, TestIgnoreConfiguredEndpointUrls)
+{
+    TempFile configFile(std::ios_base::out | std::ios_base::trunc);
+    ASSERT_TRUE(configFile.good());
+
+    configFile << "[profile default]\n";
+    configFile << "ignore_configured_endpoint_urls = true\n";
+    configFile << "endpoint_url = https://should-be-ignored.com\n";
+    configFile << "\n[profile test]\n";
+    configFile << "ignore_configured_endpoint_urls =  TRUE\n";
+    configFile << "\n[profile empty]\n";
+    configFile << "ignore_configured_endpoint_urls =\n";
+    configFile.flush();
+
+    AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
+    ASSERT_TRUE(loader.Load());
+    auto profiles = loader.GetProfiles();
+    
+    // Test flag is parsed and stored
+    ASSERT_STREQ("true", profiles["default"].GetValue("ignore_configured_endpoint_urls").c_str());
+    ASSERT_STREQ("TRUE", profiles["test"].GetValue("ignore_configured_endpoint_urls").c_str());
+    ASSERT_STREQ("", profiles["empty"].GetValue("ignore_configured_endpoint_urls").c_str());
+    
+    // Test absent key returns empty string
+    TempFile configFile2(std::ios_base::out | std::ios_base::trunc);
+    configFile2 << "[profile absent]\n";
+    configFile2 << "region = us-east-1\n";
+    configFile2.flush();
+    
+    AWSConfigFileProfileConfigLoader loader2(configFile2.GetFileName(), true);
+    ASSERT_TRUE(loader2.Load());
+    auto profiles2 = loader2.GetProfiles();
+    ASSERT_STREQ("", profiles2["absent"].GetValue("ignore_configured_endpoint_urls").c_str());
+}