Ensure closed changesets appear in the feed, and update some misleading comments.
authorAndy Allan <git@gravitystorm.co.uk>
Thu, 1 Jun 2017 14:43:51 +0000 (15:43 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Thu, 1 Jun 2017 14:43:51 +0000 (15:43 +0100)
.rubocop_todo.yml
app/controllers/changeset_controller.rb
test/controllers/changeset_controller_test.rb

index c8788ad..ce7e5d9 100644 (file)
@@ -64,7 +64,7 @@ Metrics/BlockNesting:
 # Offense count: 62
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
-  Max: 1781
+  Max: 1782
 
 # Offense count: 69
 Metrics/CyclomaticComplexity:
index da066a7..77ca20f 100644 (file)
@@ -248,7 +248,7 @@ class ChangesetController < ApplicationController
   end
 
   ##
-  # list edits (open changesets) in reverse chronological order
+  # list non-empty changesets in reverse chronological order
   def list
     if request.format == :atom && params[:max_id]
       redirect_to url_for(params.merge(:max_id => nil)), :status => :moved_permanently
index 3e081a0..db0ee5e 100644 (file)
@@ -2044,9 +2044,10 @@ EOF
   end
 
   ##
-  # This should display the last 20 changesets closed
+  # This should display the last 20 non-empty changesets
   def test_feed
     changeset = create(:changeset, :num_changes => 1)
+    closed_changeset = create(:changeset, :closed, :num_changes => 1)
     _empty_changeset = create(:changeset, :num_changes => 0)
 
     get :feed, :format => :atom
@@ -2054,7 +2055,7 @@ EOF
     assert_template "list"
     assert_equal "application/atom+xml", response.content_type
 
-    check_feed_result([changeset])
+    check_feed_result([changeset, closed_changeset])
   end
 
   ##