radiant-shop-extension 0.10.0 → 0.11.0

data/spec/models/form_checkout_spec.rb

@@ -8,165 +8,29 @@ describe FormCheckout do
     mock_page_with_request_and_data
   end
   describe '#create' do
-    context 'addresses' do
-      
-      context 'sending ids with user logged in' do
+    context 'gateway' do
+      context 'order has no billing' do
         before :each do
-          login_as :customer # Will have existing addresses due to several_items
-          @order = shop_orders(:one_item)
+          login_as :customer
+          @order = shop_orders(:empty)
           mock_valid_form_checkout_request
         end
         
-        context 'both billing and shipping' do
-          it 'should assign that address to the order billing and shipping' do
-            @data[:billing]   = { :id => shop_orders(:several_items).billing.id }
-            @data[:shipping]  = { :id => shop_orders(:several_items).shipping.id }
-            
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).billing.should  === shop_addresses(:billing)
-            shop_orders(:one_item).shipping.should === shop_addresses(:shipping)
-            
-            result[:billing].should  === true
-            result[:shipping].should === true
-          end
-        end
-        
-        context 'just billing' do
-          it 'should assign shipping to be billing' do
-            # Set the order to have no shipping address
-            @data[:shipping] = {}
-            
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).billing.should  === shop_addresses(:billing)
-            shop_orders(:one_item).shipping.should === shop_addresses(:billing)
-            
-            result[:billing].should  === true
-            result[:shipping].should === true
-          end
-        end
-      end
-      
-      context 'sending ids without customer logged in' do
-        before :each do
-          @order = shop_orders(:one_item)
-          mock_valid_form_checkout_request
-        end
-        context 'both billing and shipping' do
-          it 'it should not assign those addresses' do
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).billing.should  be_nil
-            shop_orders(:one_item).shipping.should be_nil
-            
-            result[:billing].should  === false
-            result[:shipping].should === false
-          end
-        end
-      end
-      
-      context 'sending ids as a bad customer' do
-        before :each do
-          login_as :bad_customer
-          @order = shop_orders(:one_item)
-          mock_valid_form_checkout_request
-        end
-        it 'should not assign those addresses' do
+        it 'should return the standard response' do
           @checkout = FormCheckout.new(@form, @page)
           result = @checkout.create
-          
-          shop_orders(:one_item).billing.should  be_nil
-          shop_orders(:one_item).shipping.should be_nil
-          
-          result[:billing].should  === false
-          result[:shipping].should === false
+                    
+          result[:gateway].should be_nil
+          result[:payment].should be_nil
+          result[:card].should    be_nil
+          result[:message].should be_nil
         end
       end
       
-      context 'updating existing addresses' do
+      context 'configured correctly' do
         before :each do
           login_as :customer
           @order = shop_orders(:several_items)
-          mock_valid_form_checkout_request
-        end
-        it 'should keep them assigned and update them' do
-          @data[:billing]   = { :name => 'new billing' }
-          @data[:shipping]  = { :name => 'new shipping' }
-          
-          @checkout = FormCheckout.new(@form, @page)
-          result = @checkout.create
-          
-          shop_addresses(:billing).name.should  === 'new billing'
-          shop_addresses(:shipping).name.should === 'new shipping'
-          
-          result[:billing].should  === true
-          result[:shipping].should === true
-        end
-      end
-          
-      context 'new addresses' do
-        before :each do
-          login_as :customer
-          @order = shop_orders(:one_item)
-          mock_valid_form_checkout_request
-          
-          @data[:billing][:id]  = nil
-          @data[:shipping][:id] = nil
-        end
-        
-        context 'both billing and shipping sent' do
-          it 'should create new addresses' do            
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).billing.name.should  === @data[:billing][:name]
-            shop_orders(:one_item).shipping.name.should === @data[:shipping][:name]
-            
-            result[:billing].should  === true
-            result[:shipping].should === true
-          end
-        end
-        
-        context 'only billing sent' do
-          it 'should copy billing to shipping' do
-            @data[:shipping] = {}
-            
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).billing.name.should  === @data[:billing][:name]
-            shop_orders(:one_item).shipping.name.should === @data[:billing][:name]
-            
-            result[:billing].should  === true
-            result[:shipping].should === true
-          end
-        end
-        
-        context 'billing sent with same shipping' do
-          it 'should copy billing to shipping' do
-            @data[:shipping] = @data[:billing]
-            
-            @checkout = FormCheckout.new(@form, @page)
-            result = @checkout.create
-            
-            shop_orders(:one_item).shipping.should      === shop_orders(:one_item).billing
-            
-            result[:billing].should  === true
-            result[:shipping].should === true
-          end
-        end
-      end
-    end
-    
-    context 'gateway' do
-        context 'configured correctly' do
-        before :each do
-          login_as :customer
-          @order = shop_orders(:one_item)
           
           mock_valid_form_checkout_request
         end
