Skip to content

Commit bc750c3

Browse files
committed
kvserver: use dd helpers in TestFlowControlRaftTransportV2
Epic: none Release note: none
1 parent b826146 commit bc750c3

File tree

1 file changed

+7
-17
lines changed

1 file changed

+7
-17
lines changed

pkg/kv/kvserver/flow_control_raft_transport_test.go

Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import (
2525
"github.com/cockroachdb/cockroach/pkg/settings/cluster"
2626
"github.com/cockroachdb/cockroach/pkg/testutils"
2727
"github.com/cockroachdb/cockroach/pkg/testutils/datapathutils"
28+
"github.com/cockroachdb/cockroach/pkg/testutils/dd"
2829
"github.com/cockroachdb/cockroach/pkg/util"
2930
"github.com/cockroachdb/cockroach/pkg/util/leaktest"
3031
"github.com/cockroachdb/cockroach/pkg/util/log"
@@ -45,24 +46,21 @@ type transportControl struct {
4546
}
4647

4748
func parseNodeID(t *testing.T, d *datadriven.TestData, key string) roachpb.NodeID {
48-
var arg string
49-
d.ScanArgs(t, key, &arg)
49+
arg := dd.ScanArg[string](t, d, key)
5050
ni, err := strconv.Atoi(strings.TrimPrefix(arg, "n"))
5151
require.NoError(t, err)
5252
return roachpb.NodeID(ni)
5353
}
5454

5555
func parseStoreID(t *testing.T, d *datadriven.TestData, key string) roachpb.StoreID {
56-
var arg string
57-
d.ScanArgs(t, key, &arg)
56+
arg := dd.ScanArg[string](t, d, key)
5857
si, err := strconv.Atoi(strings.TrimPrefix(arg, "s"))
5958
require.NoError(t, err)
6059
return roachpb.StoreID(si)
6160
}
6261

6362
func parseRangeID(t *testing.T, d *datadriven.TestData, key string) roachpb.RangeID {
64-
var arg string
65-
d.ScanArgs(t, key, &arg)
63+
arg := dd.ScanArg[string](t, d, key)
6664
si, err := strconv.Atoi(strings.TrimPrefix(arg, "r"))
6765
require.NoError(t, err)
6866
return roachpb.RangeID(si)
@@ -71,9 +69,8 @@ func parseRangeID(t *testing.T, d *datadriven.TestData, key string) roachpb.Rang
7169
func parseReplicaDescriptor(
7270
t *testing.T, d *datadriven.TestData, key string,
7371
) roachpb.ReplicaDescriptor {
74-
var arg string
72+
arg := dd.ScanArg[string](t, d, key)
7573
var desc roachpb.ReplicaDescriptor
76-
d.ScanArgs(t, key, &arg)
7774
parts := strings.Split(arg, "/")
7875
require.Len(t, parts, 3)
7976
ni, err := strconv.Atoi(strings.TrimPrefix(parts[0], "n"))
@@ -200,19 +197,12 @@ func TestFlowControlRaftTransportV2(t *testing.T) {
200197

201198
case "send":
202199
// Parse range=r<int>.
203-
var arg string
204-
d.ScanArgs(t, "range", &arg)
205-
ri, err := strconv.Atoi(strings.TrimPrefix(arg, "r"))
206-
require.NoError(t, err)
207-
rangeID := roachpb.RangeID(ri)
208-
200+
rangeID := parseRangeID(t, d, "range")
209201
from := parseReplicaDescriptor(t, d, "from") // parse from=n<int>/s<int>/<int>
210202
to := parseReplicaDescriptor(t, d, "to") // parse to=n<int>/s<int>/<int>
211203

212204
// Parse commit=<int>.
213-
d.ScanArgs(t, "commit", &arg)
214-
c, err := strconv.Atoi(arg)
215-
require.NoError(t, err)
205+
c := dd.ScanArg[int](t, d, "commit")
216206

217207
testutils.SucceedsSoon(t, func() error {
218208
if !rttc.Send(from, to, rangeID, raftpb.Message{Commit: uint64(c)}) {

0 commit comments

Comments
 (0)