Fixed SubResource/Status calling logic

Author: ludydoo

pkg/client/fake/intercept.go

@@ -96,7 +96,7 @@ func (c interceptor) DeleteAllOf(ctx context.Context, obj client.Object, opts ..
 }
 
 func (c interceptor) Status() client.SubResourceWriter {
-	return c.client.Status()
+	return c.SubResource("status")
 }
 
 func (c interceptor) SubResource(subResource string) client.SubResourceClient {

pkg/client/fake/intercept_test.go

@@ -220,6 +220,33 @@ var _ = Describe("Interceptor", func() {
 		_ = client2.SubResource("")
 		Expect(called).To(BeTrue())
 	})
+	It("should call the provided SubResource function with 'status' when calling Status()", func() {
+		var called bool
+		client := Interceptor(wrappedClient, InterceptorFns{
+			SubResource: func(client client.WithWatch, subResource string) client.SubResourceClient {
+				if subResource == "status" {
+					called = true
+				}
+				return nil
+			},
+		})
+		_ = client.Status()
+		Expect(called).To(BeTrue())
+	})
+	It("should call the underlying client if the provided SubResource function is nil when calling Status", func() {
+		var called bool
+		client1 := Interceptor(wrappedClient, InterceptorFns{
+			SubResource: func(client client.WithWatch, subResource string) client.SubResourceClient {
+				if subResource == "status" {
+					called = true
+				}
+				return nil
+			},
+		})
+		client2 := Interceptor(client1, InterceptorFns{})
+		_ = client2.Status()
+		Expect(called).To(BeTrue())
+	})
 })
 
 var _ = Describe("SubResourceInterceptor", func() {