Commit c9f5fce22a356ab7f8a5ba4e1857924aaec41d97
1 parent
1cea6fe1
Exists in
master
and in
2 other branches
"Add tests for request and envs"
Showing
1 changed file
with
11 additions
and
7 deletions
Show diff stats
spec/omniauth/strategies/remote_user_spec.rb
... | ... | @@ -9,19 +9,23 @@ describe 'Test Strategy Remote_User' do |
9 | 9 | end.to_app |
10 | 10 | end |
11 | 11 | |
12 | - context 'request phase' do | |
12 | + context 'Response phase' do | |
13 | 13 | before(:each) { get '/auth/remote_user',{},{'HTTP_COOKIE' => '_gitlab_session=user@myuser','HTTP_REMOTE_USER' => "user@myuser" }} |
14 | - it 'check rack response' do | |
14 | + it 'check my rack response' do | |
15 | 15 | expect(last_response.body).to eq('My body') |
16 | 16 | expect(last_response.status).to eq(200) |
17 | 17 | expect(last_response.original_headers).to eq({'HTTP_REMOTE_USER' => 'myuser' }) |
18 | 18 | expect(last_response.errors).to eq('') |
19 | 19 | end |
20 | - it 'check my env request' do | |
21 | - expect(last_request.env['HTTP_COOKIE']).to eq("_gitlab_session=user@myuser") | |
22 | - expect(last_request.env['HTTP_REMOTE_USER']).to eq("user@myuser") | |
23 | - expect(last_request.request_method).to eq("GET") | |
24 | - expect(last_request.path_info).to eq("/auth/remote_user") | |
20 | + | |
21 | + context 'Request phase' do | |
22 | + before(:each) { get '/auth/remote_user',{},{'HTTP_COOKIE' => '_gitlab_session=user@myuser','HTTP_REMOTE_USER' => "user@myuser" }} | |
23 | + it 'check my env request' do | |
24 | + expect(last_request.env['HTTP_COOKIE']).to eq("_gitlab_session=user@myuser") | |
25 | + expect(last_request.env['HTTP_REMOTE_USER']).to eq("user@myuser") | |
26 | + expect(last_request.request_method).to eq("GET") | |
27 | + expect(last_request.path_info).to eq("/auth/remote_user") | |
28 | + end | |
25 | 29 | end |
26 | 30 | end |
27 | 31 | ... | ... |