Remove {Before,After} Suite specific timeout

Author: acumino

pkg/builder/builder_suite_test.go

@@ -63,7 +63,7 @@ var _ = BeforeSuite(func() {
 
 	webhook.DefaultPort, _, err = addr.Suggest("")
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/cache/cache_suite_test.go

@@ -50,7 +50,7 @@ var _ = BeforeSuite(func() {
 
 	clientset, err = kubernetes.NewForConfig(cfg)
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/certwatcher/certwatcher_suite_test.go

@@ -40,10 +40,10 @@ func TestSource(t *testing.T) {
 
 var _ = BeforeSuite(func() {
 	logf.SetLogger(zap.New(zap.WriteTo(GinkgoWriter), zap.UseDevMode(true)))
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	for _, file := range []string{certPath, keyPath} {
 		_ = os.Remove(file)
 	}
-}, 60)
+})

pkg/client/apiutil/apiutil_suite_test.go

@@ -41,4 +41,4 @@ var _ = BeforeSuite(func() {
 
 	// for things that technically need a rest.Config for defaulting, but don't actually use them
 	cfg = &rest.Config{}
-}, 60)
+})

pkg/client/client_suite_test.go

@@ -55,7 +55,7 @@ var _ = BeforeSuite(func() {
 	Expect(err).NotTo(HaveOccurred())
 
 	Expect(pkg.AddToScheme(scheme.Scheme)).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/client/config/config_suite_test.go

@@ -35,4 +35,4 @@ func TestConfig(t *testing.T) {
 
 var _ = BeforeSuite(func() {
 	logf.SetLogger(zap.New(zap.WriteTo(GinkgoWriter), zap.UseDevMode(true)))
-}, 60)
+})

pkg/client/fake/client_suite_test.go

@@ -35,4 +35,4 @@ func TestSource(t *testing.T) {
 
 var _ = BeforeSuite(func() {
 	logf.SetLogger(zap.New(zap.WriteTo(GinkgoWriter), zap.UseDevMode(true)))
-}, 60)
+})

pkg/cluster/cluster_suite_test.go

@@ -63,7 +63,7 @@ var _ = BeforeSuite(func() {
 
 	clientset, err = kubernetes.NewForConfig(cfg)
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/controller/controller_suite_test.go

@@ -82,7 +82,7 @@ var _ = BeforeSuite(func() {
 
 	// Prevent the metrics listener being created
 	metrics.DefaultBindAddress = "0"
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/envtest/envtest_suite_test.go

@@ -44,7 +44,7 @@ var _ = BeforeSuite(func() {
 	initializeWebhookInEnvironment()
 	_, err := env.Start()
 	Expect(err).NotTo(HaveOccurred())
-}, StartTimeout)
+})
 
 func initializeWebhookInEnvironment() {
 	namespacedScopeV1 := admissionv1.NamespacedScope
@@ -133,4 +133,4 @@ func initializeWebhookInEnvironment() {
 
 var _ = AfterSuite(func() {
 	Expect(env.Stop()).NotTo(HaveOccurred())
-}, StopTimeout)
+})

pkg/internal/controller/controller_suite_test.go

@@ -50,7 +50,7 @@ var _ = BeforeSuite(func() {
 
 	clientset, err = kubernetes.NewForConfig(cfg)
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/internal/recorder/recorder_suite_test.go

@@ -50,7 +50,7 @@ var _ = BeforeSuite(func() {
 
 	clientset, err = kubernetes.NewForConfig(cfg)
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/manager/manager_suite_test.go

@@ -72,7 +72,7 @@ var _ = BeforeSuite(func() {
 
 	// Prevent the metrics listener being created
 	metrics.DefaultBindAddress = "0"
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	Expect(testenv.Stop()).To(Succeed())

pkg/source/source_suite_test.go

@@ -64,9 +64,9 @@ var _ = BeforeSuite(func() {
 		defer GinkgoRecover()
 		Expect(icache.Start(ctx)).NotTo(HaveOccurred())
 	}()
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	cancel()
 	Expect(testenv.Stop()).To(Succeed())
-}, 5)
+})

pkg/webhook/admission/admission_suite_test.go

@@ -35,4 +35,4 @@ func TestAdmissionWebhook(t *testing.T) {
 
 var _ = BeforeSuite(func() {
 	logf.SetLogger(zap.New(zap.WriteTo(GinkgoWriter), zap.UseDevMode(true)))
-}, 60)
+})

pkg/webhook/authentication/authentication_suite_test.go

@@ -35,4 +35,4 @@ func TestAuthenticationWebhook(t *testing.T) {
 
 var _ = BeforeSuite(func() {
 	logf.SetLogger(zap.New(zap.WriteTo(GinkgoWriter), zap.UseDevMode(true)))
-}, 60)
+})

pkg/webhook/webhook_suite_test.go

@@ -50,12 +50,12 @@ var _ = BeforeSuite(func() {
 	var err error
 	cfg, err = testenv.Start()
 	Expect(err).NotTo(HaveOccurred())
-}, 60)
+})
 
 var _ = AfterSuite(func() {
 	fmt.Println("stopping?")
 	Expect(testenv.Stop()).To(Succeed())
-}, 60)
+})
 
 func initializeWebhookInEnvironment() {
 	namespacedScopeV1 := admissionv1.NamespacedScope