omnicontacts 0.3.4 → 0.3.5

data/spec/omnicontacts/importer/gmail_spec.rb

@@ -5,6 +5,19 @@ describe OmniContacts::Importer::Gmail do
 
   let(:gmail) { OmniContacts::Importer::Gmail.new({}, "client_id", "client_secret") }
 
+  let(:self_response) {
+    '{
+      "id":"16482944006464829443",
+      "email":"chrisjohnson@gmail.com",
+      "name":"Chris Johnson",
+      "given_name":"Chris",
+      "family_name":"Johnson",
+      "picture":"https://lh3.googleusercontent.com/-b8aFbTBM/AAAAAAI/IWA/vsek/photo.jpg",
+      "gender":"male",
+      "birthday":"1982-06-21"
+    }'
+  }
+
   let(:contacts_as_json) {
     '{"version":"1.0","encoding":"UTF-8",
         "feed":{
@@ -31,7 +44,7 @@ describe OmniContacts::Importer::Gmail do
             {"rel":"self","type":"application/atom+xml","href":"https://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/full?alt\u003djson\u0026max-results\u003d1"},
             {"rel":"next","type":"application/atom+xml","href":"https://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/full?alt\u003djson\u0026start-index\u003d2\u0026max-results\u003d1"}
           ],
-          "author":[{"name":{"$t":"Asma"},"email":{"$t":"logged_in_user@gmail.com"}}],
+          "author":[{"name":{"$t":"Edward"},"email":{"$t":"logged_in_user@gmail.com"}}],
           "generator":{"version":"1.0","uri":"http://www.google.com/m8/feeds","$t":"Contacts"},
           "openSearch$totalResults":{"$t":"1007"},
           "openSearch$startIndex":{"$t":"1"},
@@ -54,10 +67,37 @@ describe OmniContacts::Importer::Gmail do
               "gd$givenName":{"$t":"Edward"},
               "gd$familyName":{"$t":"Bennet"}
             },
+            "gd$organization":[{"rel":"http://schemas.google.com/g/2005#other","gd$orgName":{"$t":"Google"},"gd$orgTitle":{"$t":"Master Developer"}}],
             "gContact$birthday":{"when":"1954-07-02"},
             "gContact$relation":{"rel":"father"},
             "gContact$gender":{"value":"male"},
+            "gContact$event":[{"rel":"anniversary","gd$when":{"startTime":"1983-04-21"}},{"label":"New Job","gd$when":{"startTime":"2014-12-01"}}],
             "gd$email":[{"rel":"http://schemas.google.com/g/2005#other","address":"bennet@gmail.com","primary":"true"}],
+            "gContact$groupMembershipInfo":[{"deleted":"false","href":"http://www.google.com/m8/feeds/groups/logged_in_user%40gmail.com/base/6"}],
+            "gd$structuredPostalAddress":[{"rel":"http://schemas.google.com/g/2005#home","gd$formattedAddress":{"$t":"1313 Trashview Court\nApt. 13\nNowheresville, OK 66666"},"gd$street":{"$t":"1313 Trashview Court\nApt. 13"},"gd$postcode":{"$t":"66666"},"gd$country":{"code":"VA","$t":"Valoran"},"gd$city":{"$t":"Nowheresville"},"gd$region":{"$t":"OK"}}],
+            "gd$phoneNumber":[{"rel":"http://schemas.google.com/g/2005#mobile","uri":"tel:+34-653-15-76-88","$t":"653157688"}]
+          },
+          {
+            "gd$etag":"\"R3oyfDVSLyt7I2A9WhBTSEULRA0.\"",
+            "id":{"$t":"http://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/base/1"},
+            "updated":{"$t":"2013-02-15T22:36:36.494Z"},
+            "app$edited":{"xmlns$app":"http://www.w3.org/2007/app","$t":"2013-02-15T22:36:36.494Z"},
+            "category":[{"scheme":"http://schemas.google.com/g/2005#kind","term":"http://schemas.google.com/contact/2008#contact"}],
+            "title":{"$t":"Emilia Fox"},
+            "link":[
+              {"rel":"http://schemas.google.com/contacts/2008/rel#photo","type":"image/*","href":"https://www.google.com/m8/feeds/photos/media/logged_in_user%40gmail.com/1"},
+              {"rel":"self","type":"application/atom+xml","href":"https://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/full/1"},
+              {"rel":"edit","type":"application/atom+xml","href":"https://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/full/1"}
+            ],
+            "gd$name":{
+              "gd$fullName":{"$t":"Emilia Fox"},
+              "gd$givenName":{"$t":"Emilia"},
+              "gd$familyName":{"$t":"Fox"}
+            },
+            "gContact$birthday":{"when":"1974-02-10"},
+            "gContact$relation":[{"rel":"spouse"}],
+            "gContact$gender":{"value":"female"},
+            "gd$email":[{"rel":"http://schemas.google.com/g/2005#other","address":"emilia.fox@gmail.com","primary":"true"}],
             "gContact$groupMembershipInfo":[{"deleted":"false","href":"http://www.google.com/m8/feeds/groups/logged_in_user%40gmail.com/base/6"}]
           }]
         }
