muck-commerce 0.1.9 → 0.2.0

data/test/rails_root/config/environment.rb

@@ -1,4 +1,4 @@
-RAILS_GEM_VERSION = '2.3.4' unless defined? RAILS_GEM_VERSION
+RAILS_GEM_VERSION = '2.3.5' unless defined? RAILS_GEM_VERSION
 
 require File.join(File.dirname(__FILE__), 'boot')
 

data/test/rails_root/db/migrate/20100408035338_move_customer_profile_id.rb

@@ -0,0 +1,18 @@
+class MoveCustomerProfileId < ActiveRecord::Migration
+  def self.up
+    add_column :users, :customer_profile_id, :string, :limit => 512
+    BillingInformation.all.each do |bi|
+      bi.billable.update_attribute(:customer_profile_id, bi.customer_profile_id)
+    end
+    remove_column :billing_informations, :customer_profile_id
+  end
+
+  def self.down
+    add_column :billing_informations, :customer_profile_id, :string, :limit => 512
+    BillingInformation.all.each do |bi|
+      bi.update_attribute(:customer_profile_id, bi.billable.customer_profile_id)
+    end
+    remove_column :users, :customer_profile_id
+  end
+  
+end

data/test/rails_root/remote/functional/remote_orders_controller_test.rb

@@ -7,28 +7,38 @@ class RemoteOrdersControllerTest < ActionController::TestCase
   should_require_login :index => :get, :login_url => '/login'
 
   context "orders controller" do
+    
     setup do
       MuckCommerce::Configure.enable_live_test_gateway
     end
-
+    
+    teardown do
+      MuckCommerce::Configure.disable_live_test_gateway
+    end
+    
     context "logged in without billing information" do
-
+    
       setup do
         @user = Factory(:user)
         populate_cart(@user)
+        activate_authlogic
         login_as @user
       end
-
+    
       context "POST create order with new billing information" do
         setup do
           post_create_live_billing
         end
-         should_redirect_to("user_order_path(@user, assigns['order'])") { user_order_path(@user, assigns['order']) }
+        teardown do
+          cleanup_cim(assigns(:billing_information))
+        end
+        should_not_set_the_flash
+        should_redirect_to("new order path") { user_order_path(@user, assigns(:order), :new_order => true) }
         should "not have errors on the billing object" do
