Merge pull request #13 from cakejelly/user-identities

Adds methods for managing user identities

Author: cakejelly

lib/layer/resources/user.rb

@@ -16,6 +16,36 @@ def conversations
       def messages
         Layer::ResourceProxy.new(client, self, Layer::Resources::Message)
       end
+
+      def create_identity(params)
+        client.post(identity_url,  body: params.to_json)
+      end
+
+      def identity
+        client.get(identity_url)
+      end
+
+      def update_identity(params)
+        client.patch(
+          identity_url,
+          body: params.to_json,
+          headers: client.layer_patch_header
+        )
+      end
+
+      def replace_identity(params)
+        client.put(identity_url, body: params.to_json)
+      end
+
+      def destroy_identity
+        client.delete(identity_url)
+      end
+
+      private
+
+      def identity_url
+        "#{url}/identity"
+      end
     end
   end
 end

spec/layer/resources/rich_content_spec.rb

@@ -1,5 +1,4 @@
 require "spec_helper"
-require "webmock/rspec"
 
 describe Layer::Resources::RichContent do
   let(:client) { Layer::Platform::Client.new.client }

spec/layer/resources/user_spec.rb

@@ -2,22 +2,26 @@
 
 describe Layer::Resources::User do
   let(:client) { Layer::Platform::Client.new }
+  let(:http_client) { instance_double("Layer::HttpClient") }
+  let(:user) { client.users.find("some_user") }
+
+  before do
+    allow(client).to receive(:client).and_return(http_client)
+  end
 
   describe ".find" do
     it "should return instance of User" do
-      user = client.users.find("jake")
       expect(user).to be_instance_of(described_class)
     end
 
     it "shouldn't send any request" do
       expect(Layer::HttpClient).to_not receive(:find)
-      client.users.find("jake")
+      user
     end
   end
 
   describe "#blocks" do
     it "should instantiate new ResourceProxy for Block" do
-      user = client.users.find("jake")
       blocks = user.blocks
       base = blocks.instance_variable_get("@base")
       resource = blocks.instance_variable_get("@resource")
@@ -30,7 +34,6 @@
 
   describe "#conversations" do
     it "should instantiate new ResourceProxy for Conversation" do
-      user = client.users.find("jake")
       conversations = user.conversations
       base = conversations.instance_variable_get("@base")
       resource = conversations.instance_variable_get("@resource")
@@ -43,7 +46,6 @@
 
   describe "#messages" do
     it "should instantiate new ResourceProxy for Message" do
-      user = client.users.find("jake")
       messages = user.messages
       base = messages.instance_variable_get("@base")
       resource = messages.instance_variable_get("@resource")
@@ -53,4 +55,70 @@
       expect(resource).to eq(Layer::Resources::Message)
     end
   end
+
+  describe "#create_identity" do
+    it "should create identity for a user" do
+      allow(http_client).to receive(:post).and_return("")
+      expect(http_client).to receive(:post).
+                              with("users/#{user.id}/identity", body: user_identity_params.to_json)
+
+      user.create_identity(user_identity_params)
+    end
+  end
+
+  describe "#identity" do
+    before do
+      allow(http_client).to receive(:get).and_return(user_identity_params)
+      expect(http_client).to receive(:get).with("users/#{user.id}/identity")
+    end
+
+    it "should retrieve the users identity" do
+      user.identity
+    end
+
+    it "should return a hash containing the users identity" do
+      identity = user.identity
+      expect(identity).to be_instance_of(Hash)
+    end
+  end
+
+  describe "#update_identity" do
+    let(:patch_header) { { 'Content-Type' => 'application/vnd.layer-patch+json' } }
+
+    it "should update the users identity" do
+      allow(http_client).to receive(:patch).and_return("")
+      allow(http_client).to receive(:layer_patch_header).and_return(patch_header)
+
+      expect(http_client).to receive(:patch).
+                              with(
+                                "users/#{user.id}/identity",
+                                body: user_identity_operations.to_json,
+                                headers: patch_header
+                              )
+
+      user.update_identity(user_identity_operations)
+    end
+  end
+
+  describe "#replace_identity" do
+    it "should replace the users identity" do
+      allow(http_client).to receive(:put).and_return("")
+      expect(http_client).to receive(:put).
+                              with(
+                                "users/#{user.id}/identity",
+                                body: user_identity_params.to_json
+                              )
+
+      user.replace_identity(user_identity_params)
+    end
+  end
+
+  describe "#delete_identity" do
+    it "should make a request to delete the users identity" do
+      allow(http_client).to receive(:delete).and_return("")
+      expect(http_client).to receive(:delete).with("users/#{user.id}/identity")
+
+      user.destroy_identity
+    end
+  end
 end

spec/layer_helper.rb

@@ -68,4 +68,27 @@ def webhook_params
       config: {:key1=>"value1", :key2=>"value2"}
     }
   end
+
+  def user_identity_params
+    {
+      display_name: "Frodo the Dodo",
+      avatar_url: "http://sillylordoftheringspictures.com/frodo-riding-a-dodo.png",
+      first_name: "Frodo",
+      last_name: "Baggins",
+      phone_number: "13791379137",
+      email_address: "frodo@sillylordoftheringspictures.com",
+      metadata: {
+        level: "35",
+        race: "Dodo"
+      }
+    }
+  end
+
+  def user_identity_operations
+    [
+      {operation: "set", property: "last_name", value: "Dodo"},
+      {operation: "set", property: "phone_number", value: ""},
+      {operation: "set", property: "metadata.level", value: "2"}
+    ]
+  end
 end

spec/spec_helper.rb

@@ -2,6 +2,7 @@
 require 'layer/api'
 require 'vcr'
 require 'layer_helper'
+require "webmock/rspec"
 
 VCR.configure do |c|
   c.cassette_library_dir = "spec/cassettes"