@@ -68,7 +108,16 @@ describe OmniContacts::Importer::Gmail do
     let(:token) { "token" }
     let(:token_type) { "token_type" }
 
+    before(:each) do
+      gmail.instance_variable_set(:@env, {})
+    end
+
     it "should request the contacts by specifying version and code in the http headers" do
+      gmail.should_receive(:https_get) do |host, path, params, headers|
+        headers["GData-Version"].should eq("3.0")
+        headers["Authorization"].should eq("#{token_type} #{token}")
+        self_response
+      end
       gmail.should_receive(:https_get) do |host, path, params, headers|
         headers["GData-Version"].should eq("3.0")
         headers["Authorization"].should eq("#{token_type} #{token}")
@@ -77,10 +126,12 @@ describe OmniContacts::Importer::Gmail do
       gmail.fetch_contacts_using_access_token token, token_type
     end
 
-    it "should correctly parse id, name,email,gender, birthday, image source and relation" do
+    it "should correctly parse id, name, email, gender, birthday, profile picture and relation for 1st contact" do
+      gmail.should_receive(:https_get)
       gmail.should_receive(:https_get).and_return(contacts_as_json)
       result = gmail.fetch_contacts_using_access_token token, token_type
-      result.size.should be(1)
+
+      result.size.should be(2)
       result.first[:id].should eq('http://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/base/1')
       result.first[:first_name].should eq('Edward')
       result.first[:last_name].should eq('Bennet')
@@ -89,7 +140,49 @@ describe OmniContacts::Importer::Gmail do
       result.first[:gender].should eq("male")
       result.first[:birthday].should eq({:day=>02, :month=>07, :year=>1954})
       result.first[:relation].should eq('father')
+      result.first[:profile_picture].should eq("https://profiles.google.com/s2/photos/profile/bennet")
+      result.first[:dates][0][:name].should eq("anniversary")
+    end
+
+    it "should correctly parse id, name, email, gender, birthday, profile picture, snailmail address, phone and relation for 2nd contact" do
+      gmail.should_receive(:https_get)
+      gmail.should_receive(:https_get).and_return(contacts_as_json)
+      result = gmail.fetch_contacts_using_access_token token, token_type
+      result.size.should be(2)
+      result.last[:id].should eq('http://www.google.com/m8/feeds/contacts/logged_in_user%40gmail.com/base/1')
+      result.last[:first_name].should eq('Emilia')
+      result.last[:last_name].should eq('Fox')
+      result.last[:name].should eq("Emilia Fox")
+      result.last[:email].should eq("emilia.fox@gmail.com")
+      result.last[:gender].should eq("female")
+      result.last[:birthday].should eq({:day=>10, :month=>02, :year=>1974})
+      result.last[:profile_picture].should eq("https://profiles.google.com/s2/photos/profile/emilia.fox")
+      result.last[:relation].should eq('spouse')
+      result.first[:address_1].should eq('1313 Trashview Court')
+      result.first[:address_2].should eq('Apt. 13')
+      result.first[:city].should eq('Nowheresville')
+      result.first[:region].should eq('OK')
+      result.first[:country].should eq('VA')
+      result.first[:postcode].should eq('66666')
+      result.first[:phone_number].should eq('653157688')
     end
 
