radiant-shop-extension 0.10.0 → 0.11.0

data/spec/lib/shop/tags/product_spec.rb

@@ -2,13 +2,13 @@ require 'spec/spec_helper'
 
 describe Shop::Tags::Product do
   
-  dataset :pages, :shop_products
+  dataset :pages, :shop_products, :shop_product_attachments
   
   it 'should describe these tags' do
     Shop::Tags::Product.tags.sort.should == [
-      'shop:if_products',
-      'shop:unless_products',
       'shop:products',
+      'shop:products:if_products',
+      'shop:products:unless_products',      
       'shop:products:each',
       'shop:product',
       'shop:product:id',
@@ -36,40 +36,42 @@ describe Shop::Tags::Product do
     @images   = [ images(:soft_bread_front), images(:soft_bread_back), images(:soft_bread_top) ]
   end
   
-  describe '<r:shop:if_products>' do
-    context 'success' do
+  describe '<r:shop:products>' do
+    context 'no products exist' do
       before :each do
-        mock(Shop::Tags::Helpers).current_products(anything) { @products }
+        mock(Shop::Tags::Helpers).current_products(anything) { [] }
       end
       
       it 'should render' do
-        tag = %{<r:shop:if_products>success</r:shop:if_products>}
+        tag = %{<r:shop:products>success</r:shop:products>}
         exp = %{success}
+      
         @page.should render(tag).as(exp)
       end
     end
     
-    context 'failure' do
+    context 'products exist' do
       before :each do
-        mock(Shop::Tags::Helpers).current_products(anything) { [] }
+        mock(Shop::Tags::Helpers).current_products(anything) { @products }
       end
       
-      it 'should not render' do
-        tag = %{<r:shop:if_products>failure</r:shop:if_products>}
-        exp = %{}
+      it 'should render' do
+        tag = %{<r:shop:products>success</r:shop:products>}
+        exp = %{success}
+      
         @page.should render(tag).as(exp)
       end
     end
   end
   
-  describe '<r:shop:unless_products>' do
+  describe '<r:shop:products:if_products>' do
     context 'success' do
       before :each do
-        mock(Shop::Tags::Helpers).current_products(anything) { [] }
+        mock(Shop::Tags::Helpers).current_products(anything) { @products }
       end
       
       it 'should render' do
-        tag = %{<r:shop:unless_products>success</r:shop:unless_products>}
+        tag = %{<r:shop:products:if_products>success</r:shop:products:if_products>}
         exp = %{success}
         @page.should render(tag).as(exp)
       end
@@ -77,40 +79,38 @@ describe Shop::Tags::Product do
     
     context 'failure' do
       before :each do
-        mock(Shop::Tags::Helpers).current_products(anything) { @products }
+        mock(Shop::Tags::Helpers).current_products(anything) { [] }
       end
       
       it 'should not render' do
-        tag = %{<r:shop:unless_products>failure</r:shop:unless_products>}
+        tag = %{<r:shop:products:if_products>failure</r:shop:products:if_products>}
         exp = %{}
         @page.should render(tag).as(exp)
       end
     end
   end
   
-  describe '<r:shop:products>' do
-    context 'no products exist' do
+  describe '<r:shop:unless_products>' do
+    context 'success' do
       before :each do
         mock(Shop::Tags::Helpers).current_products(anything) { [] }
       end
       
       it 'should render' do
-        tag = %{<r:shop:products>success</r:shop:products>}
+        tag = %{<r:shop:products:unless_products>success</r:shop:products:unless_products>}
         exp = %{success}
-      
         @page.should render(tag).as(exp)
       end
     end
     
-    context 'products exist' do
+    context 'failure' do
       before :each do
         mock(Shop::Tags::Helpers).current_products(anything) { @products }
       end
       
-      it 'should render' do
-        tag = %{<r:shop:products>success</r:shop:products>}
-        exp = %{success}
-      
+      it 'should not render' do
+        tag = %{<r:shop:products:unless_products>failure</r:shop:products:unless_products>}
+        exp = %{}
         @page.should render(tag).as(exp)
       end
     end
@@ -204,6 +204,7 @@ describe Shop::Tags::Product do
         it 'should render a textile filtered description' do
           tag = %{<r:shop:product:description />}
           exp = TextileFilter.filter(@product.description)
+          
           @page.should render(tag).as(exp)
         end
       end
@@ -248,6 +249,8 @@ describe Shop::Tags::Product do
         @page.should render(tag).as(exp)
       end
       
+      it 'should not be testing precision here'
+      
       it 'should render a high precision price' do
         tag = %{<r:shop:product:price precision="8"/>}
         exp = %{$1,234.34567890}

data/spec/lib/shop/tags/product_variant_spec.rb

@@ -0,0 +1,175 @@
+require 'spec/spec_helper'
+
+describe Shop::Tags::ProductVariant do
+  
+  dataset :pages, :shop_product_variants, :shop_products
+  
+  it 'should describe these tags' do
+    Shop::Tags::ProductVariant.tags.sort.should == [
+      'shop:product:variants',
+      'shop:product:variants:each',
+      'shop:product:variants:if_variants',
+      'shop:product:variants:unless_variants',
+      'shop:product:variant',
+      'shop:product:variant:id',
+      'shop:product:variant:name',
+      'shop:product:variant:sku',
+      'shop:product:variant:price'].sort
+  end
+  
+  before :all do
+    @page = pages(:home)
+  end
+  
+  context 'tags' do
+    
+    before :each do
+      @product = shop_products(:crusty_bread)
+      @variant = shop_product_variants(:mouldy_crusty_bread)
+      
+      stub(Shop::Tags::Helpers).current_product(anything) { @product }
+    end
+    
+    describe '<r:shop:product:variants>' do
+      context 'variants exist' do
+        it 'should render' do
+          tag = %{<r:shop:product:variants>success</r:shop:product:variants>}
+          exp = %{success}
+
+          @page.should render(tag).as(exp)
+        end
+      end
+      
+      context 'variants do not exist' do
+        it 'should render' do
+          @product.variants = []
+          
+          tag = %{<r:shop:product:variants>success</r:shop:product:variants>}
+          exp = %{success}
+
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+    describe '<r:shop:product:variants:if_variants>' do
+      context 'success' do
+        it 'should render' do
+          tag = %{<r:shop:product:variants:if_variants>success</r:shop:product:variants:if_variants>}
+          exp = %{success}
+          @page.should render(tag).as(exp)
+        end
+      end
+
+      context 'failure' do
+        it 'should not render' do
+          mock(Shop::Tags::Helpers).current_product_variants(anything) { [] }
+            
+          tag = %{<r:shop:product:variants:if_variants>failure</r:shop:product:variants:if_variants>}
+          exp = %{}
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+    describe '<r:shop:product:variants:unless_variants>' do
+      context 'success' do
+        it 'should render' do
+          mock(Shop::Tags::Helpers).current_product_variants(anything) { [] }
+          
+          tag = %{<r:shop:product:variants:unless_variants>success</r:shop:product:variants:unless_variants>}
+          exp = %{success}
+          @page.should render(tag).as(exp)
+        end
+      end
+
+      context 'failure' do
+        it 'should not render' do
+          tag = %{<r:shop:product:variants:unless_variants>failure</r:shop:product:variants:unless_variants>}
+          exp = %{}
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+    describe '<r:shop:product:variants:each>' do
+      context 'success' do
+        it 'should not render' do
+          tag = %{<r:shop:product:variants:each><r:product:id /></r:shop:product:variants:each>}
+          exp = @product.variants.map{ |p| p.product.id }.join('')
+          @page.should render(tag).as(exp)
+        end      
+      end
+
+      context 'failure' do
+        it 'should not render' do
+          mock(Shop::Tags::Helpers).current_product_variants(anything) { [] }
+          
+          tag = %{<r:shop:product:variants:each>failure</r:shop:product:variants:each>}
+          exp = %{}
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+    describe '<r:shop:product:variant>' do
+      context 'variant exists in the context' do
+        it 'should render' do
+          mock(Shop::Tags::Helpers).current_product_variant(anything) { @variant }
+          
+          tag = %{<r:shop:product:variant>success</r:shop:product:variant>}
+          exp = %{success}
+          @page.should render(tag).as(exp)
+        end
+      end
+
+      context 'product does not exist in the context' do
+        it 'should not render' do
+          mock(Shop::Tags::Helpers).current_product_variant(anything) { nil }
+          
+          tag = %{<r:shop:product:variant>failure</r:shop:product:variant>}
+          exp = %{}
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+    context '#attributes' do
+      before :each do
+        mock(Shop::Tags::Helpers).current_product_variant(anything) { @variant }
+      end
+      
+      describe '<r:id />' do
+        it 'should render the product id' do
+          tag = %{<r:shop:product:variant:id />}
+          exp = @variant.id.to_s
+          @page.should render(tag).as(exp)
+        end
+      end
+      describe '<r:name />' do
+        it 'should render the product name' do
+          tag = %{<r:shop:product:variant:name />}
+          exp = @variant.name
+          @page.should render(tag).as(exp)
+        end
+      end
+      describe '<r:sku />' do
+        it 'should render the product sku' do
+          tag = %{<r:shop:product:variant:sku />}
+          exp = @variant.sku
+          @page.should render(tag).as(exp)
+        end
+      end
+      describe '<r:price />' do
+        it 'should render a standard price' do
+          tag = %{<r:shop:product:variant:price />}
+          exp = Shop::Tags::Helpers.currency(@variant.price,{})
+        
+          @page.should render(tag).as(exp)
+        end
+      end
+    end
+    
+  end
+  
+end

data/spec/models/form_address_spec.rb

@@ -0,0 +1,166 @@
+require 'spec/spec_helper'
+
+describe FormAddress do
+
+  dataset :shop_orders, :pages, :forms, :shop_addresses
+  
+  before :each do
+    mock_page_with_request_and_data
+  end  
+  describe '#create' do
+    context 'addresses' do     
+      context 'sending ids with user logged in' do
+        before :each do
+          login_as :customer # Will have existing addresses due to several_items
+          @order = shop_orders(:one_item)
+          mock_valid_form_address_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 }
+            
+            @address = FormAddress.new(@form, @page)
+            result = @address.create
+            
+            shop_orders(:one_item).billing.should  === shop_addresses(:billing)
+            shop_orders(:one_item).shipping.should === shop_addresses(:shipping)
+            
+            result[:billing].should  === shop_addresses(:billing).id
+            result[:shipping].should === shop_addresses(:shipping).id
+          end
+        end
+        
+        context 'just billing' do
+          it 'should assign shipping to be billing' do
+            # Set the order to have no shipping address
+            @data[:shipping] = {}
+            @form[:extensions][:address] = { :billing => true }
+            
+            @address = FormAddress.new(@form, @page)
+            result = @address.create
+            
+            shop_orders(:one_item).billing.should  === shop_addresses(:billing)
+            shop_orders(:one_item).shipping.should === shop_addresses(:billing)
+            
+            result[:billing].should  === shop_addresses(:billing).id
+            result[:shipping].should === shop_addresses(:billing).id
+          end
+        end
+      end
+      
+      context 'sending ids without customer logged in' do
+        before :each do
+          @order = shop_orders(:one_item)
+          mock_valid_form_address_request
+        end
+        context 'both billing and shipping' do
+          it 'it should not assign those addresses' do
+            @address = FormAddress.new(@form, @page)
+            result = @address.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_address_request
+        end
+        it 'should not assign those addresses' do
+          @address = FormAddress.new(@form, @page)
+          result = @address.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
+      
+      context 'updating existing addresses' do
+        before :each do
+          login_as :customer
+          @order = shop_orders(:several_items)
+          mock_valid_form_address_request
+        end
+        it 'should keep them assigned and update them' do
+          @data[:billing]   = { :name => 'new billing' }
+          @data[:shipping]  = { :name => 'new shipping' }
+          
+          @address = FormAddress.new(@form, @page)
+          result = @address.create
+          
+          shop_addresses(:billing).name.should  === 'new billing'
+          shop_addresses(:shipping).name.should === 'new shipping'
+          
+          result[:billing].should  === shop_addresses(:billing).id
+          result[:shipping].should === shop_addresses(:shipping).id
+        end
+      end
+          
+      context 'new addresses' do
+        before :each do
+          login_as :customer
+          @order = shop_orders(:one_item)
+          mock_valid_form_address_request
+          
+          @data[:billing][:id]  = nil
+          @data[:shipping][:id] = nil
+        end
+        
+        context 'both billing and shipping sent' do
+          it 'should create new addresses' do            
+            @address = FormAddress.new(@form, @page)
+            result = @address.create
+            
+            shop_orders(:one_item).billing.name.should  === @data[:billing][:name]
+            shop_orders(:one_item).shipping.name.should === @data[:shipping][:name]
+            
+            result[:billing].should  === shop_orders(:one_item).billing.id
+            result[:shipping].should === shop_orders(:one_item).shipping.id
+          end
+        end
+        
+        context 'only billing sent' do
+          it 'should copy billing to shipping' do
+            @data[:shipping] = {}
+            
+            @address = FormAddress.new(@form, @page)
+            result = @address.create
+            
+            shop_orders(:one_item).billing.name.should  === @data[:billing][:name]
+            shop_orders(:one_item).shipping.name.should === @data[:billing][:name]
+            
+            result[:billing].should  === shop_orders(:one_item).billing.id
+            result[:shipping].should === shop_orders(:one_item).shipping.id
+          end
+        end
+        
+        context 'billing sent with same shipping' do
+          it 'should copy billing to shipping' do
+            @data[:shipping] = @data[:billing]
+            
+            @address = FormAddress.new(@form, @page)
+            result = @address.create
+            
+            shop_orders(:one_item).shipping.should      === shop_orders(:one_item).billing
+            
+            result[:billing].should  === shop_orders(:one_item).billing.id
+            result[:shipping].should === shop_orders(:one_item).shipping.id
+          end
+        end
+      end
+    end
+  end
+  
+end