Fix PUT api/project/:code/members/:id (#297)

* Fix PUT api/project/:code/members/:id

Before that commit, every PUT *must* change the name of the members, so that was :
- no idempotence,
- no ability to change only weight

fix #295

* Remove redundant comment

1 files changed, 11 insertions, 2 deletions

diff --git a/ihatemoney/tests/tests.py b/ihatemoney/tests/tests.py
index 6708ca8..97e9df3 100644
--- a/ihatemoney/tests/tests.py
+++ b/ihatemoney/tests/tests.py
@@ -1155,7 +1155,8 @@ class APITestCase(IhatemoneyTestCase):
 
         # edit this member
         req = self.client.put("/api/projects/raclette/members/1", data={
-            "name": "Fred"
+            "name": "Fred",
+            "weight": 2,
         }, headers=self.get_auth("raclette"))
 
         self.assertStatus(200, req)
@@ -1166,6 +1167,15 @@ class APITestCase(IhatemoneyTestCase):
 
         self.assertStatus(200, req)
         self.assertEqual("Fred", json.loads(req.data.decode('utf-8'))["name"])
+        self.assertEqual(2, json.loads(req.data.decode('utf-8'))["weight"])
+
+        # edit this member with same information
+        # (test PUT idemopotence)
+        req = self.client.put("/api/projects/raclette/members/1", data={
+            "name": "Fred"
+        }, headers=self.get_auth("raclette"))
+
+        self.assertStatus(200, req)
 
         # delete a member
 
@@ -1175,7 +1185,6 @@ class APITestCase(IhatemoneyTestCase):
         self.assertStatus(200, req)
 
         # the list of members should be empty
-        # get the list of members (should be empty)
         req = self.client.get("/api/projects/raclette/members",
                               headers=self.get_auth("raclette"))