Merge pull request #186 from ydb-platform/ratelimiter-error-check

* Fixed bug with check acquire error over `ydb.IsRatelimiterAcquireEr…

Author: asmyasnikov

CHANGELOG.md

@@ -1,3 +1,4 @@
+* Fixed bug with check acquire error over `ydb.IsRatelimiterAcquireError`
 * Added full changelog link to github release description 
 
 ## v3.16.11

internal/ratelimiter/errors/errors.go

@@ -32,7 +32,7 @@ func NewAcquire(amoount uint64, err error) ratelimiter.AcquireError {
 }
 
 func IsAcquireError(err error) bool {
-	ae := acquireError{}
+	var ae *acquireError
 	return errors.As(err, &ae)
 }
 

internal/ratelimiter/errors/errors_test.go

@@ -0,0 +1,37 @@
+package errors
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/ydb-platform/ydb-go-sdk/v3/internal/errors"
+)
+
+func TestIsAcquireError(t *testing.T) {
+	for _, err := range []error{
+		&acquireError{},
+		errors.WithStackTrace(&acquireError{}),
+		errors.WithStackTrace(fmt.Errorf("%w", &acquireError{})),
+	} {
+		t.Run("", func(t *testing.T) {
+			if !IsAcquireError(err) {
+				t.Errorf("not acquire error: %v", err)
+			}
+		})
+	}
+}
+
+func TestToAcquireError(t *testing.T) {
+	for _, err := range []error{
+		&acquireError{},
+		errors.WithStackTrace(&acquireError{}),
+		errors.WithStackTrace(fmt.Errorf("%w", &acquireError{})),
+	} {
+		t.Run("", func(t *testing.T) {
+			ae := ToAcquireError(err)
+			if ae == nil {
+				t.Errorf("not acquire error: %v", err)
+			}
+		})
+	}
+}