solidus_api 2.1.1 → 2.2.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1de6bd3074f0732a5a2981a9357923fdafeed688
-  data.tar.gz: f9c892017071238dd6677d65850650e104d179e8
+  metadata.gz: b950a6d9f7c428116e5210e27a72d5fb07df830a
+  data.tar.gz: 2ce2d4e97ae5f016c16adc9769a70f1cd52e5317
 SHA512:
-  metadata.gz: c019fe80b3975012c74245da544d6ff96c13644ba271cd9ecfb34bdf993c09250d897a6ab255ade2a816cce55951ef21f8eb1c244991df9f95e2a8ef12f7a5dc
-  data.tar.gz: 663ae49c937ad78351ba05979a49504fa02d1925eb185b8c794ba7ae1fd76ec67f122f3e974639efb51d8780d1a10f43db9e5fb642f8a7fc333fbde1ba2922d0
+  metadata.gz: 2685ca4dcbc794549d239d4c3129f76d9b1acf56903a5d4cf2ecab8d64e2778deac7dee07bbd923b42f405778db49c7164f4a32f22747100dddb6c6409e8ada6
+  data.tar.gz: 74a05940e48b28abd813beaa2fc56902b1f59657128dd312fe77016c22e2abf79887311dca22e7417d21b9064d039728dae9776254a96c8c4677bb62220335d9

data/app/controllers/spree/api/base_controller.rb

@@ -116,13 +116,13 @@ module Spree
 
       def product_scope
         if can?(:admin, Spree::Product)
-          scope = Product.with_deleted.accessible_by(current_ability, :read).includes(*product_includes)
+          scope = Spree::Product.with_deleted.accessible_by(current_ability, :read).includes(*product_includes)
 
           unless params[:show_deleted]
             scope = scope.not_deleted
           end
         else
-          scope = Product.accessible_by(current_ability, :read).available.includes(*product_includes)
+          scope = Spree::Product.accessible_by(current_ability, :read).available.includes(*product_includes)
         end
 
         scope

data/app/controllers/spree/api/countries_controller.rb

@@ -4,13 +4,13 @@ module Spree
       skip_before_action :authenticate_user
 
       def index
-        @countries = Country.
+        @countries = Spree::Country.
           accessible_by(current_ability, :read).
           ransack(params[:q]).
           result.
           order('name ASC')
 
-        country = Country.order("updated_at ASC").last
+        country = Spree::Country.order("updated_at ASC").last
 
         if stale?(country)
           @countries = paginate(@countries)
@@ -19,7 +19,7 @@ module Spree
       end
 
       def show
-        @country = Country.accessible_by(current_ability, :read).find(params[:id])
+        @country = Spree::Country.accessible_by(current_ability, :read).find(params[:id])
         respond_with(@country)
       end
     end

data/app/controllers/spree/api/images_controller.rb

@@ -7,7 +7,7 @@ module Spree
       end
 
       def show
-        @image = Image.accessible_by(current_ability, :read).find(params[:id])
+        @image = Spree::Image.accessible_by(current_ability, :read).find(params[:id])
         respond_with(@image)
       end
 

data/app/controllers/spree/api/inventory_units_controller.rb

@@ -24,7 +24,7 @@ module Spree
       private
 
       def inventory_unit
-        @inventory_unit ||= InventoryUnit.accessible_by(current_ability, :read).find(params[:id])
+        @inventory_unit ||= Spree::InventoryUnit.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def prepare_event

data/app/controllers/spree/api/orders_controller.rb

@@ -13,7 +13,7 @@ module Spree
       around_action :lock_order, except: [:create, :mine, :current, :index]
 
       # Dynamically defines our stores checkout steps to ensure we check authorization on each step.
-      Order.checkout_steps.keys.each do |step|
+      Spree::Order.checkout_steps.keys.each do |step|
         define_method step do
           authorize! :update, @order, params[:token]
         end
@@ -27,18 +27,8 @@ module Spree
 
       def create
         authorize! :create, Order
