Skip to content

Commit 43cafb1

Browse files
committed
update comments once more
1 parent 9a5b2ad commit 43cafb1

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

flaskapi/test_app.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,9 @@ def test_update(self):
6969
self.assertEqual(response.status_code, 200)
7070
data = json.loads(response.get_data())
7171
self.assertEqual(data['item']['value'], 30)
72-
# backup items should not change = proof need for deepcopy in setUp
73-
self.assertEqual(self.backup_items[2]['value'], 20) # not 30!
72+
# proof need for deepcopy in setUp: update app.items should not affect self.backup_items
73+
# this fails when you use shallow copy
74+
self.assertEqual(self.backup_items[2]['value'], 20) # org value
7475

7576
def test_update_error(self):
7677
# cannot edit non-existing item
@@ -89,12 +90,11 @@ def test_update_error(self):
8990
def test_delete(self):
9091
response = self.app.delete(GOOD_ITEM_URL)
9192
self.assertEqual(response.status_code, 204)
92-
# cannot delete non-existing item
9393
response = self.app.delete(BAD_ITEM_URL)
9494
self.assertEqual(response.status_code, 404)
9595

9696
def tearDown(self):
97-
# to start next test with fresh copy of items
97+
# reset app.items to initial state
9898
app.items = self.backup_items
9999

100100

0 commit comments

Comments
 (0)