Updated specs related to redonation notification

app/controllers/api/payment/braintree_controller.rb

@@ -16,14 +16,20 @@ def webhook
   end
 
   def one_click
-    member  = Member.find_by_email(params[:user][:email])
     @result = client::OneClick.new(unsafe_params, cookies.signed[:payment_methods], member).run
-
     render status: :unprocessable_entity, errors: oneclick_payment_errors unless @result.success?
   end
 
   private
 
+  def member
+    if params[:user][:email].present?
+      Member.find_by_email(params[:user][:email])
+    elsif unsafe_params[:akid].present?
+      Member.find_from_request(akid: unsafe_params[:akid], id: cookies.signed[:member_id])
+    end
+  end
+
   def oneclick_payment_errors
     if @result.class == Braintree::ErrorResult
       client::ErrorProcessing.new(@result, locale: locale).process

app/lib/payment_processor/braintree/one_click.rb

@@ -33,6 +33,8 @@ def initialize(params, cookied_payment_methods, member = nil)
     end
 
     def run
+      raise PaymentProcessor::Exceptions::CustomerNotFound unless @member.try(:customer)
+
       # TODO: On the second attempt (if the member consents to duplicate donation), post with the same parameters
       # but also params[:allow_duplicate] = true
       return duplicate_donation_error_response if duplicate_donation && !payment_options.params[:allow_duplicate]

app/lib/payment_processor/braintree/one_click_from_uri.rb

@@ -14,7 +14,7 @@ def initialize(params, page:, member: nil, cookied_payment_methods: '')
     def process
       return false unless one_click?
 
-      PaymentProcessor::Braintree::OneClick.new(options, @cookied_payment_methods).run
+      PaymentProcessor::Braintree::OneClick.new(options, @cookied_payment_methods, member).run
       self
     end
 

spec/features/express_donations/page_one_click_spec.rb

@@ -41,8 +41,10 @@
     }
 
     VCR.use_cassette('feature_member_express_donation') do
-      path = api_payment_braintree_one_click_path(donation_page.id)
-      page.driver.post path, params
+      Timecop.freeze(Time.now + 15.minutes) do
+        path = api_payment_braintree_one_click_path(donation_page.id)
+        page.driver.post path, params
+      end
     end
 
     expect(Action.count).to eq(2)

spec/lib/analytics/analytics_spec.rb

@@ -105,7 +105,7 @@
           '1999-12-31 21:00:00' => 3
         }
 
-        Timecop.freeze(start_date) do
+        Timecop.freeze(start_date).utc do
           expect(
             subject.total_actions_over_time(period: :hour)
           ).to include(sample_of_expected_data)
@@ -118,7 +118,7 @@
           '1999-12-31 23:00:00' => 3
         }
 
-        Timecop.freeze(start_date) do
+        Timecop.freeze(start_date).utc do
           expect(
             subject.total_new_members_over_time(period: :hour)
           ).to include(sample_of_expected_data)