solidus_api 1.0.0.pre

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

@@ -0,0 +1,307 @@
+require 'spec_helper'
+
+module Spree
+  describe Api::StockItemsController, :type => :controller do
+    render_views
+
+    let!(:stock_location) { create(:stock_location_with_items) }
+    let!(:stock_item) { stock_location.stock_items.order(:id).first }
+    let!(:attributes) { [:id, :count_on_hand, :backorderable,
+                         :stock_location_id, :variant_id] }
+
+    before do
+      stub_authentication!
+    end
+
+    context "as a normal user" do
+      describe "#index" do
+        it "can list stock items for an active stock location" do
+          api_get :index, stock_location_id: stock_location.to_param
+          expect(response).to be_success
+          json_response['stock_items'].first.should have_attributes(attributes)
+          json_response['stock_items'].first['variant']['sku'].should match /\ASKU-\d+\z/
+        end
+
+        it "cannot list stock items for an inactive stock location" do
+          stock_location.update_attributes!(active: false)
+          api_get :index, stock_location_id: stock_location.to_param
+          expect(response).to be_not_found
+        end
+      end
+
+      describe "#show" do
+        it "can see a stock item for an active stock location" do
+          api_get :show, stock_location_id: stock_location.to_param, id: stock_item.to_param
+          json_response.should have_attributes(attributes)
+          json_response['count_on_hand'].should eq stock_item.count_on_hand
+        end
+
+        it "cannot see a stock item for an inactive stock location" do
+          stock_location.update_attributes!(active: false)
+          api_get :show, stock_location_id: stock_location.to_param, id: stock_item.to_param
+          response.status.should == 404
+        end
+      end
+
+      describe "#create" do
+        it "cannot create a stock item" do
+          variant = create(:variant)
+          params = {
+            stock_location_id: stock_location.to_param,
+            stock_item: {
+              variant_id: variant.id,
+              count_on_hand: '20'
+            }
+          }
+
+          api_post :create, params
+          response.status.should == 404
+        end
+      end
+
+      describe "#update" do
+        it "cannot update a stock item" do
+          api_put :update, stock_location_id: stock_location.to_param, id: stock_item.to_param
+          response.status.should == 404
+        end
+      end
+
+      describe "#destroy" do
+        it "cannot destroy a stock item" do
+          api_delete :destroy, stock_location_id: stock_location.to_param, id: stock_item.to_param
+          response.status.should == 404
+        end
+      end
+    end
+
+    context "as an admin" do
+      sign_in_as_admin!
+
+      it 'cannot list of stock items' do
+        api_get :index, stock_location_id: stock_location.to_param
+        expect(json_response['stock_items'].first).to have_attributes(attributes)
+        expect(json_response['stock_items'].first['variant']['sku']).to include 'SKU'
+      end
+
+      it 'requires a stock_location_id to be passed as a parameter' do
+        api_get :index
+        expect(json_response['error']).to match(/stock_location_id parameter must be provided/)
+        expect(response.status).to eq(422)
+      end
+
+      it 'can control the page size through a parameter' do
+        api_get :index, stock_location_id: stock_location.to_param, per_page: 1
+        expect(json_response['count']).to eq(1)
+        expect(json_response['current_page']).to eq(1)
+      end
+
+      it 'can query the results through a paramter' do
+        stock_item.update_column(:count_on_hand, 30)
+        api_get :index, stock_location_id: stock_location.to_param, q: { count_on_hand_eq: '30' }
+        expect(json_response['count']).to eq(1)
+        expect(json_response['stock_items'].first['count_on_hand']).to eq 30
+      end
+
+      it 'gets a stock item' do
+        api_get :show, stock_location_id: stock_location.to_param, id: stock_item.to_param
+        expect(json_response).to have_attributes(attributes)
+        expect(json_response['count_on_hand']).to eq stock_item.count_on_hand
+      end
+
+      context 'creating a stock item' do
+        let!(:variant) do
+          variant = create(:variant)
+          # Creating a variant also creates stock items.
+          # We don't want any to exist (as they would conflict with what we're about to create)
+          StockItem.delete_all
+          variant
+        end
+        let(:count_on_hand) { '20' }
+        let(:params) do
+          {
+            stock_location_id: stock_location.to_param,
+            stock_item: {
+              variant_id: variant.id,
+              count_on_hand: count_on_hand
+            }
+          }
+        end
+
+        subject { api_post :create, params }
+
+        it 'can create a new stock item' do
+          subject
+          expect(response.status).to eq 201
+          expect(json_response).to have_attributes(attributes)
+        end
+
+        it 'creates a stock movement' do
+          expect { subject }.to change { Spree::StockMovement.count }.by(1)
+          expect(assigns(:stock_movement).quantity).to eq 20
+        end
+
+        context 'variant tracks inventory' do
+          before do
+            expect(variant.track_inventory).to eq true
+          end
+
+          it "sets the stock item's count_on_hand" do
+            subject
+            expect(assigns(:stock_item).count_on_hand).to eq 20
+          end
+        end
+
+        context 'variant does not track inventory' do
+          before do
+            variant.update_attributes(track_inventory: false)
+          end
+
+          it "doesn't set the stock item's count_on_hand" do
+            subject
+            expect(assigns(:stock_item).count_on_hand).to eq 0
+          end
+        end
+
+        context "attempting to set negative inventory" do
+          let(:count_on_hand) { '-1' }
+
+          it "does not allow negative inventory for the stock item" do
+            subject
+            expect(response.status).to eq 422
+            expect(response.body).to match Spree.t(:stock_not_below_zero)
+            expect(assigns(:stock_item).count_on_hand).to eq 0
+          end
+        end
+      end
+
+      context 'updating a stock item' do
+        before do
+          expect(stock_item.count_on_hand).to eq 10
+        end
+
+        subject { api_put :update, params }
+
+        context 'adjusting count_on_hand' do
+          let(:count_on_hand) { 40 }
+          let(:params) do
+            {
+              id: stock_item.to_param,
+              stock_item: {
+                count_on_hand: count_on_hand,
+                backorderable: true
+              }
+            }
+          end
+
+          it 'can update a stock item to add new inventory' do
+            subject
+            expect(response.status).to eq 200
+            expect(json_response['count_on_hand']).to eq 50
+            expect(json_response['backorderable']).to eq true
+          end
+
+          it 'creates a stock movement for the adjusted quantity' do
+            expect { subject }.to change { Spree::StockMovement.count }.by(1)
+            expect(Spree::StockMovement.last.quantity).to eq 40
+          end
+
+          context 'tracking inventory' do
+            before do
+              expect(stock_item.should_track_inventory?).to eq true
+            end
+
+             it "sets the stock item's count_on_hand" do
+              subject
+              expect(assigns(:stock_item).count_on_hand).to eq 50
+            end
+          end
+
+          context 'not tracking inventory' do
+            before do
+              stock_item.variant.update_attributes(track_inventory: false)
+            end
+
+            it "doesn't set the stock item's count_on_hand" do
+              subject
+              expect(assigns(:stock_item).count_on_hand).to eq 10
+            end
+          end
+
+          context "attempting to set negative inventory" do
+            let(:count_on_hand) { '-11' }
+
+            it "does not allow negative inventory for the stock item" do
+              subject
+              expect(response.status).to eq 422
+              expect(response.body).to match Spree.t(:stock_not_below_zero)
+              expect(assigns(:stock_item).count_on_hand).to eq 10
+            end
+          end
+        end
+
+        context 'setting count_on_hand' do
+          let(:count_on_hand) { 40 }
+          let(:params) do
+            {
+              id: stock_item.to_param,
+              stock_item: {
+                count_on_hand: count_on_hand,
+                force: true,
+              }
+            }
+          end
+
+          it 'can set a stock item to modify the current inventory' do
+            subject
+            expect(response.status).to eq 200
+            expect(json_response['count_on_hand']).to eq 40
+          end
+
+          it 'creates a stock movement for the adjusted quantity' do
+            expect { subject }.to change { Spree::StockMovement.count }.by(1)
+            expect(assigns(:stock_movement).quantity).to eq 30
+          end
+
+          context 'tracking inventory' do
+            before do
+              expect(stock_item.should_track_inventory?).to eq true
+            end
+
+            it "updates the stock item's count_on_hand" do
+              subject
+              expect(assigns(:stock_item).count_on_hand).to eq 40
+            end
+          end
+
+          context 'not tracking inventory' do
+            before do
+              stock_item.variant.update_attributes(track_inventory: false)
+            end
+
+            it "doesn't update the stock item's count_on_hand" do
+              subject
+              expect(assigns(:stock_item).count_on_hand).to eq 10
+            end
+          end
+
+          context "attempting to set negative inventory" do
+            let(:count_on_hand) { '-1' }
+
+            it "does not allow negative inventory for the stock item" do
+              subject
+              expect(response.status).to eq 422
+              expect(response.body).to match Spree.t(:stock_not_below_zero)
+              expect(assigns(:stock_item).count_on_hand).to eq 10
+            end
+          end
+        end
+      end
+
+      it 'can delete a stock item' do
+        api_delete :destroy, id: stock_item.to_param
+        expect(response.status).to eq(204)
+        expect { Spree::StockItem.find(stock_item.id) }.to raise_error(ActiveRecord::RecordNotFound)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,172 @@
+require 'spec_helper'
+
+module Spree
+  describe Api::StockLocationsController, :type => :controller do
+    render_views
+
+    let!(:stock_location) { create(:stock_location) }
+    let!(:attributes) { [:id, :name, :address1, :address2, :city, :state_id, :state_name, :country_id, :zipcode, :phone, :active] }
+
+    before do
+      stub_authentication!
+    end
+
+    context "as a user" do
+      describe "#index" do
+        it "can see active stock locations" do
+          api_get :index
+          expect(response).to be_success
+          stock_locations = json_response['stock_locations'].map { |sl| sl['name'] }
+          expect(stock_locations).to include stock_location.name
+        end
+
+        it "cannot see inactive stock locations" do
+          stock_location.update_attributes!(active: false)
+          api_get :index
+          expect(response).to be_success
+          stock_locations = json_response['stock_locations'].map { |sl| sl['name'] }
+          expect(stock_locations).not_to include stock_location.name
+        end
+      end
+
+      describe "#show" do
+        it "can see active stock locations" do
+          api_get :show, id: stock_location.id
+          expect(response).to be_success
+          expect(json_response['name']).to eq stock_location.name
+        end
+
+        it "cannot see inactive stock locations" do
+          stock_location.update_attributes!(active: false)
+          api_get :show, id: stock_location.id
+          expect(response).to be_not_found
+        end
+      end
+
+      describe "#create" do
+        it "cannot create a new stock location" do
+          params = {
+            stock_location: {
+              name: "North Pole",
+              active: true
+            }
+          }
+
+          api_post :create, params
+          response.status.should == 401
+        end
+      end
+
+      describe "#update" do
+        it "cannot update a stock location" do
+          api_put :update, :stock_location => { :name => "South Pole" }, :id => stock_location.to_param
+          response.status.should == 401
+        end
+      end
+
+      describe "#destroy" do
+        it "cannot delete a stock location" do
+          api_put :destroy, :id => stock_location.to_param
+          response.status.should == 401
+        end
+      end
+    end
+
+    context "as an admin" do
+      sign_in_as_admin!
+
+      describe "#index" do
+        it "can see active stock locations" do
+          api_get :index
+          expect(response).to be_success
+          stock_locations = json_response['stock_locations'].map { |sl| sl['name'] }
+          expect(stock_locations).to include stock_location.name
+        end
+
+        it "can see inactive stock locations" do
+          stock_location.update_attributes!(active: false)
+          api_get :index
+          expect(response).to be_success
+          stock_locations = json_response['stock_locations'].map { |sl| sl['name'] }
+          expect(stock_locations).to include stock_location.name
+        end
+
+        it "gets stock location information" do
+          api_get :index
+          json_response['stock_locations'].first.should have_attributes(attributes)
+          json_response['stock_locations'].first['country'].should_not be_nil
+          json_response['stock_locations'].first['state'].should_not be_nil
+        end
+
+        it 'can control the page size through a parameter' do
+          create(:stock_location)
+          api_get :index, per_page: 1
+          json_response['count'].should == 1
+          json_response['current_page'].should == 1
+          json_response['pages'].should == 2
+        end
+
+        it 'can query the results through a paramter' do
+          expected_result = create(:stock_location, name: 'South America')
+          api_get :index, q: { name_cont: 'south' }
+          json_response['count'].should == 1
+          json_response['stock_locations'].first['name'].should eq expected_result.name
+        end
+
+      end
+
+      describe "#show" do
+        it "can see active stock locations" do
+          api_get :show, id: stock_location.id
+          expect(response).to be_success
+          expect(json_response['name']).to eq stock_location.name
+        end
+
+        it "can see inactive stock locations" do
+          stock_location.update_attributes!(active: false)
+          api_get :show, id: stock_location.id
+          expect(response).to be_success
+          expect(json_response['name']).to eq stock_location.name
+        end
+      end
+
+      describe "#create" do
+        it "can create a new stock location" do
+          params = {
+            stock_location: {
+              name: "North Pole",
+              active: true
+            }
+          }
+
+          api_post :create, params
+          response.status.should == 201
+          json_response.should have_attributes(attributes)
+        end
+      end
+
+      describe "#update" do
+        it "can update a stock location" do
+          params = {
+            id: stock_location.to_param,
+            stock_location: {
+              name: "South Pole"
+            }
+          }
+
+          api_put :update, params
+          response.status.should == 200
+          json_response['name'].should eq 'South Pole'
+        end
+      end
+
+      describe "#destroy" do
+        it "can delete a stock location" do
+          api_delete :destroy, id: stock_location.to_param
+          response.status.should == 204
+          lambda { stock_location.reload }.should raise_error(ActiveRecord::RecordNotFound)
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+module Spree
+  describe Api::StockMovementsController, :type => :controller do
+    render_views
+
+    let!(:stock_location) { create(:stock_location_with_items) }
+    let!(:stock_item) { stock_location.stock_items.order(:id).first }
+    let!(:stock_movement) { create(:stock_movement, stock_item: stock_item) }
+    let!(:attributes) { [:id, :quantity, :stock_item_id] }
+
+    before do
+      stub_authentication!
+    end
+
+    context 'as a user' do
+      it 'cannot see a list of stock movements' do
+        api_get :index, stock_location_id: stock_location.to_param
+        expect(response.status).to eq(401)
+      end
+
+      it 'cannot see a stock movement' do
+        api_get :show, stock_location_id: stock_location.to_param, id: stock_movement.id
+        expect(response.status).to eq(404)
+      end
+
+      it 'cannot create a stock movement' do
+        params = {
+          stock_location_id: stock_location.to_param,
+          stock_movement: {
+            stock_item_id: stock_item.to_param
+          }
+        }
+
+        api_post :create, params
+        expect(response.status).to eq(401)
+      end
+    end
+
+    context 'as an admin' do
+      sign_in_as_admin!
+
+      it 'gets list of stock movements' do
+        api_get :index, stock_location_id: stock_location.to_param
+        expect(json_response['stock_movements'].first).to have_attributes(attributes)
+        expect(json_response['stock_movements'].first['stock_item']['count_on_hand']).to eq 11
+      end
+
+      it 'can control the page size through a parameter' do
+        create(:stock_movement, stock_item: stock_item)
+        api_get :index, stock_location_id: stock_location.to_param, per_page: 1
+        expect(json_response['count']).to eq(1)
+        expect(json_response['current_page']).to eq(1)
+        expect(json_response['pages']).to eq(2)
+      end
+
+      it 'can query the results through a paramter' do
+        expected_result = create(:stock_movement, :received, quantity: 10, stock_item: stock_item)
+        api_get :index, stock_location_id: stock_location.to_param, q: { quantity_eq: '10' }
+        expect(json_response['count']).to eq(1)
+      end
+
+      it 'gets a stock movement' do
+        api_get :show, stock_location_id: stock_location.to_param, id: stock_movement.to_param
+        expect(json_response).to have_attributes(attributes)
+        expect(json_response['stock_item_id']).to eq stock_movement.stock_item_id
+      end
+
+      it 'can create a new stock movement' do
+        params = {
+          stock_location_id: stock_location.to_param,
+          stock_movement: {
+            stock_item_id: stock_item.to_param
+          }
+        }
+
+        api_post :create, params
+        expect(response.status).to eq(201)
+        expect(json_response).to have_attributes(attributes)
+      end
+    end
+  end
+end
+

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

