radiant-shop-extension 0.11.6 → 0.90.0

data/spec/models/form_checkout_spec.rb

@@ -7,6 +7,7 @@ describe FormCheckout do
   before :each do
     mock_page_with_request_and_data
   end
+  
   describe '#create' do
     context 'gateway' do
       context 'order has no billing' do
@@ -17,7 +18,7 @@ describe FormCheckout do
         end
         
         it 'should return the standard response' do
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           result = @checkout.create
                     
           result[:gateway].should be_nil
@@ -36,7 +37,7 @@ describe FormCheckout do
         end
         
         it 'should assign ActiveMerchant to testing mode' do
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           @checkout.create
           
           ActiveMerchant::Billing::Base.mode.should === :test
@@ -44,7 +45,7 @@ describe FormCheckout do
         
         it 'should return true for payment results' do
           mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           result = @checkout.create
           
           result[:gateway].should === true
@@ -54,7 +55,7 @@ describe FormCheckout do
         
         it 'should assign a payment object' do
           mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           result = @checkout.create
           
           @order.payment.card_number.should === "XXXX-XXXX-XXXX-1"
@@ -64,10 +65,8 @@ describe FormCheckout do
         
         it 'should assign session shop_order to nil' do
           mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { true }
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           result = @checkout.create
-          
-          result[:session][:shop_order].should be_nil
         end
       end
       
@@ -80,9 +79,9 @@ describe FormCheckout do
         end
         context 'no gateway' do
           it 'should return gateway false' do
-            @form[:extensions][:checkout][:gateway] = nil
+            @form[:extensions][:bogus_checkout][:gateway] = nil
             
-            @checkout = FormCheckout.new(@form, @page)
+            @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
             result = @checkout.create
             
             result[:gateway].should === false
@@ -97,7 +96,7 @@ describe FormCheckout do
           it 'should return card and payment false' do
             @data[:card] = nil
             
-            @checkout = FormCheckout.new(@form, @page)
+            @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
             result = @checkout.create
             
             result[:gateway].should === true
@@ -112,7 +111,7 @@ describe FormCheckout do
           it 'should return card and payment false' do
             mock.instance_of(ActiveMerchant::Billing::CreditCard).valid? { false }
             
-            @checkout = FormCheckout.new(@form, @page)
+            @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
             result = @checkout.create
             
             result[:gateway].should === true
@@ -138,20 +137,27 @@ describe FormCheckout do
           mock_valid_form_checkout_request
         end
         context 'payment successful' do
-          it 'should configure mail to be sent' do
-            @checkout = FormCheckout.new(@form, @page)
-            @checkout.create
+          before :each do
+            @checkout = FormCheckout.new(@form,@page,@form[:extensions][:bogus_checkout])
+          end
+          it 'should configure order and invoice to be sent' do
+            @config = @form[:extensions][:bogus_checkout][:extensions]
+            @config[:invoice].merge!({
+              :to => @order.billing.email
+            })
+            
+            @form_mail = Object.new
+            mock(@form_mail).create.times(2) { true }
+            mock(FormMail).new(@form,@page,@config[:order]) { @form_mail }
+            mock(FormMail).new(@form,@page,@config[:invoice]) { @form_mail }
             
-            @form[:extensions][:mail][:to].should         === shop_addresses(:billing).email
-            @form[:extensions][:mail][:recipient].should  === shop_addresses(:billing).email
-            @form[:extensions][:mail][:subject].should    === @form[:extensions][:checkout][:mail][:subject]
-            @form[:extensions][:mail][:bcc].should        === @form[:extensions][:checkout][:mail][:bcc]
+            @checkout.create
           end
         end
         
         context 'payment unsuccesful' do
           it 'should not configure mail to be sent' do
-            @checkout = FormCheckout.new(@form, @page)
+            @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
             stub(@checkout).success? { false }
             @checkout.create
             
@@ -166,13 +172,13 @@ describe FormCheckout do
           @order = shop_orders(:several_items)
           
           mock_valid_form_checkout_request
-          @form[:extensions][:checkout][:mail] = nil
+          @form[:extensions][:bogus_checkout][:extensions] = nil
         end
         it 'should not configure mail to be sent' do
-          @checkout = FormCheckout.new(@form, @page)
+          @checkout = FormCheckout.new(@form, @page, @form[:extensions][:bogus_checkout])
           result = @checkout.create
           
-          @form[:extensions][:mail].should be_nil
+          do_not_allow(FormMail).new(@form,@page,anything)
         end
       end
     end

data/spec/models/shop_category_spec.rb

