Skip to content

Commit

Permalink
give "Application Start with ControllerManagedBy should Reconcile Own…
Browse files Browse the repository at this point in the history
…s objects " more time to sync
  • Loading branch information
p0lyn0mial committed Oct 14, 2024
1 parent c6622f8 commit 2544595
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions pkg/builder/controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -201,9 +201,9 @@ var _ = Describe("application", func() {
Named("my_controller-0").
Owns(&appsv1.ReplicaSet{}).
Build(typedNoop)
// If we ever allow Owns() without For() we need to update the code to error
// out on Owns() if the request type is different from reconcile.Request just
// like we do in For().
// If we ever allow Owns() without For() we need to update the code to error
// out on Owns() if the request type is different from reconcile.Request just
// like we do in For().
Expect(err).To(MatchError("Owns() can only be used together with For()"))
Expect(instance).To(BeNil())
})
Expand Down Expand Up @@ -711,7 +711,7 @@ func doReconcileTest(ctx context.Context, nameSuffix string, mgr manager.Manager
Expect(err).NotTo(HaveOccurred())

By("Waiting for the Deployment Reconcile")
Eventually(ch).Should(Receive(Equal(reconcile.Request{
Eventually(ch, "5s").Should(Receive(Equal(reconcile.Request{
NamespacedName: types.NamespacedName{Namespace: "default", Name: deployName}})))

By("Creating a ReplicaSet")
Expand Down

0 comments on commit 2544595

Please sign in to comment.