solidus_multi_domain 1.0.0

data/solidus_multi_domain.gemspec

@@ -0,0 +1,31 @@
+# encoding: UTF-8
+
+Gem::Specification.new do |s|
+  s.platform    = Gem::Platform::RUBY
+  s.name        = "solidus_multi_domain"
+  s.version     = "1.0.0"
+  s.summary     = "Adds multiple site support to Solidus"
+  s.description = "Multiple Solidus stores on different domains - single unified backed for processing orders."
+  s.required_ruby_version = ">= 2.1"
+
+  s.author       = "Solidus Team"
+  s.email        = "contact@solidus.io"
+  s.homepage     = "https://solidus.io"
+  s.license      = %q{BSD-3}
+
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.require_path = "lib"
+  s.requirements << "none"
+
+  s.add_dependency "solidus", [">= 1.0.0.pre", "< 2"]
+
+  s.add_development_dependency "rspec-rails",  "~> 3.2"
+  s.add_development_dependency "simplecov"
+  s.add_development_dependency "sqlite3"
+  s.add_development_dependency "sass-rails", "~> 4.0.2"
+  s.add_development_dependency "coffee-rails", "~> 4.0.0"
+  s.add_development_dependency "factory_girl", "~> 4.5.0"
+  s.add_development_dependency "capybara", "~> 2.4"
+  s.add_development_dependency "ffaker"
+end

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

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe Spree::Admin::ProductsController do
+  stub_authorization!
+
+  describe "on :index" do
+    it "renders index" do
+      spree_get :index
+      expect(response).to be_success
+    end
+  end
+
+  describe "on a PUT to :update" do
+    before(:each) do
+      @product = FactoryGirl.create(:product)
+      @store = FactoryGirl.create(:store)
+    end
+
+    describe "when no stores are selected" do
+      it "clears stores if they previously existed" do
+        @product.stores << @store
+
+        spree_put :update, :id => @product.to_param,
+                      :product => {:name => @product.name}, update_store_ids: 'true'
+
+        expect(@product.reload.store_ids).to be_empty
+      end
+    end
+
+    describe "when a store is selected" do
+      it "clears stores" do
+        spree_put :update, :id => @product.to_param,
+                      :product => {:name => @product.name, :store_ids => [@store.id]}, update_store_ids: 'true'
+
+        expect(@product.reload.store_ids).to eq [@store.id]
+      end
+    end
+  end
+end

data/spec/controllers/spree/admin/stores_controller_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe Spree::Admin::StoresController do
+  stub_authorization!
+
+  describe "on :index" do
+    it "renders index" do
+      spree_get :index
+      expect(response).to be_success
+    end
+  end
+end

data/spec/controllers/spree/api/line_items_controller_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Spree::Api::LineItemsController do
+
+  before(:each) do
+    stub_authentication!
+  end
+
+  describe "POST create" do
+    let(:user) { create(:user) }
+    let(:current_api_user) { user }
+    let(:order) { create(:order, user: user) }
+    let(:line_item) { build(:line_item, order: order) }
+
+    subject { spree_post :create, line_item: line_item, order_id: line_item.order.number }
+
+    context "A SpreeMultiDomain::LineItemDecorator::ProductDoesNotBelongToStoreError is raised" do
+      before(:each) do
+        def controller.create
+          raise SpreeMultiDomain::LineItemDecorator::ProductDoesNotBelongToStoreError
+        end
+      end
+
+      it "sets the correct status" do
+        subject
+        expect(response.status).to eq 422
+      end
+
+      it "contains the correct error message" do
+        subject
+        expect(JSON.parse(response.body)["message"]).to eq "Products from different stores may not be added to this order."
+      end
+    end
+  end
+end

data/spec/controllers/spree/api/orders_controller_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+
+describe Spree::Api::OrdersController do
+  let!(:user) { FactoryGirl.create(:user) }
+  let!(:store1) { FactoryGirl.create(:store) }
+  let!(:store2) { FactoryGirl.create(:store) }
+
+  before(:each) do
+    controller.stub(current_api_user: user)
+  end
+
+  describe '#mine' do
+    it 'should return only orders from the correct store' do
+      FactoryGirl.create(:order, store: store1, user: user)
+      FactoryGirl.create(:order, store: store2, user: user)
+
+      controller.stub(current_store: store1)
+      controller.mine
+
+      expect(assigns(:orders).length).to eq(1)
+      expect(assigns(:orders).first.store_id).to eq(store1.id)
+
+      controller.stub(current_store: store2)
+      controller.mine
+
+      expect(assigns(:orders).length).to eq(1)
+      expect(assigns(:orders).first.store_id).to eq(store2.id)
+    end
+  end
+
+  describe '#find_current_order' do
+
+    it 'should return the latest incomplete order from the correct store' do
+      order1 = FactoryGirl.create(:order, store: store1, state: :cart, user: user)
+      order2 = FactoryGirl.create(:order, store: store2, state: :cart, user: user)
+
+      controller.stub(current_store: store1)
+      expect(controller.send(:find_current_order)).to eql(order1)
+
+      controller.stub(current_store: store2)
+      expect(controller.send(:find_current_order)).to eql(order2)
+    end
+  end
+end

