Fetch members if needed for membership#owner

Author: liggitt

features/members_feature.rb

@@ -17,7 +17,7 @@
         r = rhc 'show-domain', domain.name
         r.status.should == 0
         r.stdout.should_not match "Members:"
-        r.stdout.should match "owned by #{domain.owner}"
+        r.stdout.should match "owned by #{domain.owner.name}"
       end
 
       it "should prevent leaving the domain for the owner" do

lib/rhc/output_helpers.rb

@@ -3,7 +3,7 @@ module OutputHelpers
 
     def display_team(team, ids=false)
       paragraph do
-        header ["Team #{team.name}", ("(owned by #{team.owner})" if team.owner.present?)] do
+        header ["Team #{team.name}", ("(owned by #{team.owner.name})" if team.owner.present?)] do
           section(:bottom => 1) do
             say format_table \
               nil,
@@ -21,7 +21,7 @@ def display_team(team, ids=false)
 
     def display_domain(domain, applications=nil, ids=false)
       paragraph do
-        header ["Domain #{domain.name}", ("(owned by #{domain.owner})" if domain.owner.present?)] do
+        header ["Domain #{domain.name}", ("(owned by #{domain.owner.name})" if domain.owner.present?)] do
           section(:bottom => 1) do
             say format_table \
               nil,

lib/rhc/rest/membership.rb

@@ -142,9 +142,7 @@ def leave(options={})
     end
 
     def owner
-      if o = Array(attributes['members']).find{ |m| m['owner'] == true }
-        o['name'] || o['login']
-      end
+      members.find(&:owner?)
     end
   end
 end

lib/rhc/rest/mock.rb

@@ -608,8 +608,8 @@ def add_domain(id, extra=false)
       d = MockRestDomain.new(self, id)
       if extra
         d.attributes['creation_time'] = '2013-07-21T15:00:44Z'
-        d.attributes['members'] = [{'owner' => true, 'name' => 'a_user_name'}]
         d.attributes['allowed_gear_sizes'] = ['small']
+        d.add_member(RHC::Rest::Membership::Member.new(:id => '1', :role => 'admin', :explicit_role => 'admin', :owner => true, :login => 'a_user_name', :type => 'user'))
       end
       @domains << d
       d
@@ -723,6 +723,7 @@ def initialize(client, id)
       @name = id
       @applications = []
       self.attributes = {:links => mock_response_links(mock_domain_links(id))}
+      init_members
     end
 
     def rename(id)