a429024013052687 Add test for tenderPeriod #68
Conversation
| @@ -224,6 +224,27 @@ def create_tender_invalid(self): | |||
| {u'description': [u'tenderPeriod.startDate should be in greater than current date'], u'location': u'body', u'name': u'tenderPeriod'} | |||
| ]) | |||
|
|
|||
| response = self.app.get('/tenders') | |||
There was a problem hiding this comment.
Цього перевіряти непортібно. Якщо в базі буде присутній рендер, це якось вплине на перевірку довжини періоду в наступному запиті?
There was a problem hiding this comment.
Цей тест я не писав. Але якщо його не буде, то цей код буде не покритий тестом.
There was a problem hiding this comment.
А хто його писав? ПР від тебе прийшов.
There was a problem hiding this comment.
Я не на той код подивився. Зараз виправлю
| owner_token = response.json['access']['token'] | ||
| now = get_now() | ||
| response = self.app.patch_json('/tenders/{}?acc_token={}'.format( | ||
| tender['id'], owner_token), {'data': {'_rev': '1-29ce9443d256c3e145f4212a31806937', |
There was a problem hiding this comment.
Звідки взявся _rev? Що означає це поле і його значення?
There was a problem hiding this comment.
Рандомно. rev - MVCC-токен
There was a problem hiding this comment.
Його потрібно передавати в PATCH-i?
There was a problem hiding this comment.
Так. По іншому не вийде.
There was a problem hiding this comment.
Що буде якщо його не передавати?
There was a problem hiding this comment.
не виконається той raise який я покриваю тестом.
andrii-shchudlo
force-pushed
the
a429024013052687_Add_test_for_tenderPeriod
branch
from
6794824 to
13589c3
Compare
andrii-shchudlo
force-pushed
the
a429024013052687_Add_test_for_tenderPeriod
branch
from
13589c3 to
8398abf
Compare
No description provided.