data/spec/controllers/spree/api/products_controller_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe Spree::Api::ProductsController do
+
+  let!(:product) { FactoryGirl.create(:product) }
+
+  let(:user)  { create(:user) }
+  before { allow(controller).to receive(:current_spree_user) { user } }
+
+  describe 'on :show to a product without any stores' do
+    let!(:store) { FactoryGirl.create(:store) }
+
+    it 'should return 404' do
+      spree_get :show, :id => product.to_param, format: :json
+
+      expect(response.response_code) == 404
+    end
+  end
+
+  # Regression test for #75
+  describe 'on :show to a product in the wrong store' do
+    let!(:store_1) { FactoryGirl.create(:store) }
+    let!(:store_2) { FactoryGirl.create(:store) }
+
+    before(:each) do
+      product.stores << store_1
+    end
+
+    it 'should return 404' do
+      controller.stub(:current_store => store_2)
+      spree_get :show, :id => product.to_param, format: :json
+
+      expect(response.response_code) == 404
+    end
+  end
+
+  describe 'on :show to a product w/ store' do
+    let!(:store) { FactoryGirl.create(:store) }
+
+    before(:each) do
+      product.stores << store
+    end
+
+    it 'should return 200' do
+      controller.stub(:current_store => store)
+      spree_get :show, :id => product.to_param, format: :json
+
+      expect(response.response_code) == 200
+    end
+  end
+
+end

data/spec/controllers/spree/api/shipments_controller_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe Spree::Api::ShipmentsController do
+  describe '#mine' do
+    let!(:user) { FactoryGirl.create(:user) }
+    let!(:store1) { FactoryGirl.create(:store) }
+    let!(:store2) { FactoryGirl.create(:store) }
+    let!(:order_from_store1) { FactoryGirl.create(:order, store: store1, user: user) }
+    let!(:order_from_store2) { FactoryGirl.create(:order, store: store2, user: user) }
+
+    before(:each) do
+      controller.stub(current_api_user: user)
+    end
+
+    it 'should return only shipments from the correct store' do
+      FactoryGirl.create(:shipment, order: order_from_store1)
+      FactoryGirl.create(:shipment, order: order_from_store2)
+
+      controller.stub(current_store: store1)
+      controller.mine
+
+      expect(assigns(:shipments).length).to eq(1)
+      expect(assigns(:shipments).first.order.store_id).to eq(store1.id)
+
+      controller.stub(current_store: store2)
+      controller.mine
+
+      expect(assigns(:shipments).length).to eq(1)
+      expect(assigns(:shipments).first.order.store_id).to eq(store2.id)
+    end
+  end
+
+  describe "PUT add" do
+
+    let(:current_api_user) { create(:admin_user) }
+    let(:shipment) { create(:shipment) }
+    let(:variant) { create(:variant) }
+
+    subject { spree_put :add, variant_id: variant.id, id: shipment.number, quantity: 1 }
+
+    before(:each) do
+      stub_authentication!
+    end
+
+    context "A SpreeMultiDomain::LineItemDecorator::ProductDoesNotBelongToStoreError is raised" do
+      before(:each) do
+        def controller.add
+          raise SpreeMultiDomain::LineItemDecorator::ProductDoesNotBelongToStoreError
+        end
+      end
+
+      it "sets the correct status" do
+        subject
+        expect(response.status).to eq 422
+      end
+
+      it "contains the correct error message" do
+        subject
+        expect(JSON.parse(response.body)["message"]).to eq "Products from different stores may not be added to this order."
+      end
+    end
+  end
+end

