Merge pull request marknadig#11 from hengyiliu/add-prompt

allow prompt parameter

Author: marknadig

lib/omniauth/azure_oauth2/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module AzureOauth2
-    VERSION = "0.0.6"
+    VERSION = "0.0.7"
   end
 end

lib/omniauth/strategies/azure_oauth2.rb

@@ -29,6 +29,7 @@ def client
           provider.respond_to?(:tenant_id) ? provider.tenant_id : 'common'
 
         options.authorize_params.domain_hint = provider.domain_hint if provider.respond_to?(:domain_hint) && provider.domain_hint
+        options.authorize_params.prompt = request.params['prompt'] if request.params['prompt']
         options.client_options.authorize_url = "#{BASE_AZURE_URL}/#{options.tenant_id}/oauth2/authorize"
         options.client_options.token_url = "#{BASE_AZURE_URL}/#{options.tenant_id}/oauth2/token"
 

spec/omniauth/strategies/azure_oauth2_spec.rb

@@ -31,26 +31,31 @@ module JWT; end
 
     describe '#client' do
       it 'has correct authorize url' do
+        allow(subject).to receive(:request) { request }
         expect(subject.client.options[:authorize_url]).to eql('https://login.microsoftonline.com/tenant/oauth2/authorize')
       end
 
       it 'has correct authorize params' do
+        allow(subject).to receive(:request) { request }
         subject.client
         expect(subject.authorize_params[:domain_hint]).to be_nil
       end
 
       it 'has correct token url' do
+        allow(subject).to receive(:request) { request }
         expect(subject.client.options[:token_url]).to eql('https://login.microsoftonline.com/tenant/oauth2/token')
       end
 
       it 'has correct token params' do
+        allow(subject).to receive(:request) { request }
         subject.client
         expect(subject.token_params[:resource]).to eql('00000002-0000-0000-c000-000000000000')
       end
 
       describe "overrides" do
         it 'should override domain_hint' do
           @options = {domain_hint: 'hint'}
+          allow(subject).to receive(:request) { request }
           subject.client
           expect(subject.authorize_params[:domain_hint]).to eql('hint')
         end
@@ -65,6 +70,10 @@ module JWT; end
       OmniAuth::Strategies::AzureOauth2.new(app, {client_id: 'id', client_secret: 'secret'}.merge(options))
     end
 
+    before do
+      allow(subject).to receive(:request) { request }
+    end
+
     describe '#client' do
       it 'has correct authorize url' do
         expect(subject.client.options[:authorize_url]).to eql('https://login.microsoftonline.com/common/oauth2/authorize')
@@ -101,6 +110,10 @@ def tenant_id
       OmniAuth::Strategies::AzureOauth2.new(app, provider_klass)
     end
 
+    before do
+      allow(subject).to receive(:request) { request }
+    end
+
     describe '#client' do
       it 'has correct authorize url' do
         expect(subject.client.options[:authorize_url]).to eql('https://login.microsoftonline.com/tenant/oauth2/authorize')
@@ -152,6 +165,10 @@ def client_secret
       OmniAuth::Strategies::AzureOauth2.new(app, provider_klass)
     end
 
+    before do
+      allow(subject).to receive(:request) { request }
+    end
+
     describe '#client' do
       it 'has correct authorize url' do
         expect(subject.client.options[:authorize_url]).to eql('https://login.microsoftonline.com/common/oauth2/authorize')
@@ -176,8 +193,9 @@ def client_secret
       double(:token => token)
     end
 
-    before :each do
+    before do
       allow(subject).to receive(:access_token) { access_token }
+      allow(subject).to receive(:request) { request }
     end
 
     it "does not clash if JWT strategy is used" do