Update api.py #10

ri0day · 2018-02-09T03:25:36Z

when create a new topic ,user's uid is neccessary

add creater uid to topics table.

using session during the test, according to prev commit

adjust test order ,prevent celery_test run first.

danidee10 · 2018-02-09T10:41:54Z

Hi, @ri0day you don't need to update the tests. The test_zelery function was named zelery for a reason. Read the article and you'll understand why (I eventually changed it back to celery after explaining).

Also, you shouldn't use self.session to make the requests for all test cases. because it stores cookies and auth information that you may not want to re-use for some tests.

using a fresh requests object gives us a blank slate with no auth information or cookies attached to it.

ri0day added 3 commits February 9, 2018 11:19

Update api.py

450845e

add creater uid to topics table.

Update test_api.py

45902b2

using session during the test, according to prev commit

Update test_api.py

a442bc5

adjust test order ,prevent celery_test run first.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update api.py #10

Update api.py #10

ri0day commented Feb 9, 2018

danidee10 commented Feb 9, 2018 •

edited

Update api.py #10

Are you sure you want to change the base?

Update api.py #10

Conversation

ri0day commented Feb 9, 2018

danidee10 commented Feb 9, 2018 • edited

danidee10 commented Feb 9, 2018 •

edited