From 73a806108e4a51cb2b5770325cf74a8950ad2410 Mon Sep 17 00:00:00 2001 From: Joel Speed Date: Thu, 13 Sep 2018 15:06:23 +0100 Subject: [PATCH] Make sure multiple controllers can be registered --- pkg/controller/controller_test.go | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/pkg/controller/controller_test.go b/pkg/controller/controller_test.go index 2606ca0a75..e4ccd7970b 100644 --- a/pkg/controller/controller_test.go +++ b/pkg/controller/controller_test.go @@ -75,6 +75,21 @@ var _ = Describe("controller.Controller", func() { close(done) }) + + It("should not return an error if two controllers are registered with different names", func(done Done) { + m, err := manager.New(cfg, manager.Options{}) + Expect(err).NotTo(HaveOccurred()) + + c1, err := controller.New("c1", m, controller.Options{Reconciler: rec}) + Expect(err).NotTo(HaveOccurred()) + Expect(c1).ToNot(BeNil()) + + c2, err := controller.New("c2", m, controller.Options{Reconciler: rec}) + Expect(err).NotTo(HaveOccurred()) + Expect(c2).ToNot(BeNil()) + + close(done) + }) }) })