allow passing display parameter with request [fixes #3]
Mark Dodwell
12 years ago
62 | 62 |
options.access_token_options.inject({}) { |h,(k,v)| h[k.to_sym] = v; h }
|
63 | 63 |
end
|
64 | 64 |
|
|
65 |
def authorize_params
|
|
66 |
super.tap do |params|
|
|
67 |
params.merge!(:display => request.params['display']) if request.params['display']
|
|
68 |
end
|
|
69 |
end
|
|
70 |
|
65 | 71 |
private
|
66 | 72 |
|
67 | 73 |
def prune!(hash)
|
1 | 1 |
require 'omniauth-facebook'
|
2 | 2 |
|
3 | 3 |
describe OmniAuth::Strategies::Facebook do
|
|
4 |
before :each do
|
|
5 |
@request = double('Request')
|
|
6 |
@request.stub(:params) { {} }
|
|
7 |
end
|
|
8 |
|
4 | 9 |
subject do
|
5 | |
OmniAuth::Strategies::Facebook.new(nil, @options || {})
|
|
10 |
OmniAuth::Strategies::Facebook.new(nil, @options || {}).tap do |strategy|
|
|
11 |
strategy.stub(:request) { @request }
|
|
12 |
end
|
6 | 13 |
end
|
7 | 14 |
|
8 | 15 |
it_should_behave_like 'an oauth2 strategy'
|
|
24 | 31 |
describe '#authorize_params' do
|
25 | 32 |
it 'is empty by default' do
|
26 | 33 |
subject.authorize_params.should be_empty
|
|
34 |
end
|
|
35 |
|
|
36 |
it 'includes display parameter from request when present' do
|
|
37 |
@request.stub(:params) { { 'display' => 'touch' } }
|
|
38 |
subject.authorize_params.should be_a(Hash)
|
|
39 |
subject.authorize_params[:display].should eq('touch')
|
27 | 40 |
end
|
28 | 41 |
end
|
29 | 42 |
|