Try polymorphism/partials for winning bidder

If there is not a winning bid, the bidder is a NullBidder

app/models/bid.rb

@@ -2,11 +2,11 @@ class Bid < ActiveRecord::Base
   belongs_to :auction
   belongs_to :bidder, class_name: 'User'
 
-  def bidder?
-    !bidder.nil?
-  end
-
   def bidder_name
     bidder.name
   end
+
+  def decorated_bidder
+    UserPresenter.new(bidder)
+  end
 end

app/models/null_bid.rb

@@ -1,8 +1,4 @@
 class NullBid
-  def bidder?
-    false
-  end
-
   def amount
     nil
   end
@@ -18,4 +14,8 @@ def bidder_id
   def bidder_name
     nil
   end
+
+  def decorated_bidder
+    NullBidder.new
+  end
 end

app/models/null_bidder.rb

@@ -0,0 +1,9 @@
+class NullBidder
+  def name
+    nil
+  end
+
+  def admin_user_page_link_partial
+    'components/null'
+  end
+end

app/presenters/user_presenter.rb

@@ -27,6 +27,10 @@ def welcome_message_partial
     'components/null'
   end
 
+  def admin_user_page_link_partial
+    'components/admin_user_page_link'
+  end
+
   def admin_edit_auction_partial
     if admin?
       'auctions/edit_auction_link'

app/view_models/admin/action_item_list_item.rb

@@ -13,16 +13,8 @@ def winning_bid
     @_winner ||= WinningBid.new(auction).find
   end
 
-  def winning_bidder?
-    winning_bid.bidder?
-  end
-
-  def winning_bidder_id
-    winning_bid.bidder_id
-  end
-
-  def winning_bidder_name
-    winning_bid.bidder_name
+  def winning_bidder
+    winning_bid.decorated_bidder
   end
 
   def id

app/views/admin/action_items/_rejected_auctions.html.erb

@@ -15,7 +15,7 @@
       <td><%= link_to(auction.title, admin_auction_path(auction.id)) %></td>
       <td><%= auction.delivery_due_at %></td>
       <td><%= auction.delivery_url %></td>
-      <td><% if auction.winning_bidder? %><%= link_to auction.winning_bidder_name, admin_user_path(auction.winning_bidder_id) %><% end %></td>
+      <td><%= render partial: auction.winning_bidder.admin_user_page_link_partial, locals: {bidder: auction.winning_bidder} %></td>
       <td><%= auction.rejected_at %></td>
       <td><%= link_to('edit', edit_admin_auction_path(auction.id)) %></td>
     </tr>

app/views/components/_admin_user_page_link.html.erb

@@ -0,0 +1 @@
+<%= link_to bidder.name, admin_user_path(bidder.id) %>

features/step_definitions/admin_views_action_items_steps.rb

@@ -8,7 +8,7 @@
   end
 
   [auction.title, auction.delivery_due_at, auction.delivery_url,
-   auction.winning_bidder_name, auction.rejected_at].each_with_index do |expected, i|
+   auction.winning_bidder.name, auction.rejected_at].each_with_index do |expected, i|
     within(:xpath, cel_xpath(table_id: 'table-rejected', column: i + 1)) do
       expect(page).to have_content(expected)
     end