motion-addressbook 0.1.0 → 0.1.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    motion-addressbook (0.1.0)
+    motion-addressbook (0.1.1)
       bubble-wrap
 
 GEM

data/lib/motion-addressbook/version.rb

@@ -1,5 +1,5 @@
 module Motion
   module Addressbook
-    VERSION = "0.1.0"
+    VERSION = "0.1.1"
   end
 end

data/motion/address_book/multi_value.rb

@@ -1,30 +1,98 @@
 module AddressBook
   class MultiValue
-    attr_reader :multivalueIdentifier, :ab_multi_values
+    attr_reader :attributes, :ab_multi_values
 
-    def initialize(ab_multi_values=nil)
-      @ab_multi_values = ab_multi_values ? ABMultiValueCreateMutableCopy(ab_multi_values) : ABMultiValueCreateMutable(KABMultiStringPropertyType)
-      # @ab_multi_values = ABMultiValueCreateMutable(KABMultiStringPropertyType)
+    def self.attribute_map
+      { :mobile   => KABPersonPhoneMobileLabel ,
+        :iphone   => KABPersonPhoneIPhoneLabel ,
+        :main     => KABPersonPhoneMainLabel   ,
+        :home_fax => KABPersonPhoneHomeFAXLabel,
+        :work_fax => KABPersonPhoneWorkFAXLabel,
+        :pager    => KABPersonPhonePagerLabel  ,
+        :work     => KABWorkLabel              ,
+        :home     => KABHomeLabel              ,
+        :other    => KABOtherLabel
+      }
+    end
+    def attribute_map
+      self.class.attribute_map
+    end
+    
+    def alex
+      ABMultiValueGetIdentifierAtIndex @ab_multi_values, 0
     end
 
-    def set(ab_label, value)
-      return if values.include? value  #temp because blank? not working
-      if blank?(ab_label)
-        ABMultiValueAddValueAndLabel(@ab_multi_values, value, ab_label, multivalueIdentifier) unless value.nil?
+    def initialize(attributes={}, existing_ab_multi_values=nil)
+      @attributes = {}
+      if existing_ab_multi_values 
+        @ab_multi_values = ABMultiValueCreateMutableCopy(existing_ab_multi_values)
+        load_attributes_from_ab
+      else
+        @ab_multi_values = ABMultiValueCreateMutable(KABMultiStringPropertyType)
+      end
+      attributes.each do |attribute, value|
+        send("#{attribute}=", value)
+      end
+    end
+
+    def method_missing(name, *args)
+      if attribute_name = getter?(name)
+        get(attribute_name)
+      elsif attribute_name = setter?(name)
+        set(attribute_name, args.first)
+      else
+        super
+      end
+    end
+
+    def self.is_attribute?(attribute_name)
+      return false if attribute_name.nil?
+      attribute_map.include?(attribute_name.to_sym) || [:email, :phone_number].include?( attribute_name.to_sym)
+    end
+    
+    def getter?(method_name)
+      if self.class.is_attribute? method_name
+        method_name
+      else
+        nil
+      end
+    end
+    def setter?(method_name)
+      method_name.to_s =~ /^(\w*)=$/
+      if self.class.is_attribute? $1
+        $1
       else
-        # untested!!!
-        ABMultiValueReplaceValueAtIndex(@ab_multi_values, value, index_for(ab_label))
+        nil
       end
     end
 
-    def set_many(new_values)
-      new_values.each do |ab_label, value|
-        set(ab_label, value)
+    def get(attribute_name)
+      attributes[attribute_name.to_sym] ||= get_field(attribute_map[attribute_name])
+    end
+    
+    def set(attribute_name, value)
+      set_field(attribute_map[attribute_name.to_sym], value)
+      attributes[attribute_name.to_sym] = value
+    end
+
+    def set_field(ab_label, value)
+      if blank?(ab_label)
+        ABMultiValueAddValueAndLabel(@ab_multi_values, value, ab_label, nil) unless value.nil?
+      else
+        ABMultiValueReplaceValueAtIndex(@ab_multi_values, value, 0)
       end
     end
 
