simple_cart 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 667e8f74d3686ff61c680e9fb301007a6aec1fe0
-  data.tar.gz: eca4f915a9bdc4d2ed799b12fadd04f9d0385017
+  metadata.gz: c077dcd7475c05441cf80c2e110eb16cbd448e67
+  data.tar.gz: 7ea287086d37c383a5f1ee7ac33df4afe5adef9f
 SHA512:
-  metadata.gz: 0cdb9eee395fea45613eef68ff4d59c3f6c505a20e89a456d9c448aef7c2aebb8a8283a6bbe31a36685d660a092b667b860dd5e665dd8faf7a6cd4f711612366
-  data.tar.gz: 2b477cff84e67bdedbf1d8550f33c87498b9ac1d9ab8a29fcab50cb20dfb342bf7bc11896b5f43c9fb9758f074140ce6d3d5eb13c2aa49422429f03c2bee94c6
+  metadata.gz: 5b8ba6afe81a8bfced0bdec02e51cc49fe36170c1d206254bbc95a42718be32e14ac26101d9525e57e6dc2849e80314fbecfecdde85147d10518d12130d9ba1e
+  data.tar.gz: a52d65f59679bf1ee0efd275048e0ce4c7a669b62fe103dfcb3719c06ad586620118125054a9a821e1c2c48904d4fa20238adf6fa853d6bf8f5fe172adb254fd

data/README.md