-
-        if can?(:admin, Order)
-          @order = Spree::Core::Importer::Order.import(determine_order_user, order_params)
-          respond_with(@order, default_template: :show, status: 201)
-        else
-          @order = Spree::Order.create!(user: current_api_user, store: current_store)
-          if OrderUpdateAttributes.new(@order, order_params).apply
-            respond_with(@order, default_template: :show, status: 201)
-          else
-            invalid_resource!(@order)
-          end
-        end
+        @order = Spree::Core::Importer::Order.import(determine_order_user, order_params)
+        respond_with(@order, default_template: :show, status: 201)
       end
 
       def empty
@@ -49,7 +39,7 @@ module Spree
 
       def index
         authorize! :index, Order
-        @orders = paginate(Order.ransack(params[:q]).result)
+        @orders = paginate(Spree::Order.ransack(params[:q]).result)
         respond_with(@orders)
       end
 

data/app/controllers/spree/api/payments_controller.rb

@@ -16,7 +16,6 @@ module Spree
       end
 
       def create
-        @order.validate_payments_attributes(payment_params)
         @payment = PaymentCreate.new(@order, payment_params).build
         if @payment.save
           respond_with(@payment, status: 201, default_template: :show)

data/app/controllers/spree/api/products_controller.rb

@@ -123,7 +123,7 @@ module Spree
 
       def set_up_shipping_category
         if shipping_category = params[:product].delete(:shipping_category)
-          id = ShippingCategory.find_or_create_by(name: shipping_category).id
+          id = Spree::ShippingCategory.find_or_create_by(name: shipping_category).id
           params[:product][:shipping_category_id] = id
         end
       end

data/app/controllers/spree/api/shipments_controller.rb

@@ -95,11 +95,11 @@ module Spree
       private
 
       def load_transfer_params
-        @original_shipment         = Spree::Shipment.where(number: params[:original_shipment_number]).first
+        @original_shipment         = Spree::Shipment.find_by!(number: params[:original_shipment_number])
         @order                     = @original_shipment.order
         @variant                   = Spree::Variant.find(params[:variant_id])
         @quantity                  = params[:quantity].to_i
-        authorize! :read, @original_shipment
+        authorize! [:update, :destroy], @original_shipment
         authorize! :create, Shipment
       end
 

data/app/controllers/spree/api/states_controller.rb

@@ -23,10 +23,10 @@ module Spree
 
       def scope
         if params[:country_id]
-          @country = Country.accessible_by(current_ability, :read).find(params[:country_id])
+          @country = Spree::Country.accessible_by(current_ability, :read).find(params[:country_id])
           return @country.states.accessible_by(current_ability, :read)
         else
-          return State.accessible_by(current_ability, :read)
+          return Spree::State.accessible_by(current_ability, :read)
         end
       end
     end

data/app/controllers/spree/api/stock_items_controller.rb

@@ -28,7 +28,7 @@ module Spree
       end
 
       def update
-        @stock_item = StockItem.accessible_by(current_ability, :update).find(params[:id])
+        @stock_item = Spree::StockItem.accessible_by(current_ability, :update).find(params[:id])
         @stock_location = @stock_item.stock_location
 
         adjustment = count_on_hand_adjustment
@@ -46,7 +46,7 @@ module Spree
       end
 
       def destroy
-        @stock_item = StockItem.accessible_by(current_ability, :destroy).find(params[:id])
+        @stock_item = Spree::StockItem.accessible_by(current_ability, :destroy).find(params[:id])
         @stock_item.destroy
         respond_with(@stock_item, status: 204)
       end
@@ -54,7 +54,7 @@ module Spree
       private
 
       def load_stock_location
-        @stock_location ||= StockLocation.accessible_by(current_ability).find(params.fetch(:stock_location_id))
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability).find(params.fetch(:stock_location_id))
       end
 
       def scope

data/app/controllers/spree/api/stock_locations_controller.rb

@@ -21,7 +21,7 @@ module Spree
 
       def create
         authorize! :create, StockLocation
-        @stock_location = StockLocation.new(stock_location_params)
+        @stock_location = Spree::StockLocation.new(stock_location_params)
         if @stock_location.save
           respond_with(@stock_location, status: 201, default_template: :show)
         else
@@ -47,7 +47,7 @@ module Spree
       private
 
       def stock_location
-        @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:id])
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def stock_location_params

