Bug fix for bounty #521: Redirect to settings page rather than home page after

linking twitter/github/linkedin account
Added test case
Added omniauth mock credentials to development.rb for testing through browser

Author: lokst

app/controllers/sessions_controller.rb

@@ -27,7 +27,7 @@ def create
       current_user.apply_oauth(oauth)
       current_user.save!
       flash[:notice] = "#{oauth[:provider].humanize} account linked"
-      redirect_to(destination_url)
+      redirect_to(edit_user_url(current_user))
     else
       @user = User.find_with_oauth(oauth)
       if @user && !@user.new_record?

config/environments/development.rb

@@ -36,4 +36,28 @@
   BetterErrors::Middleware.allow_ip! ENV['TRUSTED_IP'] if ENV['TRUSTED_IP']
   #Rails.logger = Logger.new(STDOUT)
   #Rails.logger.level = Logger::DEBUG
+
+  # Mock account credentials
+  OmniAuth.config.test_mode = true
+  OmniAuth.config.mock_auth[:linkedin] = OmniAuth::AuthHash.new({
+    :provider => 'linkedin',
+    :uid => 'linkedin12345',
+    :info => {:nickname => 'linkedinuser'},
+    :credentials => {
+      :token => 'linkedin1',
+      :secret => 'secret'}})
+  OmniAuth.config.mock_auth[:twitter] = OmniAuth::AuthHash.new({
+    :provider => 'twitter', 
+    :uid => 'twitter123545', 
+    :info => {:nickname => 'twitteruser'}, 
+    :credentials => {
+      :token => 'twitter1', 
+      :secret => 'secret'}})
+  OmniAuth.config.mock_auth[:github] = OmniAuth::AuthHash.new({
+    :provider => 'github', 
+    :uid => 'github123545', 
+    :info => {:nickname => 'githubuser'}, 
+    :credentials => {
+      :token => 'github1', 
+      :secret => 'secret'}})
 end

spec/controllers/sessions_controller_spec.rb

@@ -106,12 +106,12 @@
       expect(response).to redirect_to(new_user_url)
     end
 
-    it 'redirects back to profile page if user is already signed in' do
+    it 'redirects back to settings page if user is already signed in' do
       sign_in(user = Fabricate(:user, username: 'darth'))
       request.env['omniauth.auth'] = OmniAuth.config.mock_auth[:github] = github_response
       get :create
       expect(flash[:notice]).to include('linked')
-      expect(response).to redirect_to(badge_url(username: 'darth'))
+      expect(response).to redirect_to(edit_user_url(controller.send :current_user))
     end
   end
 
@@ -202,6 +202,15 @@
       get :create
       expect(flash[:error]).to include('already associated with a different member')
     end
+
+    it 'successful linking of an account should redirect to settings page' do
+      user = Fabricate(:user, twitter: 'mdeiters', twitter_id: '6271932')
+      sign_in(user)
+      request.env['omniauth.auth'] = OmniAuth.config.mock_auth[:twitter] = twitter_response
+      get :create
+      expect(flash[:notice]).to include('linked')
+      expect(response).to redirect_to(edit_user_url(controller.send :current_user))
+    end
   end
 
 end