fix(billing): omit empty product description for Stripe

billing/product/service.go

@@ -81,21 +81,26 @@ func (s *Service) Create(ctx context.Context, product Product) (Product, error)
 	}
 	product.Name = strings.ToLower(product.Name)
 
-	_, err := s.stripeClient.Products.New(&stripe.ProductParams{
+	providerParams := &stripe.ProductParams{
 		Params: stripe.Params{
 			Context: ctx,
 		},
-		ID:          &product.ProviderID,
-		Name:        &product.Title,
-		Description: &product.Description,
+		ID:   &product.ProviderID,
+		Name: &product.Title,
 		Metadata: map[string]string{
 			"name":          product.Name,
 			"credit_amount": fmt.Sprintf("%d", product.Config.CreditAmount),
 			"behavior":      product.Behavior.String(),
 			"product_id":    product.ID,
 			"managed_by":    "frontier",
 		},
-	})
+	}
+
+	if product.Description != "" {
+		providerParams.Description = &product.Description
+	}
+
+	_, err := s.stripeClient.Products.New(providerParams)
 	if err != nil {
 		return Product{}, fmt.Errorf("failed to create product at billing provider: %w", billingerrors.TranslateStripeError(err))
 	}

billing/product/service_test.go

@@ -40,6 +40,51 @@ func TestService_Create(t *testing.T) {
 		wantErr bool
 		setup   func() *product.Service
 	}{
+		{
+			name: "should create product with empty description",
+			args: args{
+				product: product.Product{
+					ID:          "1",
+					Name:        "product1",
+					Description: "",
+				},
+			},
+			want: product.Product{
+				ID:          "1",
+				Name:        "product1",
+				Description: "",
+			},
+			wantErr: false,
+			setup: func() *product.Service {
+				stripeClient, mockStripeBackend, mockProductRepo, mockPriceRepo, mockFeatureRepo := mockService(t)
+				mockProductRepo.EXPECT().Create(ctx, product.Product{
+					ID:          "1",
+					Name:        "product1",
+					Description: "",
+					Behavior:    product.BasicBehavior,
+				}).Return(product.Product{
+					ID:          "1",
+					Name:        "product1",
+					Description: "",
+				}, nil)
+				mockStripeBackend.EXPECT().Call("POST", "/v1/products", "key_123", &stripe.ProductParams{
+					Params: stripe.Params{
+						Context: ctx,
+					},
+					ID:          new(""),
+					Name:        new(""),
+					Description: nil,
+					Metadata: map[string]string{
+						"behavior":      "basic",
+						"credit_amount": "0",
+						"managed_by":    "frontier",
+						"name":          "product1",
+						"product_id":    "1",
+					},
+				}, &stripe.Product{}).Return(nil)
+				return product.NewService(stripeClient, mockProductRepo, mockPriceRepo, mockFeatureRepo)
+			},
+		},
 		{
 			name: "should create product in repo and billing provider with no price and features",
 			args: args{