data/app/controllers/spree/api/stock_movements_controller.rb

@@ -27,7 +27,7 @@ module Spree
       private
 
       def stock_location
-        @stock_location ||= StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
+        @stock_location ||= Spree::StockLocation.accessible_by(current_ability, :read).find(params[:stock_location_id])
       end
 
       def scope

data/app/controllers/spree/api/stores_controller.rb

@@ -5,13 +5,13 @@ module Spree
 
       def index
         authorize! :read, Store
-        @stores = Store.accessible_by(current_ability, :read).all
+        @stores = Spree::Store.accessible_by(current_ability, :read).all
         respond_with(@stores)
       end
 
       def create
         authorize! :create, Store
-        @store = Store.new(store_params)
+        @store = Spree::Store.new(store_params)
         @store.code = params[:store][:code]
         if @store.save
           respond_with(@store, status: 201, default_template: :show)
@@ -43,7 +43,7 @@ module Spree
       private
 
       def get_store
-        @store = Store.find(params[:id])
+        @store = Spree::Store.find(params[:id])
       end
 
       def store_params

data/app/controllers/spree/api/taxonomies_controller.rb

@@ -20,7 +20,7 @@ module Spree
 
       def create
         authorize! :create, Taxonomy
-        @taxonomy = Taxonomy.new(taxonomy_params)
+        @taxonomy = Spree::Taxonomy.new(taxonomy_params)
         if @taxonomy.save
           respond_with(@taxonomy, status: 201, default_template: :show)
         else
@@ -55,7 +55,7 @@ module Spree
       end
 
       def taxonomy
-        @taxonomy ||= Taxonomy.accessible_by(current_ability, :read).find(params[:id])
+        @taxonomy ||= Spree::Taxonomy.accessible_by(current_ability, :read).find(params[:id])
       end
 
       def taxonomy_params

data/app/controllers/spree/api/transfer_items_controller.rb

@@ -3,7 +3,7 @@ module Spree
     class TransferItemsController < Spree::Api::BaseController
       def create
         authorize! :create, TransferItem
-        stock_transfer = StockTransfer.accessible_by(current_ability, :update).find_by(number: params[:stock_transfer_id])
+        stock_transfer = Spree::StockTransfer.accessible_by(current_ability, :update).find_by(number: params[:stock_transfer_id])
         @transfer_item = stock_transfer.transfer_items.build(transfer_item_params)
         if @transfer_item.save
           respond_with(@transfer_item, status: 201, default_template: :show)
@@ -14,7 +14,7 @@ module Spree
 
       def update
         authorize! :update, TransferItem
-        @transfer_item = TransferItem.accessible_by(current_ability, :update).find(params[:id])
+        @transfer_item = Spree::TransferItem.accessible_by(current_ability, :update).find(params[:id])
         if @transfer_item.update_attributes(transfer_item_params)
           respond_with(@transfer_item, status: 200, default_template: :show)
         else
@@ -24,7 +24,7 @@ module Spree
 
       def destroy
         authorize! :destroy, TransferItem
-        @transfer_item = TransferItem.accessible_by(current_ability, :destroy).find(params[:id])
+        @transfer_item = Spree::TransferItem.accessible_by(current_ability, :destroy).find(params[:id])
         if @transfer_item.destroy
           respond_with(@transfer_item, status: 200, default_template: :show)
         else

data/app/controllers/spree/api/variants_controller.rb

@@ -58,7 +58,7 @@ module Spree
         if @product
           variants = @product.variants_including_master
         else
-          variants = Variant
+          variants = Spree::Variant
         end
 
         if current_ability.can?(:manage, Variant) && params[:show_deleted]

data/app/controllers/spree/api/zones_controller.rb

@@ -3,7 +3,7 @@ module Spree
     class ZonesController < Spree::Api::BaseController
       def create
         authorize! :create, Zone
-        @zone = Zone.new(zone_params)
+        @zone = Spree::Zone.new(zone_params)
         if @zone.save
           respond_with(@zone, status: 201, default_template: :show)
         else
@@ -18,7 +18,7 @@ module Spree
       end
 
       def index
