radiant-shop-extension 0.10.0 → 0.11.0

data/spec/controllers/admin/shop/products/variant_templates_controller_spec.rb

@@ -0,0 +1,53 @@
+require 'spec/spec_helper'
+
+describe Admin::Shop::Products::VariantTemplatesController do
+  
+  dataset :users, :shop_variants, :shop_product_variants
+  
+  before :each do
+    login_as :admin
+  end
+  
+  describe '#create' do
+    before :each do
+      @product = shop_products(:crusty_bread)
+      @variant = shop_variants(:milk_states)
+      @initial_product_variants = @product.variants.all
+    end
+    context 'successfully created' do
+      before :each do
+        stub(ShopProduct).find.with_any_args { @product }
+        put :update, :product_id => @product.id, :id => @variant.id
+      end
+      it 'should add the variant templates to product' do
+        final_variants = @product.variants.all
+        final_variants.should_not == @initial_product_variants
+        
+        names = final_variants.map(&:name)
+        @variant.options.each do |name|
+          names.include?(name).should be_true
+        end
+      end
+      it 'should redirect to edit product path' do
+        response.should redirect_to(edit_admin_shop_product_path(@product))
+      end
+    end
+    context 'could not create' do
+      before :each do
+        stub(ShopProduct).find.with_any_args { @product }
+        mock(@product).apply_variant_template(@variant) { false }
+        put :update, :product_id => @product.id, :id => @variant.id
+      end
+      it 'should not add the variant templates to product' do
+        @product.variants.all.should == @initial_product_variants
+      end
+      it 'should assign flash error' do
+        flash.now[:error].should_not be_nil
+      end
+      it 'should redirect to edit product path' do
+        response.should redirect_to(edit_admin_shop_product_path(@product))
+      end
+    end
+  end
+  
+end

data/spec/controllers/admin/shop/products/variants_controller_spec.rb

@@ -0,0 +1,116 @@
+require 'spec/spec_helper'
+
+describe Admin::Shop::Products::VariantsController do
+  
+  dataset :users, :shop_variants, :shop_product_variants
+  
+  before :each do
+    login_as :admin
+  end
+  
+  describe '#create' do
+    before :all do
+      @product = shop_products(:crusty_bread)
+    end
+    context 'successfully created' do
+      before :each do
+        mock.instance_of(ShopProductVariant).save! { true }
+      end
+      context 'html' do
+        before :each do
+          post :create, :product_id => @product.id, :shop_product_variant => { }
+        end
+        it 'should redirect to product path' do
+          response.should redirect_to(edit_admin_shop_product_path(@product))
+        end
+      end
+      context 'js' do
+        before :each do
+          post :create, :product_id => @product.id, :shop_product_variant => { }, :format => 'js'
+        end
+        it 'should render the partial' do
+          response.should render_template('/admin/shop/products/edit/shared/_variant')
+        end
+      end
+    end
+    context 'could not be created' do
+      before :each do
+        mock.instance_of(ShopProductVariant).save! { raise ActiveRecord::RecordNotSaved }
+      end
+      context 'html' do
+        before :each do
+          post :create, :product_id => @product.id, :shop_product_variant => { }
+        end
+        it 'should assign flash error' do
+          flash.now[:error].should_not be_nil
+        end
+        it 'should redirect to product path' do
+          response.should redirect_to(edit_admin_shop_product_path(@product))
+        end
+      end
+      context 'js' do
+        before :each do
+          post :create, :product_id => @product.id, :shop_product_variant => { }, :format => 'js'
+        end
+        it 'should not be success' do
+          response.should_not be_success
+        end
+      end
+    end
+  end
+  
+  describe '#destroy' do
+    before :each do
+      @product = shop_products(:crusty_bread)
+      @product_variant = shop_product_variants(:mouldy_crusty_bread)
+    end
+    describe 'successfully destroyed' do
+      before :each do
+        mock(ShopProductVariant).find(@product_variant.id.to_s) { @product_variant }
+        stub(@product_variant).destroy { true }
+      end
+      context 'html' do
+        before :each do
+          delete :destroy, :product_id => @product.id, :id => @product_variant.id
+        end
+        it 'should redirect to product path' do
+          response.should redirect_to(edit_admin_shop_product_path(@product_variant.product))
+        end
+      end
+      context 'js' do
+        before :each do
+          delete :destroy, :product_id => @product.id, :id => @product_variant.id, :format => 'js'
+        end
+        it 'should render the partial' do
+          response.should be_success
+        end
+      end
+    end
+    describe 'could not be destroyed' do
+      before :each do
+        mock(ShopProductVariant).find(@product_variant.id.to_s) { @product_variant }
+        stub(@product_variant).destroy { false }
+      end
+      context 'html' do
+        before :each do
+          delete :destroy, :product_id => @product.id, :id => @product_variant.id
+        end
+        it 'should assign flash error' do
+          flash.now[:error].should_not be_nil
+        end
+        it 'should redirect to product path' do
+          response.should redirect_to(edit_admin_shop_product_path(@product_variant.product))
+        end
+      end
+      context 'js' do
+        before :each do
+          delete :destroy, :product_id => @product.id, :id => @product_variant.id, :format => 'js'
+        end
+        it 'should not be success' do
+          response.should_not be_success
+        end
+      end
+    end
+  end
+  
+end