+    it "should correctly parse and set logged in user information" do
+      gmail.should_receive(:https_get).and_return(self_response)
+      gmail.should_receive(:https_get).and_return(contacts_as_json)
+
+      gmail.fetch_contacts_using_access_token token, token_type
+
+      user = gmail.instance_variable_get(:@env)["omnicontacts.user"]
+      user.should_not be_nil
+      user[:id].should eq("16482944006464829443")
+      user[:first_name].should eq("Chris")
+      user[:last_name].should eq("Johnson")
+      user[:name].should eq("Chris Johnson")
+      user[:email].should eq("chrisjohnson@gmail.com")
+      user[:gender].should eq("male")
+      user[:birthday].should eq({:day=>21, :month=>06, :year=>1982})
+      user[:profile_picture].should eq("https://profiles.google.com/s2/photos/profile/16482944006464829443")
+    end
   end
-end
+end

data/spec/omnicontacts/importer/hotmail_spec.rb

@@ -3,7 +3,22 @@ require "omnicontacts/importer/hotmail"
 
 describe OmniContacts::Importer::Hotmail do
 
-  let(:hotmail) { OmniContacts::Importer::Hotmail.new({}, "client_id", "client_secret") }
+  let(:permissions) { "perm1, perm2" }
+  let(:hotmail) { OmniContacts::Importer::Hotmail.new({}, "client_id", "client_secret", {:permissions => permissions}) }
+
+  let(:self_response) {
+    '{
+      "id": "4502de12390223d0",
+      "name": "Chris Johnson",
+      "first_name": "Chris",
+      "last_name": "Johnson",
+      "birth_day": 21,
+      "birth_month": 6,
+      "birth_year": 1982,
+      "gender": null,
+      "emails": {"preferred":"chrisjohnson@gmail.com", "account":"chrisjohn@gmail.com", "personal":null, "business":null}
+    }'
+  }
 
   let(:contacts_as_json) {
     '{
@@ -19,7 +34,8 @@ describe OmniContacts::Importer::Hotmail do
          "is_favorite": false,
          "birth_day": 5,
          "birth_month": 6,
-         "birth_year":1952
+         "birth_year":1952,
+         "email_hashes":["1234567890"]
       }
     ]}'
   }
@@ -29,7 +45,16 @@ describe OmniContacts::Importer::Hotmail do
     let(:token) { "token" }
     let(:token_type) { "token_type" }
 
+    before(:each) do
+      hotmail.instance_variable_set(:@env, {"HTTP_HOST" => "http://example.com"})
+    end
+
     it "should request the contacts by providing the token in the url" do
+      hotmail.should_receive(:https_get) do |host, path, params, headers|
+        params[:access_token].should eq(token)
+        self_response
+      end
+
       hotmail.should_receive(:https_get) do |host, path, params, headers|
         params[:access_token].should eq(token)
         contacts_as_json
@@ -37,9 +62,15 @@ describe OmniContacts::Importer::Hotmail do
       hotmail.fetch_contacts_using_access_token token, token_type
     end
 
