From 9901a386c372df290d015b412e0e33b783e36b95 Mon Sep 17 00:00:00 2001
From: deads2k <deads@redhat.com>
Date: Mon, 8 Feb 2016 16:32:36 -0500
Subject: [PATCH] remove ResourceIsValid

---
 pkg/api/meta/interfaces.go                    |  3 --
 pkg/api/meta/multirestmapper.go               | 10 -----
 pkg/api/meta/restmapper.go                    |  6 ---
 pkg/kubectl/kubectl.go                        |  6 ---
 pkg/registry/thirdpartyresourcedata/codec.go  |  4 --
 .../thirdpartyresourcedata/codec_test.go      | 41 -------------------
 6 files changed, 70 deletions(-)

diff --git a/pkg/api/meta/interfaces.go b/pkg/api/meta/interfaces.go
index cc81aafa13..f1402e7ad7 100644
--- a/pkg/api/meta/interfaces.go
+++ b/pkg/api/meta/interfaces.go
@@ -172,7 +172,4 @@ type RESTMapper interface {
 
 	AliasesForResource(resource string) ([]string, bool)
 	ResourceSingularizer(resource string) (singular string, err error)
-
-	// ResourceIsValid takes a partial resource and returns back whether or not the resource matches at least one kind
-	ResourceIsValid(resource unversioned.GroupVersionResource) bool
 }
diff --git a/pkg/api/meta/multirestmapper.go b/pkg/api/meta/multirestmapper.go
index eb63c01878..4d284e62a9 100644
--- a/pkg/api/meta/multirestmapper.go
+++ b/pkg/api/meta/multirestmapper.go
@@ -119,13 +119,3 @@ func (m MultiRESTMapper) AliasesForResource(alias string) (aliases []string, ok
 	}
 	return nil, false
 }
-
-// ResourceIsValid takes a string (either group/kind or kind) and checks if it's a valid resource
-func (m MultiRESTMapper) ResourceIsValid(resource unversioned.GroupVersionResource) bool {
-	for _, t := range m {
-		if t.ResourceIsValid(resource) {
-			return true
-		}
-	}
-	return false
-}
diff --git a/pkg/api/meta/restmapper.go b/pkg/api/meta/restmapper.go
index dc209d0c39..7b1856bc4b 100644
--- a/pkg/api/meta/restmapper.go
+++ b/pkg/api/meta/restmapper.go
@@ -522,9 +522,3 @@ func (m *DefaultRESTMapper) AliasesForResource(alias string) ([]string, bool) {
 	}
 	return nil, false
 }
-
-// ResourceIsValid takes a partial resource and checks if it's valid
-func (m *DefaultRESTMapper) ResourceIsValid(resource unversioned.GroupVersionResource) bool {
-	_, err := m.KindFor(resource)
-	return err == nil
-}
diff --git a/pkg/kubectl/kubectl.go b/pkg/kubectl/kubectl.go
index aa3db5ed08..2f9c7c4e87 100644
--- a/pkg/kubectl/kubectl.go
+++ b/pkg/kubectl/kubectl.go
@@ -86,12 +86,6 @@ func (e ShortcutExpander) KindFor(resource unversioned.GroupVersionResource) (un
 	return e.RESTMapper.KindFor(resource)
 }
 
-// ResourceIsValid takes a string (kind) and checks if it's a valid resource.
-// It expands the resource first, then invokes the wrapped mapper.
-func (e ShortcutExpander) ResourceIsValid(resource unversioned.GroupVersionResource) bool {
-	return e.RESTMapper.ResourceIsValid(expandResourceShortcut(resource))
-}
-
 // ResourceSingularizer expands the named resource and then singularizes it.
 func (e ShortcutExpander) ResourceSingularizer(resource string) (string, error) {
 	return e.RESTMapper.ResourceSingularizer(expandResourceShortcut(unversioned.GroupVersionResource{Resource: resource}).Resource)
diff --git a/pkg/registry/thirdpartyresourcedata/codec.go b/pkg/registry/thirdpartyresourcedata/codec.go
index 2e80c8c00d..5ca3943d38 100644
--- a/pkg/registry/thirdpartyresourcedata/codec.go
+++ b/pkg/registry/thirdpartyresourcedata/codec.go
@@ -126,10 +126,6 @@ func (t *thirdPartyResourceDataMapper) ResourceSingularizer(resource string) (si
 	return t.mapper.ResourceSingularizer(resource)
 }
 
-func (t *thirdPartyResourceDataMapper) ResourceIsValid(resource unversioned.GroupVersionResource) bool {
-	return t.isThirdPartyResource(resource) || t.mapper.ResourceIsValid(resource)
-}
-
 func NewMapper(mapper meta.RESTMapper, kind, version, group string) meta.RESTMapper {
 	return &thirdPartyResourceDataMapper{
 		mapper:  mapper,
diff --git a/pkg/registry/thirdpartyresourcedata/codec_test.go b/pkg/registry/thirdpartyresourcedata/codec_test.go
index 2fdd4e9c4b..28cb84ee51 100644
--- a/pkg/registry/thirdpartyresourcedata/codec_test.go
+++ b/pkg/registry/thirdpartyresourcedata/codec_test.go
@@ -178,44 +178,3 @@ func TestCreater(t *testing.T) {
 
 	}
 }
-
-func TestResourceIsValid(t *testing.T) {
-	tests := []struct {
-		kind     string
-		resource string
-		valid    bool
-		name     string
-	}{
-		{
-			kind:     "Foo",
-			resource: "foos",
-			valid:    true,
-			name:     "basic",
-		},
-		{
-			kind:     "Party",
-			resource: "parties",
-			valid:    true,
-			name:     "fun",
-		},
-		{
-			kind:     "bus",
-			resource: "buses",
-			valid:    true,
-			name:     "transport",
-		},
-		{
-			kind:     "Foo",
-			resource: "fooies",
-			name:     "bad",
-		},
-	}
-	for _, test := range tests {
-		mapper := &thirdPartyResourceDataMapper{kind: test.kind}
-		mapper.mapper = api.RESTMapper
-		valid := mapper.ResourceIsValid(unversioned.GroupVersionResource{Resource: test.resource})
-		if valid != test.valid {
-			t.Errorf("%s: expected: %v, actual: %v", test.name, test.valid, valid)
-		}
-	}
-}