@@ -197,26 +61,33 @@ describe FormCheckout do
           @order.payment.card_type.should   === @data[:card][:type]
           @order.payment.amount.should      === @order.price
         end
+        
+        it 'should assign session shop_order to nil' do
+          mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
+          @checkout = FormCheckout.new(@form, @page)
+          result = @checkout.create
+          
+          result[:session][:shop_order].should be_nil
+        end
       end
       
       context 'configured incorrectly' do
         before :each do
           login_as :customer
-          @order = shop_orders(:one_item)
+          @order = shop_orders(:several_items)
           
           mock_valid_form_checkout_request
         end
         context 'no gateway' do
           it 'should return gateway false' do
-            mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
             @form[:extensions][:checkout][:gateway] = nil
             
             @checkout = FormCheckout.new(@form, @page)
             result = @checkout.create
             
             result[:gateway].should === false
-            result[:card].should    === false
-            result[:payment].should === false
+            result[:card].should    be_nil
+            result[:payment].should be_nil
             
             @order.payment.should be_nil
           end
@@ -224,15 +95,14 @@ describe FormCheckout do
         
         context 'no card' do
           it 'should return card and payment false' do
-            mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
             @data[:card] = nil
             
             @checkout = FormCheckout.new(@form, @page)
             result = @checkout.create
             
             result[:gateway].should === true
-            result[:card].should    === false
-            result[:payment].should === false
+            result[:card].should    be_nil
+            result[:payment].should be_nil
             
             @order.payment.should be_nil
           end
@@ -247,7 +117,7 @@ describe FormCheckout do
             
             result[:gateway].should === true
             result[:card].should    === false
-            result[:payment].should === false
+            result[:payment].should be_nil
             
             @order.payment.should be_nil
           end
@@ -263,7 +133,7 @@ describe FormCheckout do
       context 'mail configured' do
         before :each do
           login_as :customer
-          @order = shop_orders(:one_item)
+          @order = shop_orders(:several_items)
           
           mock_valid_form_checkout_request
         end
@@ -293,7 +163,7 @@ describe FormCheckout do
       context 'mail not configured' do
         before :each do
           login_as :customer
-          @order = shop_orders(:one_item)
+          @order = shop_orders(:several_items)
           
           mock_valid_form_checkout_request
           @form[:extensions][:checkout][:mail] = nil

data/spec/models/shop_address_spec.rb

@@ -0,0 +1,81 @@
+require 'spec/spec_helper'
+
+describe ShopAddress do
+  
+  dataset :shop_addresses, :shop_orders
+
+  describe 'relationships' do
+    before :each do
+      @address = shop_addresses(:billing)
+    end
+    
+    context 'billing orders' do
+      before :each do
+        @order = shop_orders(:empty)
+      end
+      it 'should have many' do
+        @address.billings << @order
+        @address.billings.include?(@order).should === true
+      end
+    end
+    
+    context 'shipping orders' do
+      before :each do
+        @order = shop_orders(:empty)
+      end
+      it 'should have many' do
+        @address.shippings << @order
+        @address.shippings.include?(@order).should === true
+      end
+    end
+  end
+  
+  describe 'validations' do
+    before :each do
+      @address = shop_addresses(:billing)
+    end
+    context 'unit' do
+      it 'should not require' do
+        @address.unit = nil
+        @address.valid?.should === true
+      end
+    end
+    context 'name' do
+      it 'should require' do
+        @address.name = nil
+        @address.valid?.should === false
+      end
+    end
+    context 'street' do
+      it 'should require' do
+        @address.street = nil
+        @address.valid?.should === false        
+      end
+    end
+    context 'city' do
+      it 'should require' do
+        @address.city = nil
+        @address.valid?.should === false        
+      end
+    end
+    context 'postcode' do
+      it 'should require' do
+        @address.postcode = nil
+        @address.valid?.should === false        
+      end
+    end
+    context 'state' do
+      it 'should require' do
+        @address.state = nil
+        @address.valid?.should === false        
+      end
+    end
+    context 'country' do
+      it 'should require' do
+        @address.country = nil
+        @address.valid?.should === false        
+      end
+    end
+  end
+  
+end

data/spec/models/shop_category_spec.rb

@@ -1,58 +1,141 @@
 require 'spec/spec_helper'
 
 describe ShopCategory do
