Removing AuctionQuery wrappers for AuctionScopes

While we might revisit whether or not to abstract away all scopes in the
AuctionQuery object in the future, not necessary for the current PR

app/models/auction_query.rb

@@ -25,20 +25,12 @@ def active_auction_count
   end
 
   def needs_attention_count
-    unpublished.count +
+    relation.unpublished.count +
       pending_delivery.count +
-      pending_acceptance.count +
+      relation.pending_acceptance.count +
       payment_needed.count
   end
 
-  def unpublished
-    relation.unpublished
-  end
-
-  def pending_acceptance
-    relation.pending_acceptance
-  end
-
   def completed
     relation.accepted_or_rejected
   end

app/view_models/admin/needs_attention_auctions_view_model.rb

@@ -12,7 +12,7 @@ def drafts_partial
   end
 
   def drafts
-    @_drafts ||= AuctionQuery.new.unpublished.map do |auction|
+    @_drafts ||= Auction.unpublished.map do |auction|
       Admin::DraftListItem.new(auction)
     end
   end
@@ -38,7 +38,7 @@ def evaluation_needed_partial
   end
 
   def evaluation_needed
-    @_evaluation_needed ||= list_items(AuctionQuery.new.pending_acceptance)
+    @_evaluation_needed ||= list_items(Auction.pending_acceptance)
   end
 
   def payment_needed_partial