controller: don't run informers in unit tests when unnecessary

This commit is contained in:
Michail Kargakis 2017-01-30 11:35:59 +01:00
parent 7ee9abe447
commit 5ccc6d2727

View File

@ -497,16 +497,13 @@ func TestPodDeletionEnqueuesRecreateDeployment(t *testing.T) {
f.rsLister = append(f.rsLister, rs) f.rsLister = append(f.rsLister, rs)
f.objects = append(f.objects, foo, rs) f.objects = append(f.objects, foo, rs)
c, informers := f.newController() c, _ := f.newController()
enqueued := false enqueued := false
c.enqueueDeployment = func(d *extensions.Deployment) { c.enqueueDeployment = func(d *extensions.Deployment) {
if d.Name == "foo" { if d.Name == "foo" {
enqueued = true enqueued = true
} }
} }
stopCh := make(chan struct{})
defer close(stopCh)
informers.Start(stopCh)
c.deletePod(pod) c.deletePod(pod)
@ -532,16 +529,13 @@ func TestPodDeletionDoesntEnqueueRecreateDeployment(t *testing.T) {
// return a non-empty list. // return a non-empty list.
f.podLister = append(f.podLister, pod) f.podLister = append(f.podLister, pod)
c, informers := f.newController() c, _ := f.newController()
enqueued := false enqueued := false
c.enqueueDeployment = func(d *extensions.Deployment) { c.enqueueDeployment = func(d *extensions.Deployment) {
if d.Name == "foo" { if d.Name == "foo" {
enqueued = true enqueued = true
} }
} }
stopCh := make(chan struct{})
defer close(stopCh)
informers.Start(stopCh)
c.deletePod(pod) c.deletePod(pod)