crud: fix Get options

Before this patch, `vshard_router`, `fields`, `bucket_id`, `mode`,
`prefer_replica`, `balance` were either ignored or had a wrong name.
This patch fixes the issue.

Author: DifferentialOrange

CHANGELOG.md

@@ -55,6 +55,8 @@ Versioning](http://semver.org/spec/v2.0.0.html) except to the first release.
 - Incorrect decoding of an MP_DECIMAL when the `scale` value is negative (#314)
 - Incorrect options (`after`, `batch_size` and `force_map_call`) setup for
   crud.SelectRequest (#320)
+- Incorrect options (`vshard_router`, `fields`, `bucket_id`, `mode`,
+  `prefer_replica`, `balance`) setup for crud.GetRequest (#335)
 
 ## [1.12.0] - 2023-06-07
 

crud/get.go

@@ -42,10 +42,11 @@ func (opts GetOpts) EncodeMsgpack(enc *msgpack.Encoder) error {
 	exists := [optsCnt]bool{}
 	values[0], exists[0] = opts.Timeout.Get()
 	values[1], exists[1] = opts.VshardRouter.Get()
-	values[1], exists[1] = opts.BucketId.Get()
-	values[2], exists[2] = opts.Mode.Get()
-	values[3], exists[3] = opts.PreferReplica.Get()
-	values[4], exists[4] = opts.Balance.Get()
+	values[2], exists[2] = opts.Fields.Get()
+	values[3], exists[3] = opts.BucketId.Get()
+	values[4], exists[4] = opts.Mode.Get()
+	values[5], exists[5] = opts.PreferReplica.Get()
+	values[6], exists[6] = opts.Balance.Get()
 
 	return encodeOptions(enc, names[:], values[:], exists[:])
 }

crud/tarantool_test.go

@@ -808,6 +808,27 @@ func TestStorageInfoResult(t *testing.T) {
 	}
 }
 
+func TestGetAdditionalOpts(t *testing.T) {
+	conn := connect(t)
+	defer conn.Close()
+
+	req := crud.MakeGetRequest(spaceName).Key(key).Opts(crud.GetOpts{
+		Timeout:       crud.MakeOptUint(1),
+		Fields:        crud.MakeOptTuple([]interface{}{"name"}),
+		Mode:          crud.MakeOptString("read"),
+		PreferReplica: crud.MakeOptBool(true),
+		Balance:       crud.MakeOptBool(true),
+	})
+	resp := crud.Result{}
+
+	testCrudRequestPrepareData(t, conn)
+
+	err := conn.Do(req).GetTyped(&resp)
+	if err != nil {
+		t.Fatalf("Failed to Do CRUD request: %s", err)
+	}
+}
+
 // runTestMain is a body of TestMain function
 // (see https://pkg.go.dev/testing#hdr-Main).
 // Using defer + os.Exit is not works so TestMain body