-    it "should correctly parse id, name,email,gender, birthday, profile picture and relation" do
+    it "should set requested permissions in the authorization url" do
+      hotmail.authorization_url.should match(/scope=#{Regexp.quote(CGI.escape(permissions))}/)
+    end
+
+    it "should correctly parse id, name, email, gender, birthday, profile picture, relation and email hashes" do
+      hotmail.should_receive(:https_get).and_return(self_response)
       hotmail.should_receive(:https_get).and_return(contacts_as_json)
       result = hotmail.fetch_contacts_using_access_token token, token_type
+
       result.size.should be(1)
       result.first[:id].should eq('123456')
       result.first[:first_name].should eq("John")
@@ -50,6 +81,25 @@ describe OmniContacts::Importer::Hotmail do
       result.first[:birthday].should eq({:day=>5, :month=>6, :year=>1952})
       result.first[:profile_picture].should eq('https://apis.live.net/v5.0/123456/picture')
       result.first[:relation].should be_nil
+      result.first[:email_hashes].should eq(["1234567890"])
+    end
+
+    it "should correctly parse and set logged in user information" do
+      hotmail.should_receive(:https_get).and_return(self_response)
+      hotmail.should_receive(:https_get).and_return(contacts_as_json)
+
+      hotmail.fetch_contacts_using_access_token token, token_type
+
+      user = hotmail.instance_variable_get(:@env)["omnicontacts.user"]
+      user.should_not be_nil
+      user[:id].should eq('4502de12390223d0')
+      user[:first_name].should eq('Chris')
+      user[:last_name].should eq('Johnson')
+      user[:name].should eq('Chris Johnson')
+      user[:gender].should be_nil
+      user[:birthday].should eq({:day=>21, :month=>06, :year=>1982})
+      user[:email].should eq('chrisjohn@gmail.com')
+      user[:profile_picture].should eq('https://apis.live.net/v5.0/4502de12390223d0/picture')
     end
   end
 

data/spec/omnicontacts/importer/yahoo_spec.rb

@@ -4,6 +4,19 @@ require "omnicontacts/importer/yahoo"
 describe OmniContacts::Importer::Yahoo do
 
   describe "fetch_contacts_from_token_and_verifier" do
+    let(:self_response) {
+      '{"profile":{
+                  "guid":"PCLASP523T3E2R5TFMHDW9KWQQ",
+                  "birthdate": "06/21",
+                  "emails":[{"handle":"chrisjohnson@gmail.com", "id":10, "primary":true, "type":"HOME"}, {"handle":"xyz@xyz.com", "id":11, "type":"HOME"}],
+                  "familyName": "Johnson",
+                  "gender":"M",
+                  "givenName":"Chris",
+                  "image":{"imageUrl":"https://avatars.zenfs.com/users/23T3E2R5TFMHDW-AFE-I7lUpIsGQ==.large.png"}
+                }
+      }'
+    }
+
     let(:contacts_as_json) {
       '{
         "contacts": {
@@ -16,7 +29,8 @@ describe OmniContacts::Importer::Yahoo do
                 {"id":819, "type":"email", "value":"johnny@yahoo.com"},
                 {"id":806,"type":"name","value":{"givenName":"John","middleName":"","familyName":"Smith"},"editedBy":"OWNER","categories":[]},
                 {"id":33555343,"type":"guid","value":"7ET6MYV2UQ6VR6CBSNMCLFJIVI"},
-                {"id":946,"type":"birthday","value":{"day":"22","month":"2","year":"1952"},"editedBy":"OWNER","categories":[]}
+                {"id":946,"type":"birthday","value":{"day":"22","month":"2","year":"1952"},"editedBy":"OWNER","categories":[]},
+                {"id":21, "type":"address", "value":{"street":"1313 Trashview Court\nApt. 13", "city":"Nowheresville", "stateOrProvince":"OK", "postalCode":"66666", "country":"", "countryCode":""}, "editedBy":"OWNER", "flags":["HOME"], "categories":[]}
               ]
             }
           ]
@@ -25,9 +39,25 @@ describe OmniContacts::Importer::Yahoo do
 
     let(:yahoo) { OmniContacts::Importer::Yahoo.new({}, "consumer_key", "consumer_secret") }
 
+    before(:each) do
+      yahoo.instance_variable_set(:@env, {})
+    end
+
     it "should request the contacts by specifying all required parameters" do
       yahoo.should_receive(:fetch_access_token).and_return(["access_token", "access_token_secret", "guid"])
-      yahoo.should_receive(:http_get) do |host, path, params|
+
+      yahoo.should_receive(:https_get) do |host, path, params|
+        params[:format].should eq("json")
+        params[:oauth_consumer_key].should eq("consumer_key")
+        params[:oauth_nonce].should_not be_nil
+        params[:oauth_signature_method].should eq("HMAC-SHA1")
+        params[:oauth_timestamp].should_not be_nil
+        params[:oauth_token].should eq("access_token")
+        params[:oauth_version].should eq("1.0")
+        self_response
+      end
+
+      yahoo.should_receive(:https_get) do |host, path, params|
         params[:format].should eq("json")
         params[:oauth_consumer_key].should eq("consumer_key")
         params[:oauth_nonce].should_not be_nil
@@ -35,16 +65,17 @@ describe OmniContacts::Importer::Yahoo do
         params[:oauth_timestamp].should_not be_nil
         params[:oauth_token].should eq("access_token")
         params[:oauth_version].should eq("1.0")
-        params[:view].should eq("compact")
         contacts_as_json
       end
       yahoo.fetch_contacts_from_token_and_verifier "auth_token", "auth_token_secret", "oauth_verifier"
     end
 
-    it "should correctly parse id, name,email,gender, birthday, image source and relation" do
+    it "should correctly parse id, name,email,gender, birthday, snailmail address, image source and relation for contact and logged in user" do
       yahoo.should_receive(:fetch_access_token).and_return(["access_token", "access_token_secret", "guid"])
