test(allowed_hosts_validator_test.go): add test case

add test case
microsoft · Jan 18, 2024 · a32f7e5 · a32f7e5
1 parent 47d9018
commit a32f7e5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/authentication/allowed_hosts_validator.go b/authentication/allowed_hosts_validator.go
@@ -25,7 +25,7 @@ func NewAllowedHostsValidator(validHosts []string) AllowedHostsValidator {
 func NewAllowedHostsValidatorErrorCheck(validHosts []string) (*AllowedHostsValidator, error) {
 	result := &AllowedHostsValidator{}
 	if err := result.SetAllowedHostsErrorCheck(validHosts); err != nil {
-		return result, err
+		return nil, err
 	}
 	return result, nil
 }

diff --git a/authentication/allowed_hosts_validator_test.go b/authentication/allowed_hosts_validator_test.go
@@ -15,6 +15,7 @@ func TestItValidatesHostsUseNewAllowedHostsValidator(t *testing.T) {
 }
 
 func TestItValidatesHostsUseNewAllowedHostsValidatorErrorCheck(t *testing.T) {
-	_, err := NewAllowedHostsValidatorErrorCheck([]string{"http://graph.microsoft.com"})
+	validator, err := NewAllowedHostsValidatorErrorCheck([]string{"http://graph.microsoft.com"})
 	assert.EqualValues(t, ErrInvalidHostPrefix, err)
+	assert.Nil(t, validator)
 }