crud: replace usage of optional types in crud with go-option

fixed #492

Author: New year

crud/compile_test.go

@@ -0,0 +1,86 @@
+// crud/compile_test.go
+package crud
+
+import (
+	"testing"
+	"github.com/tarantool/go-option"
+)
+
+// TestOptionTypesCompilation проверяет что все типы option компилируются правильно
+func TestOptionTypesCompilation(t *testing.T) {
+	// Test BaseOpts
+	baseOpts := BaseOpts{
+		Timeout:      option.SomeFloat64(1.5),
+		VshardRouter: option.SomeString("router"),
+	}
+	
+	// Проверяем что Get() работает
+	if timeout, exists := baseOpts.Timeout.Get(); !exists || timeout != 1.5 {
+		t.Errorf("BaseOpts.Timeout.Get() failed")
+	}
+
+	// Test SimpleOperationOpts
+	simpleOpts := SimpleOperationOpts{
+		Timeout:             option.SomeFloat64(2.0),
+		VshardRouter:        option.SomeString("router2"),
+		Fields:              MakeOptAny([]interface{}{"field1", "field2"}),
+		BucketId:            option.SomeUint(456),  // Теперь это правильный тип
+		FetchLatestMetadata: option.SomeBool(true),
+		Noreturn:            option.SomeBool(false),
+	}
+
+	// Проверяем все поля
+	if bucket, exists := simpleOpts.BucketId.Get(); !exists || bucket != 456 {
+		t.Errorf("BucketId.Get() failed: got %v, %v", bucket, exists)
+	}
+	
+	if fields, exists := simpleOpts.Fields.Get(); !exists {
+		t.Errorf("Fields.Get() failed")
+	} else {
+		t.Logf("Fields: %v", fields)
+	}
+
+	// Test OperationManyOpts  
+	manyOpts := OperationManyOpts{
+		Timeout:     option.SomeFloat64(3.0),
+		StopOnError: option.SomeBool(true),
+	}
+
+	if stop, exists := manyOpts.StopOnError.Get(); !exists || !stop {
+		t.Errorf("StopOnError.Get() failed")
+	}
+}
+
+// TestMakeOptAny проверяет работу MakeOptAny (замена MakeOptTuple)
+func TestMakeOptAny(t *testing.T) {
+	// Test с различными типами данных
+	testCases := []struct {
+		name  string
+		value interface{}
+	}{
+		{"slice", []interface{}{"id", "name"}},
+		{"string", "test"},
+		{"number", 42},
+		{"nil", nil},
+	}
+	
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			opt := MakeOptAny(tc.value)
+			val, exists := opt.Get()
+			
+			if tc.value == nil {
+				if exists {
+					t.Errorf("Expected no value for nil input")
+				}
+			} else {
+				if !exists {
+					t.Errorf("Expected value for %v", tc.value)
+				}
+				if val != tc.value {
+					t.Errorf("Expected %v, got %v", tc.value, val)
+				}
+			}
+		})
+	}
+}

crud/options.go

@@ -183,7 +183,7 @@ type SimpleOperationOpts struct {
 	// Fields is field names for getting only a subset of fields.
 	Fields OptAny
 	// BucketId is a bucket ID.
-	BucketId option.Uint64
+	BucketId option.Uint
 	// FetchLatestMetadata guarantees the up-to-date metadata (space format)
 	// in first return value, otherwise it may not take into account
 	// the latest migration of the data format. Performance overhead is up to 15%.
@@ -225,7 +225,7 @@ type SimpleOperationObjectOpts struct {
 	// Fields is field names for getting only a subset of fields.
 	Fields OptAny
 	// BucketId is a bucket ID.
-	BucketId option.Uint64
+	BucketId option.Uint
 	// SkipNullabilityCheckOnFlatten is a parameter to allow
 	// setting null values to non-nullable fields.
 	SkipNullabilityCheckOnFlatten option.Bool