fix: revert all the renamed variables

Author: Hyunuk Lim

component/experimental/component/factory.go

@@ -44,7 +44,7 @@ type ConfigSourceFactory interface {
 	// configuration and should not cause side-effects that prevent the creation
 	// of multiple instances of the Source.
 	// The object returned by this method needs to pass the checks implemented by
-	// 'configtest.CheckConfigStruct'. It is recommended to have such check in the
+	// 'configtest.ValidateConfig'. It is recommended to have such check in the
 	// tests of any implementation of the ConfigSourceFactory interface.
 	CreateDefaultConfig() config.Source
 

component/exporter.go

@@ -70,7 +70,7 @@ type ExporterFactory interface {
 	// configuration and should not cause side-effects that prevent the creation
 	// of multiple instances of the Exporter.
 	// The object returned by this method needs to pass the checks implemented by
-	// 'configtest.CheckConfigStruct'. It is recommended to have these checks in the
+	// 'configtest.ValidateConfig'. It is recommended to have these checks in the
 	// tests of any implementation of the Factory interface.
 	CreateDefaultConfig() config.Exporter
 

component/extension.go

@@ -69,7 +69,7 @@ type ExtensionFactory interface {
 	// configuration and should not cause side-effects that prevent the creation
 	// of multiple instances of the Extension.
 	// The object returned by this method needs to pass the checks implemented by
-	// 'configtest.CheckConfigStruct'. It is recommended to have these checks in the
+	// 'configtest.ValidateConfig'. It is recommended to have these checks in the
 	// tests of any implementation of the Factory interface.
 	CreateDefaultConfig() config.Extension
 

component/processor.go

@@ -75,7 +75,7 @@ type ProcessorFactory interface {
 	// configuration and should not cause side-effects that prevent the creation
 	// of multiple instances of the Processor.
 	// The object returned by this method needs to pass the checks implemented by
-	// 'configtest.CheckConfigStruct'. It is recommended to have these checks in the
+	// 'configtest.ValidateConfig'. It is recommended to have these checks in the
 	// tests of any implementation of the Factory interface.
 	CreateDefaultConfig() config.Processor
 

component/receiver.go

@@ -79,7 +79,7 @@ type ReceiverFactory interface {
 	// configuration and should not cause side-effects that prevent the creation
 	// of multiple instances of the Receiver.
 	// The object returned by this method needs to pass the checks implemented by
-	// 'configtest.CheckConfigStruct'. It is recommended to have these checks in the
+	// 'configtest.ValidateConfig'. It is recommended to have these checks in the
 	// tests of any implementation of the Factory interface.
 	CreateDefaultConfig() config.Receiver
 

config/configcheck/configcheck.go

@@ -20,28 +20,28 @@ import (
 	"go.opentelemetry.io/collector/consumer/consumererror"
 )
 
-// CheckConfigStructFromFactories checks if all configurations for the given factories
+// ValidateConfigFromFactories checks if all configurations for the given factories
 // are satisfying the patterns used by the collector.
-func CheckConfigStructFromFactories(factories component.Factories) error {
+func ValidateConfigFromFactories(factories component.Factories) error {
 	var errs []error
 
 	for _, factory := range factories.Receivers {
-		if err := configtest.CheckConfigStruct(factory.CreateDefaultConfig()); err != nil {
+		if err := configtest.ValidateConfig(factory.CreateDefaultConfig()); err != nil {
 			errs = append(errs, err)
 		}
 	}
 	for _, factory := range factories.Processors {
-		if err := configtest.CheckConfigStruct(factory.CreateDefaultConfig()); err != nil {
+		if err := configtest.ValidateConfig(factory.CreateDefaultConfig()); err != nil {
 			errs = append(errs, err)
 		}
 	}
 	for _, factory := range factories.Exporters {
-		if err := configtest.CheckConfigStruct(factory.CreateDefaultConfig()); err != nil {
+		if err := configtest.ValidateConfig(factory.CreateDefaultConfig()); err != nil {
 			errs = append(errs, err)
 		}
 	}
 	for _, factory := range factories.Extensions {
-		if err := configtest.CheckConfigStruct(factory.CreateDefaultConfig()); err != nil {
+		if err := configtest.ValidateConfig(factory.CreateDefaultConfig()); err != nil {
 			errs = append(errs, err)
 		}
 	}

config/configcheck/configcheck_test.go

@@ -25,23 +25,23 @@ import (
 	"go.opentelemetry.io/collector/service/defaultcomponents"
 )
 
-func TestCheckConfigStructFromFactories_Success(t *testing.T) {
+func TestValidateConfigFromFactories_Success(t *testing.T) {
 	factories, err := defaultcomponents.Components()
 	require.NoError(t, err)
 
-	err = CheckConfigStructFromFactories(factories)
+	err = ValidateConfigFromFactories(factories)
 	require.NoError(t, err)
 }
 
-func TestCheckConfigStructFromFactories_Failure(t *testing.T) {
+func TestValidateConfigFromFactories_Failure(t *testing.T) {
 	factories, err := defaultcomponents.Components()
 	require.NoError(t, err)
 
 	// Add a factory returning config not following pattern to force error.
 	f := &badConfigExtensionFactory{}
 	factories.Extensions[f.Type()] = f
 
-	err = CheckConfigStructFromFactories(factories)
+	err = ValidateConfigFromFactories(factories)
 	require.Error(t, err)
 }
 

config/configtest/configtest.go

@@ -50,12 +50,12 @@ func LoadConfigAndValidate(fileName string, factories component.Factories) (*con
 	return cfg, cfg.Validate()
 }
 
-// CheckConfigStruct enforces that given configuration object is following the patterns
+// ValidateConfig enforces that given configuration object is following the patterns
 // used by the collector. This ensures consistency between different implementations
 // of components and extensions. It is recommended for implementers of components
 // to call this function on their tests passing the default configuration of the
 // component factory.
-func CheckConfigStruct(config interface{}) error {
+func ValidateConfig(config interface{}) error {
 	t := reflect.TypeOf(config)
 	if t.Kind() == reflect.Ptr {
 		t = t.Elem()
@@ -65,18 +65,18 @@ func CheckConfigStruct(config interface{}) error {
 		return fmt.Errorf("config must be a struct or a pointer to one, the passed object is a %s", t.Kind())
 	}
 
-	return CheckConfigStructDataType(t)
+	return validateConfigDataType(t)
 }
 
-// CheckConfigStructDataType performs a descending validation of the given type.
+// validateConfigDataType performs a descending validation of the given type.
 // If the type is a struct it goes to each of its fields to check for the proper
 // tags.
-func CheckConfigStructDataType(t reflect.Type) error {
+func validateConfigDataType(t reflect.Type) error {
 	var errs []error
 
 	switch t.Kind() {
 	case reflect.Ptr:
-		if err := CheckConfigStructDataType(t.Elem()); err != nil {
+		if err := validateConfigDataType(t.Elem()); err != nil {
 			errs = append(errs, err)
 		}
 	case reflect.Struct:
@@ -158,11 +158,11 @@ func checkStructFieldTags(f reflect.StructField) error {
 	switch f.Type.Kind() {
 	case reflect.Struct:
 		// It is another struct, continue down-level.
-		return CheckConfigStructDataType(f.Type)
+		return validateConfigDataType(f.Type)
 
 	case reflect.Map, reflect.Slice, reflect.Array:
 		// The element of map, array, or slice can be itself a configuration object.
-		return CheckConfigStructDataType(f.Type.Elem())
+		return validateConfigDataType(f.Type.Elem())
 
 	default:
 		fieldTag := tagParts[0]

config/configtest/configtest_test.go

@@ -82,15 +82,15 @@ func TestLoadConfigAndValidate(t *testing.T) {
 	assert.Equal(t, cfg, cfgValidate)
 }
 
-func TestCheckConfigStructPointerAndValue(t *testing.T) {
+func TestValidateConfigPointerAndValue(t *testing.T) {
 	config := struct {
 		SomeFiled string `mapstructure:"test"`
 	}{}
-	assert.NoError(t, CheckConfigStruct(config))
-	assert.NoError(t, CheckConfigStruct(&config))
+	assert.NoError(t, ValidateConfig(config))
+	assert.NoError(t, ValidateConfig(&config))
 }
 
-func TestCheckConfigStruct(t *testing.T) {
+func TestValidateConfig(t *testing.T) {
 	type BadConfigTag struct {
 		BadTagField int `mapstructure:"test-dash"`
 	}
@@ -206,7 +206,7 @@ func TestCheckConfigStruct(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			err := CheckConfigStruct(tt.config)
+			err := ValidateConfig(tt.config)
 			if tt.wantErrMsgSubStr == "" {
 				assert.NoError(t, err)
 			} else {

exporter/loggingexporter/factory_test.go

@@ -28,7 +28,7 @@ func TestCreateDefaultConfig(t *testing.T) {
 	factory := NewFactory()
 	cfg := factory.CreateDefaultConfig()
 	assert.NotNil(t, cfg, "failed to create default config")
-	assert.NoError(t, configtest.CheckConfigStruct(cfg))
+	assert.NoError(t, configtest.ValidateConfig(cfg))
 }
 
 func TestCreateMetricsExporter(t *testing.T) {