Contact#inspect output should be nice even without an email

mislav · mislav · commit 461962f70684 · 2009-01-09T01:52:09.000+01:00
diff --git a/lib/contacts.rb b/lib/contacts.rb
@@ -20,7 +20,7 @@ def email
     end
     
     def inspect
-      %!#<Contacts::Contact "#{name}" (#{email})>!
+      %!#<Contacts::Contact "#{name}"#{email ? " (#{email})" : ''}>!
     end
   end
   
diff --git a/spec/contact_spec.rb b/spec/contact_spec.rb
@@ -24,10 +24,18 @@
     it "should have username" do
       @contact.username.should == 'maxpower'
     end
-    
-    it "should have nice inspect" do
+  end
+  
+  describe '#inspect' do
+    it "should be nice" do
+      @contact = Contacts::Contact.new('max@example.com', 'Max Power', 'maxpower')
       @contact.inspect.should == '#<Contacts::Contact "Max Power" (max@example.com)>'
     end
+    
+    it "should be nice without email" do
+      @contact = Contacts::Contact.new(nil, 'Max Power', 'maxpower')
+      @contact.inspect.should == '#<Contacts::Contact "Max Power">'
+    end
   end
   
 end