+    def get_field(ab_label)
+      puts
+      puts [__FILE__, __LINE__, ab_label].inspect
+      index = ABMultiValueGetIndexForIdentifier(@ab_multi_values, ab_label)
+      puts [__FILE__, __LINE__, index].inspect
+      ABMultiValueCopyValueAtIndex(@ab_multi_values, index)
+    end
+
     def values
-      ABMultiValueCopyArrayOfAllValues(@ab_multi_values) || []
+      attributes.values
     end
 
     def include? value
@@ -32,14 +100,20 @@ module AddressBook
       values.include? value
     end
 
-    private
-    def blank?(ab_label)
-      index_for(ab_label) == -1
+    def size
+      ABMultiValueGetCount(@ab_multi_values)
     end
 
-    def index_for(ab_label)
-      return -1 # Keep getting this error :(  <ArgumentError: invalid value for Integer: "_$!<Work>!$_">
-      # ABMultiValueGetIndexForIdentifier(@ab_multi_values, ab_label)
+    def load_attributes_from_ab
+      (0...size).to_a.each do |i|
+        label = ABMultiValueCopyLabelAtIndex(@ab_multi_values, i)
+        @attributes[attribute_map.invert[label]] = ABMultiValueCopyValueAtIndex(@ab_multi_values, i)
+      end
+    end
+
+    private
+    def blank?(ab_label)
+      attributes[ab_label].nil?
     end
 
   end

data/motion/address_book/person.rb

@@ -2,12 +2,12 @@ module AddressBook
   class Person
     attr_reader :attributes, :error, :ab_person
 
-    def initialize(attributes={}, ab_person = nil)
+    def initialize(attributes={}, existing_ab_person = nil)
       @attributes = attributes
-      if ab_person.nil?
+      if existing_ab_person.nil?
         @new_record = true
       else
-        @ab_person = ab_person
+        @ab_person = existing_ab_person
         load_ab_person
         @new_record = false
       end
@@ -175,7 +175,38 @@ module AddressBook
       @attributes[:phone_number] ||= phone_number_values.first
     end
 
-    # private
+    def ab_person
+      if @ab_person.nil?
+        @ab_person = ABPersonCreate()
+        load_ab_person
+      end
+      @ab_person
+    end
+
+    def find_or_new
+      if new_record?
+        new_ab_person
+      else
+        existing_record
+      end
+    end
+
+    def new_record?
+      !!@new_record
+    end
+    def exists?
+      !new_record?
+    end
+    def inspect
+      # ensure all attributes loaded
+      attribute_map.keys.each do |attribute|
+        self.send(attribute)
+      end
+
+      super
+    end
+
+    private
 
     def load_ab_person
       set_field(KABPersonFirstNameProperty,    attributes[:first_name  ]) unless attributes[:first_name  ].nil?
@@ -183,8 +214,8 @@ module AddressBook
       set_field(KABPersonJobTitleProperty,     attributes[:job_title   ]) unless attributes[:job_title   ].nil?
       set_field(KABPersonDepartmentProperty,   attributes[:department  ]) unless attributes[:department  ].nil?
       set_field(KABPersonOrganizationProperty, attributes[:organization]) unless attributes[:organization].nil?
-      set_multi_field(KABPersonPhoneProperty,  KABPersonPhoneMobileLabel => attributes[:mobile_phone], KABWorkLabel => attributes[:office_phone])
-      set_multi_field(KABPersonEmailProperty,  KABWorkLabel => attributes[:email])
+      set_multi_field(KABPersonPhoneProperty,  :mobile => attributes[:mobile_phone], :work => attributes[:office_phone])
+      set_multi_field(KABPersonEmailProperty,  :work => attributes[:email])
     end
 
     def set_field(field, value)
@@ -195,38 +226,11 @@ module AddressBook
     end
 
     def set_multi_field(field, values)
