raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-27 01:43:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

removed Cart route, controller, model and specs: dead code

Browse Source
This commit is contained in:
luisramos0
2018-08-11 22:17:12 +01:00
parent 9dcbd9076e
commit 0b9061df28
10 changed files with 0 additions and 313 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
app/controllers/open_food_network/cart_controller.rb
View File

@@ -1,39 +0,0 @@
module OpenFoodNetwork
class CartController < ApplicationController
respond_to :json
# before_filter :authorize_read!, :except => [:index, :search, :create]
def new
@cart = Cart.new(current_api_user)
if @cart.save
respond_with(@cart, :status => 201)
else
invalid_resource!(@cart)
end
end
def show
@cart = Cart.find(params[:id])
respond_with(@cart)
end
def add_variant
@cart = Cart.find(params[:cart_id])
distributor = Enterprise.find_by_permalink(params[:distributor_id])
order_cycle = OrderCycle.find(params[:order_cycle_id]) if params[:order_cycle_id]
if @cart.add_variant params[:variant_id], params[:quantity], distributor, order_cycle, current_currency
respond_with(@cart)
else
respond_with(@cart.populate_errors)
end
end
private
def current_currency
Spree::Config[:currency]
end
end
end

28
app/models/cart.rb
View File

@@ -1,28 +0,0 @@
class Cart < ActiveRecord::Base
has_many :orders, :class_name => 'Spree::Order'
belongs_to :user, :class_name => Spree.user_class
def add_variant variant_id, quantity, distributor, order_cycle, currency
order = create_or_find_order_for_distributor distributor, order_cycle, currency
@populator = Spree::OrderPopulator.new(order, currency)
@populator.populate({ :variants => { variant_id => quantity } })
end
def create_or_find_order_for_distributor distributor, order_cycle, currency
order_for_distributor = orders.find { |order| order.distributor == distributor && order.order_cycle == order_cycle }
unless order_for_distributor
order_for_distributor = Spree::Order.create(:currency => currency, :distributor => distributor)
order_for_distributor.distributor = distributor
order_for_distributor.order_cycle = order_cycle
order_for_distributor.save!
orders << order_for_distributor
end
order_for_distributor
end
def populate_errors
@populator.errors
end
end

6
app/views/open_food_network/cart/show.v1.rabl
View File

@@ -1,6 +0,0 @@
object @cart
attributes :id
node( :orders ) do |p|
partial '/open_food_network/orders/index', object: p.orders
end

3
app/views/open_food_network/line_items/index.v1.rabl
View File

@@ -1,3 +0,0 @@
collection @line_items
extends "open_food_network/line_items/show"

4
app/views/open_food_network/line_items/show.v1.rabl
View File

@@ -1,4 +0,0 @@
object @line_item
attributes :id, :quantity
node(:name) { |p| p.variant.name }

3
app/views/open_food_network/orders/index.v1.rabl
View File

@@ -1,3 +0,0 @@
collection @orders
extends "open_food_network/orders/show"

7
app/views/open_food_network/orders/show.v1.rabl
View File

@@ -1,7 +0,0 @@
object @order
attributes :id
node( :distributor ) { |p| p.distributor.blank? ? "" : p.distributor.name }
node( :line_items ) do |p|
partial '/open_food_network/line_items/index', object: p.line_items
end

6
config/routes.rb
View File

@@ -100,12 +100,6 @@ Openfoodnetwork::Application.routes.draw do
post '/product_images/:product_id', to: 'product_images#update_product_image'
end
namespace :open_food_network do
resources :cart do
post :add_variant
end
end
get 'sitemap.xml', to: 'sitemap#index', defaults: { format: 'xml' }
# Mount Spree's routes

86
spec/controllers/cart_controller_spec.rb
View File

@@ -1,86 +0,0 @@
require 'spec_helper'
require 'spree/api/testing_support/helpers'
module OpenFoodNetwork
describe CartController, type: :controller do
render_views
let(:user) { FactoryBot.create(:user) }
let(:product1) { FactoryBot.create(:product) }
let(:cart) { Cart.create(user: user) }
let(:distributor) { FactoryBot.create(:distributor_enterprise) }
before do
end
context "as a normal user" do
context 'with an existing cart' do
it "retrieves an empty cart" do
get :show, {id: cart, :format => :json }
json_response = JSON.parse(response.body)
json_response['id'].should == cart.id
json_response['orders'].size.should == 0
end
context 'with an empty order' do
let(:order) { FactoryBot.create(:order, distributor: distributor) }
before(:each) do
cart.orders << order
cart.save!
end
it "retrieves a cart with a single order and line item" do
get :show, {id: cart, :format => :json }
json_response = JSON.parse(response.body)
json_response['orders'].size.should == 1
json_response['orders'].first['distributor'].should == order.distributor.name
json_response['orders'].first['line_items'].size.should == 0
end
end
context 'an order with line items' do
let(:product) { FactoryBot.create(:product, distributors: [ distributor ]) }
let(:order) { FactoryBot.create(:order, { distributor: distributor } ) }
let(:line_item) { FactoryBot.create(:line_item, { variant: product.master }) }
before(:each) do
order.line_items << line_item
order.save
cart.orders << order
cart.save!
end
it "retrieves a cart with a single order and line item" do
get :show, {id: cart, :format => :json }
json_response = JSON.parse(response.body)
json_response['orders'].size.should == 1
json_response['orders'].first['distributor'].should == order.distributor.name
json_response['orders'].first['line_items'].first["name"].should == product.name
json_response['orders'].first['line_items'].first["quantity"].should == line_item.quantity
end
end
context 'adding a variant' do
it 'should add variant to new order and return the order' do
product1.distributors << distributor
product1.save
variant = product1.variants.first
put :add_variant, { cart_id: cart, variant_id: variant.id, quantity: (variant.on_hand-1), distributor_id: distributor, order_cycle_id: nil, max_quantity: nil }
cart.orders.size.should == 1
cart.orders.first.line_items.size.should == 1
cart.orders.first.line_items.first.product.should == product1
end
end
end
end
end
end

