spree 0.9.4 → 0.10.0.beta

data/test/functional/admin/checkouts_controller_test.rb

@@ -0,0 +1,86 @@
+require 'test_helper'
+
+class Admin::CheckoutsControllerTest < ActionController::TestCase
+  fixtures :countries, :states
+
+  context "given order" do
+    setup do
+      UserSession.create(Factory(:admin_user))
+      @order = create_complete_order
+    end
+
+    context "on GET to :show" do
+      setup do
+        @checkout =  @order.checkout
+        @params = { :id => @checkout.id, :order_id => @order.number, }
+        get :show
+      end
+
+      should_assign_to :order
+      should_assign_to :checkout
+      should_respond_with :success
+      should_render_template "show"
+
+      should "render address details" do
+        assert_select "table" do
+          assert_select "th", :text => I18n.t("billing_address")
+        end
+
+        assert_select "table" do
+          assert_select "th", :text => I18n.t("shipping_address")
+
+        end
+      end
+    end
+
+    context "on GET to :edit" do
+      setup do
+        @checkout =  @order.checkout
+        @params = { :id => @checkout.id, :order_id => @order.number, }
+
+        @checkout.bill_address.state_name = nil
+        @checkout.bill_address.state = @checkout.bill_address.country.states.last
+
+        @checkout.ship_address = Factory(:address)
+        @checkout.ship_address.country = Factory.create(:country, :states => [])
+
+        @checkout.save
+        get :edit
+      end
+
+      should_assign_to :order
+      should_assign_to :checkout
+      should_respond_with :success
+      should_render_template "edit"
+
+      should "render address details" do
+        assert_select "table" do
+          assert_select "th", :text => I18n.t("billing_address")
+          assert_select "input[id='checkout_bill_address_attributes_firstname'][value=?]", @checkout.bill_address.firstname
+          assert_select "input[id='checkout_bill_address_attributes_lastname'][value=?]", @checkout.bill_address.lastname
+          assert_select "input[id='checkout_bill_address_attributes_address1'][value=?]", @checkout.bill_address.address1
+          assert_select "input[id='checkout_bill_address_attributes_address2'][value=?]", @checkout.bill_address.address2
+          assert_select "input[id='checkout_bill_address_attributes_city'][value=?]", @checkout.bill_address.city
+          assert_select "input[id='checkout_bill_address_attributes_zipcode'][value=?]", @checkout.bill_address.zipcode
+
+          assert_select "input[id='checkout_bill_address_attributes_state_name'][disabled='disabled']"
+          assert_select "select[id='checkout_bill_address_attributes_state_id'] option[selected='selected'][value=?]", @checkout.bill_address.state.id.to_s
+
+        end
+
+        assert_select "table" do
+          assert_select "th", :text => I18n.t("shipping_address")
+          assert_select "input[id='checkout_ship_address_attributes_firstname'][value=?]", @checkout.ship_address.firstname
+          assert_select "input[id='checkout_ship_address_attributes_lastname'][value=?]", @checkout.ship_address.lastname
+          assert_select "input[id='checkout_ship_address_attributes_address1'][value=?]", @checkout.ship_address.address1
+          assert_select "input[id='checkout_ship_address_attributes_address2'][value=?]", @checkout.ship_address.address2
+          assert_select "input[id='checkout_ship_address_attributes_city'][value=?]", @checkout.ship_address.city
+          assert_select "input[id='checkout_ship_address_attributes_zipcode'][value=?]", @checkout.ship_address.zipcode
+
+          assert_select "input[id='checkout_ship_address_attributes_state_name'][value=?]", @checkout.ship_address.state_name
+          assert_select "select[id='checkout_ship_address_attributes_state_id'][disabled='disabled']"
+        end
+      end
+    end
+  end
+end

data/test/functional/admin/configurations_controller_test.rb

@@ -0,0 +1,41 @@
+require 'test_helper'
+
+class Admin::ConfigurationsControllerTest < ActionController::TestCase
+  setup do
+    UserSession.create(Factory(:admin_user))
+  end
+
+  context "on GET to :index" do
+    setup do
+      get :index
+    end
+
+    should_respond_with :success
+    should_render_template "index"
+
+    should "display a Mail Server Settings link" do
+      assert_select('a[href=?]', admin_mail_settings_path)
+    end
+
+    should "display a Tax Categories link" do
+      assert_select 'a[href=?]', admin_tax_categories_path
+    end
+
+    should "display a Zones link" do
+      assert_select 'a[href=?]', admin_zones_path
+    end
+
+    should "display a States link" do
+      assert_select 'a[href=?]', admin_country_states_path(214)
+    end
+
+    should "display a Payment Methods link" do
+      assert_select 'a[href=?]', admin_payment_methods_path
+    end
+
+    should "display an Inventory Settings link" do
+      assert_select 'a[href=?]', admin_inventory_settings_path
+    end
+
+  end
+end

data/test/functional/admin/line_items_controller_test.rb

