diff options
| -rw-r--r-- | src/gobang.go | 8 | ||||
| -rw-r--r-- | 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() |