-      multi_field = MultiValue.new(ABRecordCopyValue(ab_person, field))
-      multi_field.set_many(values)
+      multi_field = MultiValue.new(values, ABRecordCopyValue(ab_person, field))
       ABRecordSetValue(ab_person, field, multi_field.ab_multi_values, error )
     end
     def get_multi_field(field)
-      MultiValue.new(ABRecordCopyValue(ab_person, field))
-    end
-
-    def ab_person
-      if @ab_person.nil?
-        @ab_person = new_ab_person
-        load_ab_person
-      end
-      @ab_person
-    end
-
-    def find_or_new
-      if new_record?
-        new_ab_person
-      else
-        existing_record
-      end
-    end
-
-    def new_ab_person
-      ab_person = ABPersonCreate()
-      # ABAddressBookAddRecord(address_book, ab_person, error )
-      ab_person
-    end
-    
-    def new_record?
-      !!@new_record
+      MultiValue.new({}, ABRecordCopyValue(ab_person, field))
     end
 
     def existing_records
@@ -237,12 +241,6 @@ module AddressBook
       end
     end
 
-    def exists?
-      !new_record?
-    end
-    # def new_record?
-    #   existing_record.nil?
-    # end
     def existing_record
       # what if there are more than one match? email should be unique but ...
       existing_records.first
@@ -252,17 +250,5 @@ module AddressBook
       @address_book ||= ABAddressBookCreate()
     end
 
-    def inspect
-      # ensure all attributes loaded
-      attribute_map.keys.each do |attribute|
-        self.send(attribute)
-      end
-
-      super
-    end
-
-    # def define_some_constants
-    #   [KABWorkLabel, KABOtherLabel, KABHomeLabel]
-    # end
   end
 end

data/spec/address_book/multi_value_spec.rb

@@ -1,66 +1,125 @@
 describe AddressBook::MultiValue do
-  describe 'new field' do
-    before do
-      @multi_value = AddressBook::MultiValue.new
+  describe 'properties on a new multivalue' do
+    describe 'initializing with values' do
+      before do
+        @attributes = { :mobile   => '123-456-7890',   :iphone   => '222-333-4444',   :main  => '555-1212', 
+                        :home_fax => '1-617-555-8000', :work_fax => '1-212-555-0000', :pager => '99-9999-9999',
+                        :work     => 'alex@work.com',  :home     => 'alex@home.com',  :other => 'alex@other.com'}
+        @multi_value = AddressBook::MultiValue.new @attributes
+      end
+    
+      it 'should be able to get each of the single value fields' do
+        @multi_value.mobile.should.equal   @attributes[:mobile  ]
+        @multi_value.iphone.should.equal   @attributes[:iphone  ]
+        @multi_value.main.should.equal     @attributes[:main    ]
+        @multi_value.home_fax.should.equal @attributes[:home_fax]
+        @multi_value.work_fax.should.equal @attributes[:work_fax]
+        @multi_value.pager.should.equal    @attributes[:pager   ]
+        @multi_value.work.should.equal     @attributes[:work    ]
+        @multi_value.home.should.equal     @attributes[:home    ]
+        @multi_value.other.should.equal    @attributes[:other   ]
+      end
+      
+      it 'should give all the values in a list' do
+        @multi_value.values.should == ["123-456-7890", "222-333-4444", "555-1212", "1-617-555-8000", "1-212-555-0000", "99-9999-9999", "alex@work.com", "alex@home.com", "alex@other.com"]
+      end
     end
 
-    describe 'setting one value' do
-      before { @multi_value.set(KABWorkLabel, 'alex@work.com') }
-      it 'gives access to the values' do
-        @multi_value.values.should == ['alex@work.com']
-      end
-      it 'should know if the value exists' do
-        @multi_value.include?('alex@work.com').should.be.true 
+    describe 'setting each field' do
+      before do
+        @multi_value = AddressBook::MultiValue.new
       end
 
-      it 'should know if the value exists' do
-        @multi_value.include?('not@there.com').should.be.false 
+      it 'should be able to set the first name' do
+        @multi_value.mobile =            '123456789'
+        @multi_value.mobile.should.equal '123456789'
       end
