ppl 1.23.0 → 1.24.0

data/spec/ppl/format/contact/phone_number_spec.rb

@@ -4,7 +4,7 @@ describe Ppl::Format::Contact::PhoneNumber do
   describe "#initialize" do
     it "should pass the colors through to the table" do
       colors = {"number" => "blue"}
-      Ppl::Format::Table.should_receive(:new).with([:phone_numbers, :type], colors)
+      Ppl::Format::Table.should_receive(:new).with([:star, :phone_numbers, :type], colors)
       format = Ppl::Format::Contact::PhoneNumber.new(colors)
     end
   end
@@ -22,6 +22,7 @@ describe Ppl::Format::Contact::PhoneNumber do
 
     it "should always include the phone number" do
       @table.should_receive(:add_row).with({
+        :star          => " ",
         :phone_numbers => "01234567890",
         :type          => nil,
       })
@@ -31,12 +32,23 @@ describe Ppl::Format::Contact::PhoneNumber do
     it "should put the type in parentheses" do
       @number.type = "work"
       @table.should_receive(:add_row).with({
+        :star          => " ",
         :phone_numbers => "01234567890",
         :type          => "(work)",
       })
       @format.process(@contact)
     end
 
+    it "should mark preferred numbers with a star" do
+      @number.preferred = true
+      @table.should_receive(:add_row).with({
+        :star          => "*",
+        :phone_numbers => "01234567890",
+        :type          => nil,
+      })
+      @format.process(@contact)
+    end
+
   end
 
 end

data/spec/ppl/service/email_address_spec.rb

@@ -0,0 +1,73 @@
+
+describe Ppl::Service::EmailAddress do
+
+  before(:each) do
+    @service = Ppl::Service::EmailAddress.new
+    @contact = Ppl::Entity::Contact.new
+    @storage = double(Ppl::Adapter::Storage)
+    @storage.stub(:save_contact)
+    @service.storage = @storage
+    @contact.email_addresses << Ppl::Entity::EmailAddress.new("one@example.org")
+  end
+
+  describe "#add" do
+
+    it "should add the email address to the contact" do
+      @storage.stub(:save_contact) do |contact|
+        contact.email_addresses[1].address.should eq "two@example.org"
+      end
+      @service.add(@contact, "two@example.org", {})
+    end
+
+    it "should set the new address as preferred if asked" do
+      @service.add(@contact, "two@example.org", {:preferred => true})
+      @contact.email_addresses[1].preferred.should eq true
+    end
+
+    it "should store the contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.add(@contact, "", {})
+    end
+
+  end
+
+  describe "#update" do
+    it "should store the updated contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.update(@contact, "", {})
+    end
+
+    it "should mark the address as preferred if asked to" do
+      @storage.stub(:save_contact) do |contact|
+        contact.email_addresses.first.preferred.should eq true
+      end
+      @service.update(@contact, "one@example.org", {:preferred => true})
+    end
+
+    it "should mark the address as not preferred if asked to" do
+      @contact.email_addresses.first.preferred = true
+      @storage.stub(:save_contact) do |contact|
+        contact.email_addresses.first.preferred.should eq false
+      end
+      @service.update(@contact, "one@example.org", {:preferred => false})
+    end
+  end
+
+  describe "#remove" do
+
+    it "should remove the email address from the contact" do
+      @storage.stub(:save_contact) do |contact|
+        contact.email_addresses.length.should eq 0
+      end
+      @service.remove(@contact, "one@example.org")
+    end
+
+    it "should store the contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.remove(@contact, "")
+    end
+
+  end
+
+end
+

data/spec/ppl/service/phone_number_spec.rb

