[chore] [receiver/purefa] Use confighttp.NewDefaultClientConfig inste…

…ad of manually creating struct (#35645) **Description:** This PR makes usage of `NewDefaultClientConfig` instead of manually creating the confighttp.ClientConfig struct. **Link to tracking Issue:** #35457 --------- Co-authored-by: Andrzej Stencel <[email protected]>
open-telemetry · Nov 26, 2024 · 7a3fade · 7a3fade
1 parent b6a7c7b
commit 7a3fade
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/receiver/purefareceiver/config_test.go b/receiver/purefareceiver/config_test.go
@@ -11,6 +11,7 @@ import (
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"go.opentelemetry.io/collector/component"
+	"go.opentelemetry.io/collector/config/confighttp"
 	"go.opentelemetry.io/collector/confmap/confmaptest"
 
 	"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/purefareceiver/internal/metadata"
@@ -22,14 +23,17 @@ func TestLoadConfig(t *testing.T) {
 	cm, err := confmaptest.LoadConf(filepath.Join("testdata", "config.yaml"))
 	require.NoError(t, err)
 
+	clientConfig := confighttp.NewDefaultClientConfig()
+
 	tests := []struct {
 		id       component.ID
 		expected component.Config
 	}{
 		{
 			id: component.NewID(metadata.Type),
 			expected: &Config{
-				ArrayName: "foobar.example.com",
+				ClientConfig: clientConfig,
+				ArrayName:    "foobar.example.com",
 				Settings: &Settings{
 					ReloadIntervals: &ReloadIntervals{
 						Array:       15 * time.Second,

diff --git a/receiver/purefareceiver/factory.go b/receiver/purefareceiver/factory.go
@@ -29,7 +29,7 @@ func NewFactory() receiver.Factory {
 func createDefaultConfig() component.Config {
 	return &Config{
 		ArrayName:    "foobar.example.com",
-		ClientConfig: confighttp.ClientConfig{},
+		ClientConfig: confighttp.NewDefaultClientConfig(),
 		Settings: &Settings{
 			ReloadIntervals: &ReloadIntervals{
 				Array:       15 * time.Second,