Ensure only subscriptions with status "canceled" are excluded

lib/subscription_server/providers/stripe.rb

@@ -30,7 +30,7 @@ def subscriptions(product_ids, resource_name)
       sub_hash = sub.to_h
       price = sub_hash[:items][:data][0][:price]
 
-      if product_ids.include?(price[:product]) && sub_hash[:canceled_at].blank?
+      if product_ids.include?(price[:product]) && sub_hash[:status] == 'active'
         product = ::Stripe::Product.retrieve(price[:product])
         subscription = SubscriptionServer::Subscription.new(
           resource: resource_name,

spec/components/subscription_server/user_subscriptions_spec.rb

@@ -81,6 +81,16 @@ def response_error(message)
           expect(subscription.price_name).to eq(Stripe::PRICE_NAME)
         end
 
+        context "with a cancelled subscription" do
+          it "does not load cancelled subscriptions" do
+            Stripe::Customer.has_subscription = true
+            Stripe::Customer.subscription_attrs = { "status" => "canceled" }
+            @instance.load(resources)
+            expect(@instance.errors).to include(response_error("no subscriptions found for #{resource}"))
+            expect(@instance.subscriptions).to eq([])
+          end
+        end
+
         context "with domain limit" do
           before do
             Stripe::Customer.has_subscription = true

spec/fixtures/providers/stripe_plugin/plugin.rb

@@ -16,6 +16,7 @@ module ::Stripe
 
   class Customer
     cattr_accessor :has_subscription
+    cattr_accessor(:subscription_attrs) { {} }
 
     def self.list(email: nil, expand: nil)
       stripe_path = "#{Rails.root}/plugins/discourse-subscription-server/spec/fixtures/providers/stripe"
@@ -29,6 +30,11 @@ def self.list(email: nil, expand: nil)
         subscription["items"]["data"][0]["price"]["product"] = PRODUCT_ID
         subscription["items"]["data"][0]["price"]["id"] = PRICE_ID
         subscription["items"]["data"][0]["price"]["nickname"] = PRICE_NAME
+        if subscription_attrs.present?
+          subscription_attrs.each do |k, v|
+            subscription[k] = v
+          end
+        end
         customer_list["data"][0]["subscriptions"]["data"] << subscription
       end