Applied comments from the review

This commit is contained in:
Piotr Szczesniak 2016-02-18 07:18:37 +01:00
parent 2b614179c7
commit c02e966d65
3 changed files with 2 additions and 3 deletions

View File

@ -180,7 +180,7 @@ func doRoundTripTest(group testapi.TestGroup, kind string, t *testing.T) {
if api.Scheme.Recognizes(group.GroupVersion().WithKind(kind)) { if api.Scheme.Recognizes(group.GroupVersion().WithKind(kind)) {
roundTripSame(t, group, item, nonRoundTrippableTypesByVersion[kind]...) roundTripSame(t, group, item, nonRoundTrippableTypesByVersion[kind]...)
} }
if !nonInternalRoundTrippableTypes.Has(kind) { if !nonInternalRoundTrippableTypes.Has(kind) && api.Scheme.Recognizes(group.GroupVersion().WithKind(kind)) {
roundTrip(t, group.Codec(), fuzzInternalObject(t, group.InternalGroupVersion(), item, rand.Int63())) roundTrip(t, group.Codec(), fuzzInternalObject(t, group.InternalGroupVersion(), item, rand.Int63()))
} }
} }

View File

@ -75,7 +75,7 @@ func init() {
if _, ok := Groups[autoscaling.GroupName]; !ok { if _, ok := Groups[autoscaling.GroupName]; !ok {
Groups[autoscaling.GroupName] = TestGroup{ Groups[autoscaling.GroupName] = TestGroup{
externalGroupVersion: unversioned.GroupVersion{Group: autoscaling.GroupName, Version: registered.GroupOrDie(autoscaling.GroupName).GroupVersion.Version}, externalGroupVersion: unversioned.GroupVersion{Group: autoscaling.GroupName, Version: registered.GroupOrDie(autoscaling.GroupName).GroupVersion.Version},
internalGroupVersion: autoscaling.SchemeGroupVersion, internalGroupVersion: extensions.SchemeGroupVersion,
} }
} }
if _, ok := Groups[extensions.GroupName]; !ok { if _, ok := Groups[extensions.GroupName]; !ok {

View File

@ -168,7 +168,6 @@ func (c *codec) Decode(data []byte, defaultGVK *unversioned.GroupVersionKind, in
targetGV.Group = group targetGV.Group = group
targetGV.Version = runtime.APIVersionInternal targetGV.Version = runtime.APIVersionInternal
} else { } else {
fmt.Printf("looking for %v in %#v\n", group, c.decodeVersion)
gv, ok := c.decodeVersion[group] gv, ok := c.decodeVersion[group]
if !ok { if !ok {
// unknown objects are left in their original version // unknown objects are left in their original version