saucy 0.10.8 → 0.10.9

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+0.10.9
+
+* Follow-up to 0.10.8: Actually work irrespective of system TZ
+
 0.10.8
 
 Backport merchant account TZ fixes from 0.13.1

data/lib/saucy/subscription.rb

@@ -86,10 +86,10 @@ module Saucy
       end
 
       def update_subscription_cache!
+        zone = ActiveSupport::TimeZone[Saucy::Configuration.merchant_account_time_zone]
         flush_cache :subscription
         update_attribute(:subscription_status, subscription.status)
-        update_attribute(:next_billing_date, Time.parse(subscription.next_billing_date).in_time_zone(
-          Saucy::Configuration.merchant_account_time_zone))
+        update_attribute(:next_billing_date, zone.parse(subscription.next_billing_date))
       end
 
       def changing_plan?(attributes)
@@ -198,9 +198,9 @@ module Saucy
       def create_subscription
         result = Braintree::Subscription.create(subscription_attributes)
         if result.success?
+          zone = ActiveSupport::TimeZone[Saucy::Configuration.merchant_account_time_zone]
           self.subscription_token = result.subscription.id
-          self.next_billing_date = Time.parse(result.subscription.next_billing_date).in_time_zone(
-            Saucy::Configuration.merchant_account_time_zone)
+          self.next_billing_date = zone.parse(result.subscription.next_billing_date)
           self.subscription_status = result.subscription.status
         else
           false
@@ -226,10 +226,9 @@ module Saucy
       def update_subscriptions!
         recently_billed = where("next_billing_date <= ?", Time.now)
         recently_billed.each do |account|
+          zone = ActiveSupport::TimeZone[Saucy::Configuration.merchant_account_time_zone]
           account.subscription_status = account.subscription.status
-          account.next_billing_date = account.subscription.next_billing_date
-          account.next_billing_date = Time.parse(account.subscription.next_billing_date).in_time_zone(
-            Saucy::Configuration.merchant_account_time_zone)
+          account.next_billing_date = zone.parse(account.subscription.next_billing_date)
           account.save!
           if account.past_due?
             BillingMailer.problem(account, account.subscription.transactions.last).deliver!

data/spec/models/subscription_spec.rb

@@ -351,6 +351,9 @@ describe Account, "with a paid subscription" do
     end
 
     before do
+      @old_env_tz = ENV['TZ']
+      ENV['TZ'] = "Europe/Paris"
+
       Saucy::Configuration.merchant_account_time_zone = "Eastern Time (US & Canada)"
       subject.next_billing_date = "2011-08-15"
       subject.save!
@@ -365,6 +368,10 @@ describe Account, "with a paid subscription" do
       subject.reload.next_billing_date.should == Time.parse("2011-09-15 00:00 -0400")
     end
 
+    after do
+      ENV['TZ'] = @old_env_tz
+    end
+
     it "does not update accounts if their billing date hasn't elapsed from the merchant account's perspective" do
       Time.stubs(:now => Time.parse("2011-09-15 02:00 -0000"))
       Account.update_subscriptions!
@@ -381,6 +388,7 @@ describe Account, "with a paid subscription" do
   context "and an active subscription due 2 months from now" do
     let(:subscription) { FakeBraintree.subscriptions[subject.subscription_token] }
     let(:next_billing_date_string) { 2.months.from_now.to_s(:braintree_date) }
+    let(:zone) { ActiveSupport::TimeZone[Saucy::Configuration.merchant_account_time_zone] }
 
     before do
       subscription["status"] = Braintree::Subscription::Status::Active
@@ -394,8 +402,7 @@ describe Account, "with a paid subscription" do
       Timecop.travel(subject.next_billing_date + 1.day) do
         Account.update_subscriptions!
         subject.reload.subscription_status.should == Braintree::Subscription::Status::Active
-        subject.next_billing_date.should == Time.parse(next_billing_date_string).in_time_zone(
-          Saucy::Configuration.merchant_account_time_zone)
+        subject.next_billing_date.should == zone.parse(next_billing_date_string)
       end
     end
 
@@ -464,6 +471,8 @@ describe Account, "with a paid subscription that is past due" do
     subject.reload
   end
 
+  let(:zone) { ActiveSupport::TimeZone[Saucy::Configuration.merchant_account_time_zone] }
+
   it "retries the subscription charge and updates the subscription when the billing information is correctly updated" do
     next_billing_date_string = 1.day.from_now.to_s(:braintree_date)
     subscription = FakeBraintree.subscriptions[subject.subscription_token]
@@ -486,8 +495,7 @@ describe Account, "with a paid subscription that is past due" do
                                             :expiration_year => 2012).should be
 
     subject.reload.subscription_status.should == Braintree::Subscription::Status::Active
-    subject.next_billing_date.should == Time.parse(next_billing_date_string).in_time_zone(
-      Saucy::Configuration.merchant_account_time_zone)
+    subject.next_billing_date.should == zone.parse(next_billing_date_string)
   end
 
   it "retries the subscription charge and updates the subscription when the payment processing fails" do
@@ -513,8 +521,7 @@ describe Account, "with a paid subscription that is past due" do
 
     subject.errors[:card_number].should include("was denied by the payment processor with the message: no good")
     subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
-    subject.next_billing_date.should == Time.parse(next_billing_date_string).in_time_zone(
-      Saucy::Configuration.merchant_account_time_zone)
+    subject.next_billing_date.should == zone.parse(next_billing_date_string)
   end
 
   it "retries the subscription charge and updates the subscription when the settlement fails" do
@@ -540,8 +547,7 @@ describe Account, "with a paid subscription that is past due" do
 
     subject.errors[:card_number].should include("no good")
     subject.reload.subscription_status.should == Braintree::Subscription::Status::PastDue
-    subject.next_billing_date.should == Time.parse(next_billing_date_string).in_time_zone(
-      Saucy::Configuration.merchant_account_time_zone)
+    subject.next_billing_date.should == zone.parse(next_billing_date_string)
   end
 end
 

metadata

@@ -2,7 +2,7 @@
 name: saucy
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.10.8
+  version: 0.10.9
 platform: ruby
 authors: 
 - thoughtbot, inc.
@@ -287,7 +287,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 209378144174508178
+      hash: -2922648281484856857
       segments: 
       - 0
       version: "0"