g5_authentication_client 0.2.1 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9da8c28deb98850c7e6b1174c64163c2dc694104
-  data.tar.gz: b8530a5eb42cab81dc1ceb2c3cc32a6010f56115
+  metadata.gz: 631343ac9b439ce2f1e1cbad1e8fa52999ff30ad
+  data.tar.gz: 903d0901198ef0057473475b33eeb74a13b61317
 SHA512:
-  metadata.gz: d3a70a125ba261f7580db5ba0873c21a3b6c9b12857b80908ee45ed5a6b1324a610261056c3dfb57eb6752ae34597b73bd7a641ffbf23a6ed94d60ec8082c7ad
-  data.tar.gz: 0a5f34d64d22f3194ad5e7b9ec90383c15e305e3c3668125360929b118aa70b2245f667b6561dc54a80dc710b146d92f69461fbd59a1e0432661d91b5ad0e256
+  metadata.gz: 08bb76c84e52b228ff9790d83dc7282869e7128a72ecd855a97255a396ac8cdf241866dd9a5b194e0d96a7fed589f71eba8cc42ed74faaae35babf8eb90888c8
+  data.tar.gz: 8ab1e14d06f37eb166c15d05b921d8b8be1bb2bfe60dc1a239ae45b035c732f5c8961f0db63c3b6cb77b9abc437d1832df17b56dae10aeac59ca3f2d9bd44eec

data/.ruby-version

@@ -1 +1 @@
-2.1.0
+2.1.2

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## v0.2.3 (2014-07-31)
+* remove id from body in create/update user
+
 ## v0.2.0 (2014-03-10)
 
 * First open source release to [RubyGems](http://rubygems.org/)

data/lib/g5_authentication_client/client.rb

@@ -98,7 +98,7 @@ module G5AuthenticationClient
     def create_user(options={})
       user=User.new(options)
       user.validate_for_create!
-      response=oauth_access_token.post('/v1/users', params: {user: user.to_hash})
+      response=oauth_access_token.post('/v1/users', body: user_hash(user.to_hash))
       User.new(response.parsed)
     end
 
@@ -111,10 +111,22 @@ module G5AuthenticationClient
     def update_user(options={})
       user=User.new(options)
       user.validate!
-      response=oauth_access_token.put("/v1/users/#{user.id}", params: {user: user.to_hash})
+      response=oauth_access_token.put("/v1/users/#{user.id}", body: user_hash(user.to_hash))
       User.new(response.parsed)
     end
 
+    # Find a user by email
+    # @param [String] email address
+    # @return [G5AuthenticationClient::User]
+    def find_user_by_email(email)
+      response = oauth_access_token.get('/v1/users', params: { email: email })
+      user = response.parsed.first
+      if user
+        user=User.new(user)
+      end
+      user
+    end
+
     # Get a user
     # @param [Integer] id the user ID in the remote service
     # @return [G5AuthenticationClient::User]
@@ -158,6 +170,11 @@ module G5AuthenticationClient
     end
 
     private
+
+    def user_hash(h)
+      { user: h.reject{ |k,v| k == 'id' } }
+    end
+
     def oauth_client
       OAuth2::Client.new(client_id, client_secret, site: endpoint)
     end

data/lib/g5_authentication_client/user.rb

@@ -54,5 +54,6 @@ module G5AuthenticationClient
       validate!
       raise ArgumentError.new("Password required for new user.") unless !password.nil?
     end
+
   end
 end

data/lib/g5_authentication_client/version.rb

@@ -1,3 +1,3 @@
 module G5AuthenticationClient
-  VERSION = '0.2.1'
+  VERSION = '0.2.3'
 end

data/spec/g5_authentication_client/client_spec.rb

@@ -45,6 +45,19 @@ describe G5AuthenticationClient::Client do
     id: user_id}
   end
 
