<td> tags must be closed, move browse index test over to changeset, lazy fix for new format

test/functional/browse_controller_test.rb

@@ -18,30 +18,6 @@ class BrowseControllerTest < ActionController::TestCase
   
   end
   
-  # This should display the last 20 changesets closed.
-  def test_index
-    @changesets = Changeset.find(:all, :order => "closed_at DESC", :conditions => ['closed_at < ?', DateTime.now], :limit=> 20)
-    assert @changesets.size <= 20
-    get :index
-    assert_response :success
-    assert_template "index"
-    # Now check that all 20 (or however many were returned) changesets are in the html
-    assert_select "h2", :text => "#{@changesets.size} Recently Closed Changesets", :count => 1
-    assert_select "ul[id='recently_changed'] li a", :count => @changesets.size
-    @changesets.each do |changeset|
-      if changeset.user.data_public?
-        user = changeset.user.display_name
-      else
-        user = "(anonymous)"
-      end
-    
-      cmt = changeset.tags_as_hash['comment'].to_s
-      cmt = "(no comment)" if cmt.length == 0
-      text = "#{changeset.id} by #{user} - #{cmt}"
-      assert_select "ul[id='recently_changed'] li a[href=/browse/changeset/#{changeset.id}]", :text => text
-    end
-  end
-  
   # Test reading a relation
   def test_read_relation
     

test/functional/changeset_controller_test.rb

@@ -995,6 +995,21 @@ EOF
            "element limit.")
   end
   
+  # This should display the last 20 changesets closed.
+  def test_list
+    @changesets = Changeset.find(:all, :order => "created_at DESC", :conditions => ['min_lat IS NOT NULL'], :limit=> 20)
+    assert @changesets.size <= 20
+    get :list
+    assert_response :success
+    assert_template "list"
+    # Now check that all 20 (or however many were returned) changesets are in the html
+    assert_select "h1", :text => "Recent Changes", :count => 1
+    assert_select "table[id='keyvalue'] tr", :count => @changesets.size + 1
+    @changesets.each do |changeset|
+      # FIXME this test needs rewriting - test for table contents
+    end
+  end
+  
   #------------------------------------------------------------
   # utility functions
   #------------------------------------------------------------