Fix the blog post order for the list page.

handroll/extensions/blog.py

@@ -98,7 +98,8 @@ def on_post_composition(self, director):
         """Generate blog output."""
         if not self._should_generate:
             return
-        blog_posts = sorted(self.posts.values(), key=lambda p: p.date)
+        blog_posts = sorted(
+            self.posts.values(), key=lambda p: p.date, reverse=True)
         self._generate_atom_feed(director, blog_posts)
         if self.list_template is not None:
             self._generate_list_page(director, blog_posts)
@@ -117,8 +118,7 @@ def _generate_atom_feed(self, director, blog_posts):
         """Generate the atom feed."""
         logger.info(_('Generating Atom XML feed ...'))
         builder = FeedBuilder(self.atom_metadata)
-        # The feed expects oldest entries first.
-        builder.add(reversed(blog_posts))
+        builder.add(blog_posts)
         output_file = os.path.join(director.outdir, self.atom_output)
         builder.write_to(output_file)
 

handroll/tests/test_blog_extension.py

@@ -198,7 +198,7 @@ def test_adds_post(self, builder_add):
         post = self.factory.make_blog_post()
         extension.posts[post.source_file] = post
         extension.on_post_composition(director)
-        received_post = next(builder_add.call_args[0][0])
+        received_post = builder_add.call_args[0][0][0]
         self.assertEqual(post, received_post)
 
     def test_date_in_post(self):
@@ -230,8 +230,8 @@ def test_posts_added_to_builder_by_date(self, builder_add):
         os.mkdir(director.outdir)
         extension.on_post_composition(director)
         posts = builder_add.call_args[0][0]
-        self.assertEqual(older, next(posts))
-        self.assertEqual(current, next(posts))
+        self.assertEqual(older, posts[0])
+        self.assertEqual(current, posts[1])
 
     def test_list_template_not_required(self):
         director = self.factory.make_director()