Refactor error variable names and update associated tests in metricsf…

…orwarder config

src/autoscaler/metricsforwarder/config/config.go

@@ -18,14 +18,13 @@ import (
 )
 
 // There are 3 type of errors that this package can return:
-// - ReadYamlError
-// - ReadFileError
-// - ReadEnvironmentError
+// - ErrReadYaml
+// - ErrReadEnvironment
+// - ErrReadVCAPEnvironment
 
-var ReadYamlError = errors.New("failed to read config file")
-var ReadFileError = errors.New("failed to open config file")
-var ReadEnvironmentError = errors.New("failed to read environment variables")
-var ReadVCAPEnvironmentError = errors.New("failed to read VCAP environment variables")
+var ErrReadYaml = errors.New("failed to read config file")
+var ErrReadEnvironment = errors.New("failed to read environment variables")
+var ErrReadVCAPEnvironment = errors.New("failed to read VCAP environment variables")
 
 const (
 	DefaultMetronAddress        = "127.0.0.1:3458"
@@ -116,20 +115,20 @@ func LoadConfig(filepath string) (*Config, error) {
 		err = dec.Decode(&conf)
 
 		if err != nil {
-			return nil, fmt.Errorf("%w: %w", ReadYamlError, err)
+			return nil, fmt.Errorf("%w: %w", ErrReadYaml, err)
 		}
 
 		defer r.Close()
 	}
 
 	err = envconfig.Process("", &conf)
 	if err != nil {
-		return nil, fmt.Errorf("%w: %w", ReadEnvironmentError, err)
+		return nil, fmt.Errorf("%w: %w", ErrReadEnvironment, err)
 	}
 
 	err = loadVCAPEnvs(&conf)
 	if err != nil {
-		return nil, fmt.Errorf("%w: %w", ReadVCAPEnvironmentError, err)
+		return nil, fmt.Errorf("%w: %w", ErrReadVCAPEnvironment, err)
 	}
 
 	return &conf, nil

src/autoscaler/metricsforwarder/config/config_test.go

@@ -119,7 +119,7 @@ cred_helper_impl: default
 					})
 
 					It("return invalid port error", func() {
-						Expect(err).To(MatchError(config.ReadEnvironmentError))
+						Expect(err).To(MatchError(config.ErrReadEnvironment))
 						Expect(err).To(MatchError(MatchRegexp("converting 'NAN' to type int")))
 					})
 
@@ -206,7 +206,7 @@ server:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal.*into int")))
 			})
 		})
@@ -220,7 +220,7 @@ health:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal.*into int")))
 			})
 		})
@@ -246,7 +246,7 @@ health:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal.*into int")))
 			})
 		})
@@ -272,7 +272,7 @@ health:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal.*into int")))
 			})
 		})
@@ -298,7 +298,7 @@ health:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal .* into time.Duration")))
 			})
 		})
@@ -327,7 +327,7 @@ rate_limit:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal .* into int")))
 			})
 		})
@@ -356,7 +356,7 @@ rate_limit:
 			})
 
 			It("should error", func() {
-				Expect(err).To(MatchError(config.ReadYamlError))
+				Expect(err).To(MatchError(config.ErrReadYaml))
 				Expect(err).To(MatchError(MatchRegexp("cannot unmarshal .* into time.Duration")))
 			})
 		})