Skip to content

Commit 901ca7f

Browse files
committed
Fix TestTranslator to set mock provider before GetMetadataInfo call
The test was resetting the global provider but then calling GetMetadataInfo which falls through to the Azure path on Azure CI runners. Now we set the mock provider first so GetMetadataInfo uses it instead of Azure IMDS.
1 parent aef107a commit 901ca7f

1 file changed

Lines changed: 11 additions & 1 deletion

File tree

translator/translate/otel/exporter/awscloudwatchlogs/translator_test.go

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,24 @@ func testMetadata() *logsutil.Metadata {
3434
func TestTranslator(t *testing.T) {
3535
cloudmetadata.ResetGlobalProvider()
3636
defer cloudmetadata.ResetGlobalProvider()
37+
38+
// Set mock provider to ensure consistent behavior across all environments (including Azure CI)
39+
mock := &cloudmetadata.MockProvider{
40+
InstanceID: "some_instance_id",
41+
Hostname: "some_hostname",
42+
PrivateIP: "some_private_ip",
43+
AccountID: "some_account_id",
44+
}
45+
cloudmetadata.SetGlobalProviderForTest(mock)
46+
3747
t.Setenv(envconfig.AWS_CA_BUNDLE, "/ca/bundle")
3848
agent.Global_Config.Region = "us-east-1"
3949
agent.Global_Config.Role_arn = "global_arn"
4050
agent.Global_Config.Credentials = map[string]any{
4151
"profile": "some_profile",
4252
"shared_credential_file": "/some/credentials",
4353
}
44-
globallogs.GlobalLogConfig.MetadataInfo = logsutil.GetMetadataInfo(testMetadata)
54+
globallogs.GlobalLogConfig.MetadataInfo = logsutil.GetMetadataInfo(nil)
4555
tt := NewTranslatorWithName(common.PipelineNameEmfLogs)
4656
require.EqualValues(t, "awscloudwatchlogs/emf_logs", tt.ID().String())
4757
testCases := map[string]struct {

0 commit comments

Comments
 (0)