@@ -1,3 +1,5 @@
+[![Build Status](https://travis-ci.org/leksster/SimpleCart-engine.svg?branch=master)](https://travis-ci.org/leksster/SimpleCart-engine)
+
 ## SimpleCart
 
 ### Installation:
@@ -8,29 +10,38 @@ gem 'simple_cart'
 
 ### Usage:
 
-Create or update your model with fields:
+##### Your model should contain fields:
   - name (sting)
   - price (decimal)
   - qty (integer)
+```html
+<%= add_to_cart(object) %>
+```
 
-Add to your route.rb file
+##### After you install SimpleCart and add it to your Gemfile, you need to run the generator:
 ```ruby
-mount SimpleCart::Engine => "/simple_cart"
+rails generate simple_cart:install
 ```
-Run SimpleCart migrations
+
+##### Run migrations task
 ```ruby
-rake simple_cart:install:migrations
 rake db:migrate
 ```
-You can manage delivery companies in the database using rake tasks. You need to create one in order to use checkout procedure properly.
+
+##### You can manage delivery services in the database using rake tasks. You need to create one in order to use checkout procedure properly.
 ```ruby
 rake simple_cart:create_delivery [NAME] [COSTS]
 rake simple_cart:delete_delivery [NAME]
 rake simple_cart:delete_all
 ```
 
-Use view helper method to add 'buy' button with required object:
+##### Use view helper method to add 'buy' button with required object:
 ```html
 <%= add_to_cart(object) %>
 ```
+
+##### If you want to customize views
+```ruby
+rake simple_cart:views
+```
 :trollface:

data/Rakefile

@@ -14,7 +14,7 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-APP_RAKEFILE = File.expand_path("../test/dummy/Rakefile", __FILE__)
+APP_RAKEFILE = File.expand_path("../spec/dummy/Rakefile", __FILE__)
 load 'rails/tasks/engine.rake'
 
 
@@ -24,14 +24,5 @@ load 'rails/tasks/statistics.rake'
 
 Bundler::GemHelper.install_tasks
 
-require 'rake/testtask'
-
-Rake::TestTask.new(:test) do |t|
-  t.libs << 'lib'
-  t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
-  t.verbose = false
-end
-
 
 task default: :test

data/app/controllers/simple_cart/checkout_controller.rb

@@ -31,7 +31,7 @@ module SimpleCart
       when :shipping
         @order.update(order_params)
       when :delivery
-        @order.delivery = Delivery.find(params[:order][:delivery_id])
+        @order.delivery = SimpleCart::Delivery.find(params[:order][:delivery_id])
       when :payment
         @order.update(order_params)
       end
@@ -41,7 +41,7 @@ module SimpleCart
     private
 
     def set_order
-      @order = Order.find(params[:order_id])
+      @order = SimpleCart::Order.find(params[:order_id])
     end
 
     def order_params

data/app/models/simple_cart/cart.rb

@@ -11,8 +11,8 @@ module SimpleCart
     def subtotal
       subtotal = 0
       unless @session.nil?
-        @session.each do |k, v|
-          subtotal += k.split('_')[0].constantize.find(k.split('_')[1]).price * v
+        @session.each do |klass_with_id, qty|
+          subtotal += find_in_model(klass_with_id).price * qty
         end
         subtotal -= @discount unless @discount.nil?
         subtotal
@@ -22,8 +22,8 @@ module SimpleCart
     def items
       unless @session.nil?
         arr = []
-        @session.each do |k, v|
-          arr << k.split('_')[0].constantize.find(k.split('_')[1])
+        @session.each do |klass_with_id, qty|
+          arr << find_in_model(klass_with_id)
         end
         arr
       end
@@ -70,5 +70,10 @@ module SimpleCart
       coupon = SimpleCart.coupon_class.find_by(code: code)
       self.subtotal * (coupon.discount / 100.0) unless coupon.nil?
     end
+
+    private 
+    def find_in_model(klass_with_id)
+      klass_with_id.split('_')[0].constantize.find(klass_with_id.split('_')[1])
+    end
   end
 end

data/app/views/simple_cart/carts/_buy.html.haml

@@ -1,7 +1,6 @@
-= form_for item, :url => simple_cart.add_to_cart_path(item), :html => {:class => 'form-inline one-line text-center'}, method: :post do |f|
+= form_for item, :url => simple_cart.add_to_cart_path(item, klass: item.class), :html => {:class => 'form-inline one-line text-center'}, method: :post do |f|
   %h2
     = number_to_currency(item.price)
   .form-group.integer.required.one-line
     = number_field_tag :qty, '1', class: 'form-control one-line', value: '1', min: '1', max: '99'
-    = hidden_field_tag(:klass, :item, :value => item.class)
     = f.submit 'Add to cart', class: 'btn btn-default btn btn-primary'

data/app/views/simple_cart/checkout/payment.html.haml

@@ -64,6 +64,6 @@
     .col-md-4.col-sm-4.text-center
       %h2
         Order Summary
-      / = render 'summary'
+      = render 'summary'
 
 

data/lib/generators/simple_cart/install_generator.rb

@@ -0,0 +1,19 @@
+require 'rails/generators'
+
+module SimpleCart
+  class InstallGenerator < Rails::Generators::Base
+    source_root File.expand_path('../templates', __FILE__)
+
+    def test
+      puts 'hi'
+    end
+
+    def copy_migration 
+      rake 'simple_cart:install:migrations'
+    end
+
+    def init_routes
+      route 'mount SimpleCart::Engine, at: "/shop"'
+    end
+  end
+end

data/lib/generators/simple_cart/views_generator.rb

@@ -0,0 +1,12 @@
+require 'rails/generators'
+
+module SimpleCart
+  class ViewsGenerator < Rails::Generators::Base
+    source_root File.expand_path("../../../../app/views/simple_cart", __FILE__)    
+    def copy_views
+      directory 'carts', 'app/views/simple_cart/carts'
+      directory 'checkout', 'app/views/simple_cart/checkout'
+      directory 'orders', 'app/views/simple_cart/orders'
+    end
+  end
+end

data/lib/simple_cart.rb

@@ -1,8 +1,9 @@
 require "simple_cart/engine"
+require "simple_cart/acts_as_simple_cart"
 
 module SimpleCart
   mattr_accessor :coupon_class
-  
+
   def self.coupon_class
     @@coupon_class.constantize
   end

data/lib/simple_cart/acts_as_simple_cart.rb

@@ -0,0 +1,15 @@
+module SimpleCart
+  module ActsAsSimpleCart
+    extend ActiveSupport::Concern
+
+    module ClassMethods
+      def acts_as_product
+        class_eval do
+          has_many :order_items, class_name: 'SimpleCart::OrderItem', as: :itemable
+        end
+      end
+    end
+  end
+end
+
+ActiveRecord::Base.include(SimpleCart::ActsAsSimpleCart)

data/lib/simple_cart/engine.rb

@@ -6,7 +6,6 @@ require 'aasm'
 require 'jquery-rails'
 require 'turbolinks'
 
-
 module SimpleCart
   class Engine < ::Rails::Engine
     config.before_initialize do

data/lib/simple_cart/version.rb

@@ -1,3 +1,3 @@
 module SimpleCart
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/controllers/{carts_controller_spec.rb → simple_cart/carts_controller_spec.rb}

@@ -120,6 +120,5 @@ module SimpleCart
         expect(assigns(:order)).not_to be_nil
       end
     end
-
   end
 end

data/spec/controllers/simple_cart/checkout_controller_spec.rb

@@ -0,0 +1,206 @@
+require 'byebug'
+module SimpleCart
+  RSpec.describe CheckoutController, type: :controller do
+    let(:order) do
+      FactoryGirl.create( :order, 
+      order_items: [FactoryGirl.create(:order_item)], 
+      billing_address: billing_address, 
+      shipping_address: shipping_address, 
+      delivery: FactoryGirl.create(:delivery), 
+      credit_card: FactoryGirl.create(:credit_card) ) 
+    end
+    
+    let(:shipping_address) { FactoryGirl.create(:shipping_address) }
+    let(:billing_address) { FactoryGirl.create(:billing_address) }
+    let(:credit_card) { FactoryGirl.create(:credit_card) }
+    let(:order_in_queue) { FactoryGirl.create(:order, :aasm_state => 'in_queue') }
+    let(:steps) { [:billing, :shipping, :delivery, :payment, :confirm] }
+
+    routes { SimpleCart::Engine.routes }
+
+    describe "GET #show" do
+
+      before(:each) do 
+        allow(SimpleCart::Order).to receive(:find).and_return(order)
+      end
+
+      shared_examples "receives build_or_find methods" do
+        it "#build_or_find_billing_address" do
+          expect(order).to receive(:build_or_find_billing_address)
+          get(:show, step_params)
+        end
+
+        it "#build_or_find_shipping_address" do
+          expect(order).to receive(:build_or_find_shipping_address)
+          get(:show, step_params)
+        end
+
+        it "#build_or_find_credit_card" do
+          expect(order).to receive(:build_or_find_credit_card)
+          get(:show, step_params)
+        end
+
+        it "assignes @order" do
+          get :show, step_params
+          expect(assigns(:order)).to be_a(SimpleCart::Order)
+        end
+      end
+
+      context "Steps" do
+        
+        it_behaves_like "receives build_or_find methods" do
+          [:billing, :shipping, :delivery, :payment, :confirm].each do |step|
+            let(:step_params) {{ :order_id => order.id, :id => step }}
+          end
+        end
+
+        context "http success" do
+          it "returns http success on every step" do
+            steps.each do |step|
+              get :show, { :order_id => order.id, :id => step }
+              expect(response).to have_http_status(:success)
+            end
+          end
+        end
+
+        context "render the page" do
+          it "renders specific view for the current step" do
+            steps.each do |step|
+              get :show, { :order_id => order.id, :id => step }
+              expect(response).to render_template step
+            end
+          end
+        end
+
+        context "redirects" do
+          it "redirects to previous step if current is not valid" do
+            steps = [:delivery, :payment, :confirm]
+            steps.each_with_index do |step, index|
+              order = FactoryGirl.create(:order)
+              allow(Order).to receive(:find).and_return(order)
+              get :show, { :order_id => order.id, :id => step }
+              expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => [:billing, :delivery, :payment][index])
+            end
+          end
+        end
+      end
+    end
+
+    describe "PUT #update" do
+
+      before(:each) do 
+        allow(SimpleCart::Order).to receive(:find).and_return(order)
+        allow(order).to receive(:update).and_return(true)
+        allow(order).to receive(:build_shipping_address).and_return(shipping_address)
+        allow(order).to receive(:billing_address).and_return(billing_address)
+      end
+
+      context ":billing step" do
+        let(:step_params) do 
+          {
+            :order_id => order.id, 
+            :id       => :billing, 
+            :order    => { :billing_address_attributes => FactoryGirl.attributes_for(:billing_address) },
+            :shipping  => checked 
+          } 
+        end
+
+        before do
+          allow(order).to receive(:update).and_return(true)
+          put :update, step_params
+        end
+
+        after { put :update, step_params }
+                            
+        context "without billing address checkbox checked" do
+          let(:checked) { nil }
+
+          it "#update order" do
+            expect(order).to receive(:update).with(step_params[:order].with_indifferent_access)
+          end
+
+          it "redirects to :shipping" do
+            expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => :shipping)
+          end
+        end
+
+        context "with billing address checkbox checked" do
+          let(:checked) { true }
+
+          it "#update order" do
+            expect(order).to receive(:update).with(step_params[:order])
+          end
+
+          it "#build_shipping_address" do
+            expect(order).to receive(:build_shipping_address)
+            
+          end
+
+          it "redirects to :delivery" do
+            expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => :delivery)
+          end
+        end
+      end
+
+      context ":shipping step" do
+        let(:step_params) do
+          { 
+            :order_id => order.id, :id => :shipping, 
+            :order => { :shipping_address_attributes => FactoryGirl.attributes_for(:shipping_address) } 
+          } 
+        end
+
+        before { put :update, step_params }
+
+        it "#update order" do
+          expect(order).to receive(:update).with(step_params[:order].with_indifferent_access)
+          put :update, step_params
+        end
+
+        it "redirects to :delivery" do
+          expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => :delivery)
+        end    
+      end
+
+      context ":delivery step" do
+        let(:step_params) do
+          {
+            :order_id => order.id, 
+            :id       => :delivery, 
+            :order    => { :delivery_id => FactoryGirl.create(:delivery).id }
+          } 
+        end 
+
+        before { put :update, step_params }
+
+        it "sets delivery for order" do
+          expect(order.delivery).not_to be_nil
+        end
+
+        it "redirects to :payment" do
+          expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => :payment)
+        end
+      end
+
+      context ":payment step" do
+        let(:step_params) do
+          {
+            :order_id => order.id, 
+            :id       => :payment, 
+            :order    => { :credit_card_attributes => FactoryGirl.attributes_for(:credit_card) }
+          } 
+        end
+
+        it "#update order" do
+          expect(order).to receive(:update).with(step_params[:order].with_indifferent_access)
+          put :update, step_params
+        end
+
+        it "redirects to :confirm" do
+          put :update, step_params
+          expect(response).to redirect_to order_checkout_path(:order_id => order.id, :id => :confirm)
+        end
+      end
+    end
+  end
+end