added more tests

Author: kai-ion

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

@@ -74,46 +74,136 @@ TEST_F(ServiceEndpointsConfigFileLoaderTest, TestServiceSpecificEndpointsOnly)
     TempFile configFile(std::ios_base::out | std::ios_base::trunc);
     ASSERT_TRUE(configFile.good());
 
-    configFile << "[profile test]\n";
-    configFile << "region = us-east-1\n";
-    configFile << "services = testservices\n";
-    configFile << "\n[services testservices]\n";
+    configFile << "[services s3-minio]\n";
     configFile << "s3 =\n";
-    configFile << "  endpoint_url = http://localhost:9000\n";
+    configFile << "  endpoint_url = https://play.min.io:9000\n";
+    configFile << "\n[profile dev-minio]\n";
+    configFile << "services = s3-minio\n";
     configFile.flush();
 
     AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
     ASSERT_TRUE(loader.Load());
 
     // Test that global endpoint is null when not set
-    auto globalEndpoint = loader.GetGlobalEndpointUrl("test");
+    auto globalEndpoint = loader.GetGlobalEndpointUrl("dev-minio");
     ASSERT_EQ(nullptr, globalEndpoint);
 
-    // Test service-specific endpoint still works
-    auto s3Endpoint = loader.GetServiceEndpointUrl("test", "s3");
+    // Test service-specific endpoint
+    auto s3Endpoint = loader.GetServiceEndpointUrl("dev-minio", "s3");
     ASSERT_NE(nullptr, s3Endpoint);
-    ASSERT_STREQ("http://localhost:9000", s3Endpoint->c_str());
+    ASSERT_STREQ("https://play.min.io:9000", s3Endpoint->c_str());
+    
+    // Test case insensitive lookup
+    auto s3EndpointUpper = loader.GetServiceEndpointUrl("dev-minio", "S3");
+    ASSERT_NE(nullptr, s3EndpointUpper);
+    ASSERT_STREQ("https://play.min.io:9000", s3EndpointUpper->c_str());
 }
 
 TEST_F(ServiceEndpointsConfigFileLoaderTest, TestGlobalEndpointOnly)
 {
     TempFile configFile(std::ios_base::out | std::ios_base::trunc);
     ASSERT_TRUE(configFile.good());
 
-    configFile << "[profile test]\n";
-    configFile << "endpoint_url = https://global.example.com\n";
+    configFile << "[profile dev-global]\n";
+    configFile << "endpoint_url = https://play.min.io:9000\n";
     configFile.flush();
 
     AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
     ASSERT_TRUE(loader.Load());
 
     // Test global endpoint
-    auto globalEndpoint = loader.GetGlobalEndpointUrl("test");
+    auto globalEndpoint = loader.GetGlobalEndpointUrl("dev-global");
     ASSERT_NE(nullptr, globalEndpoint);
-    ASSERT_STREQ("https://global.example.com", globalEndpoint->c_str());
+    ASSERT_STREQ("https://play.min.io:9000", globalEndpoint->c_str());
 
     // Test that service-specific endpoint is null when not set
-    auto s3Endpoint = loader.GetServiceEndpointUrl("test", "s3");
+    auto s3Endpoint = loader.GetServiceEndpointUrl("dev-global", "s3");
     ASSERT_EQ(nullptr, s3Endpoint);
 }
 
+TEST_F(ServiceEndpointsConfigFileLoaderTest, TestServiceSpecificAndGlobalEndpoints)
+{
+    TempFile configFile(std::ios_base::out | std::ios_base::trunc);
+    ASSERT_TRUE(configFile.good());
+
+    configFile << "[services s3-specific-and-global]\n";
+    configFile << "s3 =\n";
+    configFile << "  endpoint_url = https://play.min.io:9000\n";
+    configFile << "\n[profile dev-s3-specific-and-global]\n";
+    configFile << "endpoint_url = http://localhost:1234\n";
+    configFile << "services = s3-specific-and-global\n";
+    configFile.flush();
+
+    AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
+    ASSERT_TRUE(loader.Load());
+    
+    // Test service-specific S3 endpoint
+    auto s3Endpoint = loader.GetServiceEndpointUrl("dev-s3-specific-and-global", "s3");
+    ASSERT_NE(nullptr, s3Endpoint);
+    ASSERT_STREQ("https://play.min.io:9000", s3Endpoint->c_str());
+    
+    // Test global endpoint for other services
+    auto globalEndpoint = loader.GetGlobalEndpointUrl("dev-s3-specific-and-global");
+    ASSERT_NE(nullptr, globalEndpoint);
+    ASSERT_STREQ("http://localhost:1234", globalEndpoint->c_str());
+    
+    // Test that non-configured service returns null (would fall back to global)
+    auto dynamoEndpoint = loader.GetServiceEndpointUrl("dev-s3-specific-and-global", "dynamodb");
+    ASSERT_EQ(nullptr, dynamoEndpoint);
+}
+
+TEST_F(ServiceEndpointsConfigFileLoaderTest, TestMultipleServicesInDefinition)
+{
+  TempFile configFile(std::ios_base::out | std::ios_base::trunc);
+  ASSERT_TRUE(configFile.good());
+
+  configFile << "[services testing-s3-and-eb]\n";
+  configFile << "s3 =\n";
+  configFile << "  endpoint_url = http://localhost:4567\n";
+  configFile << "elastic_beanstalk =\n";
+  configFile << "  endpoint_url = http://localhost:8000\n";
+  configFile << "\n[profile dev]\n";
+  configFile << "services = testing-s3-and-eb\n";
+  configFile.flush();
+
+  AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
+  ASSERT_TRUE(loader.Load());
+
+  // Test S3 endpoint
+  auto s3Endpoint = loader.GetServiceEndpointUrl("dev", "s3");
+  ASSERT_NE(nullptr, s3Endpoint);
+  ASSERT_STREQ("http://localhost:4567", s3Endpoint->c_str());
+
+  // Test Elastic Beanstalk endpoint with normalized service ID
+  auto ebEndpoint = loader.GetServiceEndpointUrl("dev", "Elastic Beanstalk");
+  ASSERT_NE(nullptr, ebEndpoint);
+  ASSERT_STREQ("http://localhost:8000", ebEndpoint->c_str());
+
+  // Test direct normalized lookup
+  auto ebEndpointDirect = loader.GetServiceEndpointUrl("dev", "elastic_beanstalk");
+  ASSERT_NE(nullptr, ebEndpointDirect);
+  ASSERT_STREQ("http://localhost:8000", ebEndpointDirect->c_str());
+}
+
+TEST_F(ServiceEndpointsConfigFileLoaderTest, TestIgnoreGlobalEndpointInServicesSection)
+{
+    TempFile configFile(std::ios_base::out | std::ios_base::trunc);
+    ASSERT_TRUE(configFile.good());
+
+    configFile << "[profile testing]\n";
+    configFile << "services = bad-service-definition\n";
+    configFile << "\n[services bad-service-definition]\n";
+    configFile << "endpoint_url = http://do-not-use.aws/\n";
+    configFile.flush();
+
+    AWSConfigFileProfileConfigLoader loader(configFile.GetFileName(), true);
+    ASSERT_TRUE(loader.Load());
+    
+    // Test that global endpoint in services section is ignored
+    auto globalEndpoint = loader.GetGlobalEndpointUrl("testing");
+    ASSERT_EQ(nullptr, globalEndpoint);
+    
+    // Test that service-specific endpoints return null (no services configured)
+    auto s3Endpoint = loader.GetServiceEndpointUrl("testing", "s3");
+    ASSERT_EQ(nullptr, s3Endpoint);
+}