Use correct syntax for change matcher in spec/controllers/spree/admin/shipping_categories_controller_spec.rb

Co-authored-by: Maikel <maikel@email.org.au>
This commit is contained in:
Shen
2021-06-01 20:51:10 +01:00
committed by Shen Sat
parent 99a4be134b
commit 80ae211b72

View File

@@ -13,7 +13,7 @@ module Spree
it "creates a shipping shipping category" do
expect {
spree_post :create, shipping_category: { name: "Frozen" }
}.to change(Spree::ShippingCategory.all, :count).by(1)
}.to change { Spree::ShippingCategory.count }.by(1)
expect(response).to redirect_to spree.admin_shipping_categories_url
end
@@ -31,7 +31,7 @@ module Spree
shipping_category = create(:shipping_category)
expect {
spree_delete :destroy, id: shipping_category.id
}.to change(Spree::ShippingCategory.all, :count).by(-1)
}.to change { Spree::ShippingCategory.count }.by(-1)
expect(response).to redirect_to spree.admin_shipping_categories_url
end