feat(publicip): better error messages stating the provider type if unknown

This commit is contained in:
Quentin McGaw
2024-02-04 14:37:21 +00:00
parent 7aa14c8737
commit 133956f082
7 changed files with 7 additions and 7 deletions

View File

@@ -53,7 +53,7 @@ func Test_SetProviders(t *testing.T) {
expectedSettings: settings{
providers: []Provider{Cloudflare},
},
err: errors.New("unknown provider: invalid"),
err: errors.New("unknown public IP echo DNS provider: invalid"),
},
}

View File

@@ -22,7 +22,7 @@ func ListProviders() []Provider {
}
}
var ErrUnknownProvider = errors.New("unknown provider")
var ErrUnknownProvider = errors.New("unknown public IP echo DNS provider")
func ValidateProvider(provider Provider) error {
for _, possible := range ListProviders() {

View File

@@ -22,7 +22,7 @@ func Test_ValidateProvider(t *testing.T) {
},
"invalid provider": {
provider: Provider("invalid"),
err: errors.New("unknown provider: invalid"),
err: errors.New("unknown public IP echo DNS provider: invalid"),
},
}

View File

@@ -66,7 +66,7 @@ func Test_New(t *testing.T) {
SetProvidersIP(Provider("invalid")),
},
err: ErrUnknownProvider,
errMessage: "unknown provider: invalid",
errMessage: "unknown public IP echo HTTP provider: invalid",
},
}

View File

@@ -42,7 +42,7 @@ func ListProvidersForVersion(version ipversion.IPVersion) (providers []Provider)
}
var (
ErrUnknownProvider = errors.New("unknown provider")
ErrUnknownProvider = errors.New("unknown public IP echo HTTP provider")
ErrProviderIPVersion = errors.New("provider does not support IP version")
)

View File

@@ -65,7 +65,7 @@ func Test_ValidateProvider(t *testing.T) {
"unknown": {
provider: Provider("unknown"),
version: ipversion.IP4,
err: errors.New("unknown provider: unknown"),
err: errors.New("unknown public IP echo HTTP provider: unknown"),
},
}

View File

@@ -20,7 +20,7 @@ func ListProviders() []Provider {
}
}
var ErrUnknownProvider = errors.New("unknown provider")
var ErrUnknownProvider = errors.New("unknown public IP information provider")
func ValidateProvider(provider Provider) error {
for _, possible := range ListProviders() {