-  dataset :shop_categories
-  
-  before(:each) do
-    @category = shop_categories(:bread)
-    @categories = [
-      shop_categories(:bread),
-      shop_categories(:milk)
-    ]
-  end
+  dataset :shop_categories, :users
   
-  context 'attributes' do
+  describe 'relationships' do
+    before :each do
+      @category = shop_categories(:bread)
+    end
     
-    it 'should have a title' do
-      @category.name.should == 'bread'
+    it 'should have a page Page' do
+      @category.page.is_a?(Page).should be_true
     end
     
-    it 'should have a handle' do
-      @category.handle.should == 'bread'
+    it 'should have a created_by User' do
+      @category.created_by = users(:admin)
+      @category.created_by.is_a?(User).should be_true
     end
     
-    it 'should have a position' do
-      @category.position.should == 1
+    it 'should have a updated_by User' do
+      @category.updated_by = users(:admin)
+      @category.updated_by.is_a?(User).should be_true
     end
     
-    it 'should have a products array' do
-      @category.products.class.should == Array
+    it 'should have a product_layout Layout' do
+      @category.product_layout.is_a?(Layout).should be_true
+    end
+  end
+  
+  describe 'validations' do
+    before :each do
+      @category = shop_categories(:bread)
     end
     
+    context 'page' do
+      it 'should require a page' do
+        @category.page = nil
+        @category.valid?.should be_false
+      end
+    end 
   end
   
-  context 'validation' do
+  describe 'methods' do
+    before :each do
+      @category = shop_categories(:bread)
+    end
     
-    it 'should generate a valid handle on validation' do
-      @product = ShopCategory.new({ :name => 'delicious_ _:_;_=_+_._~_bread' })
-      @product.valid? == true
-      @product.handle.should == 'delicious_______________bread'
+    context '#name' do
+      it 'should return page title' do
+        @category.name.should === @category.page.title
+      end
+    end
+    
+    context '#handle' do
+      it 'should return a handle formatted url' do
+        @category.handle.should === ShopProduct.to_sku(@category.page.url)
+      end
+    end
+    
+    context 'url' do
+      it 'should return a standard url path' do
+        @category.url.should === @category.page.url
+      end
+    end
+    
+    context 'slug' do
+      it 'should return the page slug' do
+        @category.slug.should === @category.page.slug
+      end
+    end
+    
+    context 'to_json' do
+      it 'should overload standard to_json' do
+        pending 'not sure how to write this'
+        # @category.to_json.should === ShopCategory.params
+      end
     end
     
-  end
-
-  it 'should find a category by handle' do
-    ShopCategory.find_by_handle(@category.handle).should == @category
   end
   
-  context 'Class Methods' do
+  describe 'filters' do
+    before :each do
+      @category = shop_categories(:bread)
+    end
     
-    describe '#attrs' do
-      it 'should have a set of standard parameters' do
-        ShopCategory.attrs.should === [ :id, :handle, :description, :created_at, :updated_at ]
+    context 'handle' do
+      context 'slug' do
+        it 'should generate on validation' do
+          @category.page.slug = 'delicious_ _:_;_=_+_._~_bread'
+          @category.valid?
+          @category.page.slug.should === 'delicious_______________bread'
+        end
+      end
+      context 'breadcrumb' do
+        context 'has not been set' do
+          it 'should generate from the slug on validation' do
+            @category.page.slug = 'delicious_ _:_;_=_+_._~_bread'
+            @category.page.breadcrumb = nil
+            @category.valid?
+            @category.page.breadcrumb.should === 'delicious_______________bread'          
+          end
+        end
+        context 'has been set' do
+          it 'should generate on validation' do
+            @category.page.breadcrumb = 'delicious_ _:_;_=_+_._~_bread'
+            @category.valid?
+            @category.page.breadcrumb.should === 'delicious_______________bread'
+          end        
+        end
       end
     end
     
+    context 'product layout' do
+      it 'should select on validation' do
+        @category = shop_categories(:bread)
+        @category.product_layout_id = nil
+        @category.valid?
+        @category.product_layout.should === Layout.find_by_name(Radiant::Config['shop.product_layout'])
+      end
+    end
+  end
+  
+  describe '#attrs' do
+    it 'should return attribute set' do
+      ShopCategory.attrs.should === [ :id, :product_layout_id, :page_id, :created_at, :updated_at ]
+    end
+  end
+  
+  describe '#methds' do
+    it 'should return method set' do
+      ShopCategory.methds.should === [ :name, :description, :handle, :url, :created_at, :updated_at ]
+    end
+  end
+  
+  describe '#params' do
+    it 'should return parameter set' do
+      ShopCategory.params.should === { :only => ShopCategory.attrs, :methods => ShopCategory.methds }
+    end
   end
   
 end