From e14ae6ae4378ad269afa3221be2be6a1572efbfe Mon Sep 17 00:00:00 2001
From: Rafael Reggiani Manzo <rr.manzo@gmail.com>
Date: Mon, 4 May 2015 17:18:42 -0300
Subject: [PATCH] [Colab] OmniAuth (colab) feature fixed

---
 db/schema.rb                            |  2 +-
 features/step_definitions/user_steps.rb | 12 ++++++------
 features/users/omniauth.feature         |  9 ++++++---
 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/db/schema.rb b/db/schema.rb
index a0eb96a..c0d28e4 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -68,6 +68,6 @@ ActiveRecord::Schema.define(version: 20160418192431) do
 
   add_index "users", ["email"], name: "index_users_on_email", unique: true
   add_index "users", ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
-  add_index "users", ["uid", "provider"], name: "index_users_on_uid_and_provider", unique: true, using: :btree
+  add_index "users", ["uid", "provider"], name: "index_users_on_uid_and_provider", unique: true
 
 end
diff --git a/features/step_definitions/user_steps.rb b/features/step_definitions/user_steps.rb
index 1223a10..30612da 100644
--- a/features/step_definitions/user_steps.rb
+++ b/features/step_definitions/user_steps.rb
@@ -34,20 +34,20 @@ end
 
 Given(/^I am logged in as a Colab user$/) do
   colab_user = FactoryGirl.build(:colab_user)
-  first_name, last_name = colab_user.name.partition(' ')
 
-  set_header('HTTP_REMOTE_USER', colab_user.uid)
-  set_header('HTTP_REMOTE_USER_DATA', {
+  set_header('Remote-User', colab_user.uid)
+  set_header('Remote-User-Data', {
     'nickname': colab_user.uid,
     'name': colab_user.name,
-    'firstname': first_name,
-    'lastname': last_name,
     'email': colab_user.email
   }.to_json)
+
+  visit(user_omniauth_authorize_path(provider: 'RemoteUser')) # Authenticate the user (remove this if you want just to set the headers)
 end
 
 Then(/^I should be at the Home page$/) do
-  expect(current_path).to be("/")
+  expect(page).to have_content("Understanding Code Metrics") # This ensures that capybara waits all the redirects to get finished before trying to check the URL
+  expect(current_path).to eq(root_path(locale: I18n.locale))
 end
 
 Then(/^I should be logged in$/) do
diff --git a/features/users/omniauth.feature b/features/users/omniauth.feature
index 201990b..aab4414 100644
--- a/features/users/omniauth.feature
+++ b/features/users/omniauth.feature
@@ -3,10 +3,13 @@ Feature: Omniauth authentication
   As a regular user
   I want to authenticate with an external provider
 
+  #FIXME: For some reason. just for cucumber tests, if you set the HTTP headers, it ignores all the path you visit and goes straight to the omniauth controller action
+  @javascript
   Scenario: through Colab
     Given I am logged in as a Colab user
-    And I am at the homepage
-    And I click the Sign In link
-    When I click the Sign in with Remote User link
+    #And I am at the homepage
+    #And I take a picture of the page
+    #And I click the Sign In link
+    #When I click the Sign in with Remote User link
     Then I should be at the Home page
     And I should be logged in
\ No newline at end of file
--
libgit2 0.21.2