const: add 'concurrent schema update' error constant

Closes #404, TNTP-3336

Author: bigbes

CHANGELOG.md

@@ -12,6 +12,8 @@ Versioning](http://semver.org/spec/v2.0.0.html) except to the first release.
 
 - Implemented all box.schema.user operations requests and sugar interface (#426).
 - Implemented box.session.su request and sugar interface only for current session granting (#426).
+- Defined `ErrConcurrentSchemaUpdate` constant for "concurrent schema update" error.
+  Now you can check this error with `errors.Is(err, tarantool.ErrConcurrentSchemaUpdate)`.
 
 ### Changed
 

schema.go

@@ -19,6 +19,10 @@ const (
 	vspaceSpFormatFieldNum = 7
 )
 
+var (
+	ErrConcurrentSchemaUpdate = errors.New("concurrent schema update")
+)
+
 func msgpackIsUint(code byte) bool {
 	return code == msgpcode.Uint8 || code == msgpcode.Uint16 ||
 		code == msgpcode.Uint32 || code == msgpcode.Uint64 ||
@@ -415,7 +419,7 @@ func GetSchema(doer Doer) (Schema, error) {
 			schema.SpacesById[spaceId].IndexesById[index.Id] = index
 			schema.SpacesById[spaceId].Indexes[index.Name] = index
 		} else {
-			return Schema{}, errors.New("concurrent schema update")
+			return Schema{}, ErrConcurrentSchemaUpdate
 		}
 	}
 

schema_test.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"github.com/vmihailenco/msgpack/v5"
 
@@ -160,3 +161,7 @@ func TestResolverNotCalledWithNameSupport(t *testing.T) {
 			resolver.indexResolverCalls)
 	}
 }
+
+func TestErrConcurrentSchemaUpdate(t *testing.T) {
+	assert.EqualError(t, tarantool.ErrConcurrentSchemaUpdate, "concurrent schema update")
+}

tarantool_test.go

@@ -3999,14 +3999,13 @@ func TestConnect_schema_update(t *testing.T) {
 	for i := 0; i < 100; i++ {
 		fut := conn.Do(NewCallRequest("create_spaces"))
 
-		if conn, err := Connect(ctx, dialer, opts); err != nil {
-			if err.Error() != "concurrent schema update" {
-				t.Errorf("unexpected error: %s", err)
-			}
-		} else if conn == nil {
-			t.Errorf("conn is nil")
-		} else {
-			conn.Close()
+		switch conn, err := Connect(ctx, dialer, opts); {
+		case err != nil:
+			assert.ErrorIs(t, err, ErrConcurrentSchemaUpdate)
+		case conn == nil:
+			assert.Fail(t, "conn is nil")
+		default:
+			_ = conn.Close()
 		}
 
 		if _, err := fut.Get(); err != nil {