+  let(:new_user_request) do
+    {
+      "email"=>email,
+      "first_name"=>"",
+      "last_name"=>"",
+      "organization_name"=>"",
+      "password"=>"#{password}x",
+      "password_confirmation"=>"",
+      "phone_number"=>"",
+      "title"=>""
+    }
+  end
+
   let(:email){'foo@blah.com'}
   let(:password){'mybigtestpasswored'}
   let(:user_id){1}
@@ -273,8 +286,8 @@ describe G5AuthenticationClient::Client do
     subject(:create_user) { client.create_user(new_user_options) }
 
     before do
-      stub_request(:post, /#{endpoint}\/v1\/users/).
-        with(headers: {'Authorization' => auth_header_value}).
+      stub_request(:post, "#{endpoint}/v1/users").
+        with(:body => {"user"=>new_user_request}, headers: {'Authorization' => auth_header_value}).
         to_return(status: 200,
                   body: returned_user.to_json,
                   headers: {'Content-Type' => 'application/json'})
@@ -288,7 +301,7 @@ describe G5AuthenticationClient::Client do
 
     before do
       stub_request(:put, /#{endpoint}\/v1\/users\/#{user_id}/).
-        with(headers: {'Authorization' => auth_header_value}).
+        with(:body => { "user" => new_user_request }, headers: {'Authorization' => auth_header_value}).
         to_return(status: 200,
                   body: returned_user.to_json,
                   headers: {'Content-Type' => 'application/json'})
@@ -297,6 +310,30 @@ describe G5AuthenticationClient::Client do
     it_should_behave_like 'an oauth protected resource', G5AuthenticationClient::User
   end
 
+  describe '#find_user_by_email' do
+    subject(:find_user_by_email) { client.find_user_by_email(email) }
+
+    let(:response_body) { [returned_user].to_json}
+
+    before do
+      stub_request(:get, /#{endpoint}\/v1\/users/).
+        with(query: {'email' => email}, headers: {'Authorization' => auth_header_value}).
+        to_return(status: 200,
+                  body: response_body,
+                  headers: {'Content-Type' => 'application/json'})
+    end
+
+    context 'when there is no user' do
+      let(:response_body) { [].to_json }
+
+      it 'should return nil' do
+        expect(find_user_by_email).to be_nil
+      end
+    end
+
+    it_should_behave_like 'an oauth protected resource', G5AuthenticationClient::User
+  end
+
   describe '#get_user' do
     subject(:get_user) { client.get_user(user_id) }
 

data/spec/g5_authentication_client/user_spec.rb

@@ -181,4 +181,36 @@ describe G5AuthenticationClient::User do
       end
     end
   end
+
+  describe '#to_hash' do
+    subject(:to_hash) { user.to_hash }
+
+    it 'should have a first name' do
+      expect(to_hash['first_name']).to eq(first_name)
+    end
+
+    it 'should have a last name' do
+      expect(to_hash['last_name']).to eq(last_name)
+    end
+
+    it 'should have an email' do
+      expect(to_hash['email']).to eq(email)
+    end
+
+    it 'should have a title ' do
+      expect(to_hash['title']).to eq(title)
+    end
+
+    it 'should have an organization name' do
+      expect(to_hash['organization_name']).to eq(organization_name)
+    end
+
+    it 'should have an phone_number' do
+      expect(to_hash['phone_number']).to eq(phone_number)
+    end
+
+    it 'should have an id' do
+      expect(to_hash['id']).to eq(id)
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: g5_authentication_client
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.3
 platform: ruby
 authors:
 - Rob Revels
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-08 00:00:00.000000000 Z
+date: 2014-07-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: modelish
@@ -203,7 +203,6 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
-- ".ruby-gemset"
 - ".ruby-version"
 - CHANGELOG.md
 - Gemfile
@@ -244,7 +243,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: g5_authentication_client
-rubygems_version: 2.1.11
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Client for the G5 Auth service

data/.ruby-gemset

@@ -1 +0,0 @@
-g5_authentication_client