-        @zones = Zone.
+        @zones = Spree::Zone.
           accessible_by(current_ability, :read).
           order('name ASC').
           ransack(params[:q]).

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

@@ -163,19 +163,6 @@ module Spree
         expect(response.status).to eq(200)
       end
 
-      context "with disallowed payment method" do
-        it "returns not found" do
-          order.update_column(:state, "payment")
-          allow_any_instance_of(Spree::Gateway::Bogus).to receive(:source_required?).and_return(false)
-          @payment_method.update!(available_to_users: false)
-          expect {
-            api_put :update, id: order.to_param, order_token: order.guest_token, order: { payments_attributes: [{ payment_method_id: @payment_method.id }] }
-          }.not_to change { Spree::Payment.count }
-          expect(response.status).to eq(404)
-        end
-      end
-
-
       it "returns errors when source is required and missing" do
         order.update_column(:state, "payment")
         api_put :update, id: order.to_param, order_token: order.guest_token,
@@ -281,7 +268,7 @@ module Spree
               payments_attributes: [
                 {
                   source_attributes: {
-                    existing_card_id: credit_card.id.to_s,
+                    wallet_payment_source_id: wallet_payment_source.id.to_param,
                     verification_value: '456'
                   }
                 }
@@ -290,7 +277,11 @@ module Spree
           }
         end
 
-        let!(:credit_card) do
+        let!(:wallet_payment_source) do
+          order.user.wallet.add(credit_card)
+        end
+
+        let(:credit_card) do
           create(:credit_card, user_id: order.user_id, payment_method_id: @payment_method.id)
         end
 
@@ -307,6 +298,34 @@ module Spree
           expect(response.status).to eq 200
           expect(order.credit_cards).to match_array [credit_card]
         end
+
+        context 'with deprecated existing_card_id param' do
+          let(:params) do
+            {
+              id: order.to_param,
+              order_token: order.guest_token,
+              order: {
+                payments_attributes: [
+                  {
+                    source_attributes: {
+                      existing_card_id: credit_card.id.to_param,
+                      verification_value: '456'
+                    }
+                  }
+                ]
+              }
+            }
+          end
+
+          it 'succeeds' do
+            Spree::Deprecation.silence do
+              api_put(:update, params)
+            end
+
+            expect(response.status).to eq 200
+            expect(order.credit_cards).to match_array [credit_card]
+          end
+        end
       end
 
       it "returns the order if the order is already complete" do
@@ -402,7 +421,7 @@ module Spree
         end
 
         it "returns a sensible error when no payment method is specified" do
-          # api_put :complete, :id => order.to_param, :order_token => order.token, :order => {}
+          # api_put :complete, id: order.to_param, order_token: order.token, order: {}
           subject
           expect(json_response["errors"]["base"]).to include(Spree.t(:no_payment_found))
         end
@@ -411,7 +430,7 @@ module Spree
           let(:params) { super().merge(expected_total: order.total + 1) }
 
           it "returns an error if expected_total is present and does not match actual total" do
-            # api_put :complete, :id => order.to_param, :order_token => order.token, :expected_total => order.total + 1
+            # api_put :complete, id: order.to_param, order_token: order.token, expected_total: order.total + 1
             subject
             expect(response.status).to eq(400)
             expect(json_response['errors']['expected_total']).to include(Spree.t(:expected_total_mismatch, scope: 'api.order'))

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

@@ -32,9 +32,8 @@ module Spree
     describe "POST create" do
       let(:target_user) { create :user }
       let(:date_override) { Time.parse('2015-01-01') }
-      let(:attributes) { { user_id: target_user.id, created_at: date_override, email: target_user.email } }
 
-      subject { api_post :create, order: attributes }
+      subject { api_post :create, order: { user_id: target_user.id, created_at: date_override, email: target_user.email } }
 
       context "when the current user cannot administrate the order" do
         stub_authorization! do |_|
@@ -43,37 +42,12 @@ module Spree
 
         it "does not include unpermitted params, or allow overriding the user", focus: true do
           subject
-          expect(response).to be_success
           order = Spree::Order.last
           expect(order.user).to eq current_api_user
           expect(order.email).to eq target_user.email
         end
 
         it { is_expected.to be_success }