@@ -0,0 +1,83 @@
+require 'spec_helper'
+
+module Spree
+  describe Api::StockTransfersController do
+    render_views
+
+    let!(:stock_transfer) { create(:stock_transfer_with_items) }
+    let(:transfer_item)   { stock_transfer.transfer_items.first }
+
+    before do
+      stub_authentication!
+    end
+
+    context "as a normal user" do
+      describe "#receive" do
+        it "cannot receive transfer items from a stock transfer" do
+          api_post :receive, id: stock_transfer.to_param, variant_id: transfer_item.variant.to_param
+          expect(response.status).to eq 401
+        end
+      end
+    end
+
+    context "as an admin" do
+      sign_in_as_admin!
+
+      describe "#receive" do
+        subject do
+          api_post :receive, id: stock_transfer.to_param, variant_id: variant_id
+        end
+
+        context "valid parameters" do
+          let(:variant_id) { transfer_item.variant.to_param }
+
+          it "can receive a transfer items from a stock transfer" do
+            subject
+            expect(response.status).to eq 200
+          end
+
+          it "increments the received quantity for the transfer_item" do
+            expect { subject }.to change { transfer_item.reload.received_quantity }.by(1)
+          end
+
+          it "returns the received transfer item in the response" do
+            subject
+            expect(JSON.parse(response.body)["received_item"]["id"]).to eq transfer_item.id
+          end
+        end
+
+        context "transfer item has been fully received" do
+          let(:variant_id) { transfer_item.variant.to_param }
+
+          before do
+            transfer_item.update_attributes!(expected_quantity: 1, received_quantity: 1)
+          end
+
+          it "returns a 422" do
+            subject
+            expect(response.status).to eq 422
+          end
+
+          it "returns a specific error message" do
+            subject
+            expect(JSON.parse(response.body)["errors"]["received_quantity"]).to eq ["must be less than or equal to 1"]
+          end
+        end
+
+        context "variant is not in the transfer order" do
+          let(:variant_id) { create(:variant).to_param }
+
+          it "returns a 422" do
+            subject
+            expect(response.status).to eq 422
+          end
+
+          it "returns a specific error message" do
+            subject
+            expect(JSON.parse(response.body)["error"]).to eq Spree.t(:item_not_in_stock_transfer)
+          end
+        end
+      end
+    end
+  end
+end