From 179f40292f64debb6b3ea2fe5e950f62f5a05a18 Mon Sep 17 00:00:00 2001 From: Taras Madan Date: Mon, 16 Sep 2024 14:29:26 +0200 Subject: pkg/validator: target value is not empty by default The consistency make the code more predictable. We want all the values to be non-empty. If "empty" is an option - use validator.AnyOk(validator.EmptyStr(target), otherCheck(target)). --- pkg/validator/validator_test.go | 2 ++ 1 file changed, 2 insertions(+) (limited to 'pkg/validator/validator_test.go') diff --git a/pkg/validator/validator_test.go b/pkg/validator/validator_test.go index ef04d9cde..6397081e4 100644 --- a/pkg/validator/validator_test.go +++ b/pkg/validator/validator_test.go @@ -24,6 +24,7 @@ func TestIsCommitHash(t *testing.T) { validator.CommitHash("!311c1b497e51a628aa89e7cb954481e5f9dced2", "valName").Err.Error()) } +// nolint: dupl func TestIsNamespaceName(t *testing.T) { assert.True(t, validator.NamespaceName("upstream").Ok) assert.False(t, validator.NamespaceName("up").Ok) @@ -58,6 +59,7 @@ func TestIsDashboardClientKey(t *testing.T) { func TestIsKernelFilePath(t *testing.T) { assert.True(t, validator.KernelFilePath("io_uring/advise.c").Ok) assert.False(t, validator.KernelFilePath("io-uring/advise.c").Ok) + assert.False(t, validator.KernelFilePath("").Ok) assert.Equal(t, "not a kernel file path", validator.KernelFilePath("io-uring").Err.Error()) assert.Equal(t, "kernelPath: not a kernel file path", -- cgit mrf-deployment