--- a/server/python/django2/renkanmanager/tests/v1_0/tests_revision.py Wed Jun 15 16:31:43 2016 +0200
+++ b/server/python/django2/renkanmanager/tests/v1_0/tests_revision.py Mon Jun 20 14:44:40 2016 +0200
@@ -67,14 +67,32 @@
get_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : "bad-renkan-guid", "revision_guid": self.test_initial_revision_guid})
get_response = self.client.get(get_url, {}, format="json")
+ self.assertEqual(get_response.status_code, status.HTTP_400_BAD_REQUEST)
+
+ ###################################################
+ # GETting with wrong renkan guid
+ ###################################################
+
+ get_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : "00000000-0000-0000-0000-000000000000", "revision_guid": self.test_initial_revision_guid})
+ get_response = self.client.get(get_url, {}, format="json")
self.assertEqual(get_response.status_code, status.HTTP_404_NOT_FOUND)
+
###################################################
# GETting with wrong revision guid
###################################################
get_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : self.test_renkan_guid, "revision_guid": "bad-revision-guid"})
get_response = self.client.get(get_url, {}, format="json")
+ self.assertEqual(get_response.status_code, status.HTTP_400_BAD_REQUEST)
+
+
+ ###################################################
+ # GETting with wrong revision guid
+ ###################################################
+
+ get_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : self.test_renkan_guid, "revision_guid": "00000000-0000-0000-0000-000000000000"})
+ get_response = self.client.get(get_url, {}, format="json")
self.assertEqual(get_response.status_code, status.HTTP_404_NOT_FOUND)
def test_get_revision_list(self):
@@ -138,8 +156,6 @@
delete_response = self.client.delete(delete_url, {}, format="json")
self.assertEqual(delete_response.status_code, status.HTTP_400_BAD_REQUEST)
- # Restoring rightful user
- _ = self.client.login(username="blop", password="blop")
###################################################
# Try to DELETE the initial revision (should 204 now that we added a revision)
@@ -155,7 +171,7 @@
delete_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : "bad-renkan-guid", "revision_guid": self.test_initial_revision_guid})
delete_response = self.client.delete(delete_url, {}, format="json")
- self.assertEqual(delete_response.status_code, status.HTTP_404_NOT_FOUND)
+ self.assertEqual(delete_response.status_code, status.HTTP_400_BAD_REQUEST)
###################################################
# Try to DELETE with wrong revision guid
@@ -163,4 +179,4 @@
delete_url = reverse("v1.0:revision_detail", kwargs={"renkan_guid" : self.test_renkan_guid, "revision_guid": "bad-revision-guid"})
delete_response = self.client.delete(delete_url, {}, format="json")
- self.assertEqual(delete_response.status_code, status.HTTP_404_NOT_FOUND)
+ self.assertEqual(delete_response.status_code, status.HTTP_400_BAD_REQUEST)