-
-        context 'creating payment' do
-          let(:attributes) { super().merge(payments_attributes: [{ payment_method_id: payment_method.id }]) }
-
-          context "with allowed payment method" do
-            let!(:payment_method) { create(:check_payment_method, name: "allowed" ) }
-            it { is_expected.to be_success }
-            it "creates a payment" do
-              expect {
-                subject
-              }.to change { Spree::Payment.count }.by(1)
-            end
-          end
-
-          context "with disallowed payment method" do
-            let!(:payment_method) { create(:check_payment_method, name: "forbidden", available_to_users: false) }
-            it { is_expected.to be_not_found }
-            it "creates no payments" do
-              expect {
-                subject
-              }.not_to change { Spree::Payment.count }
-            end
-          end
-        end
       end
 
       context "when the current user can administrate the order" do
@@ -123,30 +97,6 @@ module Spree
             subject
           }.to_not change{ order.reload.number }
         end
-
-        context 'creating payment' do
-          let(:order_params) { super().merge(payments_attributes: [{ payment_method_id: payment_method.id }]) }
-
-          context "with allowed payment method" do
-            let!(:payment_method) { create(:check_payment_method, name: "allowed" ) }
-            it { is_expected.to be_success }
-            it "creates a payment" do
-              expect {
-                subject
-              }.to change { Spree::Payment.count }.by(1)
-            end
-          end
-
-          context "with disallowed payment method" do
-            let!(:payment_method) { create(:check_payment_method, name: "forbidden", available_to_users: false) }
-            it { is_expected.to be_not_found }
-            it "creates no payments" do
-              expect {
-                subject
-              }.not_to change { Spree::Payment.count }
-            end
-          end
-        end
       end
 
       context "when the user can administer the order" do
@@ -383,7 +333,10 @@ module Spree
 
     # Regression test for https://github.com/spree/spree/issues/3404
     it "can specify additional parameters for a line item" do