data/spec/controllers/admin/shop/products_controller_spec.rb

@@ -2,71 +2,45 @@ require 'spec/spec_helper'
 
 describe Admin::Shop::ProductsController do
   
-  dataset :users
+  dataset :users, :shop_categories, :shop_products
     
   before(:each) do
     login_as  :admin
     
-    @shop_category = ShopCategory.new
-    @shop_categories = [ @shop_category ]
-    
-    @shop_product = ShopProduct.new
-    @shop_products = [ @shop_product ]
-    
-    stub(@shop_category).id { 1 }
-    stub(@shop_product).id { 1 }
-    stub(@shop_product).products { @shop_products }
+    @product  = shop_products(:crusty_bread)
+    @products = @product.category.products
   end
   
   describe '#index' do
-    before :each do
-      mock(ShopCategory).all { @shop_categories }
-      mock(ShopProduct).find(:all) { @shop_products } # Resource Controller
-      mock(ShopProduct).search('search') { @shop_products }
-    end
-    
     context 'html' do
-      before :each do
-        get :index, :search => 'search'
-      end
-      
       it 'should render index' do
+        get :index
+        
         response.should render_template(:index)
       end
       
-      it 'should assign the shop_products and shop_categories instance variables' do
-        assigns(:shop_products).should === @shop_products
-        assigns(:shop_categories).should === @shop_categories
+      it 'should also assign shop_categories' do
+        get :index
+        
+        assigns(:shop_categories).should == ShopCategory.all
       end
     end
     
     context 'js' do
-      before :each do
-        get :index, :search => 'search', :format => 'js'
-      end
-      
       it 'should render the collection partial and success status' do
+        get :index, :format => 'js'
+        
         response.should be_success
         response.should render_template('/admin/shop/products/index/_product')
       end
-      
-      it 'should assign the shop_products instance variable' do
-        assigns(:shop_products).should === @shop_products
-      end
     end
     
     context 'json' do
-      before :each do
-        get :index, :search => 'search', :format => 'json'
-      end
-      
       it 'should return a json object of the array and success status' do
+        get :index, :format => 'json'
+        
         response.should be_success
-        response.body.should === @shop_products.to_json(ShopProduct.params)
-      end
-      
-      it 'should assign the shop_product_images instance variable' do
-        assigns(:shop_products).should === @shop_products
+        response.body.should === assigns(:shop_products).to_json
       end
     end
     
@@ -74,16 +48,18 @@ describe Admin::Shop::ProductsController do
   
   describe '#sort' do
     before :each do