@@ -0,0 +1,75 @@
+
+describe Ppl::Service::PhoneNumber do
+
+  before(:each) do
+    @service = Ppl::Service::PhoneNumber.new
+    @contact = Ppl::Entity::Contact.new
+    @storage = double(Ppl::Adapter::Storage)
+    @storage.stub(:save_contact)
+    @service.storage = @storage
+    @contact.phone_numbers << Ppl::Entity::PhoneNumber.new("01234567890")
+  end
+
+  describe "#add" do
+
+    it "should add the phone number to the contact" do
+      @service.add(@contact, "10987654321", {})
+      @contact.phone_numbers[1].number.should eq "10987654321"
+    end
+
+    it "should process the input options against the new contact" do
+      @service.add(@contact, "10987654321", {:preferred => true})
+      @contact.phone_numbers[1].preferred.should eq true
+    end
+
+    it "should store the contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.add(@contact, "", {})
+    end
+
+  end
+
+  describe "#update" do
+    
+    it "should update the number's type" do
+      @service.update(@contact, "01234567890", {:type => "cell"})
+      @contact.phone_numbers.first.type.should eq "cell"
+    end
+
+    it "should update the number's preferred status" do
+      @contact.phone_numbers << Ppl::Entity::PhoneNumber.new("109876543210")
+      @contact.phone_numbers[1].preferred = true
+      @service.update(@contact, "01234567890", {:preferred => true})
+      @contact.phone_numbers[0].preferred.should eq true
+      @contact.phone_numbers[1].preferred.should eq false
+    end
+
+    it "should mark numbers as not preferred" do
+      @contact.phone_numbers[0].preferred = true
+      @service.update(@contact, "01234567890", {:preferred => false})
+      @contact.phone_numbers[0].preferred.should eq false
+    end
+
+    it "should store the contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.update(@contact, "", {})
+    end
+
+  end
+
+  describe "#remove" do
+
+    it "should remove the phone number from the contact" do
+      @service.remove(@contact, "01234567890")
+      @contact.phone_numbers.length.should eq 0
+    end
+
+    it "should store the contact" do
+      @storage.should_receive(:save_contact).with(@contact)
+      @service.remove(@contact, "")
+    end
+
+  end
+
+end
+

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ppl
 version: !ruby/object:Gem::Version
-  version: 1.23.0
+  version: 1.24.0
 platform: ruby
 authors:
 - Henry Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-21 00:00:00.000000000 Z
+date: 2013-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colored
@@ -226,6 +226,7 @@ files:
 - lib/ppl/command/version.rb
 - lib/ppl/entity/address_book.rb
 - lib/ppl/entity/contact.rb
+- lib/ppl/entity/email_address.rb
 - lib/ppl/entity/phone_number.rb
 - lib/ppl/entity/postal_address.rb
 - lib/ppl/error/completion_not_found.rb
@@ -257,6 +258,8 @@ files:
 - lib/ppl/format/postal_address.rb
 - lib/ppl/format/postal_address/one_line.rb
 - lib/ppl/format/table.rb
+- lib/ppl/service/email_address.rb
+- lib/ppl/service/phone_number.rb
 - ppl.gemspec
 - spec/ppl/adapter/color/colored_spec.rb
 - spec/ppl/adapter/color_spec.rb
@@ -301,6 +304,7 @@ files:
 - spec/ppl/command/version_spec.rb
 - spec/ppl/entity/address_book_spec.rb
 - spec/ppl/entity/contact_spec.rb
+- spec/ppl/entity/email_address_spec.rb
 - spec/ppl/entity/phone_number_spec.rb
 - spec/ppl/entity/postal_address_spec.rb
 - spec/ppl/format/address_book/ages_spec.rb
@@ -329,6 +333,8 @@ files:
 - spec/ppl/format/postal_address/one_line_spec.rb
 - spec/ppl/format/postal_address_spec.rb
 - spec/ppl/format/table_spec.rb
+- spec/ppl/service/email_address_spec.rb
+- spec/ppl/service/phone_number_spec.rb
 - spec/spec_helper.rb
 homepage: http://ppladdressbook.org
 licenses: