From 502a89275dd4c4dbdf1b6852e18766589921fd0a Mon Sep 17 00:00:00 2001 From: Amol Deodhar Date: Wed, 24 Apr 2024 09:41:40 -0400 Subject: [PATCH] add test for nil nb config --- controller/linodecluster_controller_test.go | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/controller/linodecluster_controller_test.go b/controller/linodecluster_controller_test.go index e59edb6ee..7878193c3 100644 --- a/controller/linodecluster_controller_test.go +++ b/controller/linodecluster_controller_test.go @@ -134,6 +134,25 @@ var _ = Describe("cluster-lifecycle", Ordered, Label("cluster", "cluster-lifecyc Expect(err.Error()).To(ContainSubstring("nodeBalancer created was nil")) }), ), + Path( + Call("cluster is not created because nb config was nil", func(ctx context.Context, m Mock) { + cScope.LinodeClient = m.NodeBalancerClient + getNB := m.NodeBalancerClient.EXPECT().ListNodeBalancers(gomock.Any(), gomock.Any()).Return(nil, nil) + m.NodeBalancerClient.EXPECT().CreateNodeBalancer(gomock.Any(), gomock.Any()). + After(getNB). + Return(&linodego.NodeBalancer{ + ID: nodebalancerID, + IPv4: &controlPlaneEndpointHost, + }, nil) + m.NodeBalancerClient.EXPECT().CreateNodeBalancerConfig(gomock.Any(), gomock.Any(), gomock.Any()). + After(getNB). + Return(nil, errors.New("nodeBalancer config created was nil")) + }), + Result("created nb config is nil", func(ctx context.Context, m Mock) { + _, err := reconciler.reconcile(ctx, cScope, logr.Logger{}) + Expect(err.Error()).To(ContainSubstring("nodeBalancer config created was nil")) + }), + ), Path( Call("cluster is not created because there is no capl cluster", func(ctx context.Context, m Mock) { cScope.LinodeClient = m.NodeBalancerClient