From e9d16c2b627cc64cd172bac855909cb2a968c3a3 Mon Sep 17 00:00:00 2001 From: Ed Bartosh Date: Tue, 21 Jul 2020 05:18:52 +0300 Subject: [PATCH] etcdserverpb: add TestInvalidGoTypeIntPanic test (#12116) Tested conditions that cause panic: invalid Go type int for field k8s_io.kubernetes.vendor.go_etcd_io.etcd.etcdserver.etcdserverpb.loggablePutRequest.value_size Signed-off-by: Ed Bartosh --- .../raft_internal_stringer_test.go | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 etcdserver/etcdserverpb/raft_internal_stringer_test.go diff --git a/etcdserver/etcdserverpb/raft_internal_stringer_test.go b/etcdserver/etcdserverpb/raft_internal_stringer_test.go new file mode 100644 index 000000000..49e38e77c --- /dev/null +++ b/etcdserver/etcdserverpb/raft_internal_stringer_test.go @@ -0,0 +1,31 @@ +// Copyright 2020 The etcd Authors +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package etcdserverpb_test + +import ( + "testing" + + pb "go.etcd.io/etcd/v3/etcdserver/etcdserverpb" +) + +// TestInvalidGoYypeIntPanic tests conditions that caused +// panic: invalid Go type int for field k8s_io.kubernetes.vendor.go_etcd_io.etcd.etcdserver.etcdserverpb.loggablePutRequest.value_size +// See https://github.com/kubernetes/kubernetes/issues/91937 for more details +func TestInvalidGoTypeIntPanic(t *testing.T) { + result := pb.NewLoggablePutRequest(&pb.PutRequest{}).String() + if result != "" { + t.Errorf("Got result: %s, expected empty string", result) + } +}