We don't really need to query all of Updates for a user's page... not…

… sure if this gets any performance benefits but it feels nicer.

app/controllers/users_controller.rb

@@ -28,8 +28,8 @@ def show
         redirect_to "/users/#{user.username}"
       end
     end
-    @author = user.author
-    @updates = Update.where(:feed_id => user.feed.id).order(['created_at', 'descending'])
+    @author  = user.author
+    @updates = user.updates
     @updates = @updates.paginate(:page => params[:page], :per_page => params[:per_page])
     set_pagination_buttons(@updates)
 

app/models/user.rb

@@ -45,6 +45,10 @@ def feed
     self.author.feed
   end
 
+  def updates
+    self.author.feed.updates.sort(:created_at.desc)
+  end
+
   # Before a user is created, we will generate some RSA keys
   def generate_rsa_pair
     key = RSA::KeyPair.generate(2048)

test/acceptance/profile_test.rb

@@ -11,6 +11,23 @@
     assert_equal url, page.current_url
   end
 
+  it "has the user's updates on the page in reverse chronological order" do
+    u = Factory(:user)
+    update1 = Factory(:update,
+                      :text       => "This is a message posted yesterday",
+                      :author     => u.author,
+                      :created_at => 1.day.ago)
+    update2 = Factory(:update,
+                      :text       => "This is a message posted last week",
+                      :author     => u.author,
+                      :created_at => 1.week.ago)
+    u.feed.updates << update1
+    u.feed.updates << update2
+
+    visit "/users/#{u.username}"
+    assert_match /#{update1.text}.*#{update2.text}/m, page.body
+  end
+
   it "has a link to edit your own profile" do
     u = Factory(:user)
     a = Factory(:authorization, :user => u)