solidus_backend 1.4.2 → 2.0.0.beta1

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

@@ -8,7 +8,7 @@ describe Spree::Admin::ShippingMethodsController, type: :controller do
     shipping_method = stub_model(Spree::ShippingMethod)
     allow(Spree::ShippingMethod).to receive_messages find: shipping_method
     expect(shipping_method.deleted_at).to be_nil
-    delete :destroy, id: 1
+    delete :destroy, params: { id: 1 }
     expect(shipping_method.reload.deleted_at).not_to be_nil
   end
 end

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

@@ -15,7 +15,7 @@ module Spree
         before { request.env["HTTP_REFERER"] = "product_admin_page" }
 
         subject do
-          post :create, { variant_id: variant, stock_location_id: stock_location, stock_movement: { quantity: 1, stock_item_id: stock_item.id } }
+          post :create, params: { variant_id: variant, stock_location_id: stock_location, stock_movement: { quantity: 1, stock_item_id: stock_item.id } }
         end
 
         it "creates a stock movement with originator" do
@@ -31,7 +31,7 @@ module Spree
 
         context "with product_slug param" do
           it "scopes the variants by the product" do
-            get :index, product_slug: variant_1.product.slug
+            get :index, params: { product_slug: variant_1.product.slug }
             expect(assigns(:variants)).to include variant_1
             expect(assigns(:variants)).not_to include variant_2
           end

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

@@ -46,18 +46,18 @@ module Spree
       end
 
       it "searches by stock location" do
-        get :index, q: { source_location_id_or_destination_location_id_eq: ny_store.id }
+        get :index, params: { q: { source_location_id_or_destination_location_id_eq: ny_store.id } }
         expect(assigns(:stock_transfers).count).to eq 1
         expect(assigns(:stock_transfers)).to include(stock_transfer1)
       end
 
       it "filters the closed stock transfers" do
-        get :index, q: { closed_at_null: '1' }
+        get :index, params: { q: { closed_at_null: '1' } }
         expect(assigns(:stock_transfers)).to match_array [stock_transfer1]
       end
 
       it "doesn't filter any stock transfers" do
-        get :index, q: { closed_at_null: '0' }
+        get :index, params: { q: { closed_at_null: '0' } }
         expect(assigns(:stock_transfers)).to match_array [stock_transfer1, stock_transfer2]
       end
     end
@@ -66,7 +66,7 @@ module Spree
       let(:warehouse) { StockLocation.create(name: "Warehouse", active: false) }
 
       subject do
-        post :create, stock_transfer: { source_location_id: warehouse.id, description: nil }
+        post :create, params: { stock_transfer: { source_location_id: warehouse.id, description: nil } }
       end
 
       context "user doesn't have read access to the selected stock location" do
@@ -101,7 +101,7 @@ module Spree
       # Regression spec for Solidus issue #1087
       context "missing source_stock_location parameter" do
         subject do
-          post :create, stock_transfer: { source_location_id: nil, description: nil }
+          post :create, params: { stock_transfer: { source_location_id: nil, description: nil } }
         end
 
         it "sets a flash error" do
@@ -118,7 +118,7 @@ module Spree
       let(:parameters)           { { id: transfer_with_items.to_param } }
 
       subject do
-        get :receive, parameters
+        get :receive, params: parameters
       end
 
       context 'stock transfer is not receivable' do
@@ -171,7 +171,7 @@ module Spree
       end
 
       subject do
-        put :finalize, id: transfer_with_items.to_param
+        put :finalize, params: { id: transfer_with_items.to_param }
       end
 
       context 'stock transfer is not finalizable' do
@@ -229,7 +229,7 @@ module Spree
       end
 
       subject do
-        put :close, id: transfer_with_items.to_param
+        put :close, params: { id: transfer_with_items.to_param }
       end
 
       context 'stock transfer is not receivable' do
@@ -321,7 +321,7 @@ module Spree
       let(:warehouse_stock_item) { warehouse.stock_items.find_by(variant: transfer_variant) }
       let(:ny_stock_item) { ny_store.stock_items.find_by(variant: transfer_variant) }
 
-      subject { put :ship, id: stock_transfer.number }
+      subject { put :ship, params: { id: stock_transfer.number } }
 
       before do
         warehouse_stock_item.set_count_on_hand(1)

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

@@ -20,7 +20,7 @@ describe Spree::Admin::StoreCreditsController do
     let!(:store_credit) { create(:store_credit, user: user, category: a_credit_category) }
     let!(:event)        { create(:store_credit_auth_event, store_credit: store_credit, created_at: 5.days.ago) }
 
-    before { get :show, user_id: user.id, id: store_credit.id }
+    before { get :show, params: { user_id: user.id, id: store_credit.id  } }
 
     it "sets the store_credit variable to a new store credit model" do
       expect(assigns(:store_credit)).to eq store_credit
@@ -33,12 +33,12 @@ describe Spree::Admin::StoreCreditsController do
   end
 
   describe "#new" do
-    before { get :new, user_id: create(:user).id }
+    before { get :new, params: { user_id: create(:user).id } }
     it { expect(assigns(:credit_categories)).to eq [a_credit_category, b_credit_category] }
   end
 
   describe "#create" do
-    subject { post :create, parameters }
+    subject { post :create, params: parameters }
 
     before  {
       allow(controller).to receive_messages(try_spree_current_user: admin_user)
@@ -94,7 +94,7 @@ describe Spree::Admin::StoreCreditsController do
   describe "#edit_amount" do
     let!(:store_credit) { create(:store_credit, user: user, category: a_credit_category) }
 
-    before { get :edit_amount, user_id: user.id, id: store_credit.id }
+    before { get :edit_amount, params: { user_id: user.id, id: store_credit.id } }
 
     it_behaves_like "update reason loader"
 
@@ -106,7 +106,7 @@ describe Spree::Admin::StoreCreditsController do
   describe "#edit_validity" do
     let!(:store_credit) { create(:store_credit, user: user, category: a_credit_category) }
 
-    before { get :edit_validity, user_id: user.id, id: store_credit.id }
+    before { get :edit_validity, params: { user_id: user.id, id: store_credit.id } }
 
     it_behaves_like "update reason loader"
 
@@ -119,7 +119,7 @@ describe Spree::Admin::StoreCreditsController do
     let(:memo)          { "New memo" }
     let!(:store_credit) { create(:store_credit, user: user) }
 
-    subject { put :update, parameters.merge(format: :json) }
+    subject { put :update, params: parameters.merge(format: :json) }
 
     before  { allow(controller).to receive_messages(try_spree_current_user: admin_user) }
 
@@ -182,7 +182,7 @@ describe Spree::Admin::StoreCreditsController do
       }
     end
 
-    subject { put :update_amount, parameters }
+    subject { put :update_amount, params: parameters }
 
     before  { allow(controller).to receive_messages(try_spree_current_user: admin_user) }
 
@@ -272,7 +272,7 @@ describe Spree::Admin::StoreCreditsController do
       }
     end
 
-    subject { put :invalidate, parameters }
+    subject { put :invalidate, params: parameters }
 
     it "attempts to invalidate the store credit" do
       expect { subject }.to change { store_credit.reload.invalidated_at }.from(nil)

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

@@ -24,7 +24,7 @@ describe Spree::Admin::UsersController, type: :controller do
     end
 
     it "redirects to edit" do
-      get :show, id: user.id
+      get :show, params: { id: user.id }
       expect(response).to redirect_to spree.edit_admin_user_path(user)
     end
   end
@@ -42,7 +42,7 @@ describe Spree::Admin::UsersController, type: :controller do
 
       it "allows admins to update a user's API key" do
         expect {
-          put :generate_api_key, id: user.id
+          put :generate_api_key, params: { id: user.id }
         }.to change { user.reload.spree_api_key }
         expect(response).to redirect_to(spree.edit_admin_user_path(user))
       end
@@ -50,7 +50,7 @@ describe Spree::Admin::UsersController, type: :controller do
       it "allows admins to clear a user's API key" do
         user.generate_spree_api_key!
         expect {
-          put :clear_api_key, id: user.id
+          put :clear_api_key, params: { id: user.id }
         }.to change{ user.reload.spree_api_key }.to(nil)
         expect(response).to redirect_to(spree.edit_admin_user_path(user))
       end
@@ -86,42 +86,42 @@ describe Spree::Admin::UsersController, type: :controller do
       end
 
       it "can create user with roles" do
-        post :create, { user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
+        post :create, params: { user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
         expect(user.spree_roles).to eq([dummy_role])
       end
 
       it "can create user without roles" do
-        post :create, { user: { first_name: "Bob" } }
+        post :create, params: { user: { first_name: "Bob" } }
         expect(user.spree_roles).to eq([])
       end
     end
 
     context "when the user cannot manage roles" do
       it "cannot assign users roles" do
-        post :create, { user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
+        post :create, params: { user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
         expect(user.spree_roles).to eq([])
       end
 
       it "can create user without roles" do
-        post :create, { user: { first_name: "Bob" } }
+        post :create, params: { user: { first_name: "Bob" } }
         expect(user.spree_roles).to eq([])
       end
     end
 
     it "can create a shipping_address" do
-      post :create, { user: { ship_address_attributes: valid_address_attributes } }
+      post :create, params: { user: { ship_address_attributes: valid_address_attributes } }
       expect(user.reload.ship_address.city).to eq('New York')
     end
 
     it "can create a billing_address" do
-      post :create, { user: { bill_address_attributes: valid_address_attributes } }
+      post :create, params: { user: { bill_address_attributes: valid_address_attributes } }
       expect(user.reload.bill_address.city).to eq('New York')
     end
 
     it "can set stock locations" do
       location = Spree::StockLocation.create(name: "my_location")
       location_2 = Spree::StockLocation.create(name: "my_location_2")
-      post :create, { user: { stock_location_ids: [location.id, location_2.id] } }
+      post :create, params: { user: { stock_location_ids: [location.id, location_2.id] } }
       expect(user.stock_locations).to match_array([location, location_2])
     end
   end
@@ -142,14 +142,14 @@ describe Spree::Admin::UsersController, type: :controller do
 
       it "can set roles" do
         expect {
-          put :update, { id: user.id, user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
+          put :update, params: { id: user.id, user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
         }.to change { user.reload.spree_roles.to_a }.to([dummy_role])
       end
 
       it "can clear roles" do
         user.spree_roles << dummy_role
         expect {
-          put :update, { id: user.id, user: { first_name: "Bob", spree_role_ids: [] } }
+          put :update, params: { id: user.id, user: { first_name: "Bob", spree_role_ids: [] } }
         }.to change { user.reload.spree_roles.to_a }.to([])
       end
     end
@@ -157,14 +157,14 @@ describe Spree::Admin::UsersController, type: :controller do
     context "when the user cannot manage roles" do
       it "cannot set roles" do
         expect {
-          put :update, { id: user.id, user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
+          put :update, params: { id: user.id, user: { first_name: "Bob", spree_role_ids: [dummy_role.id] } }
         }.not_to change { user.reload.spree_roles.to_a }
       end
 
       it "cannot clear roles" do
         user.spree_roles << dummy_role
         expect {
-          put :update, { id: user.id, user: { first_name: "Bob" } }
+          put :update, params: { id: user.id, user: { first_name: "Bob" } }
         }.not_to change { user.reload.spree_roles.to_a }
       end
     end
@@ -176,7 +176,7 @@ describe Spree::Admin::UsersController, type: :controller do
 
       it "can change email of a user" do
         expect {
-          put :update, { id: user.id, user: { email: "bob@example.com" } }
+          put :update, params: { id: user.id, user: { email: "bob@example.com" } }
         }.to change { user.reload.email }.to("bob@example.com")
       end
     end
@@ -188,25 +188,25 @@ describe Spree::Admin::UsersController, type: :controller do
 
       it "cannot change email of a user" do
         expect {
-          put :update, { id: user.id, user: { email: "bob@example.com" } }
+          put :update, params: { id: user.id, user: { email: "bob@example.com" } }
         }.not_to change { user.reload.email }
       end
     end
 
     it "can update ship_address attributes" do
-      post :update, { id: user.id, user: { ship_address_attributes: valid_address_attributes } }
+      post :update, params: { id: user.id, user: { ship_address_attributes: valid_address_attributes } }
       expect(user.reload.ship_address.city).to eq('New York')
     end
 
     it "can update bill_address attributes" do
-      post :update, { id: user.id, user: { bill_address_attributes: valid_address_attributes } }
+      post :update, params: { id: user.id, user: { bill_address_attributes: valid_address_attributes } }
       expect(user.reload.bill_address.city).to eq('New York')
     end
 
     it "can set stock locations" do
       location = Spree::StockLocation.create(name: "my_location")
       location_2 = Spree::StockLocation.create(name: "my_location_2")
-      post :update, { id: user.id, user: { stock_location_ids: [location.id, location_2.id] } }
+      post :update, params: { id: user.id, user: { stock_location_ids: [location.id, location_2.id] } }
       expect(user.stock_locations).to match_array([location, location_2])
     end
   end
@@ -220,13 +220,13 @@ describe Spree::Admin::UsersController, type: :controller do
     before { user.orders << order }
 
     it "assigns a list of the users orders" do
-      get :orders, { id: user.id }
+      get :orders, params: { id: user.id }
       expect(assigns[:orders].count).to eq 1
       expect(assigns[:orders].first).to eq order
     end
 
     it "assigns a ransack search for Spree::Order" do
-      get :orders, { id: user.id }
+      get :orders, params: { id: user.id }
       expect(assigns[:search]).to be_a Ransack::Search
       expect(assigns[:search].klass).to eq Spree::Order
     end
@@ -241,13 +241,13 @@ describe Spree::Admin::UsersController, type: :controller do
     before { user.orders << order }
 
     it "assigns a list of the users orders" do
-      get :items, { id: user.id }
+      get :items, params: { id: user.id }
       expect(assigns[:orders].count).to eq 1
       expect(assigns[:orders].first).to eq order
     end
 
     it "assigns a ransack search for Spree::Order" do
-      get :items, { id: user.id }
+      get :items, params: { id: user.id }
       expect(assigns[:search]).to be_a Ransack::Search
       expect(assigns[:search].klass).to eq Spree::Order
     end

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

@@ -14,7 +14,7 @@ module Spree
 
         context "deleted is not requested" do
           it "does not assign deleted variants for a requested product" do
-            get :index, product_id: product.slug
+            get :index, params: { product_id: product.slug }
             expect(assigns(:collection)).to include variant_1
             expect(assigns(:collection)).not_to include variant_2
           end
@@ -22,7 +22,7 @@ module Spree
 
         context "deleted is requested" do
           it "assigns deleted along with non-deleted variants for a requested product" do
-            get :index, product_id: product.slug, deleted: "on"
+            get :index, params: { product_id: product.slug, deleted: "on" }
             expect(assigns(:collection)).to include variant_1
             expect(assigns(:collection)).to include variant_2
           end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_backend
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 2.0.0.beta1
 platform: ruby
 authors:
 - Solidus Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-12 00:00:00.000000000 Z
+date: 2016-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solidus_api
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 2.0.0.beta1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 2.0.0.beta1
 - !ruby/object:Gem::Dependency
   name: solidus_core
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 2.0.0.beta1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: 2.0.0.beta1
 - !ruby/object:Gem::Dependency
   name: sass-rails
   requirement: !ruby/object:Gem::Requirement
@@ -900,13 +900,13 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements:
 - none
 rubyforge_project: solidus_backend
-rubygems_version: 2.6.11
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Admin interface for the Solidus e-commerce framework.