diff options
author | Jeremy Cline <jcline@redhat.com> | 2019-10-15 17:30:11 -0400 |
---|---|---|
committer | Stephen Finucane <stephen@that.guru> | 2019-11-30 17:00:39 +0000 |
commit | e0f3220b3698fa3f9383b8c536a01f57cad8df15 (patch) | |
tree | 8e3d3c17047254213851d02801ebe2f59e91cccf /patchwork/tests | |
parent | e53d7985a15a90617d653645d1bb0c2693b73ff6 (diff) | |
download | patchwork-e0f3220b3698fa3f9383b8c536a01f57cad8df15.tar patchwork-e0f3220b3698fa3f9383b8c536a01f57cad8df15.tar.gz |
Allow ordering events by date
By default, the events API orders events by date in descending order
(newest first). However, it's useful to be able to order the events by
oldest events first. For example, when a client is polling the events
API for new events since a given date and wishes to process them in
chronological order.
Signed-off-by: Jeremy Cline <jcline@redhat.com>
Reviewed-by: Stephen Finucane <stephen@that.guru>
Diffstat (limited to 'patchwork/tests')
-rw-r--r-- | patchwork/tests/api/test_event.py | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/patchwork/tests/api/test_event.py b/patchwork/tests/api/test_event.py index 8816538..bff8f40 100644 --- a/patchwork/tests/api/test_event.py +++ b/patchwork/tests/api/test_event.py @@ -149,6 +149,24 @@ class TestEventAPI(utils.APITestCase): resp = self.client.get(self.api_url(), {'series': 999999}) self.assertEqual(0, len(resp.data)) + def test_order_by_date_default(self): + """Assert the default ordering is by date descending.""" + self._create_events() + + resp = self.client.get(self.api_url()) + events = Event.objects.order_by("-date").all() + for api_event, event in zip(resp.data, events): + self.assertEqual(api_event["id"], event.id) + + def test_order_by_date_ascending(self): + """Assert the default ordering is by date descending.""" + self._create_events() + + resp = self.client.get(self.api_url(), {'order': 'date'}) + events = Event.objects.order_by("date").all() + for api_event, event in zip(resp.data, events): + self.assertEqual(api_event["id"], event.id) + def test_create(self): """Ensure creates aren't allowed""" user = create_maintainer() |