@@ -0,0 +1,177 @@
+require 'test_helper'
+
+class Admin::LineItemsControllerTest < ActionController::TestCase
+  context "given order" do
+    setup do
+      UserSession.create(Factory(:admin_user))
+      @order = Factory(:order)
+      @order.checkout.shipping_method = Factory(:shipping_method)
+      @order.checkout.save
+      @order.shipment.reload
+      @variant = Factory(:variant)
+    end
+
+    context "on POST to :create" do
+      setup do
+        post :create, {
+             "order_id" => @order.number,
+             "line_item" => { "quantity" => 3,
+                               "variant_id" => @variant.id}
+        }
+        @order.reload
+      end
+
+      should_assign_to :order
+      should_assign_to :line_item
+      should_respond_with :success
+
+      should_change("Order.line_items.size", :by => 1) { @order.line_items.size }
+
+      should_change("@order.total", :by => (19.99 * 3)) { @order.item_total.to_f }
+
+      should "render a form" do
+        assert_select "form[id='edit_order_#{@order.id}']" do
+          @order.reload.line_items.each do |line_item|
+            assert_select "tr[id='line_item_#{line_item.id}']"
+          end
+
+        end
+      end
+
+    end
+
+    context "with existing line_item" do
+      setup do
+        @line_item = Factory(:line_item, :variant => @variant, :order => @order, :price => 19.99, :quantity => 3)
+        @order.line_items.reload
+        @order.update_totals
+      end
+
+      context "on POST to :create" do
+        setup do
+          post :create, {
+               "order_id" => @order.number,
+               "line_item" => { "quantity" => 3,
+                                 "variant_id" => @variant.id}
+          }
+          @order.reload
+        end
+
+        should_assign_to :order
+        should_respond_with :success
+
+        should_not_change("Order.line_items.size") { @order.line_items.size }
+
+        should_change("@order.line_items.total", :by => (19.99 * 3)) { @order.line_items.total.to_f }
+        should_change("@order.total", :by => (19.99 * 3)) { @order.item_total.to_f }
+
+        should "render a form" do
+          assert_select "form[id='edit_order_#{@order.id}']" do
+            @order.reload.line_items.each do |line_item|
+              assert_select "tr[id='line_item_#{line_item.id}']"
+            end
+
+          end
+        end
+
+      end
+
+      context "on PUT to :update with quantity = 2" do
+        setup do
+
+          put :update, {
+            "id" => @line_item.id,
+            "order_id" => @order.number,
+            "line_item" => {
+              "quantity" => 2}
+          }
+
+          @order.line_items.reload
+          @order.update_totals
+        end
+
+        should_assign_to :order
+        should_assign_to :line_item
+        should_respond_with :success
+
+        should_not_change("Order.line_items.size") { @order.line_items.size }
+
+        should_change("@order.line_items.total", :to => (19.99 * 2)) { @order.line_items.total.to_f }
+        should_change("@order.total", :to => (19.99 * 2)) { @order.total.to_f }
+
+        should "render a form" do
+          assert_select "form[id='edit_order_#{@order.id}']" do
+            @order.reload.line_items.each do |line_item|
+              assert_select "tr[id='line_item_#{line_item.id}']"
+            end
+
+          end
+        end
+      end
+
+      context "on PUT to :update with quantity = 0" do
+        setup do
+          put :update, {
+            "id" => @line_item.id,
+            "order_id" => @order.number,
+            "line_item" => {
+              "quantity" => 0}
+          }
+
+          @order.line_items.reload
+          @order.update_totals
+        end
+
+        should_assign_to :order
+        should_assign_to :line_item
+        should_respond_with :success
+
+        should_change("Order.line_items.size", :by => -1) { @order.line_items.size }
+
+        should_change("@order.line_items.total", :by => (19.99 * -3)) { @order.line_items.total.to_f }
+        should_change("@order.total", :by => (19.99 * -3)) { @order.total.to_f }
+
+        should "render a form" do
+          assert_select "form[id='edit_order_#{@order.id}']" do
+            @order.reload.line_items.each do |line_item|
+              assert_select "tr[id='line_item_#{line_item.id}']"
+            end
+
+          end
+        end
+      end
+
+      context "on DELETE to :destroy" do
+        setup do
+          delete :destroy, {
+            "id" => @line_item.id,
+            "order_id" => @order.number
+          }
+
+          @order.line_items.reload
+          @order.update_totals
+
+          should_assign_to :order
+          should_assign_to :line_item
+          should_respond_with :success
+
+          should_change("Order.line_items.size", :by => -1) { @order.line_items.size }
+
+          should_change("@order.line_items.total", :by => (19.99 * -3)) { @order.line_items.total.to_f }
+          should_change("@order.total", :by => (19.99 * -3)) { @order.total.to_f }
+
+          should "render a form" do
+            assert_select "form[id='edit_order_#{@order.id}']" do
+              @order.reload.line_items.each do |line_item|
+                assert_select "tr[id='line_item_#{line_item.id}']"
+              end
+
+            end
+          end
+        end
+      end
+
+    end
+
+  end
+end