-      expect_any_instance_of(Spree::LineItem).to receive(:special=).with("foo")
+      expect(Order).to receive(:create!).and_return(order = Spree::Order.new)
+      allow(order).to receive(:associate_user!)
+      allow(order).to receive_message_chain(:contents, :add).and_return(line_item = double('LineItem'))
+      expect(line_item).to receive(:update_attributes!).with(hash_including("special" => "foo"))
 
       allow(controller).to receive_messages(permitted_line_item_attributes: [:id, :variant_id, :quantity, :special])
       api_post :create, order: {

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

@@ -45,17 +45,6 @@ module Spree
             expect(response.status).to eq(201)
             expect(json_response).to have_attributes(attributes)
           end
-
-          context "disallowed payment method" do
-            it "does not create a new payment" do
-              PaymentMethod.first.update!(available_to_users: false)
-
-              expect {
-                api_post :create, payment: { payment_method_id: PaymentMethod.first.id, amount: 50 }
-              }.not_to change { Spree::Payment.count }
-              expect(response.status).to eq(404)
-            end
-          end
         end
 
         context "payment source is required" do

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

@@ -0,0 +1,136 @@
+require 'spec_helper'
+
+RSpec.describe Spree::Api::ShipmentsController, type: :request do
+  let(:user) { create(:admin_user, spree_api_key: 'abc123') }
+  let(:stock_item) { create(:stock_item, backorderable: false) }
+  let(:variant) { stock_item.variant }
+
+  let(:order) do
+    create(
+      :completed_order_with_totals,
+      user: user,
+      line_items_attributes: [
+        {
+          variant: variant
+        }
+      ]
+    )
+  end
+
+  let(:shipment) { order.shipments.first }
+
+  describe "POST /api/shipments/transfer_to_location" do
+    let(:stock_location) { create(:stock_location) }
+    let(:source_shipment) { order.shipments.first }
+    let(:parsed_response) { JSON.parse(response.body) }
+    let(:stock_location_id) { stock_location.id }
+
+    subject do
+      post "/api/shipments/transfer_to_location.json",
+        params: {
+          original_shipment_number: source_shipment.number,
+          stock_location_id: stock_location_id,
+          quantity: 1,
+          variant_id: variant.id,
+          token: user.spree_api_key
+        }
+    end
+
+    context "for a successful transfer" do
+      before do
+        stock_location.restock(variant, 1)
+      end
+
+      it "returns the correct message" do
+        subject
+        expect(response).to be_success
+        expect(parsed_response["success"]).to be true
+        expect(parsed_response["message"]).to eq("Variants successfully transferred")
+      end
+    end
+
+    context "if the source shipment can not be found" do
+      let(:stock_location_id) { 9999 }
+
+      it "returns a 404" do
+        subject
+        expect(response).to be_not_found
+        expect(parsed_response["error"]).to eq("The resource you were looking for could not be found.")
+      end
+    end
+
+    context "if the user can not update shipments" do
+      let(:user) { create(:user, spree_api_key: 'abc123') }
+
+      custom_authorization! do |_|
+        can :read, Spree::Shipment
+        cannot :update, Spree::Shipment
+        can :create, Spree::Shipment
+        can :destroy, Spree::Shipment
+      end
+
+      it "is not authorized" do
+        subject
+        expect(response).to be_unauthorized
+      end
+    end
+
+    context "if the user can not destroy shipments" do
+      let(:user) { create(:user, spree_api_key: 'abc123') }
+
+      custom_authorization! do |_|
+        can :read, Spree::Shipment
+        can :update, Spree::Shipment
+        cannot :destroy, Spree::Shipment
+        can :create, Spree::Shipment
+      end
+
+      it "is not authorized" do
+        subject
+        expect(response).to be_unauthorized
+      end
+    end
+  end
+
+  describe "POST /api/shipments/transfer_to_shipment" do
+    let(:stock_location) { create(:stock_location) }
+    let(:source_shipment) { order.shipments.first }
+    let(:target_shipment) { order.shipments.create(stock_location: stock_location) }
+    let(:parsed_response) { JSON.parse(response.body) }
+    let(:source_shipment_number) { source_shipment.number }
+
+    subject do
+      post "/api/shipments/transfer_to_shipment.json",
+        params: {
+          original_shipment_number: source_shipment_number,
+          target_shipment_number: target_shipment.number,
+          quantity: 1,
+          variant_id: variant.id,
+          token: user.spree_api_key
+        }
+    end
+
+    context "for a successful transfer" do
+      before do
+        stock_location.restock(variant, 1)
+      end
+
+      it "returns the correct message" do
+        subject
+        expect(response).to be_success
+        expect(parsed_response["success"]).to be true
+        expect(parsed_response["message"]).to eq("Variants successfully transferred")
+      end
+    end
+
+    context "if the source shipment can not be found" do
+      let(:source_shipment_number) { 9999 }
+
+      it "returns a 404" do
+        subject
+        expect(response).to be_not_found
+        expect(parsed_response["error"]).to eq("The resource you were looking for could not be found.")
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_api
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0.beta1
 platform: ruby
 authors:
 - Solidus Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-12 00:00:00.000000000 Z
+date: 2017-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solidus_core
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.1.1
+        version: 2.2.0.beta1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.1.1
+        version: 2.2.0.beta1
 - !ruby/object:Gem::Dependency
   name: rabl
   requirement: !ruby/object:Gem::Requirement
@@ -252,6 +252,7 @@ files:
 - spec/requests/api/address_books_spec.rb
 - spec/requests/rabl_cache_spec.rb
 - spec/requests/ransackable_attributes_spec.rb
+- spec/requests/spree/api/shipments_controller_spec.rb
 - spec/shared_examples/protect_product_actions.rb
 - spec/spec_helper.rb
 - spec/support/controller_hacks.rb
@@ -280,7 +281,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.8.23
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.10
 signing_key: 
 specification_version: 4
 summary: REST API for the Solidus e-commerce framework.
@@ -327,6 +328,7 @@ test_files:
 - spec/requests/api/address_books_spec.rb
 - spec/requests/rabl_cache_spec.rb
 - spec/requests/ransackable_attributes_spec.rb
+- spec/requests/spree/api/shipments_controller_spec.rb
 - spec/shared_examples/protect_product_actions.rb
 - spec/spec_helper.rb
 - spec/support/controller_hacks.rb