-    end
-    describe 'setting many values' do
-      before { @multi_value.set_many(KABWorkLabel => 'alex@work.com', KABHomeLabel => 'alex@home.com') }
-      it 'gives access to the values' do
-        @multi_value.values.should == ['alex@work.com', 'alex@home.com']
+      it 'should be able to set the iphone' do
+        @multi_value.iphone =            '123456789'
+        @multi_value.iphone.should.equal '123456789'
       end
-      it 'should know both values exist' do
-        @multi_value.include?('alex@work.com').should.be.true 
-        @multi_value.include?('alex@home.com').should.be.true 
+      it 'should be able to set the main' do
+        @multi_value.main =            '123456789'
+        @multi_value.main.should.equal '123456789'
       end
-
-      it 'should know if the value exists' do
-        @multi_value.include?('not@there.com').should.be.false 
+      it 'should be able to set the home fax' do
+        @multi_value.home_fax =            '123456789'
+        @multi_value.home_fax.should.equal '123456789'
+      end
+      it 'should be able to set the work fax' do
+        @multi_value.work_fax =            '123456789'
+        @multi_value.work_fax.should.equal '123456789'
+      end
+      it 'should be able to set the pager' do
+        @multi_value.pager =            '123456789'
+        @multi_value.pager.should.equal '123456789'
+      end
+      it 'should be able to set the work' do
+        @multi_value.work =            'a@work.com'
+        @multi_value.work.should.equal 'a@work.com'
+      end
+      it 'should be able to set the home' do
+        @multi_value.home =            'a@home.com'
+        @multi_value.home.should.equal 'a@home.com'
+      end
+      it 'should be able to set the other' do
+        @multi_value.other =            'a@other.com'
+        @multi_value.other.should.equal 'a@other.com'
       end
     end
   end
-
-  describe 'existing field' do
+  
+  describe 'an existing multivalue' do
     before do
-      old_multi_value = AddressBook::MultiValue.new
-      old_multi_value.set(KABWorkLabel, 'alex@work.com')
-      
-      @multi_value = AddressBook::MultiValue.new old_multi_value.ab_multi_values
-    end
-    
-    it 'should have the value' do
-      @multi_value.values.should == ['alex@work.com']
+      @first_multi_value = AddressBook::MultiValue.new :mobile => '123-456-7890', :iphone => '99-8888888-7777777-66'
+      @multi_value = AddressBook::MultiValue.new({:mobile => '987654321', :home_fax => '777-6666-4444'}, @first_multi_value.ab_multi_values)
     end
     
-    it 'should be able to update the value' do
-      @multi_value.set(KABWorkLabel, 'alex@new_work.com')
-      puts '******PENDING******  Waiting for RubyMotion ABMultiValueGetIndexForIdentifier fix'
-      @multi_value.values.should == ['alex@work.com', 'alex@new_work.com'] # ['alex@new_work.com']
+    it 'should ' do
+      @multi_value.mobile.should == '987654321'
+      @multi_value.iphone.should == '99-8888888-7777777-66'
+      @multi_value.home_fax.should == '777-6666-4444'
     end
+  end
 
-    it 'should ignore when updating an existing value to the same value' do
-      @multi_value.set(KABWorkLabel, 'alex@work.com')
-      @multi_value.values.should == ['alex@work.com']
+describe 'method missing magic' do
+  before do
+    @multi_value = AddressBook::MultiValue.new
+  end
+  describe 'getters' do
+    it 'should have a getter for each attribute' do
+      @multi_value.getter?('mobile'  ).should.be truthy
+      @multi_value.getter?('iphone'  ).should.be truthy
+      @multi_value.getter?('main'    ).should.be truthy
+      @multi_value.getter?('home_fax').should.be truthy
+      @multi_value.getter?('work_fax').should.be truthy
+      @multi_value.getter?('pager'   ).should.be truthy
+      @multi_value.getter?('work'    ).should.be truthy
+      @multi_value.getter?('home'    ).should.be truthy
+      @multi_value.getter?('other'   ).should.be truthy
     end
