[exporter/bmchelix] New component: BMC Helix Exporter

* Fixed ApiKey var naming (APIKey)
open-telemetry · Jan 10, 2025 · 0aadc01 · 0aadc01
1 parent 0869621
commit 0aadc01
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/exporter/bmchelixexporter/config.go b/exporter/bmchelixexporter/config.go
@@ -13,7 +13,7 @@ import (
 // Config struct is used to store the configuration of the exporter
 type Config struct {
 	Endpoint    string                    `mapstructure:"endpoint"`
-	ApiKey      string                    `mapstructure:"api_key"`
+	APIKey      string                    `mapstructure:"api_key"`
 	Timeout     time.Duration             `mapstructure:"timeout"`
 	RetryConfig configretry.BackOffConfig `mapstructure:"retry_on_failure"`
 }
@@ -23,7 +23,7 @@ func (c *Config) Validate() error {
 	if c.Endpoint == "" {
 		return errors.New("endpoint is required")
 	}
-	if c.ApiKey == "" {
+	if c.APIKey == "" {
 		return errors.New("api key is required")
 	}
 	if c.Timeout <= 0 {

diff --git a/exporter/bmchelixexporter/config_test.go b/exporter/bmchelixexporter/config_test.go
@@ -32,7 +32,7 @@ func TestLoadConfig(t *testing.T) {
 			id: component.NewIDWithName(metadata.Type, "helix1"),
 			expected: &Config{
 				Endpoint:    "https://helix1:8080",
-				ApiKey:      "api_key",
+				APIKey:      "api_key",
 				Timeout:     10 * time.Second,
 				RetryConfig: configretry.NewDefaultBackOffConfig(),
 			},
@@ -41,7 +41,7 @@ func TestLoadConfig(t *testing.T) {
 			id: component.NewIDWithName(metadata.Type, "helix2"),
 			expected: &Config{
 				Endpoint: "https://helix2:8080",
-				ApiKey:   "api_key",
+				APIKey:   "api_key",
 				Timeout:  20 * time.Second,
 				RetryConfig: configretry.BackOffConfig{
 					Enabled:             true,
@@ -80,14 +80,14 @@ func TestValidateConfig(t *testing.T) {
 			name: "valid_config",
 			config: &Config{
 				Endpoint: "https://helix:8080",
-				ApiKey:   "api_key",
+				APIKey:   "api_key",
 				Timeout:  10 * time.Second,
 			},
 		},
 		{
 			name: "invalid_config1",
 			config: &Config{
-				ApiKey: "api_key",
+				APIKey: "api_key",
 			},
 			err: "endpoint is required",
 		},
@@ -102,7 +102,7 @@ func TestValidateConfig(t *testing.T) {
 			name: "invalid_config3",
 			config: &Config{
 				Endpoint: "https://helix:8080",
-				ApiKey:   "api_key",
+				APIKey:   "api_key",
 				Timeout:  -1,
 			},
 			err: "timeout must be a positive integer",
@@ -111,7 +111,7 @@ func TestValidateConfig(t *testing.T) {
 			name: "invalid_config4",
 			config: &Config{
 				Endpoint: "https://helix:8080",
-				ApiKey:   "api_key",
+				APIKey:   "api_key",
 				Timeout:  0,
 			},
 			err: "timeout must be a positive integer",