replace __internal with runtime.APIVersionInternal

This commit is contained in:
xuzhonghu 2018-05-24 16:57:09 +08:00
parent 481b2dc7cf
commit 6f76247eb6
2 changed files with 9 additions and 9 deletions

View File

@ -122,7 +122,7 @@ func (s *Scheme) nameFunc(t reflect.Type) string {
for _, gvk := range gvks { for _, gvk := range gvks {
internalGV := gvk.GroupVersion() internalGV := gvk.GroupVersion()
internalGV.Version = "__internal" // this is hacky and maybe should be passed in internalGV.Version = APIVersionInternal // this is hacky and maybe should be passed in
internalGVK := internalGV.WithKind(gvk.Kind) internalGVK := internalGV.WithKind(gvk.Kind)
if internalType, exists := s.gvkToType[internalGVK]; exists { if internalType, exists := s.gvkToType[internalGVK]; exists {

View File

@ -129,25 +129,25 @@ func TestDecode(t *testing.T) {
}{ }{
{ {
serializer: &mockSerializer{actual: gvk1}, serializer: &mockSerializer{actual: gvk1},
convertor: &checkConvertor{groupVersion: schema.GroupVersion{Group: "other", Version: "__internal"}}, convertor: &checkConvertor{groupVersion: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal}},
expectedGVK: gvk1, expectedGVK: gvk1,
decodes: schema.GroupVersion{Group: "other", Version: "__internal"}, decodes: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal},
}, },
{ {
serializer: &mockSerializer{actual: gvk1, obj: decodable1}, serializer: &mockSerializer{actual: gvk1, obj: decodable1},
convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "other", Version: "__internal"}}, convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal}},
expectedGVK: gvk1, expectedGVK: gvk1,
sameObject: decodable2, sameObject: decodable2,
decodes: schema.GroupVersion{Group: "other", Version: "__internal"}, decodes: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal},
}, },
// defaultGVK.Group is allowed to force a conversion to the destination group // defaultGVK.Group is allowed to force a conversion to the destination group
{ {
serializer: &mockSerializer{actual: gvk1, obj: decodable1}, serializer: &mockSerializer{actual: gvk1, obj: decodable1},
defaultGVK: &schema.GroupVersionKind{Group: "force"}, defaultGVK: &schema.GroupVersionKind{Group: "force"},
convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "force", Version: "__internal"}}, convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "force", Version: runtime.APIVersionInternal}},
expectedGVK: gvk1, expectedGVK: gvk1,
sameObject: decodable2, sameObject: decodable2,
decodes: schema.GroupVersion{Group: "force", Version: "__internal"}, decodes: schema.GroupVersion{Group: "force", Version: runtime.APIVersionInternal},
}, },
// uses direct conversion for into when objects differ // uses direct conversion for into when objects differ
{ {
@ -184,10 +184,10 @@ func TestDecode(t *testing.T) {
into: &runtime.VersionedObjects{Objects: []runtime.Object{}}, into: &runtime.VersionedObjects{Objects: []runtime.Object{}},
serializer: &mockSerializer{actual: gvk1, obj: decodable1}, serializer: &mockSerializer{actual: gvk1, obj: decodable1},
convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "other", Version: "__internal"}}, convertor: &checkConvertor{in: decodable1, obj: decodable2, groupVersion: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal}},
expectedGVK: gvk1, expectedGVK: gvk1,
expectedObject: &runtime.VersionedObjects{Objects: []runtime.Object{decodable1, decodable2}}, expectedObject: &runtime.VersionedObjects{Objects: []runtime.Object{decodable1, decodable2}},
decodes: schema.GroupVersion{Group: "other", Version: "__internal"}, decodes: schema.GroupVersion{Group: "other", Version: runtime.APIVersionInternal},
}, },
// decode into the same version as the serialized object // decode into the same version as the serialized object