-      @products = [
-        'shop_category_1_products[]=2',
-        'shop_category_1_products[]=1'
+      @params = [
+        "category_#{@product.category.id}_products[]=2",
+        "category_#{@product.category.id}_products[]=1",
+        "category_#{@product.category.id}_products[]=0"
       ].join('&')
     end
     
     context 'products are not passed' do
       context 'html' do
         it 'should assign an error and redirect to admin_shop_products_path path' do
-          put :sort
+          put :sort, :category_id => @product.category.id
+          
           flash.now[:error].should_not be_nil
           response.should redirect_to(admin_shop_products_path)
         end
@@ -91,7 +67,8 @@ describe Admin::Shop::ProductsController do
       
       context 'js' do
         it 'should return an error string and failure status' do
-          put :sort, :format => 'js'
+          put :sort, :category_id => @product.category.id, :format => 'js'
+          
           response.should_not be_success
           response.body.should === 'Could not sort Products.'
         end
@@ -99,7 +76,8 @@ describe Admin::Shop::ProductsController do
       
       context 'json' do
         it 'should return a json error object and failure status' do
-          put :sort, :format => 'json'
+          put :sort, :category_id => @product.category.id, :format => 'json'
+          
           response.should_not be_success
           JSON.parse(response.body)['error'].should === 'Could not sort Products.'
         end
@@ -107,17 +85,14 @@ describe Admin::Shop::ProductsController do
     end
     
     context 'products are passed' do
-      before :each do
-        mock(ShopCategory).find('1') { @shop_category }
-      end
       context 'could not sort' do
         before :each do
-          mock(ShopProduct).find('2').stub!.update_attributes!(anything) { raise 'No Sorting' }
+          mock(ShopProduct).sort(@product.category.id.to_s, anything) { raise ActiveRecord::RecordNotFound }
         end
         
         context 'html' do
           it 'should assign an error and redirect to admin_shop_products_path path' do
-            put :sort, :products => @products, :category_id => 1
+            put :sort, :products => @params, :category_id => @product.category.id
             flash.now[:error].should === 'Could not sort Products.'
             response.should redirect_to(admin_shop_products_path)
           end
@@ -125,7 +100,7 @@ describe Admin::Shop::ProductsController do
         
         context 'js' do
           it 'should return an error string and failure status' do
-            put :sort, :products => @products, :category_id => 1, :format => 'js'
+            put :sort, :products => @params, :category_id => @product.category.id, :format => 'js'
             response.should_not be_success
             response.body.should === 'Could not sort Products.'
           end
@@ -133,7 +108,7 @@ describe Admin::Shop::ProductsController do
         
         context 'json' do
           it 'should return a json error object and failure status' do
-            put :sort, :products => @products, :category_id => 1, :format => 'json'
+            put :sort, :products => @params, :category_id => @product.category.id, :format => 'json'
             response.should_not be_success
             JSON.parse(response.body)['error'].should === 'Could not sort Products.'
           end
@@ -142,20 +117,20 @@ describe Admin::Shop::ProductsController do
       
       context 'successfully sorted' do
         before :each do
-          mock(ShopProduct).find(anything).times(2).stub!.update_attributes!(anything) { true }
+          mock(ShopProduct).sort(@product.category.id.to_s, anything) { true }
         end
         
         context 'html' do
-          it 'should assign a notice and redirect to admin_shop_products_path path' do
-            put :sort, :products => @products, :category_id => 1
-            flash.now[:notice].should === 'Products successfully sorted.'
+          it 'should redirect to admin_shop_products_path path' do
+            put :sort, :products => @params, :category_id => @product.category.id
             response.should redirect_to(admin_shop_products_path)
           end
         end
         
         context 'js' do
           it 'should return success string and success status' do
-            put :sort, :products => @products, :category_id => 1, :format => 'js'
+            put :sort, :products => @params, :category_id => @product.category.id, :format => 'js'
+            
             response.should be_success
             response.body.should === 'Products successfully sorted.'
           end
@@ -163,7 +138,8 @@ describe Admin::Shop::ProductsController do
         
         context 'json' do
           it 'should return a json success object and success status' do
-            put :sort, :products => @products, :category_id => 1, :format => 'json'
+            put :sort, :products => @params, :category_id => @product.category.id, :format => 'json'
+            
             response.should be_success
             JSON.parse(response.body)['notice'].should === 'Products successfully sorted.'
           end
@@ -173,12 +149,7 @@ describe Admin::Shop::ProductsController do
   end
   
   describe '#create' do
-    context 'product could not be created' do
-      before :each do
-        mock(ShopProduct).new()   { @shop_product }
-        stub(@shop_product).save! { raise ActiveRecord::RecordNotSaved }
-      end
-      
+    context 'product could not be created' do      
       context 'html' do
         it 'should assign a flash error and render new' do
           post :create, :shop_product => {}
@@ -209,59 +180,61 @@ describe Admin::Shop::ProductsController do
 
     context 'product successfully created' do
       before :each do
-        mock(ShopProduct).new()   { @shop_product }
-        stub(@shop_product).save! { true }
+        @params = {
+          :page_attributes => { 
+            :title     => 'name', 
+            :parent_id => @product.category.page.id,
+            :parts_attributes => [{
+              :name    => 'description',
+              :content => '*name*' 
+            }]
+          }, 
+          :price => 11.11
+        }
       end
-      
       context 'html' do
         context 'not continue' do
-          it 'should assign a notice and redirect to edit_shop_product path' do
-            post :create, :shop_product => {}
-            flash.now[:notice].should === 'Product created successfully.'
+          it 'should redirect to edit_shop_product path' do
+            post :create, :shop_product => @params
+
             response.should redirect_to(admin_shop_products_path)
           end
         end
         context 'continue' do
-          it 'should assign a notice and redirect to edit_shop_product path' do
-            post :create, :shop_product => {}, :continue => true
-            flash.now[:notice].should === 'Product created successfully.'
-            response.should redirect_to(edit_admin_shop_product_path(@shop_product))
+          it 'should redirect to edit_shop_product path' do
+            post :create, :shop_product => @params, :continue => true
+            
+            response.should redirect_to(edit_admin_shop_product_path(assigns(:shop_product)))
           end
         end
       end
       
       context 'js' do
         it 'should render the collection partial and success status' do
-          post :create, :shop_product => {}, :format => 'js'
+          post :create, :shop_product => @params, :format => 'js'
+          
           response.should be_success
-          assigns(:shop_product).should === @shop_product
           response.should render_template('/admin/shop/products/index/_product')
         end
       end
       
       context 'json' do
         it 'should render the json object and redirect to json show' do
-          post :create ,:shop_product => {}, :format => 'json'
+          post :create ,:shop_product => @params, :format => 'json'
+          
           response.should be_success
-          response.body.should === @shop_product.to_json(ShopProduct.params)
+          response.body.should === assigns(:shop_product).to_json
         end
       end
     end
   end
   
   describe '#update' do
-    before :each do
-      mock(ShopProduct).find('1') { @shop_product }
-    end
-    
     context 'could not update' do
-      before :each do
-        stub(@shop_product).update_attributes!({}) { raise ActiveRecord::RecordNotSaved }
-      end
-      
       context 'html' do
         it 'should assign a flash error and render edit' do
-          put :update, :id => @shop_product.id, :shop_product => {}        
+          put :update, :id => @product.id, :shop_product => { :title => 'failure' }
+          
           response.should render_template(:edit)
           flash.now[:error].should === 'Could not update Product.'
         end
@@ -269,7 +242,7 @@ describe Admin::Shop::ProductsController do
       
       context 'js' do
         it 'should render the error and failure status' do
-          put :update, :id => @shop_product.id, :shop_product => {}, :format => 'js'
+          put :update, :id => @product.id, :shop_product => { :title => 'failure' }, :format => 'js'
           response.should_not be_success
           response.body.should === 'Could not update Product.'
         end
@@ -277,38 +250,32 @@ describe Admin::Shop::ProductsController do
       
       context 'json' do
         it 'should assign an error json object and failure status' do
-          put :update, :id => @shop_product.id, :shop_product => {}, :format => 'json'
+          put :update, :id => @product.id, :shop_product => { :title => 'failure' }, :format => 'json'
           response.should_not be_success
           JSON.parse(response.body)['error'].should === 'Could not update Product.'
         end
       end
     end
     
-    context 'successfully updated' do
-      before :each do
-        stub(@shop_product).update_attributes!({}) { true }
-      end
-      
+    context 'successfully updated' do      
       context 'html' do
         context 'not continue' do
-          it 'should assign a notice and redirect to edit_shop_product path' do
-            put :update, :id => @shop_product.id, :shop_product => {}
-            flash.now[:notice].should === 'Product updated successfully.'
+          it 'should redirect to edit_shop_product path' do
+            put :update, :id => @product.id, :shop_product => { :page_attributes => { :title => 'success', :parent_id => @product.category.page.id } }
             response.should redirect_to(admin_shop_products_path)
           end
         end
         context 'continue' do
-          it 'should assign a notice and redirect to edit_shop_product path' do
-            put :update, :id => @shop_product.id, :shop_product => {}, :continue => true
-            flash.now[:notice].should === 'Product updated successfully.'
-            response.should redirect_to(edit_admin_shop_product_path(@shop_product))
+          it 'should redirect to edit_shop_product path' do
+            put :update, :id => @product.id, :shop_product => { :page_attributes => { :title => 'success', :parent_id => @product.category.page.id } }, :continue => true
+            response.should redirect_to(edit_admin_shop_product_path(assigns(:shop_product)))
           end
         end
       end
       
       context 'js' do
         it 'should render the partial and success status' do
-          put :update, :id => @shop_product.id, :shop_product => {}, :format => 'js'
+          put :update, :id => @product.id, :shop_product => { :page_attributes => { :title => 'success', :parent_id => @product.category.page.id } }, :format => 'js'
           response.should be_success
           response.should render_template('/admin/shop/products/index/_product')
         end
@@ -316,66 +283,30 @@ describe Admin::Shop::ProductsController do
       
       context 'json' do
         it 'should assign the json object and success status' do
-          put :update, :id => @shop_product.id, :shop_product => {}, :format => 'json'
+          put :update, :id => @product.id, :shop_product => { :page_attributes => { :title => 'success', :parent_id => @product.category.page.id } }, :format => 'json'
+          
           response.should be_success
-          response.body.should === @shop_product.to_json(ShopProduct.params)
+          response.body.should === assigns(:shop_product).to_json(ShopProduct.params)
         end
       end
     end
   end
 
   describe '#destroy' do
-    before :each do
-      mock(ShopProduct).find('1') { @shop_product }
-    end
-    
-    context 'product not destroyed' do
-      before :each do
-        stub(@shop_product).destroy { raise ActiveRecord::RecordNotFound }
-      end
-      
-      context 'html' do
-        it 'should assign a flash error and render remove' do
-          delete :destroy, :id => 1
-          flash.now[:error].should === 'Could not delete Product.'
-          response.should render_template(:remove)
-        end
-      end
-      
-      context 'js' do
-        it 'should render an error and failure status' do
-          delete :destroy, :id => 1, :format => 'js'
-          response.body.should === 'Could not delete Product.'
-          response.should_not be_success
-        end
-      end
-      
-      context 'json' do
-        it 'should render an error and failure status' do
-          delete :destroy, :id => 1, :format => 'json'
-          JSON.parse(response.body)['error'].should === 'Could not delete Product.'
-          response.should_not be_success
-        end
-      end
-      
-    end
-    
     context 'product successfully destroyed' do
-      before :each do
-        stub(@shop_product).destroy { true }
-      end
       
       context 'html' do
-        it 'should assign a flash notice and redirect to shop_products path' do
-          delete :destroy, :id => 1
-          flash.now[:notice].should === 'Product deleted successfully.'
+        it 'should redirect to shop_products path' do
+          delete :destroy, :id => @product.id
+          
           response.should redirect_to(admin_shop_products_path)
         end
       end
       
       context 'js' do
         it 'should render success message and success status' do
-          delete :destroy, :id => 1, :format => 'js'
+          delete :destroy, :id => @product.id, :format => 'js'
+          
           response.body.should === 'Product deleted successfully.'
           response.should be_success
         end
@@ -383,7 +314,8 @@ describe Admin::Shop::ProductsController do
       
       context 'json' do
         it 'should return a success json object and success status' do
-          delete :destroy, :id => 1, :format => 'json'
+          delete :destroy, :id => @product.id, :format => 'json'
+          
           JSON.parse(response.body)['notice'].should === 'Product deleted successfully.'
           response.should be_success
         end