Skip to content

Commit d85f0ed

Browse files
committed
chore: fix ut
fix golint
1 parent 739696b commit d85f0ed

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

pkg/blob/nodeserver.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -552,9 +552,9 @@ func (d *Driver) NodeGetVolumeStats(_ context.Context, req *csi.NodeGetVolumeSta
552552
return nil, status.Errorf(codes.Internal, "%v", err)
553553
}
554554
if cache != nil {
555-
resp := cache.(csi.NodeGetVolumeStatsResponse)
555+
resp := cache.(*csi.NodeGetVolumeStatsResponse)
556556
klog.V(6).Infof("NodeGetVolumeStats: volume stats for volume %s path %s is cached", req.VolumeId, req.VolumePath)
557-
return &resp, nil
557+
return resp, nil
558558
}
559559

560560
mc := metrics.NewMetricContext(blobCSIDriverName, "node_get_volume_stats", d.cloud.ResourceGroup, "", d.Name)
@@ -624,7 +624,7 @@ func (d *Driver) NodeGetVolumeStats(_ context.Context, req *csi.NodeGetVolumeSta
624624
isOperationSucceeded = true
625625
klog.V(6).Infof("NodeGetVolumeStats: volume stats for volume %s path %s is %v", req.VolumeId, req.VolumePath, resp)
626626
// cache the volume stats per volume
627-
d.volStatsCache.Set(req.VolumeId, *resp)
627+
d.volStatsCache.Set(req.VolumeId, resp)
628628
return resp, nil
629629
}
630630

pkg/blob/nodeserver_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ func TestNodePublishVolume(t *testing.T) {
248248
Exec: fakeExec,
249249
}
250250

251-
for _, test := range tests {
251+
for _, test := range tests { //nolint:govet
252252
d.cloud.ResourceGroup = "rg"
253253
if test.setup != nil {
254254
test.setup(d)
@@ -356,7 +356,7 @@ func TestNodeUnpublishVolume(t *testing.T) {
356356
Exec: fakeExec,
357357
}
358358

359-
for _, test := range tests {
359+
for _, test := range tests { //nolint:govet
360360
if test.setup != nil {
361361
test.setup(d)
362362
}
@@ -735,7 +735,7 @@ func TestNodeGetVolumeStats(t *testing.T) {
735735
_ = makeDir(fakePath)
736736
d := NewFakeDriver()
737737

738-
for _, test := range tests {
738+
for _, test := range tests { //nolint:govet
739739
_, err := d.NodeGetVolumeStats(context.Background(), &test.req)
740740
//t.Errorf("[debug] error: %v\n metrics: %v", err, metrics)
741741
if !reflect.DeepEqual(err, test.expectedErr) {

0 commit comments

Comments
 (0)