Opened 11 years ago

Closed 3 years ago

#583 closed enhancement (fixed)

Create tests for atom feeds

Reported by: Elrond Owned by:
Priority: minor Milestone:
Component: programming Keywords: test
Cc: Jessica Tallon Parent Tickets:


We need some tests for our atom feeds.

How deeply the tests go, is left to the one implementing the tests.
But IMHO each tested feed should at least have one entry.

Change History (5)

comment:1 by ShawnRisk, 10 years ago

Keywords: test added

comment:2 by ShawnRisk, 9 years ago

Cc: Jessica Tallon added

comment:3 by Elrond, 9 years ago

Hi Tsyesika, as you work on pump and activity streams, could you at least comment on this?

comment:4 by Jessica Tallon, 9 years ago

In the near future the content posted to Mediagoblin won't all be public, in fact by default it won't be public. This will require changes to the atom feeds we have now. I have spoken to Elrond, as people might still use and love the old style atom feeds they won't be removed in favour of the new pump style Activity Streams 1.0 feeds however they will need modifying to only show content listed as public.

The tests will probably just add some media which is targeted at "public" and then the feed checked to see if they're available, once the permission model is implemented there probably should also be a test to verify non-public media should not be added to the atom feed.

comment:5 by Ben Sturmfels, 3 years ago

Resolution: fixed
Status: newclosed

I added some basic tests for feeds while re-implementing on top of the feedgenerator library (previous feed classes were dropped from recent Werkzeug).

Note: See TracTickets for help on using tickets.