fix: use localhost for test and add docstrings to errors

Author: samphillips-f3

.gitignore

@@ -7,6 +7,7 @@
 
 # Test binary, built with `go test -c`
 *.test
+tools/
 
 # Output of the go coverage tool, specifically when used with LiteIDE
 *.out

config/error.go

@@ -1,5 +1,6 @@
 package config
 
+// ValidationError describes an error validating the provided config.
 type ValidationError struct {
 	message string
 }

config/loader_test.go

@@ -105,7 +105,7 @@ func TestLoadConfig(t *testing.T) {
 
 	expectedCfg := Config{
 		Proxy: ProxyConfig{
-			UpstreamTarget: "https://princesscarolyn.net",
+			UpstreamTarget: "http://localhost",
 			Signer: SignerConfig{
 				KeyId:             "6f33b219-137c-467e-9a61-f61040a03363",
 				KeyFilePath:       "/etc/form3/private/private.key",
@@ -138,7 +138,7 @@ func TestLoadConfig(t *testing.T) {
 
 	configFile := "config_test.yaml"
 	overrides := []string{
-		"proxy.upstreamTarget=https://princesscarolyn.net",
+		"proxy.upstreamTarget=http://localhost",
 		"log.level=debug",
 	}
 	_ = os.Setenv("SERVER_PORT", "9090")

proxy/error.go

@@ -2,6 +2,7 @@ package proxy
 
 import "fmt"
 
+// InvalidRequestError is raised when the contents of the request cause signing to fail.
 type InvalidRequestError struct {
 	reason error
 }

signer/signer_test.go

@@ -68,7 +68,7 @@ func TestLoadKey(t *testing.T) {
 }
 
 func TestGetSignatureHeaders(t *testing.T) {
-	dummyUrl := "https://oompaloompa.localhost:1234"
+	dummyUrl := "https://localhost:1234"
 	dummyBody := "{\"name\":\"travis\"}"
 
 	tests := []struct {