From a5ff4d6853dab7c102d1edf949cb4eeb2bc6284e Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Wed, 2 Sep 2020 20:42:50 +0100 Subject: [PATCH] Remove unneeded setup code and remove unnecessary reference to FactoryGirl --- spec/models/spree/order_spec.rb | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/spec/models/spree/order_spec.rb b/spec/models/spree/order_spec.rb index 48a1a50d7f..77fa763f46 100644 --- a/spec/models/spree/order_spec.rb +++ b/spec/models/spree/order_spec.rb @@ -6,10 +6,6 @@ describe Spree::Order do let(:user) { build(:user, email: "spree@example.com") } let(:order) { build(:order, user: user) } - before do - allow(Spree::LegacyUser).to receive_messages(current: build(:user, id: 123)) - end - context "#products" do let(:order) { create(:order_with_line_items) } @@ -36,8 +32,8 @@ describe Spree::Order do context "#associate_user!" do it "should associate a user with a persisted order" do - order = FactoryGirl.create(:order_with_line_items, created_by: nil) - user = FactoryGirl.create(:user) + order = create(:order_with_line_items, created_by: nil) + user = create(:user) order.user = nil order.email = nil @@ -55,8 +51,8 @@ describe Spree::Order do it "should not overwrite the created_by if it already is set" do creator = create(:user) - order = FactoryGirl.create(:order_with_line_items, created_by: creator) - user = FactoryGirl.create(:user) + order = create(:order_with_line_items, created_by: creator) + user = create(:user) order.user = nil order.email = nil