131
spec/models/cart_spec.rb
View File

@@ -1,131 +0,0 @@
require 'spec_helper'
# TODO this seems to be redundant
describe Cart do
describe "associations" do
it { should have_many(:orders) }
end
describe 'when adding a product' do
let(:cart) { Cart.create(user: user) }
let(:distributor) { FactoryBot.create(:distributor_enterprise) }
let(:other_distributor) { FactoryBot.create(:distributor_enterprise) }
let(:currency) { "AUD" }
let(:product) { FactoryBot.create(:product, :distributors => [distributor]) }
let(:product_with_order_cycle) { create(:product) }
let(:order_cycle) { create(:simple_order_cycle, distributors: [distributor, other_distributor], variants: [product_with_order_cycle.master]) }
describe 'to an empty cart' do
it 'should create an order for the product being added, and associate the product to the selected distribution' do
subject.add_variant product.master.id, 3, distributor, nil, currency
subject.orders.size.should == 1
order = subject.orders.first.reload
order.currency.should == currency
order.distributor.should == product.distributors.first
order.order_cycle.should be_nil
order.line_items.first.product.should == product
end
it 'should create an order for the product being added, and associate the order with an order cycle and distributor' do
subject.add_variant product_with_order_cycle.master.id, 3, distributor, order_cycle, currency
subject.orders.size.should == 1
order = subject.orders.first.reload
order.currency.should == currency
order.distributor.should == distributor
order.order_cycle.should == order_cycle
order.line_items.first.product.should == product_with_order_cycle
end
end
describe 'to a cart with an order for a distributor' do
let(:product_from_other_distributor) { FactoryBot.create(:product, :distributors => [other_distributor]) }
let(:order) { FactoryBot.create(:order, :distributor => distributor) }
before do
FactoryBot.create(:line_item, :order => order, :product => product)
order.reload
subject.orders << order
subject.save!
end
it 'should create a new order and add a line item to it when product added for different distributor' do
subject.add_variant product_from_other_distributor.master.id, 3, other_distributor, nil, currency
subject.reload
subject.orders.size.should == 2
new_order_for_other_distributor = subject.orders.find { |order| order.distributor == other_distributor }
new_order_for_other_distributor.order_cycle.should be_nil
order.line_items.size.should == 1
new_order_for_other_distributor.line_items.size.should == 1
new_order_for_other_distributor.line_items.first.product.should == product_from_other_distributor
end
it 'should group line item in existing order, when product added for the same distributor' do
subject.add_variant product.master.id, 3, distributor, nil, currency
subject.orders.size.should == 1
order = subject.orders.first.reload
order.line_items.size.should == 2
order.line_items.first.product.should == product
end
it 'should create a new order for product in an order cycle' do
subject.add_variant product_with_order_cycle.master.id, 3, distributor, order_cycle, currency
subject.orders.size.should == 2
new_order_for_distributor = subject.orders.find { |order| order.order_cycle == order_cycle }
new_order_for_distributor.reload
new_order_for_distributor.line_items.first.product.should == product_with_order_cycle
end
end
describe 'existing order for distributor and order cycle' do
let(:order) { FactoryBot.create(:order, :distributor => distributor, :order_cycle => order_cycle) }
before do
subject.orders << order
subject.save!
end
it 'should group line items in existing order when added for the same distributor and order cycle' do
subject.add_variant product_with_order_cycle.master.id, 3, distributor, order_cycle, currency
subject.orders.size.should == 1
order = subject.orders.first.reload
order.line_items.size.should == 1
order.line_items.find{ |line_item| line_item.product == product_with_order_cycle }.should_not be_nil
end
it 'should create line item in new order when product added is for a different order cycle' do
order_cycle2 = create(:simple_order_cycle, distributors: [distributor], variants: [product_with_order_cycle.master])
subject.add_variant product_with_order_cycle.master.id, 3, distributor, order_cycle2, currency
subject.orders.size.should == 2
new_order_for_second_order_cycle = subject.orders.find { |order| order.order_cycle == order_cycle2 }
new_order_for_second_order_cycle.reload
new_order_for_second_order_cycle.line_items.size.should == 1
new_order_for_second_order_cycle.line_items.first.product.should == product_with_order_cycle
new_order_for_second_order_cycle.distributor.should == distributor
new_order_for_second_order_cycle.order_cycle.should == order_cycle2
end
it 'should create line_items in new order when added with different distributor, but same order_cycle' do
subject.add_variant product_with_order_cycle.master.id, 3, other_distributor, order_cycle, currency
subject.orders.size.should == 2
new_order_for_second_order_cycle = subject.orders.find { |order| order.distributor == other_distributor }
new_order_for_second_order_cycle.reload
new_order_for_second_order_cycle.line_items.size.should == 1
new_order_for_second_order_cycle.line_items.find{ |line_item| line_item.product == product_with_order_cycle }.should_not be_nil
new_order_for_second_order_cycle.order_cycle.should == order_cycle
end
end
end
end