From 23bfa1b56718716b89134870dfd35be1d5d67fc1 Mon Sep 17 00:00:00 2001
From: Blake Erickson <o.blakeerickson@gmail.com>
Date: Thu, 25 Nov 2021 08:11:52 -0700
Subject: [PATCH] FIX: Exclude blocked charges in campaign total (#96)

The campaign total should only include payments that have succeeded.

See: https://meta.discourse.org/t/209591/14
---
 .../discourse_subscriptions/campaign.rb       |  1 +
 spec/services/campaign_spec.rb                | 24 ++++++++++++++++++-
 2 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/app/services/discourse_subscriptions/campaign.rb b/app/services/discourse_subscriptions/campaign.rb
index 280c463..3209066 100644
--- a/app/services/discourse_subscriptions/campaign.rb
+++ b/app/services/discourse_subscriptions/campaign.rb
@@ -170,6 +170,7 @@ module DiscourseSubscriptions
 
           all_invoices[:data].each do |invoice|
             customer_id = invoice[:customer]
+            next if invoice[:paid] != true
             line_item = invoice[:lines][:data][0] if invoice[:lines] && invoice[:lines][:data] # Discourse only makes single-line item charges
             # check if non-subscription and that the plan is active
             if line_item[:plan] == nil &&
diff --git a/spec/services/campaign_spec.rb b/spec/services/campaign_spec.rb
index 90f56fe..3ae8eba 100644
--- a/spec/services/campaign_spec.rb
+++ b/spec/services/campaign_spec.rb
@@ -27,6 +27,7 @@ describe DiscourseSubscriptions::Campaign do
     let(:invoice) do
       {
         id: "in_1234",
+        paid: true,
         lines: {
           data: [
             {
@@ -42,12 +43,33 @@ describe DiscourseSubscriptions::Campaign do
         }
       }
     end
+    let(:invoice2) do
+      {
+        id: "in_1235",
+        paid: false,
+        lines: {
+          data: [
+            {
+              plan: nil,
+              price: {
+                product: "prodct_65433",
+                active: true,
+                unit_amount: 600,
+                recurring: nil,
+              }
+            }
+          ]
+        }
+      }
+    end
 
     before do
       Fabricate(:product, external_id: "prodct_23456")
       Fabricate(:customer, product_id: "prodct_23456", user_id: user.id, customer_id: 'x')
       Fabricate(:product, external_id: "prodct_65432")
       Fabricate(:customer, product_id: "prodct_65432", user_id: user2.id, customer_id: 'y')
+      Fabricate(:product, external_id: "prodct_65433")
+      Fabricate(:customer, product_id: "prodct_65433", user_id: user2.id, customer_id: 'y')
       SiteSetting.discourse_subscriptions_public_key = "public-key"
       SiteSetting.discourse_subscriptions_secret_key = "secret-key"
     end
@@ -56,7 +78,7 @@ describe DiscourseSubscriptions::Campaign do
       context "for all subscription purchases" do
         it "refreshes the campaign data properly" do
           ::Stripe::Subscription.expects(:list).returns(data: [subscription], has_more: false)
-          ::Stripe::Invoice.expects(:list).returns(data: [invoice], has_more: false)
+          ::Stripe::Invoice.expects(:list).returns(data: [invoice, invoice2], has_more: false)
 
           DiscourseSubscriptions::Campaign.new.refresh_data