aboutsummaryrefslogtreecommitdiffstats
path: root/rest_framework/tests/generics.py
diff options
context:
space:
mode:
authorTom Christie2012-10-02 15:39:41 +0100
committerTom Christie2012-10-02 15:39:41 +0100
commitf010a9553ec7ab1645c2bae4c959a19bb5117a21 (patch)
tree4d0f88cb9f8f8da3657ea13319eed84405d5dc0e /rest_framework/tests/generics.py
parentab173fd8f9070ccdb70f86f400d2ffa780977ce4 (diff)
downloaddjango-rest-framework-f010a9553ec7ab1645c2bae4c959a19bb5117a21.tar.bz2
Add test to ensure that pk is readonly in PUT requests
Diffstat (limited to 'rest_framework/tests/generics.py')
-rw-r--r--rest_framework/tests/generics.py12
1 files changed, 12 insertions, 0 deletions
diff --git a/rest_framework/tests/generics.py b/rest_framework/tests/generics.py
index 79c01f9e..3c241f42 100644
--- a/rest_framework/tests/generics.py
+++ b/rest_framework/tests/generics.py
@@ -191,3 +191,15 @@ class TestInstanceView(TestCase):
}
self.assertEquals(response.status_code, status.HTTP_200_OK)
self.assertEquals(response.data, expected)
+
+ def test_put_cannot_set_id(self):
+ """
+ POST requests to create a new object should not be able to set the id.
+ """
+ content = {'id': 999, 'text': 'foobar'}
+ request = factory.put('/1', json.dumps(content), content_type='application/json')
+ response = self.view(request, pk=1).render()
+ self.assertEquals(response.status_code, status.HTTP_200_OK)
+ self.assertEquals(response.data, {'id': 1, 'text': 'foobar'})
+ updated = self.objects.get(id=1)
+ self.assertEquals(updated.text, 'foobar')