-          assert assigns['billing'].errors.empty?, assigns['billing'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.empty?, assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
-
+    
       context "POST create with bad verification value" do
         setup do
           post_create_live_billing(:verification_value => '')
@@ -36,10 +46,10 @@ class RemoteOrdersControllerTest < ActionController::TestCase
         should_respond_with :success
         should_render_template :new
         should "have errors on the billing object" do
-          assert assigns['billing'].errors.on(:verification_value), assigns['billing'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.on(:verification_value), assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
-  
+      
       context "POST create with bad credit card" do
         setup do
           post_create_live_billing(:credit_card_number => '')
@@ -47,85 +57,75 @@ class RemoteOrdersControllerTest < ActionController::TestCase
         should_respond_with :success
         should_render_template :new
         should "have errors on the billing object" do
-          assert assigns['billing'].errors.on(:credit_card_number), assigns['billing'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.on(:credit_card_number), assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
-  
+      
     end
-  
+
     context "user logged in with existing billing information" do
     
       setup do
-        @user = Factory(:user)
+        @user, @billing_information = setup_live_user_and_billing
+        # send billing info to the CIM
+        response = try_cim(@user, @billing_information) { |user, billing_information| OrderTransaction.cim_create(user, billing_information) }
+        assert response.success?, response.message
+        assert @user.customer_profile_id
+        assert @billing_information.customer_payment_profile_id
+        assert @billing_information.has_billing_profile?
+        @user.order_transactions.push(response)
+        @user.reload
         populate_cart(@user)
-        @billing = Factory.build(:live_billing_information, :billable => @user)
-        @user.billing_informations << @billing
-        @billing.vault_id = '1234'
-        @billing.save!
-      
+        activate_authlogic
         login_as @user
       end
-    
+      teardown do
+        cleanup_cim(@billing_information)
+      end
+      
       should "assign billing information" do
-        assert @billing.billable == @user, "Billing user is #{@billing.billable.login} but user is #{@user}"
+        assert @billing_information.billable == @user, "Billing user is #{@billing_information.billable.login} but user is #{@user}"
       end
-    
+          
       context "POST create order with existing billing information" do
         setup do
-          post :create
+          try_cim_post do
+            post :create, :amount => 1000
+          end
+        end
+        should_not_set_the_flash
+        should "have valid CIM profile ids" do
+          assert @user.billing_information, "User does not have any billing information."
+          assert @billing_information.has_billing_profile?
         end
-        should_redirect_to("user_order_path(@user, assigns['order'])") { user_order_path(@user, assigns['order']) }
+        should_redirect_to("user order path") { user_order_path(@user, assigns(:order), :new_order => true) }
         should "not have errors on the billing object" do
-          assert assigns['billing'].errors.empty?, assigns['billing'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.empty?, assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
-  
+        
       context "POST create order with updated billing information" do
         setup do
-          post :create, :billing_information => { 
-                          :first_name => 'test', 
-                          :last_name => 'guy', 
-                          :address1 => 'P.O. Box 377',
-                          :city => 'Somewhere',
-                          :state_id => Factory(:state).id,
-                          :zip => 99999,
-                          :country_id => Factory(:country).id,
-                          :telephone => '777-777-7777', 
-                          :payment_method => 'CC',
-                          :credit_card_type => 'visa',
-                          :credit_card_number => MuckCommerceTestDefinitions::VISA,
-                          :verification_value => '999',
-                          :credit_card_expiration => Time.now },
-                        :changed_billing => 'true'
+          post_create_live_billing({:telephone => '777-555-2222'}, true)
+        end
+        teardown do
+          cleanup_cim(assigns(:billing_information))
         end
-        should_redirect_to("user order path") { user_order_path(assigns(:user), assigns(:order)) }
+        should_not_set_the_flash
+        should_redirect_to("user order path") { user_order_path(assigns(:user), assigns(:order), :new_order => true) }
         should "not have errors on the billing object" do
-          assert assigns['billing_information'].errors.empty?, assigns['billing_information'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.empty?, assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
-    
+          
       context "POST create order with bad credit card" do
         setup do
-          post :create, :billing => { 
-                          :first_name => 'test', 
-                          :last_name => 'guy', 
-                          :address1 => 'P.O. Box 377',
-                          :city => 'Somewhere',
-                          :state_id => Factory(:state).id,
-                          :zip => 99999,
-                          :country_id => Factory(:country).id,
-                          :telephone => '777-777-7777', 
-                          :payment_method => GlobalConfig.payment_methods[0][1],
-                          :credit_card_type => 'bogus',
-                          :credit_card_number => '2232',
-                          :verification_value => '999',
-                          :credit_card_expiration => Time.now },
-                        :changed_billing => 'true'
+          post_create_live_billing({:credit_card_type => 'bogus', :credit_card_number => '2232'}, true)
         end
         should_respond_with :success
         should_render_template :new
         should "have errors on the billing object" do
-          assert assigns['billing'].errors.on(:credit_card_number), assigns['billing'].errors.full_messages.to_sentence
+          assert assigns(:billing_information).errors.on(:credit_card_number), assigns(:billing_information).errors.full_messages.to_sentence
         end
       end
     

data/test/rails_root/remote/functional/remote_subscriptions_controller_test.rb

@@ -0,0 +1,9 @@
+require File.dirname(__FILE__) + '/../remote_helper'
+
+class RemoteSubscriptionsControllerTest < ActionController::TestCase
+
+  tests Muck::SubscriptionsController
+  
+  should_require_login :edit => :get, :update => :put, :destroy => :delete, :login_url => '/login'
+  
+end

data/test/rails_root/remote/remote_helper.rb

@@ -1,20 +1,49 @@
 require File.dirname(__FILE__) + '/../test/test_helper'
 
-def post_create_live_billing(options = {})
-  post :create, :billing => { 
-                  :first_name => 'test', 
-                  :last_name => 'guy', 
-                  :address1 => '33 commerce way',
-                  :city => 'San Jose',
-                  :state_id => Factory(:state).id,
-                  :zip => 99999,
-                  :country_id => Factory(:country).id,
-                  :telephone => '777-777-7777', 
-                  :payment_method => 'CC',
-                  :credit_card_type => 'visa',
-                  :credit_card_number => MuckCommerceTestDefinitions::PAYPAL_VISA,
-                  :verification_value => '999',
-                  :credit_card_expiration => Time.now }.merge(options)
+def post_create_live_billing(options = {}, changed_billing = false, amount = 1000)
+  try_cim_post do
+    post :create, :billing_information => Factory.attributes_for(:live_billing_information).merge(options),
+                  :changed_billing => changed_billing,
+                  :amount => amount
+  end
+end
+
+# Once a customer is setup in the CIM their record remains.  This can cause problems with the 
+# tests if a CIM user is not deleted for some reason. This method will attempt to find
+# and delete a CIM user if needed after which the user is again setup.
+def try_cim(user, billing_information)
+  response = yield(user, billing_information)
+  if !response.success?
+    if response.params['messages']['message']['code'] == 'E00039'
+      billing_information.billable.customer_profile_id = parse_customer_profile_id(response.message)
+      response = OrderTransaction.cim_delete(billing_information.billable.customer_profile_id)
+      assert response.success?, response.message
+      response = yield(user, billing_information)
+    end
+  end
+  response
+end
+
+def try_cim_post
+  yield
+  if flash[:error] && flash[:error].include?('A duplicate record with id')
+    message = flash[:error]
+    flash.clear
+    response = OrderTransaction.cim_delete_by_customer_profile_id(parse_customer_profile_id(message))
+    assert response.success?, response.message
+    yield
+  end
+end
+
+def parse_customer_profile_id(message)
+  message[/\d+/]
+end
+
+def cleanup_cim(billing_information)
+  if billing_information.has_billing_profile?
+    response = OrderTransaction.cim_delete(billing_information.billable.customer_profile_id)
+    #assert response.success?, response.message
+  end
 end
 
 # Switch to paypal test gateway

data/test/rails_root/remote/unit/remote_order_test.rb

@@ -17,15 +17,14 @@ class RemoteOrderTest < ActiveSupport::TestCase
     context "valid credit card" do
       setup do
         @order = Order.new
-        @user, @billing = setup_live_user_and_billing
+        @user, @billing_information = setup_live_user_and_billing
       end
       teardown do
-        # Have to get rid of records in the CIM
-        OrderTransaction.cim_delete(@billing) if MuckCommerce::Configure.cim_enabled?
+        cleanup_cim(@billing_information)
       end
       should "succesfully checkout" do
         assert_difference "Order.count", 1 do
-          success, purchase_made, purchase = @order.checkout(@billing, @amount, @coupons, @options)
+          success, purchase_made, purchase = @order.checkout(@billing_information, @amount, @coupons, @options)
           assert success
           assert purchase_made
           assert purchase.success?
@@ -41,14 +40,14 @@ class RemoteOrderTest < ActiveSupport::TestCase
       # In response the gateway will return declined instead of throwing an exception
       setup do
         @order = Order.new
-        @user, @billing = setup_live_user_and_billing(:credit_card_number => MuckCommerceTestDefinitions::BRAIN_MASTERCARD)
+        @user, @billing_information = setup_live_user_and_billing(:credit_card_number => MuckCommerceTestDefinitions::BRAIN_MASTERCARD)
       end
 
       should "fail checkout" do
         assert_no_difference "Order.count" do
           assert_no_difference "OrderTransaction.count" do
             assert_raise(MuckCommerce::Exceptions::PaymentGatewayError, 'There was a problem processing your order') do
-              success, purchase_made, purchase = @order.checkout(@billing, @amount, @coupons, @options)
+              success, purchase_made, purchase = @order.checkout(@billing_information, @amount, @coupons, @options)
               assert !success
               assert !purchase_made
               assert !purchase.success?
@@ -62,13 +61,13 @@ class RemoteOrderTest < ActiveSupport::TestCase
     context "exception credit card" do
       setup do
         @order = Order.new
-        @user, @billing = setup_live_user_and_billing(:credit_card_number => MuckCommerceTestDefinitions::EXCEPTION_CARD)
+        @user, @billing_information = setup_live_user_and_billing(:credit_card_number => MuckCommerceTestDefinitions::EXCEPTION_CARD)
       end
 
       should "throw exception during checkout" do
         assert_no_difference "Order.count" do
           assert_raise(MuckCommerce::Exceptions::PaymentGatewayError, 'Invalid credit card information') do
-            @order.checkout(@billing, @amount, @coupons, @options)
+            @order.checkout(@billing_information, @amount, @coupons, @options)
           end
         end
         

data/test/rails_root/remote/unit/remote_order_transaction_test.rb

@@ -135,53 +135,92 @@ class RemoteOrderTransactionTest < ActiveSupport::TestCase
       context "management actions" do
         should " create and delete cim entry" do
           # create entry
-          response = OrderTransaction.cim_create(@billing_information)
-          puts response.message if !response.success?
-          assert response.success?
-          assert_equal @billing_information.customer_profile_id, response.reference
+          response = try_cim(@user, @billing_information) { |user, billing_information| OrderTransaction.cim_create(user, billing_information) }
+          assert response.success?, response.message
+          assert_equal @user.customer_profile_id, response.reference
           assert_equal @billing_information.customer_payment_profile_id, response.params["customer_payment_profile_id_list"]["numeric_string"]
           # delete entry
-          response = OrderTransaction.cim_delete(@billing_information)
+          response = OrderTransaction.cim_delete(@billing_information.billable.customer_profile_id)
           assert response.success?, response.message
         end
         context "cim entry actions" do
           setup do
-            response = OrderTransaction.cim_create(@billing_information)
+            response = try_cim(@user, @billing_information) { |user, billing_information| OrderTransaction.cim_create(user, billing_information) }
+            @customer_profile_id = response.params['customer_profile_id']
+            @customer_payment_profile_id = response.params["customer_payment_profile_id_list"]["numeric_string"]
             assert response.success?, response.message
+            assert @billing_information.has_billing_profile?
+            assert_equal @customer_profile_id, @billing_information.billable.customer_profile_id
+            assert_equal @customer_payment_profile_id, @billing_information.customer_payment_profile_id
           end
           teardown do
-            response = OrderTransaction.cim_delete(@billing_information)
+            cleanup_cim(@billing_information)
           end
           should "update cim" do
             # update entry
-            @billing_information.first_name = 'something'
-            @billing_information.last_name = 'else'
-            response = OrderTransaction.cim_update(@billing_information)
+            @user.email = 'somerandomthing@example.com'
+            response = OrderTransaction.cim_update(@user)
+            assert response.test?
+            assert response.success?, response.message
+            response = OrderTransaction.cim_get_customer_profile(@user.customer_profile_id)
+            assert response.test?
+            assert response.success?, response.message
+            assert_equal @user.email, response.params['profile']['email']
+          end
+          should "call cim_create_update and determine to update entry" do
+            # update entry
+            @billing_information.first_name = 'two'
+            @billing_information.last_name = 'first'
+            response = OrderTransaction.cim_create_update(@user, @billing_information)
             assert response.success?, response.message
+            response = OrderTransaction.cim_get_customer_profile(@user.customer_profile_id)
+            assert_equal @billing_information.first_name, response.params['profile']['payment_profiles']['bill_to']['first_name']
+            assert_equal @billing_information.last_name, response.params['profile']['payment_profiles']['bill_to']['last_name']
           end
           should "validate user cim profile" do
-            response = OrderTransaction.cim_validate_user(@billing_information)
-            puts response.message if !response.success?
+            assert_equal @user, @billing_information.billable
+            assert_equal @customer_profile_id, @billing_information.billable.customer_profile_id
+            assert_equal @customer_payment_profile_id, @billing_information.customer_payment_profile_id
+            response = OrderTransaction.cim_validate_customer_payment_profile(@billing_information)
+            assert response.success?, response.message
+            assert_equal I18n.translate('muck.commerce.cim_validate_profile'), response.action
+          end
+          should "get cim payment profile" do
+            response = OrderTransaction.cim_get_customer_payment_profile(@user.customer_profile_id, @billing_information.customer_payment_profile_id)
+            assert response.success?, response.message
+            assert response.params['payment_profile']
+          end
+          should "update cim payment profile" do
+            # update entry
+            @billing_information.first_name = 'payment'
+            @billing_information.last_name = 'update'
+            response = OrderTransaction.cim_update_payment_profile(@billing_information)
+            assert response.test?
+            assert response.success?, response.message
+            response = OrderTransaction.cim_get_customer_payment_profile(@user.customer_profile_id, @billing_information.customer_payment_profile_id)
+            assert response.test?
             assert response.success?, response.message
+            assert_equal @billing_information.first_name, response.params['payment_profile']['bill_to']['first_name']
+            assert_equal @billing_information.last_name, response.params['payment_profile']['bill_to']['last_name']
           end
         end
-        should "determine whether to create or update entry" do
+        should "call cim_create_update and determine to create a new entry" do
           # create entry
-          response = OrderTransaction.cim_create_update(@billing_information)
-          assert response.success?
-          # update entry
-          @billing_information.first_name = 'something'
-          @billing_information.last_name = 'else'
-          response = OrderTransaction.cim_create_update(@billing_information)
+          assert !@billing_information.has_billing_profile?
+          response = try_cim(@user, @billing_information) { |user, billing_information| OrderTransaction.cim_create_update(user, billing_information) }
+          assert response.success?, response.message
+          response = OrderTransaction.cim_get_customer_profile(@user.customer_profile_id)
           assert response.success?, response.message
+          cleanup_cim(@billing_information)
         end
       end
       context "payment actions" do
         setup do
-          response = OrderTransaction.cim_create(@billing_information)
+          response = try_cim(@user, @billing_information) { |user, billing_information| OrderTransaction.cim_create(user, billing_information) }
+          assert @billing_information.has_billing_profile?
         end
         teardown do
-          response = OrderTransaction.cim_delete(@billing_information)
+          cleanup_cim(@billing_information)
         end
         should "purchase" do
           payment = OrderTransaction.cim_purchase(1234, @billing_information, @options)
@@ -194,19 +233,6 @@ class RemoteOrderTransactionTest < ActiveSupport::TestCase
           assert_equal I18n.translate('muck.commerce.authorization_cim'), payment.action
         end
       end
-      context "validation" do
-        setup do
-          response = OrderTransaction.cim_create(@billing_information)
-        end
-        teardown do
-          response = OrderTransaction.cim_delete(@billing_information)
-        end
-        should "validate the user" do
-          response = OrderTransaction.cim_validate_user(@billing_information)
-          assert response.success?
-          assert_equal I18n.translate('muck.commerce.cim_validate_profile'), response.action
-        end
-      end
     end
      
   end