data/test/functional/admin/payments_controller_test.rb

@@ -0,0 +1,120 @@
+require 'test_helper'
+
+class Admin::PaymentsControllerTest < ActionController::TestCase
+  fixtures :payment_methods, :countries
+
+  context "given order" do
+    setup do
+      Spree::Config.set(:auto_capture => true) 
+      UserSession.create(Factory(:admin_user))
+      create_new_order_v2
+      @order.reload
+      # Add a charge to create an outstanding balance on the order so new payments will validate
+      @charge = Factory(:charge, :amount => 2.99, :order => @order)
+      @order.update_totals!
+      @order.reload
+    end
+
+    context "on POST to :create" do
+
+      context "entering a new creditcard" do
+        setup do
+          @params = {
+            :payment_type => 'creditcard_payment',
+            :order_id => @order.id, 
+            :card => 'new',
+            :payment => {
+              :amount => '2.99',
+              :payment_method_id => payment_methods(:bogus_test).id.to_s
+            },
+            :payment_source => {Gateway.current.id.to_s => Factory.attributes_for(:creditcard)}
+          }
+          post :create, @params
+        end
+        should_create :payment
+        should_create :creditcard_txn
+        should_respond_with :redirect
+        should "create payment with the right attributes" do
+          assert_equal 1, @order.payments.count
+          assert_equal 2.99, @order.payments.last.txns.last.amount.to_f
+        end
+      end
+
+      context "selected existing creditcard with CIM gateway" do
+        setup do
+          PaymentMethod.update_all(:active => false)
+          payment_methods(:authorize_net_cim_test).update_attribute(:active, true)
+          @creditcard = Factory(:creditcard, :gateway_customer_profile_id => '123', :gateway_payment_profile_id => '456')
+          @params = {
+            :order_id => @order.id, 
+            :card => @creditcard.id,
+            :payment => {
+              :amount => '2.99',
+              :payment_method_id => payment_methods(:authorize_net_cim_test).id.to_s
+            }
+          }
+          post :create, @params
+          @order.reload
+          @payment = @order.payments.first
+        end
+        should_create :payment
+        should_respond_with :redirect
+        should "create payment with the right attributes" do
+          assert_equal @creditcard.id, @payment.source.id
+          assert_equal 2.99, @payment.txns.last.amount.to_f
+        end
+      end
+      
+
+      context "for cheque payment" do
+        setup do
+          @params = {
+            :order_id => @order.id, 
+            :payment => {
+              :payment_method_id => payment_methods(:check_payment_method).id.to_s, 
+              :amount => '1.99'
+            }
+          }
+          post :create, @params
+        end
+        should_create :payment
+        should_respond_with :redirect
+      end
+
+    end
+
+    context "on PUT to :finalize" do
+      setup do
+        @payment = Factory(:payment, :source => nil, :payment_method => payment_methods(:check_payment_method))
+        @payment.process!
+        put :finalize, {
+          :order_id => @order.id,
+          :id => @payment.id
+        }
+        @payment.reload
+      end
+      
+      should "move payment from checkout to order" do
+        assert_equal Order, @payment.payable.class
+      end
+    end
+
+    context "on PUT to :fire for capture event" do
+      setup do
+        @payment = Factory(:payment, :source => Factory(:creditcard), :payment_method => Gateway.current)
+        @payment.process!
+        put :fire, {
+          :order_id => @order.id,
+          :id => @payment.id,
+          :e => 'capture'
+        }
+        @payment.reload
+      end
+
+      should "move payment from checkout to order" do
+        assert_equal Order, @payment.payable.class
+      end
+    end
+
+  end
+end

data/test/functional/admin/product_groups_controller_test.rb

@@ -0,0 +1,41 @@
+require 'test_helper'
+
+class Admin::ProductGroupsControllerTest < ActionController::TestCase
+  context "ProductGroupsController" do
+    setup do
+      UserSession.create(Factory(:admin_user))
+      @pg = Factory(:product_group)
+    end
+
+    context "on GET to :index" do
+      setup do
+        get :index
+      end
+
+      should_respond_with :success
+    end
+
+    context "on GET to :show" do
+      setup do
+        get :show, {:id => @pg.id}
+      end
+
+      should_respond_with :success
+    end
+
+    # TODO: test the addition of non-ordering scopes
+    context "on POST to :create" do
+      setup do
+        post :create, {
+            "product_group"=>{ "name"=>"TestableProductGroup", 
+                               "order_scope"=>"descend_by_popularity" 
+            }
+        }
+      end
+
+      should_respond_with :redirect
+      should_change("ProductGroup.count", :by => 1) { ProductGroup.count }
+      should_change("ProductScope.count", :by => 1) { ProductScope.count }
+    end
+  end
+end