Ticket #11413: add_first_and_last_aliases.diff

activerecord/test/cases/base_test.rb

@@ -1625 +1625 @@
-    assert_equal last, Developer.find(:first, :order => 'id desc')
-  end
-  
+  def test_last
+    assert_equal Developer.find(:first, :order => 'id desc'), Developer.last
+  end
+  
-  def test_find_ordered_last
-    last  = Developer.find :last, :order => 'developers.salary ASC'
-    assert_equal last, Developer.find(:all, :order => 'developers.salary ASC').last

activerecord/test/cases/finder_test.rb

@@ -143 +143 @@
-    first = Topic.find(:first, :conditions => "title = 'The First Topic!'")
-    assert_nil(first)
-  end
+  
+  def test_first
+    assert_equal topics(:second).title, Topic.first(:conditions => "title = 'The Second Topic of the day'").title
+  end
+  
+  def test_first_failing
+    assert_nil Topic.first(:conditions => "title = 'The Second Topic of the day!'")
+  end
-
-  def test_unexisting_record_exception_handling
-    assert_raises(ActiveRecord::RecordNotFound) {

activerecord/lib/active_record/base.rb

@@ -511 +511 @@
-          else             find_from_ids(args, options)
-        end
-      end
+      
+      # This is an alias for find(:first).  You can pass in all the same arguments to this method as you can
+      # to find(:first)
+      def first(*args)
+        find(:first, *args)
+      end
-
+      # This is an alias for find(:last).  You can pass in all the same arguments to this method as you can
+      # to find(:last)
+      def last(*args)
+        find(:last, *args)
+      end
+      
-      #
-      # Executes a custom sql query against your database and returns all the results.  The results will
-      # be returned as an array with columns requested encapsulated as attributes of the model you call