diff --git a/go/vt/tableacl/tableacl.go b/go/vt/tableacl/tableacl.go index dd6638e9786..2bb48ca28f2 100644 --- a/go/vt/tableacl/tableacl.go +++ b/go/vt/tableacl/tableacl.go @@ -111,7 +111,7 @@ func (tacl *tableACL) init(configFile string, aclCB func()) error { return err } if len(data) == 0 { - return error.New("tableACL config file is empty") + return errors.New("tableACL config file is empty") } config := &tableaclpb.Config{} diff --git a/go/vt/tableacl/tableacl_test.go b/go/vt/tableacl/tableacl_test.go index 388567b62e2..1f5e89a6a48 100644 --- a/go/vt/tableacl/tableacl_test.go +++ b/go/vt/tableacl/tableacl_test.go @@ -74,6 +74,21 @@ func TestInitWithValidConfig(t *testing.T) { } } +func TestInitWithEmptyConfig(t *testing.T) { + tacl := tableACL{factory: &simpleacl.Factory{}} + f, err := os.CreateTemp("", "tableacl") + if err != nil { + t.Fatal(err) + } + defer os.Remove(f.Name()) + if err := f.Close(); err != nil { + t.Fatal(err) + } + if err := tacl.init(f.Name(), func() {}); err == nil { + t.Fatal("tableACL config file is empty") + } +} + func TestInitFromProto(t *testing.T) { tacl := tableACL{factory: &simpleacl.Factory{}} readerACL := tacl.Authorized("my_test_table", READER)