-
-    it 'should be able to add an additional value' do
-      @multi_value.set(KABHomeLabel, 'alex@home.com')
-      @multi_value.values.should == ['alex@work.com', 'alex@home.com']
+    it 'should know what is not a getter' do
+      @multi_value.getter?('nonesense').should.be falsey
+      @multi_value.getter?('mobile='  ).should.be falsey
     end
-
   end
-  
+  describe 'setters' do
+    it 'should have a setter for each attribute' do
+      @multi_value.setter?('mobile='  ).should.be truthy
+      @multi_value.setter?('iphone='  ).should.be truthy
+      @multi_value.setter?('main='    ).should.be truthy
+      @multi_value.setter?('home_fax=').should.be truthy
+      @multi_value.setter?('work_fax=').should.be truthy
+      @multi_value.setter?('pager='   ).should.be truthy
+      @multi_value.setter?('work='    ).should.be truthy
+      @multi_value.setter?('home='    ).should.be truthy
+      @multi_value.setter?('other='   ).should.be truthy
+    end
+    it 'should know what is not a setter' do
+      @multi_value.setter?('nonesense=').should.be falsey
+      @multi_value.setter?('mobile'    ).should.be falsey
+    end
+  end
+end
+
 end

data/spec/address_book/person_spec.rb

@@ -1,49 +1,31 @@
-def unique_email
-  "alex_#{Time.now.to_i}@example.com"
-end
 describe AddressBook::Person do
 
-  describe '.all' do
-    before do
-      @person = AddressBook::Person.create({:first_name => 'Alex', :last_name=>'Rothenberg'})
-    end
-    it 'should have the person we created' do
-      all_names = AddressBook::Person.all.map do |person|
-        [person.first_name, person.last_name]
-      end
-      all_names.should.include? ['Alex', 'Rothenberg']
-    end
-
-    it 'should get bigger when we create another' do
-      initial_people_count = AddressBook::Person.all.size
-      @person = AddressBook::Person.create({:first_name => 'Alex2', :last_name=>'Rothenberg2'})
-      AddressBook::Person.all.size.should == (initial_people_count + 1)
-    end
-  end
-
   describe 'ways of creating and finding people' do
     describe 'new' do
       before do
-        @alex = AddressBook::Person.new(:first_name => 'Alex', :last_name => 'Testy', :email => 'alex@example.com')
+        @alex = AddressBook::Person.new(:first_name => 'Alex', :last_name => 'Testy', :email => 'alex_testy@example.com')
       end
       it 'should create but not save in the address book' do
         @alex.should.be.new_record
         @alex.first_name.should == 'Alex'
         @alex.last_name.should  == 'Testy'
-        @alex.email_values.should     == ['alex@example.com']
+        @alex.email_values.should     == ['alex_testy@example.com']
       end
     end
-
+    
     describe 'existing' do
       before do
-        AddressBook::Person.create(:first_name => 'Alex', :last_name => 'Testy', :email => 'alex@example.com')
+        @email = unique_email
+        @alex = AddressBook::Person.create(:first_name => 'Alex', :last_name => 'Testy', :email => @email)
       end
       describe '.find_by_all_email' do
         it 'should find matches' do
-          alexes = AddressBook::Person.find_all_by_email 'alex@example.com'
+          alexes = AddressBook::Person.find_all_by_email @email
           alexes.should.not.be.empty
           alexes.each do |alex|
-            alex.email.should == 'alex@example.com'
+            alex.email.should == @email
+            alex.first_name.should == 'Alex'
+            alex.last_name.should  == 'Testy'
           end
         end
         it 'should give empty list when nothing matches' do
@@ -53,8 +35,10 @@ describe AddressBook::Person do
       end
       describe '.find_by_email' do
         it 'should find match' do
