github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-08-04 01:40:07 +00:00
Code Issues Projects Releases Wiki Activity

fix non-nil ptr struct convert

Browse Source 
Signed-off-by: wrfly <mr.wrfly@gmail.com>
This commit is contained in:
wrfly 2018-03-03 15:01:02 +08:00
parent f9f5677b3e
commit 03792059cb
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
staging/src/k8s.io/apimachinery/pkg/conversion/queryparams/convert.go
View File

@ -174,6 +174,9 @@ func convertStruct(result url.Values, st reflect.Type, sv reflect.Value) {
kind = ft.Kind() kind = ft.Kind()
if !field.IsNil() { if !field.IsNil() {
field = reflect.Indirect(field) field = reflect.Indirect(field)
// If the field is non-nil, it should be added to params
// and the omitempty should be overwite to false
omitempty = false
} }
} }

6
staging/src/k8s.io/apimachinery/pkg/conversion/queryparams/convert_test.go
View File

@ -70,6 +70,7 @@ type childStructs struct {
Follow bool `json:"follow,omitempty"` Follow bool `json:"follow,omitempty"`
Previous bool `json:"previous,omitempty"` Previous bool `json:"previous,omitempty"`
SinceSeconds *int64 `json:"sinceSeconds,omitempty"` SinceSeconds *int64 `json:"sinceSeconds,omitempty"`
TailLines *int64 `json:"tailLines,omitempty"`
SinceTime *metav1.Time `json:"sinceTime,omitempty"` SinceTime *metav1.Time `json:"sinceTime,omitempty"`
EmptyTime *metav1.Time `json:"emptyTime"` EmptyTime *metav1.Time `json:"emptyTime"`
NonPointerTime metav1.Time `json:"nonPointerTime"` NonPointerTime metav1.Time `json:"nonPointerTime"`
@ -99,6 +100,7 @@ func validateResult(t *testing.T, input interface{}, actual, expected url.Values
func TestConvert(t *testing.T) { func TestConvert(t *testing.T) {
sinceSeconds := int64(123) sinceSeconds := int64(123)
tailLines := int64(0)
sinceTime := metav1.Date(2000, 1, 1, 12, 34, 56, 0, time.UTC) sinceTime := metav1.Date(2000, 1, 1, 12, 34, 56, 0, time.UTC)
tests := []struct { tests := []struct {
@ -182,6 +184,7 @@ func TestConvert(t *testing.T) {
Follow: true, Follow: true,
Previous: true, Previous: true,
SinceSeconds: &sinceSeconds, SinceSeconds: &sinceSeconds,
TailLines: nil,
SinceTime: &sinceTime, // test a custom marshaller SinceTime: &sinceTime, // test a custom marshaller
EmptyTime: nil, // test a nil custom marshaller without omitempty EmptyTime: nil, // test a nil custom marshaller without omitempty
NonPointerTime: sinceTime, NonPointerTime: sinceTime,
@ -194,10 +197,11 @@ func TestConvert(t *testing.T) {
Follow: true, Follow: true,
Previous: true, Previous: true,
SinceSeconds: &sinceSeconds, SinceSeconds: &sinceSeconds,
TailLines: &tailLines,
SinceTime: nil, // test a nil custom marshaller with omitempty SinceTime: nil, // test a nil custom marshaller with omitempty
NonPointerTime: sinceTime, NonPointerTime: sinceTime,
}, },
expected: url.Values{"container": {"mycontainer"}, "follow": {"true"}, "previous": {"true"}, "sinceSeconds": {"123"}, "emptyTime": {""}, "nonPointerTime": {"2000-01-01T12:34:56Z"}}, expected: url.Values{"container": {"mycontainer"}, "follow": {"true"}, "previous": {"true"}, "sinceSeconds": {"123"}, "tailLines": {"0"}, "emptyTime": {""}, "nonPointerTime": {"2000-01-01T12:34:56Z"}},
}, },
} }