Recommended changes from PR review

- MinLen -> MinLength
- regex -> strings.TrimSpace

Author: dopey

ui/options.go

@@ -145,8 +145,8 @@ func WithValidateNotEmpty() Option {
 
 // WithValidateMinLen adds a custom validation function to a prompt that
 // checks the input string meets the minimum length requirement.
-func WithValidateMinLen(length int) Option {
-	return WithValidateFunc(MinLen(length))
+func WithValidateMinLength(minLength int) Option {
+	return WithValidateFunc(MinLength(minLength))
 }
 
 // WithValidateYesNo adds a custom validation function to a prompt for a Yes/No

ui/validators.go

@@ -4,7 +4,6 @@ import (
 	"errors"
 	"fmt"
 	"net"
-	"regexp"
 	"strings"
 
 	"github.com/manifoldco/promptui"
@@ -75,17 +74,15 @@ func YesNo() promptui.ValidateFunc {
 	}
 }
 
-// MinLen is a validation function that checks for a minimum length.
+// MinLength is a validation function that checks for a minimum length.
 // An input length <= 0 indicates that the check should not be performed.
-func MinLen(length int) promptui.ValidateFunc {
+func MinLength(minLength int) promptui.ValidateFunc {
 	return func(s string) error {
-		if length <= 0 {
+		if minLength <= 0 {
 			return nil
 		}
-		re := regexp.MustCompile(`\s+`)
-		next := re.ReplaceAllString(s, "")
-		if len(next) < length {
-			return fmt.Errorf("input does not meet minimum length requirement; must be at least %v characters", length)
+		if len(strings.TrimSpace(s)) < minLength {
+			return fmt.Errorf("input does not meet minimum length requirement; must be at least %v characters", minLength)
 		}
 		return nil
 	}

ui/validators_test.go

@@ -91,7 +91,7 @@ func TestDNS(t *testing.T) {
 	}
 }
 
-func TestMinLen(t *testing.T) {
+func TestMinLength(t *testing.T) {
 	tests := []struct {
 		name    string
 		length  int
@@ -129,21 +129,21 @@ func TestMinLen(t *testing.T) {
 			wantErr: true,
 		},
 		{
-			name:    "ignore-whitespace-characters",
-			length:  15,
-			input:   "  p \t\n#$%@#$ a s s  ",
+			name:    "ignore-pre-post-whitespace-characters",
+			length:  5,
+			input:   "  pass  ",
 			wantErr: true,
 		},
 		{
 			name:    "ignore-whitespace-characters-ok",
-			length:  10,
-			input:   "  p \t\n#$%@#$ a s s  ",
+			length:  4,
+			input:   "  pass  ",
 			wantErr: false,
 		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			gotErr := MinLen(tt.length)(tt.input) != nil
+			gotErr := MinLength(tt.length)(tt.input) != nil
 			if gotErr != tt.wantErr {
 				t.Errorf("MinLen(%v)(%s) = %v, want %v", tt.length, tt.input, gotErr, tt.wantErr)
 			}