-          alex = AddressBook::Person.find_by_email 'alex@example.com'
-          alex.email.should == 'alex@example.com'
+          alex = AddressBook::Person.find_by_email @email
+          alex.email.should == @email
+          alex.first_name.should == 'Alex'
+          alex.last_name.should  == 'Testy'
         end
         it 'should give empty list when nothing matches' do
           alexes = AddressBook::Person.find_by_email unique_email
@@ -63,10 +47,12 @@ describe AddressBook::Person do
       end
       describe '.where' do
         it 'should find matches' do
-          alexes = AddressBook::Person.where(:email => 'alex@example.com')
+          alexes = AddressBook::Person.where(:email => @email)
           alexes.should.not.be.empty
           alexes.each do |alex|
-            alex.email.should == 'alex@example.com'
+            alex.email.should == @email
+            alex.first_name.should == 'Alex'
+            alex.last_name.should  == 'Testy'
           end
         end
         it 'should give empty list when nothing matches' do
@@ -74,27 +60,43 @@ describe AddressBook::Person do
           alexes.should == []
         end
       end
+      
+      describe '.all' do
+        it 'should have the person we created' do
+          all_names = AddressBook::Person.all.map do |person|
+            [person.first_name, person.last_name]
+          end
+          all_names.should.include? [@alex.first_name, @alex.last_name]
+        end
+    
+        it 'should get bigger when we create another' do
+          initial_people_count = AddressBook::Person.all.size
+          @person = AddressBook::Person.create({:first_name => 'Alex2', :last_name=>'Rothenberg2'})
+          AddressBook::Person.all.size.should == (initial_people_count + 1)
+        end
+      end      
     end
 
-  #   describe 'new or existing' do
-  #     before do
-  #       AddressBook::Person.create(:first_name => 'Alex', :last_name => 'Testy', :email => 'alex@example.com')
-  #     end
-  #     it 'should find an existing person' do
-  #       alex = AddressBook::Person.find_or_new_by_email('alex@example.com')
-  #       alex.should.not.be.new_record
-  #       alex.email.should      == 'alex@example.com'
-  #       alex.first_name.should == 'Alex'
-  #       alex.last_name.should  == 'Testy'
-  #     end
-  #     it 'should return new person when no match found' do
-  #       never_before_used_email = unique_email
-  #       alex = AddressBook::Person.find_or_new_by_email(never_before_used_email)
-  #       alex.should.be.new_record
-  #       alex.email.should == never_before_used_email
-  #       alex.first_name.should == nil
-  #     end
-  #   end
+    describe '.find_or_new_by_XXX - new or existing' do
+      before do
+        @email = unique_email
+        AddressBook::Person.create(:first_name => 'Alex', :last_name => 'Testy', :email => @email)
+      end
+      it 'should find an existing person' do
+        alex = AddressBook::Person.find_or_new_by_email(@email)
+        alex.should.not.be.new_record
+        alex.email.should      == @email
+        alex.first_name.should == 'Alex'
+        alex.last_name.should  == 'Testy'
+      end
+      it 'should return new person when no match found' do
+        never_before_used_email = unique_email
+        alex = AddressBook::Person.find_or_new_by_email(never_before_used_email)
+        alex.should.be.new_record
+        alex.email.should == never_before_used_email
+        alex.first_name.should == nil
+      end
+    end
   end
 
   describe 'save' do
@@ -105,17 +107,17 @@ describe AddressBook::Person do
                      :email => unique_email
                     }
     end
-
+  
     describe 'a new person' do
       before do
         @ab_person = AddressBook::Person.new(@attributes)
       end
-
+  
       it 'should not be existing' do
         @ab_person.should.be.new_record
         @ab_person.should.not.be.exists
       end
-
+  
       it 'should be able to get each of the single value fields' do
         @ab_person.first_name.should.equal   @attributes[:first_name  ]
         @ab_person.last_name.should.equal    @attributes[:last_name   ]
@@ -123,7 +125,7 @@ describe AddressBook::Person do
         @ab_person.department.should.equal   @attributes[:department  ]
         @ab_person.organization.should.equal @attributes[:organization]
       end