@@ -25,14 +25,6 @@ describe ShopCategory do
     it 'should have a product_layout Layout' do
       @category.product_layout.is_a?(Layout).should be_true
     end
-    
-    it 'should have many discountables' do
-      @category.discountables.is_a?(Array).should be_true
-    end
-    
-    it 'should have many discounts' do
-      @category.discounts.is_a?(Array).should be_true
-    end
   end
   
   describe 'validations' do
@@ -105,14 +97,14 @@ describe ShopCategory do
             @category.page.slug = 'delicious_ _:_;_=_+_._~_bread'
             @category.page.breadcrumb = nil
             @category.valid?
-            @category.page.breadcrumb.should === 'delicious_______________bread'          
+            @category.page.breadcrumb.should === @category.page.title
           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'
+            @category.page.breadcrumb.should === 'delicious_ _:_;_=_+_._~_bread'
           end        
         end
       end

data/spec/models/shop_line_item_spec.rb

@@ -2,49 +2,86 @@ require 'spec/spec_helper'
 
 describe ShopLineItem do
   
-  dataset :shop_line_items, :shop_products, :shop_discountables
-
-  it 'should calculate a weight' do
-    shop_line_items(:one).weight.should === (shop_line_items(:one).item.weight * shop_line_items(:one).quantity).to_f
-    shop_line_items(:two).weight.should === (shop_line_items(:two).item.weight * shop_line_items(:two).quantity).to_f
+  dataset :shop_line_items, :shop_products
+  
+  context 'methods' do
+  
+    describe '#weight' do
+      it 'should multiple weight by quanttiy' do
+        shop_line_items(:one).weight.should === (shop_line_items(:one).item.weight * shop_line_items(:one).quantity).to_f
+        shop_line_items(:two).weight.should === (shop_line_items(:two).item.weight * shop_line_items(:two).quantity).to_f
+      end
+    end
+  
+    describe '#price' do
+      it 'should multiple price by quantity' do
+        shop_line_items(:one).price.should === (shop_line_items(:one).item_price * shop_line_items(:one).quantity).to_f
+        shop_line_items(:two).price.should === (shop_line_items(:two).item_price * shop_line_items(:two).quantity).to_f
+      end
+      it 'should not inherit the changes of a items base price' do
+        shop_line_items(:one).price.to_f.should === shop_line_items(:one).item.price.to_f
+        shop_line_items(:one).item.update_attribute(:price, 100.00)
+        shop_line_items(:one).price.to_f.should_not === shop_line_items(:one).item.price.to_f
+      end
+    end
+  
+    describe '#to_json' do
+      it 'should overload standard to_json' do
+        pending 'not sure how to write this'
+      end
+    end
+    
   end
+  
+  context 'filters' do
+    
+    describe '#adjust_quantity' do
+      it 'should adjust quantity to 1 if less than' do
+        s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 0 })
+        s.valid?
+        s.quantity.should === 1
 
-  it 'should calculate the price' do
-    shop_line_items(:one).price.should === (shop_line_items(:one).item.price * shop_line_items(:one).quantity).to_f
-    shop_line_items(:two).price.should === (shop_line_items(:two).item.price * shop_line_items(:two).quantity).to_f
+        s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 1 })
+        s.valid?
+        s.quantity.should === 1
+
+        s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => -100 })
+        s.valid?
+        s.quantity.should === 1
+
+        s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 100 })
+        s.valid?
+        s.quantity.should === 100
+      end
+    end
+    
+    describe '#set_price' do
+      it 'should assign the items price to item_price before validations' do
+        s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 1 })
+        s.item_price.should be_nil
+        s.valid?
+        s.item_price.should === shop_products(:crusty_bread).price
+      end
+    end
+    
   end
   
-  context 'with discount' do
-    it 'should calculate a reduced price' do
-      price = shop_line_items(:one).price
-      ShopDiscountable.create(:discount => shop_discounts(:ten_percent), :discounted => shop_line_items(:one))
-      shop_line_items(:one).price.should === (price-price*0.1)
+  describe '#attrs' do
+    it 'should return attribute set' do
+      ShopLineItem.attrs.should === [ :id, :quantity ]
     end
   end
   
-  it 'should have a set of standard parameters' do
-    ShopLineItem.params.should === [
-      :id,
-      :quantity
-    ]
+  describe '#methds' do
+    it 'should return method set' do
+      ShopLineItem.methds.should === [ :price, :weight ]
+    end
   end
   