-      yahoo.should_receive(:http_get).and_return(contacts_as_json)
+      yahoo.should_receive(:https_get).and_return(self_response)
+      yahoo.should_receive(:https_get).and_return(contacts_as_json)
       result = yahoo.fetch_contacts_from_token_and_verifier "auth_token", "auth_token_secret", "oauth_verifier"
+
       result.size.should be(1)
       result.first[:id].should eq('10')
       result.first[:first_name].should eq('John')
@@ -53,16 +84,41 @@ describe OmniContacts::Importer::Yahoo do
       result.first[:email].should eq("johnny@yahoo.com")
       result.first[:gender].should be_nil
       result.first[:birthday].should eq({:day=>22, :month=>2, :year=>1952})
+      result.first[:address_1].should eq('1313 Trashview Court')
+      result.first[:address_2].should eq('Apt. 13')
+      result.first[:city].should eq('Nowheresville')
+      result.first[:region].should eq('OK')
+      result.first[:postcode].should eq('66666')
       result.first[:relation].should be_nil
     end
 
     it "should return an empty list of contacts" do
       empty_contacts_list = '{"contacts": {"start":0, "count":0}}'
       yahoo.should_receive(:fetch_access_token).and_return(["access_token", "access_token_secret", "guid"])
-      yahoo.should_receive(:http_get).and_return(empty_contacts_list)
+      yahoo.should_receive(:https_get).and_return(self_response)
+      yahoo.should_receive(:https_get).and_return(empty_contacts_list)
       result = yahoo.fetch_contacts_from_token_and_verifier "auth_token", "auth_token_secret", "oauth_verifier"
+
       result.should be_empty
     end
 
+    it "should correctly parse and set logged in user information" do
+      yahoo.should_receive(:fetch_access_token).and_return(["access_token", "access_token_secret", "guid"])
+      yahoo.should_receive(:https_get).and_return(self_response)
+      yahoo.should_receive(:https_get).and_return(contacts_as_json)
+      yahoo.fetch_contacts_from_token_and_verifier "auth_token", "auth_token_secret", "oauth_verifier"
+
+      user = yahoo.instance_variable_get(:@env)["omnicontacts.user"]
+      user.should_not be_nil
+      user[:id].should eq('PCLASP523T3E2R5TFMHDW9KWQQ')
+      user[:first_name].should eq('Chris')
+      user[:last_name].should eq('Johnson')
+      user[:name].should eq('Chris Johnson')
+      user[:gender].should eq('male')
+      user[:birthday].should eq({:day=>21, :month=>06, :year=>nil})
+      user[:email].should eq('chrisjohnson@gmail.com')
+      user[:profile_picture].should eq('https://avatars.zenfs.com/users/23T3E2R5TFMHDW-AFE-I7lUpIsGQ==.large.png')
+    end
+
   end
 end

data/spec/omnicontacts/middleware/base_oauth_spec.rb

@@ -11,7 +11,7 @@ describe OmniContacts::Middleware::BaseOAuth do
       end
       
       def redirect_path
-        "/contacts/testprovider/callback"
+        "#{ MOUNT_PATH }testprovider/callback"
       end
     end
     OmniContacts.integration_test.enabled = true
@@ -26,17 +26,23 @@ describe OmniContacts::Middleware::BaseOAuth do
   
   it "should return a preconfigured list of contacts" do
     OmniContacts.integration_test.mock(:testprovider, :email => "user@example.com")
-    get "/contacts/testprovider"    
-    get "/contacts/testprovider/callback"    
+    get "#{ MOUNT_PATH }testprovider"
+    get "#{ MOUNT_PATH }testprovider/callback"
     last_request.env["omnicontacts.contacts"].first[:email].should eq("user@example.com")
   end
 
   it "should redurect to failure url" do
     OmniContacts.integration_test.mock(:testprovider, "some_error" )
-    get "/contacts/testprovider"
-    get "/contacts/testprovider/callback"
+    get "#{ MOUNT_PATH }testprovider"
+    get "#{MOUNT_PATH }testprovider/callback"
     last_response.should be_redirect
