append posts to atom feed to keep post order from new to old

[lexming]

PR Description

Fixes #215

Looking at the code, it seems that there is the intention in ablog.posts.generate_atom_feeds to order the posts from newer to older. See:

ablog/src/ablog/post.py

Line 675 in af590e1

sorted_posts_by_date = sorted(feed_posts, key=lambda post: post.date, reverse=True)

However, later on, posts in sorted_posts_by_date are added to the feed with:

ablog/src/ablog/post.py

Line 686 in af590e1

feed_entry = feed.add_entry()

The method add_entry does prepend each post to the feed though (default behaviour), effectively reversing the order of posts a second time.

This PR explicitly sets the order of add_entry to not alter the post order set in sorted_posts_by_date.

[nabobalis]

Thanks, if you can update the unit test for this in ablog/tests/test_build.py and probably need to actually test the date of the post otherwise clearly I did not spot this bug from my previous fix.

[lexming]

@nabobalis I added a check to test_feed to explicitly test that the date of the first post in the feed is older than the second one.

[nabobalis]

Thanks!

[nabobalis]

I will get a release out soon.