data/spec/controllers/spree/products_controller_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Spree::ProductsController do
+
+  let!(:product) { FactoryGirl.create(:product) }
+
+  describe 'on :show to a product without any stores' do
+    let!(:store) { FactoryGirl.create(:store) }
+
+    it 'returns 404' do
+      spree_get :show, :id => product.to_param
+
+      expect(response.response_code).to eq 404
+    end
+  end
+
+  # Regression test for #75
+  describe 'on :show to a product in the wrong store' do
+    let!(:store_1) { FactoryGirl.create(:store) }
+    let!(:store_2) { FactoryGirl.create(:store) }
+
+    before(:each) do
+      product.stores << store_1
+    end
+
+    it 'returns 404' do
+      allow(controller).to receive_messages(:current_store => store_2)
+      spree_get :show, :id => product.to_param
+
+      expect(response.response_code).to eq 404
+    end
+  end
+
+  describe 'on :show to a product w/ store' do
+    let!(:store) { FactoryGirl.create(:store) }
+
+    before(:each) do
+      product.stores << store
+    end
+
+    it 'returns 200' do
+      allow(controller).to receive_messages(:current_store => store)
+      spree_get :show, :id => product.to_param
+
+      expect(response.response_code).to eq 200
+    end
+  end
+
+end

data/spec/helpers/products_helper_decorator_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Spree
+  describe ProductsHelper do
+    before(:each) do
+      @store     = FactoryGirl.create(:store)
+      @taxonomy  = FactoryGirl.create(:taxonomy, :store => @store)
+      @taxonomy2 = FactoryGirl.create(:taxonomy)
+
+      allow(helper).to receive(:current_store) { @store }
+    end
+
+    describe "#get_taxonomies" do
+      it "only show taxonomies on current_store" do
+        taxonomies = helper.get_taxonomies
+
+        expect(taxonomies).to include(@taxonomy)
+        expect(taxonomies).to_not include(@taxonomy2)
+      end
+    end
+  end
+end

data/spec/mailers/carton_mailer_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe Spree::CartonMailer do
+  let(:mail) { Spree::CartonMailer.shipped_email(carton.id) }
+
+  let(:carton) do
+    FactoryGirl.create(:carton, inventory_units: order.inventory_units)
+  end
+
+  let(:order) do
+    FactoryGirl.create(:order_ready_to_ship, line_items_count: 1)
+  end
+
+  describe 'from address' do
+    subject do
+      mail.from
+    end
+
+    context 'the order does not have a store' do
+      let(:order) do
+        FactoryGirl.create(:order_ready_to_ship, line_items_count: 1, store: nil)
+      end
+
+      it { is_expected.to eq [Spree::Config[:mails_from]] }
+    end
+
+    context 'the order has a store' do
+      let(:order) do
+        FactoryGirl.create(
+          :order_ready_to_ship,
+          line_items_count: 1,
+          store: store,
+        )
+      end
+
+      let(:store) { FactoryGirl.create(:store, mail_from_address: 'store@example.com') }
+      it { is_expected.to eq ['store@example.com'] }
+    end
+  end
+
+end

data/spec/models/spree/line_item_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe Spree::LineItem do
+  describe "before create" do
+
+    let(:order_store) { create(:store) }
+    let(:other_store) { create(:store) }
+    let(:order)       { create(:order, store: order_store) }
+    let(:variant)     { create(:variant, product: product) }
+    let(:line_item)   { build(:line_item, order: order, product: product) }
+
+    subject { line_item.save! }
+
+    context "the order does not have a store" do
+      before(:each) { order.update_attributes(store_id: nil) }
+      let(:product) { create(:product, stores: [order_store])}
+
+      it 'does not raise an error' do
+        expect{ subject }.to_not raise_error
+      end
+    end
+
+    context "the line item's product does not belong to the order's store" do
+      let(:product) { create(:product, stores: [other_store]) }
+
+      it 'raises the correct error' do
+        expect{ subject }.to raise_error(SpreeMultiDomain::LineItemDecorator::ProductDoesNotBelongToStoreError)
+      end
+    end
+
+    context "the line item's product belongs to the order's store" do
+      let(:product) { create(:product, stores: [order_store])}
+
+      it "does not raise an error" do
+        expect{ subject }.to_not raise_error
+      end
+    end
+  end
+end

data/spec/models/spree/order_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Spree::Order do
+
+  before(:each) do
+    @store = FactoryGirl.create(:store)
+    @order = FactoryGirl.create(:order, :store => @store)
+
+    @order2 = FactoryGirl.create(:order)
+  end
+
+  it 'should correctly find products by store' do
+    by_store = Spree::Order.by_store(@store)
+
+    expect(by_store).to include(@order)
+    expect(by_store).to_not include(@order2)
+  end
+end