From f8f36c672776bb00b2d53c5d49e92b1bfb608516 Mon Sep 17 00:00:00 2001 From: Wenjia Date: Thu, 21 Jan 2021 11:27:57 -0800 Subject: [PATCH] Remove unused expectedRV in store_test TestGet expectedRV is defined in tests struct but not set in test cases, removing the unnecessary checking --- .../src/k8s.io/apiserver/pkg/storage/etcd3/store_test.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/staging/src/k8s.io/apiserver/pkg/storage/etcd3/store_test.go b/staging/src/k8s.io/apiserver/pkg/storage/etcd3/store_test.go index 961029b8c7c..65b354d4f39 100644 --- a/staging/src/k8s.io/apiserver/pkg/storage/etcd3/store_test.go +++ b/staging/src/k8s.io/apiserver/pkg/storage/etcd3/store_test.go @@ -226,7 +226,6 @@ func TestGet(t *testing.T) { expectRVTooLarge bool expectedOut *example.Pod rv string - expectedRV string }{{ // test get on existing item name: "get existing", key: key, @@ -287,11 +286,6 @@ func TestGet(t *testing.T) { } return } - if tt.expectedRV != "" { - if tt.expectedRV != out.ResourceVersion { - t.Errorf("expecting resource version want=%s, got=%s", tt.expectedRV, out.ResourceVersion) - } - } if err != nil { t.Fatalf("Get failed: %v", err) }