-
+  
       describe 'setting each field' do
         it 'should be able to set the first name' do
           @ab_person.first_name = 'new first name'
@@ -145,7 +147,7 @@ describe AddressBook::Person do
           @ab_person.organization = 'new organization'
           @ab_person.organization.should.equal 'new organization'
         end
-
+  
         it 'should be able to set the phot' do
           image = CIImage.emptyImage
           data = UIImagePNGRepresentation(UIImage.imageWithCIImage image)
@@ -153,11 +155,11 @@ describe AddressBook::Person do
           UIImagePNGRepresentation(@ab_person.photo).should.equal data
         end
       end
-
+  
       it 'should be able to get the phone numbers' do
         @ab_person.phone_number_values.should.equal [@attributes[:mobile_phone], @attributes[:office_phone] ]
       end
-
+  
       it 'should be able to get the emails' do
         @ab_person.email_values.should.equal [@attributes[:email] ]
       end
@@ -171,22 +173,23 @@ describe AddressBook::Person do
         end
       end
     end
-
+  
     describe 'updating an existing person' do
       before do
         AddressBook::Person.new(@attributes).save
         @attributes[:job_title   ] = 'i got promoted'
         @attributes[:office_phone] = '111 222 3333'
-        # @attributes[:department  ] = nil
+        @attributes[:department  ] = nil
         @ab_person = AddressBook::Person.find_or_new_by_email(@attributes[:email])
       end
-
+  
       it 'should know it is not new' do
         @ab_person.should.not.be.new_record
         @ab_person.should.be.exists
+        @ab_person.first_name.should == 'Alex'
         @ab_person.department.should == 'Development'
       end
-
+  
       describe 'updating' do
         it 'should be able to get each of the single value fields' do
           @ab_person.save
@@ -196,11 +199,11 @@ describe AddressBook::Person do
           AddressBook::Person.find_by_email(@ab_person.email).first_name.should == 'New First Name'
         end
       end
-
+  
     end
-
+  
   end
-
+  
   describe 'method missing magic' do
     before do
       @person = AddressBook::Person.new
@@ -265,6 +268,5 @@ describe AddressBook::Person do
         AddressBook::Person.first_finder?('find_all_by_email').should.be falsey
       end
     end
-end
-
+  end
 end

data/spec/helpers/hacks.rb

@@ -0,0 +1,3 @@
+# The value of these constants is undefined until one of the following functions has been called: ABAddressBookCreate, ABPersonCreate, ABGroupCreate.
+# see https://developer.apple.com/library/ios/#documentation/AddressBook/Reference/ABPersonRef_iPhoneOS/Reference/reference.html and search for "Discussion"
+ABAddressBookCreate()

data/spec/helpers/person_helpers.rb

@@ -1,5 +1,4 @@
-def unique_email(email='alex@example.com')
+def unique_email(email='alex_testy@example.com')
   name, domain = email.split('@')
-  "#{name}_#{Time.now.to_i}@#{domain}"
+  "#{name}_#{Time.now.to_i}_#{rand(1000)}@#{domain}"
 end
-  

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: motion-addressbook
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-02 00:00:00.000000000 Z
+date: 2012-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bubble-wrap
@@ -80,6 +80,7 @@ files:
 - spec/address_book/multi_value_spec.rb
 - spec/address_book/person_spec.rb
 - spec/helpers/bacon_matchers.rb
+- spec/helpers/hacks.rb
 - spec/helpers/person_helpers.rb
 homepage: ''
 licenses: []
@@ -95,7 +96,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2222188653506619315
+      hash: 2733254916178873065
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -104,7 +105,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2222188653506619315
+      hash: 2733254916178873065
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24
@@ -115,4 +116,5 @@ test_files:
 - spec/address_book/multi_value_spec.rb
 - spec/address_book/person_spec.rb
 - spec/helpers/bacon_matchers.rb
+- spec/helpers/hacks.rb
 - spec/helpers/person_helpers.rb