VOL-3344 - Enabled default linters and fixed found issues.

Also disabled govet's variable shadowing check, since this is very useful for isolating multiple "err"ors in a row.

Change-Id: Icd70c723c406c5986b8cfe4535ad7daa0752e48c
diff --git a/pkg/mocks/mockKVClient.go b/pkg/mocks/mockKVClient.go
index 78cbef6..e45b30b 100644
--- a/pkg/mocks/mockKVClient.go
+++ b/pkg/mocks/mockKVClient.go
@@ -302,10 +302,7 @@
 func (kvclient *MockKVClient) IsConnectionUp(ctx context.Context) bool {
 	// timeout in second
 	t, _ := ctx.Deadline()
-	if t.Second()-time.Now().Second() < 1 {
-		return false
-	}
-	return true
+	return t.Second()-time.Now().Second() >= 1
 }
 
 // CloseWatch mock function implementation for KVClient