zendesk2 1.2.3 → 1.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2a5501867637d62174442f46e0f9119117c5b36b
-  data.tar.gz: a9242b5fe3c07db88175db6254be1fa1a2b4766f
+  metadata.gz: 930984502b3ffcce83fd01762bb8d48a07d760d5
+  data.tar.gz: 901281d2b7f73eeedbb0405ab259f62d6ab920aa
 SHA512:
-  metadata.gz: f3d724891660d349ec816798a226deddfe4a9cfbf4eed4f572b25678f981427f720e6ab0973213b1e89ffeefe9834f58b0495755055744606793e2889df4bf19
-  data.tar.gz: 04bbf03267e9ad8a67502d9da874be66735b6069ad2faf67059d7b8f1c7df63b211a33d1272f047110ecc0ca053d1f34aa87d95e46a5d52503de0bdb11112ee6
+  metadata.gz: f642fe7199612809c97ec24c1f3a680ac33a270944d4205c6316bf4a6a4429224f0b09d331f1af5b4d671345b1450029be7af405d96471fadfb12064d277fa40
+  data.tar.gz: 76141a4780845418663435b0ef1cd769d0fe5ba5cbcc277d31df3c8978587c5f497d3802e75e1b32497bcf5e6e627c97565241ce59e90b5c8c4fe7da917dbfe3

data/lib/zendesk2/client/requests/get_user_identities.rb

@@ -1,7 +1,7 @@
 class Zendesk2::Client
   class Real
     def get_user_identities(params={})
-      user_id   = params["user_id"]
+      user_id     = require_parameters(params, "user_id")
       page_params = Zendesk2.paging_parameters(params)
 
       request(
@@ -14,9 +14,9 @@ class Zendesk2::Client
 
   class Mock
     def get_user_identities(params={})
-      user_id = params["user_id"]
+      user_id = require_parameters(params, "user_id")
 
-      page(params, :identities, "/users/#{user_id}/identities.json", "identities", filter: lambda{|c| c.select{|a| a["user_id"] == user_id}})
+      page(params, :identities, "/users/#{user_id}/identities.json", "identities", filter: lambda { |c| c.select { |a| a["user_id"] == user_id } })
     end
   end # Mock
 end

data/lib/zendesk2/client/requests/update_user.rb

@@ -17,25 +17,36 @@ class Zendesk2::Client
       user_id = params.delete("id")
       path    = "/users/#{user_id}.json"
 
-      if (email = params["email"]) && self.data[:identities].find { |k,i| i["type"] == "email" && i["value"] == email && i["user_id"] != user_id }
-        error!(:invalid, details: {"email" => [ { "description" => "Email #{params["email"]} is already being used by another user" } ] })
+      email = params["email"]
+
+      existing_identity = self.data[:identities].values.find { |i| i["type"] == "email" && i["value"] == email }
+
+      if existing_identity && existing_identity["user_id"] != user_id
+        # email not allowed to conflict across users
+        error!(:invalid, details: { "email" => [ {
+          "description" => "Email #{params["email"]} is already being used by another user",
+        } ] })
+      elsif existing_identity && existing_identity["user_id"] == user_id
+        # no-op email already used
+      else
+        # add a new identity
+        user_identity_id = self.class.new_id # ugh
+
+        user_identity = {
+          "id"         => user_identity_id,
+          "url"        => url_for("/users/#{user_id}/identities/#{user_identity_id}.json"),
+          "created_at" => Time.now.iso8601,
+          "updated_at" => Time.now.iso8601,
+          "type"       => "email",
+          "value"      => email,
+          "verified"   => false,
+          "primary"    => false,
+          "user_id"    => user_id,
+        }
+
+        self.data[:identities][user_identity_id] = user_identity
       end
 
-      user_identity_id = self.class.new_id # ugh
-
-      user_identity = {
-        "id"         => user_identity_id,
-        "url"        => url_for("/users/#{user_id}/identities/#{user_identity_id}.json"),
-        "created_at" => Time.now.iso8601,
-        "updated_at" => Time.now.iso8601,
-        "type"       => "email",
-        "value"      => params["email"],
-        "verified"   => false,
-        "primary"    => false,
-        "user_id"    => user_id,
-      }
-
-      self.data[:identities][user_identity_id] = user_identity
       body = self.data[:users][user_id].merge!(params)
       response(
         :method => :put,
@@ -44,6 +55,7 @@ class Zendesk2::Client
           "user" => body
         },
       )
+
     end
   end
 end

data/lib/zendesk2/version.rb

@@ -1,3 +1,3 @@
 module Zendesk2
-  VERSION = "1.2.3"
+  VERSION = "1.2.4"
 end

data/spec/users_spec.rb

@@ -32,11 +32,7 @@ describe "users" do
   end
 
   describe "#save" do
-    before(:each) do
-      @user = client.users.create!(email: "zendesk2+#{Zendesk2.uuid}@example.org", name: Zendesk2.uuid)
-    end
-
-    let(:user) { @user }
+    let!(:user) { client.users.create!(email: "zendesk2+#{Zendesk2.uuid}@example.org", name: Zendesk2.uuid) }
 
     it "should update organization" do
       user.organization = organization = client.organizations.create!(name: Zendesk2.uuid)
@@ -125,22 +121,29 @@ describe "users" do
     end
 
     it "should create another identity when updating email" do
+      expect(user.identities.size).to eq(1)
+
       original_email = user.email
       user.email = (new_email = "zendesk2+#{Zendesk2.uuid}@example.org")
-      user.save!
 
-      expect((identities = user.identities.all).size).to eq(2)
-      new_identity = identities.find{|i| i.value == new_email}
-      expect(new_identity).not_to be_nil
+      expect {
+        user.save!
+      }.to change { user.identities.size }.by(1)
 
-      expect(new_identity.primary).to be_falsey
+      new_identity = user.identities.find { |i| i.value == new_email }
 
-      original_identity = identities.find{|i| i.value == original_email}
-      expect(original_identity).not_to be_nil
+      expect(new_identity).to be
+      expect(new_identity.primary).to eq(false)
 
-      expect(original_identity.primary).to be_truthy
+      original_identity = user.identities.find { |i| i.value == original_email }
 
+      expect(original_identity).to be
+      expect(original_identity.primary).to eq(true)
       expect(user.reload.email).to eq(original_email)
+
+      expect {
+        user.save!
+      }.not_to change { user.identities.size }
     end
 
     it "should form 'legacy' login url" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: zendesk2
 version: !ruby/object:Gem::Version
-  version: 1.2.3
+  version: 1.2.4
 platform: ruby
 authors:
 - Josh Lane
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-25 00:00:00.000000000 Z
+date: 2014-09-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable