From b872bf49c5d6fadbb8424a2ed45ebbd1aa621395 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Mon, 31 Aug 2020 16:56:48 +0100 Subject: [PATCH] Replace usage of helper method with a more simple approach --- lib/spree/api/testing_support/setup.rb | 11 ----------- spec/controllers/api/variants_controller_spec.rb | 2 +- 2 files changed, 1 insertion(+), 12 deletions(-) diff --git a/lib/spree/api/testing_support/setup.rb b/lib/spree/api/testing_support/setup.rb index 3ff75abc04..666b673c23 100644 --- a/lib/spree/api/testing_support/setup.rb +++ b/lib/spree/api/testing_support/setup.rb @@ -2,17 +2,6 @@ module Spree module Api module TestingSupport module Setup - def sign_in_as_user! - let!(:current_api_user) do - user = Spree::LegacyUser.new(email: "ofn@example.com") - user.stub(:has_spree_role?).with("admin").and_return(false) - user.stub(:enterprises) { [] } - user.stub(:owned_groups) { [] } - user.stub(:spree_api_key) { "spree_api_key" } - user - end - end - # enterprises is an array of variable names of let defines # eg. # let(:enterprise) { ... } diff --git a/spec/controllers/api/variants_controller_spec.rb b/spec/controllers/api/variants_controller_spec.rb index c91f25f2ec..e222382957 100644 --- a/spec/controllers/api/variants_controller_spec.rb +++ b/spec/controllers/api/variants_controller_spec.rb @@ -14,7 +14,7 @@ describe Api::VariantsController, type: :controller do end context "as a normal user" do - sign_in_as_user! + let(:current_api_user) { build(:user) } let!(:product) { create(:product) } let!(:variant) do