-  it 'should adjust quantity to 1 if less than' do
-    s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 0 })
-    s.valid?
-    s.quantity.should === 1
-    
-    s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 1 })
-    s.valid?
-    s.quantity.should === 1
-    
-    s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => -100 })
-    s.valid?
-    s.quantity.should === 1
-    
-    s = ShopLineItem.new({ :item => shop_products(:crusty_bread), :quantity => 100 })
-    s.valid?
-    s.quantity.should === 100
+  describe '#params' do
+    it 'should return parameter set' do
+      ShopLineItem.params.should === { :only => ShopLineItem.attrs, :methods => ShopLineItem.methds }
+    end
   end
   
 end

data/spec/models/shop_order_spec.rb

@@ -1,8 +1,8 @@
 require 'spec/spec_helper'
-
+  
 describe ShopOrder do
-
-  dataset :shop_orders, :shop_line_items, :shop_products, :shop_packages, :shop_payments
+  
+  dataset :shop_orders, :shop_line_items, :shop_products, :shop_payments
   
   describe '#quantity' do
     it 'should return the total items' do
@@ -133,11 +133,11 @@ describe ShopOrder do
       end
       context 'type and quantity passed' do
         it 'should assign the new type and new quantity' do
-          @order.add!(shop_packages(:all_bread).id, 2, 'ShopPackage')
+          @order.add!(@product.id, 2, 'ShopProductAlternative')
           
           @order.line_items.count.should           === 1
           @order.quantity.should                   === 2
-          @order.line_items.first.item_type.should === 'ShopPackage'
+          @order.line_items.first.item_type.should === 'ShopProductAlternative'
         end
       end
     end

data/spec/models/shop_product_attachment_spec.rb

@@ -41,6 +41,7 @@ describe ShopProductAttachment do
     
     describe '#url' do
       it 'should return its assets url' do
+        stub(@product_image).image.stub!.url { 'url' }
         @product_image.url.should === @product_image.image.url
       end
     end

data/spec/models/shop_product_spec.rb

@@ -1,7 +1,7 @@
 require 'spec/spec_helper'
 
 describe ShopProduct do
-  dataset :shop_products, :shop_line_items, :shop_discountables
+  dataset :shop_products, :shop_line_items
   
   describe 'relationships' do
     before :each do
@@ -38,30 +38,6 @@ describe ShopProduct do
       @product.images.is_a?(Array).should be_true
     end
     
-    it 'should have many packings' do
-      @product.packings.is_a?(Array).should be_true
-    end
-    
-    it 'should have many packages' do
-      @product.packages.is_a?(Array).should be_true
-    end
-    
-    it 'should have many related' do
-      @product.related.is_a?(Array).should be_true
-    end
-    
-    it 'should have many variants' do
-      @product.variants.is_a?(Array).should be_true
-    end
-    
-    it 'should have many discountables' do
-      @product.discountables.is_a?(Array).should be_true
-    end
-    
-    it 'should have many discounts' do
-      @product.discounts.is_a?(Array).should be_true
-    end
-    
   end
   
   describe 'validations' do
@@ -189,37 +165,18 @@ describe ShopProduct do
             @product.page.slug = 'delicious_ _:_;_=_+_._~_bread'
             @product.page.breadcrumb = nil
             @product.valid?
-            @product.page.breadcrumb.should === 'delicious_______________bread'          
+            @product.page.breadcrumb.should === @product.page.title          
           end
         end
         context 'has been set' do
-          it 'should generate on validation' do
+          it 'should be left as is' do
             @product.page.breadcrumb = 'delicious_ _:_;_=_+_._~_bread'
             @product.valid?
-            @product.page.breadcrumb.should === 'delicious_______________bread'
+            @product.page.breadcrumb.should === 'delicious_ _:_;_=_+_._~_bread'
           end        
         end
       end
     end
-    
-    context 'discounts' do
-      context 'after_create' do
-        it 'should assign its categories discounts to itself' do
-          @product = ShopProduct.new(
-            :price => 11.11,
-            :page_attributes => {
-              :title => 'New Page With Discounts',
-              :slug  => 'new_page_with_discounts',
-              :parent_id => shop_categories(:bread).id
-            } 
-          )
-          @product.discounts.empty?.should === true
-          @product.save
-          
-          ShopProduct.find(@product).discounts.should === shop_categories(:bread).discounts
-        end
-      end
-    end
   end
   
   describe '#attrs' do

data/spec/spec.opts

@@ -1,6 +1,4 @@
 --colour
---format
-progress
---loadby
-mtime
---reverse
+--format specdoc
+--loadby mtime
+--reverse