diff --git a/comp/trace/config/config_otlp_test.go b/comp/trace/config/config_otlp_test.go index c7429b3588547..eb092218859c1 100644 --- a/comp/trace/config/config_otlp_test.go +++ b/comp/trace/config/config_otlp_test.go @@ -20,7 +20,7 @@ import ( ) func TestFullYamlConfigWithOTLP(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() diff --git a/comp/trace/config/config_test.go b/comp/trace/config/config_test.go index 73517aec3e2e6..c17d4325221be 100644 --- a/comp/trace/config/config_test.go +++ b/comp/trace/config/config_test.go @@ -18,7 +18,6 @@ import ( "path/filepath" "reflect" "regexp" - "runtime" "strings" "testing" "text/template" @@ -149,7 +148,7 @@ func TestSplitTagRegex(t *testing.T) { func TestTelemetryEndpointsConfig(t *testing.T) { t.Run("default", func(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() require.NotNil(t, cfg) @@ -163,7 +162,7 @@ func TestTelemetryEndpointsConfig(t *testing.T) { "apm_config.telemetry.dd_url": "http://example.com/", } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -177,7 +176,7 @@ func TestTelemetryEndpointsConfig(t *testing.T) { "apm_config.telemetry.dd_url": "111://abc.com", } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -192,7 +191,7 @@ func TestTelemetryEndpointsConfig(t *testing.T) { "site": "new_site.example.com", } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -210,7 +209,7 @@ func TestTelemetryEndpointsConfig(t *testing.T) { "apm_config.telemetry.additional_endpoints": additionalEndpoints, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -233,7 +232,7 @@ func TestTelemetryEndpointsConfig(t *testing.T) { "apm_config.telemetry.additional_endpoints": additionalEndpoints, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -301,7 +300,7 @@ func TestConfigHostname(t *testing.T) { t.Skip() } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/site_override.yaml"}, Overrides: overrides, })) @@ -313,7 +312,7 @@ func TestConfigHostname(t *testing.T) { }) t.Run("file", func(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -326,7 +325,7 @@ func TestConfigHostname(t *testing.T) { t.Run("env", func(t *testing.T) { t.Setenv("DD_HOSTNAME", "onlyenv") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/site_override.yaml"}, })) cfg := config.Object() @@ -338,7 +337,7 @@ func TestConfigHostname(t *testing.T) { t.Run("file+env", func(t *testing.T) { t.Setenv("DD_HOSTNAME", "envoverride") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -352,7 +351,7 @@ func TestConfigHostname(t *testing.T) { "serverless.enabled": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/site_default.yaml"}, Overrides: overrides, })) @@ -405,7 +404,7 @@ func TestConfigHostname(t *testing.T) { bin := makeProgram(t, "host.name", 0) defer os.Remove(bin) - config := buildConfigComponent(t) + config := buildConfigComponent(t, false) cfg := config.Object() require.NotNil(t, cfg) @@ -419,7 +418,7 @@ func TestConfigHostname(t *testing.T) { bin := makeProgram(t, "", 0) defer os.Remove(bin) - config := buildConfigComponent(t) + config := buildConfigComponent(t, false) cfg := config.Object() require.NotNil(t, cfg) @@ -432,7 +431,7 @@ func TestConfigHostname(t *testing.T) { bin := makeProgram(t, "", 0) defer os.Remove(bin) - config := buildConfigComponent(t) + config := buildConfigComponent(t, false) cfg := config.Object() require.NotNil(t, cfg) @@ -447,7 +446,7 @@ func TestConfigHostname(t *testing.T) { bin := makeProgram(t, "", 1) defer os.Remove(bin) - config := buildConfigComponent(t) + config := buildConfigComponent(t, false) cfg := config.Object() require.NotNil(t, cfg) @@ -460,7 +459,7 @@ func TestConfigHostname(t *testing.T) { bin := makeProgram(t, "some text", 1) defer os.Remove(bin) - config := buildConfigComponent(t) + config := buildConfigComponent(t, false) cfg := config.Object() require.NotNil(t, cfg) @@ -483,7 +482,7 @@ func TestSite(t *testing.T) { "vector": {"./testdata/observability_pipelines_worker_override.yaml", "https://observability_pipelines_worker.domain.tld:8443"}, } { t.Run(name, func(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: tt.file}, })) cfg := config.Object() @@ -495,7 +494,7 @@ func TestSite(t *testing.T) { } func TestDefaultConfig(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() require.NotNil(t, cfg) @@ -516,7 +515,7 @@ func TestDefaultConfig(t *testing.T) { } func TestNoAPMConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/no_apm_config.yaml"}, })) cfg := config.Object() @@ -530,7 +529,7 @@ func TestNoAPMConfig(t *testing.T) { } func TestDisableLoggingConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/disable_file_logging.yaml"}, })) cfg := config.Object() @@ -541,7 +540,7 @@ func TestDisableLoggingConfig(t *testing.T) { } func TestFullYamlConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -642,7 +641,7 @@ func TestFullYamlConfig(t *testing.T) { } func TestFileLoggingDisabled(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/disable_file_logging.yaml"}, })) @@ -653,7 +652,7 @@ func TestFileLoggingDisabled(t *testing.T) { } func TestUndocumentedYamlConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/undocumented.yaml"}, })) cfg := config.Object() @@ -711,7 +710,7 @@ func TestNormalizeEnvFromDDEnv(t *testing.T) { t.Run("", func(t *testing.T) { t.Setenv("DD_ENV", in) - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/no_apm_config.yaml"}, })) cfg := config.Object() @@ -733,7 +732,7 @@ func TestNormalizeEnvFromDDTags(t *testing.T) { t.Run("", func(t *testing.T) { t.Setenv("DD_TAGS", in) - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/no_apm_config.yaml"}, })) cfg := config.Object() @@ -755,7 +754,7 @@ func TestNormalizeEnvFromConfig(t *testing.T) { "./testdata/non-normalized_env_host_tag.yaml", } { t.Run("", func(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: cfgFile}, })) @@ -784,7 +783,7 @@ func TestLoadEnv(t *testing.T) { t.Setenv(tt.envOld, "1,2,3") t.Setenv(tt.envNew, "4,5,6") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -803,7 +802,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "123") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -815,7 +814,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_SITE" t.Run(env, func(t *testing.T) { t.Setenv(env, "my-site.com") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/site_default.yaml"}, })) @@ -828,7 +827,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_APM_ENABLED" t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -840,7 +839,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_APM_DD_URL" t.Run(env, func(t *testing.T) { t.Setenv(env, "my-site.com") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -852,7 +851,7 @@ func TestLoadEnv(t *testing.T) { env = "HTTPS_PROXY" t.Run(env, func(t *testing.T) { t.Setenv(env, "my-proxy.url") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -864,7 +863,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_PROXY_HTTPS" t.Run(env, func(t *testing.T) { t.Setenv(env, "my-proxy.url") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -876,7 +875,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_HOSTNAME" t.Run(env, func(t *testing.T) { t.Setenv(env, "local.host") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -888,7 +887,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_BIND_HOST" t.Run(env, func(t *testing.T) { t.Setenv(env, "bindhost.com") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, false, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -904,7 +903,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "1234") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -918,7 +917,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "4321") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -931,7 +930,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/undocumented.yaml"}, })) cfg := config.Object() @@ -944,7 +943,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "12.3") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/undocumented.yaml"}, })) cfg := config.Object() @@ -960,7 +959,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "1,2,3") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := config.Object() @@ -974,7 +973,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "web|http.request=1,db|sql.query=0.5") - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -991,7 +990,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `[{"name":"name1", "pattern":"pattern1"}, {"name":"name2","pattern":"pattern2","repl":"replace2"}]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -1018,7 +1017,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `important1 important2:value1`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -1032,7 +1031,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["important1:value with a space"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1046,7 +1045,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `important1 important2:^value1$`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -1060,7 +1059,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["important1:^value with a space$"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) @@ -1075,7 +1074,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `bad1:value1`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1088,7 +1087,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["bad1:value with a space"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1101,7 +1100,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["bad1:value with a space","bad2:value with spaces"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1117,7 +1116,7 @@ func TestLoadEnv(t *testing.T) { env = "DD_APM_FILTER_TAGS_REGEX_REJECT" t.Run(env, func(t *testing.T) { t.Setenv(env, `bad1:^value1$`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1129,7 +1128,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["bad1:value with a space"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1146,7 +1145,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "50") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1165,7 +1164,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "6") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/deprecated-max-tps-apm.yaml"}, })) @@ -1180,7 +1179,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "6") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1201,7 +1200,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "12") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1217,7 +1216,7 @@ func TestLoadEnv(t *testing.T) { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1233,7 +1232,7 @@ func TestLoadEnv(t *testing.T) { } { t.Run(envKey, func(t *testing.T) { t.Setenv(envKey, "7") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1247,7 +1246,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "337.41") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1260,7 +1259,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `{"url1": ["key1", "key2"], "url2": ["key3"]}`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1276,7 +1275,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-site.com") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1290,7 +1289,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-site.com") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1304,7 +1303,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-key") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1318,7 +1317,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `{"url1": ["key1", "key2"], "url2": ["key3"]}`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1339,7 +1338,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-diagnostics-site.com") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1353,7 +1352,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-diagnostics-key") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1367,7 +1366,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `{"diagnostics-url1": ["diagnostics-key1", "diagnostics-key2"], "diagnostics-url2": ["diagnostics-key3"]}`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1388,7 +1387,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-site.com") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1401,7 +1400,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "my-key") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1414,7 +1413,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `{"url1": ["key1", "key2"], "url2": ["key3"]}`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1435,7 +1434,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "false") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1450,7 +1449,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "false") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1463,7 +1462,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1477,7 +1476,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["client_id", "product_id"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1494,7 +1493,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["key1", "key2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1511,7 +1510,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1525,7 +1524,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1539,7 +1538,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1553,7 +1552,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1569,7 +1568,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1583,7 +1582,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["document_id", "template_id"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1600,7 +1599,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["key1", "key2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1617,7 +1616,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1631,7 +1630,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1645,7 +1644,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1659,7 +1658,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1673,7 +1672,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["id1", "id2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1690,7 +1689,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["key1", "key2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1707,7 +1706,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, "true") - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1721,7 +1720,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["id1", "id2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1738,7 +1737,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `["key1", "key2"]`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1755,7 +1754,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `{"url1": ["key1", "key2"], "url2": ["key3"]}`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1776,7 +1775,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { assert := func(in string, _ []string) { t.Setenv(env, in) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1802,7 +1801,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `install_id_foo_bar`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1816,7 +1815,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `host_injection`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1830,7 +1829,7 @@ func TestLoadEnv(t *testing.T) { t.Run(env, func(t *testing.T) { t.Setenv(env, `1699621675`) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/full.yaml"}, })) cfg := c.Object() @@ -1862,7 +1861,7 @@ func TestFargateConfig(t *testing.T) { } { t.Run("", func(t *testing.T) { env.SetFeatures(t, data.features...) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/no_apm_config.yaml"}, })) @@ -1876,7 +1875,7 @@ func TestFargateConfig(t *testing.T) { func TestSetMaxMemCPU(t *testing.T) { t.Run("default, non-containerized", func(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() require.NotNil(t, cfg) @@ -1887,7 +1886,7 @@ func TestSetMaxMemCPU(t *testing.T) { }) t.Run("default, containerized", func(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() require.NotNil(t, cfg) @@ -1903,7 +1902,7 @@ func TestSetMaxMemCPU(t *testing.T) { "apm_config.max_memory": "200", } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -1918,7 +1917,7 @@ func TestSetMaxMemCPU(t *testing.T) { "apm_config.max_memory": "300", } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -1931,7 +1930,7 @@ func TestSetMaxMemCPU(t *testing.T) { func TestPeerTagsAggregation(t *testing.T) { t.Run("default-enabled", func(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() assert.True(t, cfg.PeerTagsAggregation) assert.Nil(t, cfg.PeerTags) @@ -1948,7 +1947,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_service_aggregation": true, "apm_config.peer_tags_aggregation": false, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.False(t, cfg.PeerTagsAggregation) @@ -1961,7 +1960,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_service_aggregation": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.True(t, cfg.PeerTagsAggregation) @@ -1978,7 +1977,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_service_aggregation": false, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.True(t, cfg.PeerTagsAggregation) @@ -1996,7 +1995,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_tags_aggregation": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.True(t, cfg.PeerTagsAggregation) @@ -2014,7 +2013,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_tags_aggregation": false, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.False(t, cfg.PeerTagsAggregation) @@ -2027,7 +2026,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_tags": []string{"user_peer_tag"}, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.False(t, cfg.PeerTagsAggregation) @@ -2039,7 +2038,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_tags": []string{"user_peer_tag"}, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() assert.True(t, cfg.PeerTagsAggregation) assert.Equal(t, []string{"user_peer_tag"}, cfg.PeerTags) @@ -2055,7 +2054,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_tags_aggregation": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.True(t, cfg.PeerTagsAggregation) @@ -2068,7 +2067,7 @@ func TestPeerTagsAggregation(t *testing.T) { "apm_config.peer_service_aggregation": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) assert.True(t, cfg.PeerTagsAggregation) @@ -2084,7 +2083,7 @@ func TestPeerTagsAggregation(t *testing.T) { func TestComputeStatsBySpanKind(t *testing.T) { t.Run("default-enabled", func(t *testing.T) { - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) cfg := config.Object() @@ -2096,7 +2095,7 @@ func TestComputeStatsBySpanKind(t *testing.T) { overrides := map[string]interface{}{ "apm_config.compute_stats_by_span_kind": false, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -2108,7 +2107,7 @@ func TestComputeStatsBySpanKind(t *testing.T) { "apm_config.compute_stats_by_span_kind": true, } - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{Overrides: overrides})) + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{Overrides: overrides})) cfg := config.Object() require.NotNil(t, cfg) @@ -2124,7 +2123,7 @@ func TestGenerateInstallSignature(t *testing.T) { err = os.WriteFile(cfgFile, cfgContent, 0644) assert.NoError(t, err) - c := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + c := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: cfgFile}, })) @@ -2153,7 +2152,7 @@ func TestGenerateInstallSignature(t *testing.T) { func TestMockConfig(t *testing.T) { t.Setenv("DD_SITE", "datadoghq.eu") - config := buildConfigComponent(t, fx.Supply(corecomp.Params{})) + config := buildConfigComponent(t, true, fx.Supply(corecomp.Params{})) cfg := config.Object() require.NotNil(t, cfg) @@ -2162,7 +2161,7 @@ func TestMockConfig(t *testing.T) { } func TestMockDefaultConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Supply(corecomp.Params{})) + config := buildConfigComponent(t, true, fx.Supply(corecomp.Params{})) cfg := config.Object() require.NotNil(t, cfg) @@ -2171,8 +2170,7 @@ func TestMockDefaultConfig(t *testing.T) { } func TestGetCoreConfigHandler(t *testing.T) { - t.Skip("skipping flaky test - APMSP-1390") - config := buildConfigComponent(t, fx.Supply(corecomp.Params{})) + config := buildConfigComponent(t, true, fx.Supply(corecomp.Params{})) handler := config.GetConfigHandler().(http.HandlerFunc) @@ -2209,7 +2207,7 @@ func TestGetCoreConfigHandler(t *testing.T) { } func TestDisableReceiverConfig(t *testing.T) { - config := buildConfigComponent(t, fx.Replace(corecomp.MockParams{ + config := buildConfigComponent(t, true, fx.Replace(corecomp.MockParams{ Params: corecomp.Params{ConfFilePath: "./testdata/disable_receiver.yaml"}, })) cfg := config.Object() @@ -2220,16 +2218,12 @@ func TestDisableReceiverConfig(t *testing.T) { } func TestOnUpdateAPIKeyCallback(t *testing.T) { - // APMSP-1494 - if runtime.GOOS == "darwin" { - t.Skip("skipping flaky test on darwin") - } var n int callback := func(_, _ string) { n++ } - config := buildConfigComponent(t) + config := buildConfigComponent(t, true) config.OnUpdateAPIKey(callback) @@ -2238,13 +2232,18 @@ func TestOnUpdateAPIKeyCallback(t *testing.T) { assert.Equal(t, 1, n) } -func buildConfigComponent(t *testing.T, coreConfigOptions ...fx.Option) Component { +func buildConfigComponent(t *testing.T, setHostnameInConfig bool, coreConfigOptions ...fx.Option) Component { t.Helper() coreConfig := fxutil.Test[corecomp.Component](t, corecomp.MockModule(), fx.Options(coreConfigOptions...), ) + // set the hostname in the config to avoid trying to create a connection to the core agent + // (This can be slow and flaky in tests that don't need to run this logic) + if setHostnameInConfig { + coreConfig.SetWithoutSource("hostname", "testhostname") + } taggerComponent := fxutil.Test[taggermock.Mock](t, fx.Replace(coreConfig),