From c691004d83d88613a4f86ad0875f18cb2885dc3b Mon Sep 17 00:00:00 2001 From: EuAndreh Date: Sun, 13 Oct 2024 06:30:23 -0300 Subject: src/gobang.go: ValidSQLTablePrefix() -> ValidateSQLTablePrefix() --- src/gobang.go | 8 ++++++-- tests/gobang.go | 13 ++++++++----- 2 files changed, 14 insertions(+), 7 deletions(-) diff --git a/src/gobang.go b/src/gobang.go index 620dc1a..dd638f2 100644 --- a/src/gobang.go +++ b/src/gobang.go @@ -63,8 +63,12 @@ var ( var _SQLTablePrefixRE = regexp.MustCompilePOSIX("^[a-zA-Z][_a-zA-z0-9]*$") -func ValidSQLTablePrefix(prefix string) bool { - return _SQLTablePrefixRE.MatchString(prefix) +func ValidateSQLTablePrefix(prefix string) error { + if !_SQLTablePrefixRE.MatchString(prefix) { + return ErrBadSQLTablePrefix + } + + return nil } func SomeError(errs ...error) error { diff --git a/tests/gobang.go b/tests/gobang.go index c7d1549..be33eb4 100644 --- a/tests/gobang.go +++ b/tests/gobang.go @@ -13,8 +13,8 @@ import ( -func test_ValidSQLTablePrefix() { - TestStart("ValidSQLTablePrefix()") +func test_ValidateSQLTablePrefix() { + TestStart("ValidateSQLTablePrefix()") Testing("simple identifiers are good", func() { input := []string{ @@ -25,7 +25,7 @@ func test_ValidSQLTablePrefix() { } for _, given := range input { - TAssertEqual(ValidSQLTablePrefix(given), true) + TAssertEqual(ValidateSQLTablePrefix(given), nil) } }) @@ -40,7 +40,10 @@ func test_ValidSQLTablePrefix() { } for _, given := range input { - TAssertEqual(ValidSQLTablePrefix(given), false) + TAssertEqual( + ValidateSQLTablePrefix(given), + ErrBadSQLTablePrefix, + ) } }) } @@ -761,7 +764,7 @@ func test_setHostname() { func MainTest() { - test_ValidSQLTablePrefix() + test_ValidateSQLTablePrefix() test_SomeError() test_SomeFnError() test_Random() -- cgit v1.2.3