Reduce LOC in third party controller test
This commit is contained in:
@@ -63,6 +63,14 @@ func (f *FakeAPIInterface) ListThirdPartyResources() []string {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func TestSyncAPIs(t *testing.T) {
|
func TestSyncAPIs(t *testing.T) {
|
||||||
|
resourcesNamed := func(names ...string) []expapi.ThirdPartyResource {
|
||||||
|
result := []expapi.ThirdPartyResource{}
|
||||||
|
for _, name := range names {
|
||||||
|
result = append(result, expapi.ThirdPartyResource{ObjectMeta: api.ObjectMeta{Name: name}})
|
||||||
|
}
|
||||||
|
return result
|
||||||
|
}
|
||||||
|
|
||||||
tests := []struct {
|
tests := []struct {
|
||||||
list *expapi.ThirdPartyResourceList
|
list *expapi.ThirdPartyResourceList
|
||||||
apis []string
|
apis []string
|
||||||
@@ -72,26 +80,14 @@ func TestSyncAPIs(t *testing.T) {
|
|||||||
}{
|
}{
|
||||||
{
|
{
|
||||||
list: &expapi.ThirdPartyResourceList{
|
list: &expapi.ThirdPartyResourceList{
|
||||||
Items: []expapi.ThirdPartyResource{
|
Items: resourcesNamed("foo.example.com"),
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.example.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
expectedInstalled: []string{"foo.example.com"},
|
expectedInstalled: []string{"foo.example.com"},
|
||||||
name: "simple add",
|
name: "simple add",
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
list: &expapi.ThirdPartyResourceList{
|
list: &expapi.ThirdPartyResourceList{
|
||||||
Items: []expapi.ThirdPartyResource{
|
Items: resourcesNamed("foo.example.com"),
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.example.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
apis: []string{
|
apis: []string{
|
||||||
"/apis/example.com",
|
"/apis/example.com",
|
||||||
@@ -101,13 +97,7 @@ func TestSyncAPIs(t *testing.T) {
|
|||||||
},
|
},
|
||||||
{
|
{
|
||||||
list: &expapi.ThirdPartyResourceList{
|
list: &expapi.ThirdPartyResourceList{
|
||||||
Items: []expapi.ThirdPartyResource{
|
Items: resourcesNamed("foo.example.com"),
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.example.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
apis: []string{
|
apis: []string{
|
||||||
"/apis/example.com",
|
"/apis/example.com",
|
||||||
@@ -123,18 +113,7 @@ func TestSyncAPIs(t *testing.T) {
|
|||||||
},
|
},
|
||||||
{
|
{
|
||||||
list: &expapi.ThirdPartyResourceList{
|
list: &expapi.ThirdPartyResourceList{
|
||||||
Items: []expapi.ThirdPartyResource{
|
Items: resourcesNamed("foo.example.com", "foo.company.com"),
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.example.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.company.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
apis: []string{
|
apis: []string{
|
||||||
"/apis/company.com",
|
"/apis/company.com",
|
||||||
@@ -145,13 +124,7 @@ func TestSyncAPIs(t *testing.T) {
|
|||||||
},
|
},
|
||||||
{
|
{
|
||||||
list: &expapi.ThirdPartyResourceList{
|
list: &expapi.ThirdPartyResourceList{
|
||||||
Items: []expapi.ThirdPartyResource{
|
Items: resourcesNamed("foo.example.com"),
|
||||||
{
|
|
||||||
ObjectMeta: api.ObjectMeta{
|
|
||||||
Name: "foo.example.com",
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
apis: []string{
|
apis: []string{
|
||||||
"/apis/company.com",
|
"/apis/company.com",
|
||||||
|
Reference in New Issue
Block a user