linter rule for allow deprecated ydb.Connection in tests only

persistend allow ydb.Connection for existed common used public functions

Author: rekby

.golangci.yml

@@ -201,7 +201,6 @@ issues:
   # Default: []
   exclude:
     - "has been deprecated since Go 1.16"
-    - "ydb.Connection is deprecated"
 
   # Independently from option `exclude` we use default exclude patterns,
   # it can be disabled by this option. To list all
@@ -236,3 +235,5 @@ issues:
       linters:
         - unused
         - unparam
+    - path: _test\.go
+      text: "ydb.Connection is deprecated"

sugar/path.go

@@ -23,7 +23,10 @@ const (
 // pathToCreate is a database root relative path
 // MakeRecursive method equal bash command `mkdir -p ~/path/to/create`
 // where `~` - is a root of database
-func MakeRecursive(ctx context.Context, db ydb.Connection, pathToCreate string) error {
+//
+// Use deprecated ydb.Connection as param for backward compatibility
+// Usually use *ydb.Driver, that implement ydb.Connection
+func MakeRecursive(ctx context.Context, db ydb.Connection, pathToCreate string) error { //nolint:staticcheck
 	pathToCreate = path.Join(db.Name(), pathToCreate)
 	for i := len(db.Name()) + 1; i < len(pathToCreate); i++ {
 		x := strings.IndexByte(pathToCreate[i:], '/')
@@ -78,7 +81,10 @@ func MakeRecursive(ctx context.Context, db ydb.Connection, pathToCreate string)
 // Empty prefix means than use root of database.
 // RemoveRecursive method equal bash command `rm -rf ~/path/to/remove`
 // where `~` - is a root of database
-func RemoveRecursive(ctx context.Context, db ydb.Connection, pathToRemove string) error {
+//
+// Use deprecated ydb.Connection as param for backward compatibility
+// Usually use *ydb.Driver, that implement ydb.Connection
+func RemoveRecursive(ctx context.Context, db ydb.Connection, pathToRemove string) error { //nolint:staticcheck
 	fullSysTablePath := path.Join(db.Name(), sysTable)
 	var list func(int, string) error
 	list = func(i int, p string) error {