raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-27 01:43:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6719 from coopdevs/stop-using-spree-req-helpers-in-users-controller-spec

Browse Source 
[Rails 5] Stop using deprecated req. helpers in users specs
This commit is contained in:
Pau Pérez Fabregat
2021-02-01 14:56:28 +01:00
committed by GitHub
parent 87bce505ec cc2e46890e
commit 640681d664
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
spec/controllers/spree/users_controller_spec.rb
View File

@@ -3,6 +3,8 @@
require 'spec_helper'
describe Spree::UsersController, type: :controller do
routes { Spree::Core::Engine.routes }
include AuthenticationHelper
describe "show" do
@@ -24,7 +26,7 @@ describe Spree::UsersController, type: :controller do
end
it "returns orders placed by the user at normal shops" do
spree_get :show
get :show
expect(orders).to include d1o1, d1o2
expect(orders).to_not include d1_order_for_u2, d1o3, d2o1
@@ -59,15 +61,17 @@ describe Spree::UsersController, type: :controller do
end
describe "registered_email" do
routes { Openfoodnetwork::Application.routes }
let!(:user) { create(:user) }
it "returns true if email corresponds to a registered user" do
spree_post :registered_email, email: user.email
post :registered_email, email: user.email
expect(json_response['registered']).to eq true
end
it "returns false if email does not correspond to a registered user" do
spree_post :registered_email, email: 'nonregistereduser@example.com'
post :registered_email, email: 'nonregistereduser@example.com'
expect(json_response['registered']).to eq false
end
end
@@ -75,14 +79,14 @@ describe Spree::UsersController, type: :controller do
context '#load_object' do
it 'should redirect to signup path if user is not found' do
allow(controller).to receive_messages(spree_current_user: nil)
spree_put :update, user: { email: 'foobar@example.com' }
put :update, user: { email: 'foobar@example.com' }
expect(response).to redirect_to('/login')
end
end
context '#create' do
it 'should create a new user' do
spree_post :create, user: { email: 'foobar@example.com', password: 'foobar123', password_confirmation: 'foobar123' }
post :create, user: { email: 'foobar@example.com', password: 'foobar123', password_confirmation: 'foobar123' }
expect(assigns[:user].new_record?).to be_falsey
end
end