-    last_response.headers["location"].should eq("/contacts/failure?error_message=internal_error&importer=testprovider")
+    last_response.headers["location"].should eq("#{ MOUNT_PATH }failure?error_message=internal_error&importer=testprovider")
+  end
+  
+  it "should pass through state query params to the failure url" do
+    OmniContacts.integration_test.mock(:testprovider, "some_error" )
+    get "#{MOUNT_PATH }testprovider/callback?state=/parent/resource/id"
+    last_response.headers["location"].should eq("#{ MOUNT_PATH }failure?error_message=internal_error&importer=testprovider&state=/parent/resource/id")
   end
   
   after(:all) do 

data/spec/omnicontacts/middleware/oauth1_spec.rb

@@ -41,32 +41,38 @@ describe OmniContacts::Middleware::OAuth1 do
   context "visiting the listening path" do
     it "should save the authorization token and redirect to the authorization url" do
       OAuth1Middleware.mock_auth_token_resp.should_receive(:body).and_return(["auth_token", "auth_token_secret"])
-      get "/contacts/oauth1middleware"
+      get "#{ MOUNT_PATH }oauth1middleware"
       last_response.should be_redirect
       last_response.headers['location'].should eq("http://www.example.com")
     end
 
+    it "should pass through state query params visiting the listening path" do
+      OAuth1Middleware.mock_auth_token_resp.should_receive(:body).and_return(["auth_token", "auth_token_secret"])
+      get "#{ MOUNT_PATH }oauth1middleware?state=/parent/resource/id"
+      last_response.headers['location'].should eq("http://www.example.com?state=/parent/resource/id")
+    end
+
     it "should redirect to failure url if fetching the request token does not succeed" do
       OAuth1Middleware.mock_auth_token_resp.should_receive(:body).and_raise("Request failed")
       get "contacts/oauth1middleware"
       last_response.should be_redirect
-      last_response.headers["location"].should eq("/contacts/failure?error_message=internal_error&importer=oauth1middleware")
+      last_response.headers["location"].should eq("#{ MOUNT_PATH }failure?error_message=internal_error&importer=oauth1middleware")
     end
   end
 
   context "visiting the callback url after authorization" do
     it "should return the list of contacts" do
       OAuth1Middleware.mock_session.should_receive(:[]).and_return("oauth_token_secret")
-      get "/contacts/oauth1middleware/callback?oauth_token=token&oauth_verifier=verifier"
+      get "#{ MOUNT_PATH }oauth1middleware/callback?oauth_token=token&oauth_verifier=verifier"
       last_response.should be_ok
       last_request.env["omnicontacts.contacts"].size.should be(1)
     end
 
     it "should redirect to failure url if oauth_token_secret is not found in the session" do
       OAuth1Middleware.mock_session.should_receive(:[]).and_return(nil)
-      get "/contacts/oauth1middleware/callback?oauth_token=token&oauth_verifier=verifier"
+      get "#{ MOUNT_PATH }oauth1middleware/callback?oauth_token=token&oauth_verifier=verifier"
       last_response.should be_redirect
-      last_response.headers["location"].should eq("/contacts/failure?error_message=not_authorized&importer=oauth1middleware")
+      last_response.headers["location"].should eq("#{ MOUNT_PATH }failure?error_message=not_authorized&importer=oauth1middleware")
     end
   end
 end

data/spec/omnicontacts/middleware/oauth2_spec.rb

@@ -40,10 +40,15 @@ describe OmniContacts::Middleware::OAuth2 do
 
   context "visiting the listening path" do
     it "should redirect to authorization site when visiting the listening path" do
-      get "/contacts/oauth2middleware"
+      get "#{ MOUNT_PATH }oauth2middleware"
       last_response.should be_redirect
       last_response.headers['location'].should eq("http://www.example.com")
     end
+
+    it "should pass through state query params visiting the listening path" do
+      get "#{ MOUNT_PATH }oauth2middleware?state=/parent/resource/id"
+      last_response.headers['location'].should eq("http://www.example.com?state=/parent/resource/id")
+    end
   end
 
   context "visiting the callback url after authorization" do
@@ -56,7 +61,7 @@ describe OmniContacts::Middleware::OAuth2 do
     it "should redirect to failure page because user did not allow access to contacts list" do
       get '/redirect_path?error=not_authorized'
       last_response.should be_redirect
-      last_response.headers["location"].should eq("/contacts/failure?error_message=not_authorized&importer=oauth2middleware")
+      last_response.headers["location"].should eq("#{ MOUNT_PATH }failure?error_message=not_authorized&importer=oauth2middleware")
     end
   end
 end