Landlease procedure by denis-quintagroup · Pull Request #1018 · openprocurement/robot_tests

denis-quintagroup · 2019-01-15T12:33:14Z

This change is

ivanka12 · 2019-01-15T13:27:37Z

op_robot_tests/tests_files/contract_signing.robot

@@ -1,5 +1,6 @@
 *** Settings ***
 Resource        keywords.robot
+Resource        base_keywords.robot


потрібно забрати keywords.robot, якщо імпортуєш base_keywords.robot, keywords.robot вже імпортується в base_keywords.robot

ivanka12 · 2019-01-15T13:28:25Z

op_robot_tests/tests_files/contract_signing.robot

 #             CONTRACT
 ##############################################################################################

+


лишній пустий рядок

ivanka12 · 2019-01-15T13:29:38Z

op_robot_tests/tests_files/contract_signing.robot

+  [Tags]   ${USERS.users['${viewer}'].broker}: Відображення основних даних угоди
+  ...      viewer
+  ...      ${USERS.users['${viewer}'].broker}
+  ...      dateSigned_view


можна лишити один тег dateSigned

ivanka12 · 2019-01-15T13:36:48Z

op_robot_tests/tests_files/initial_data.py

    return fake.dgfID()


+def create_fake_dateSigned():


може просто назвати create_fake_date

ivanka12 · 2019-01-15T13:42:43Z

op_robot_tests/tests_files/keywords.robot

-  Should Match Regexp  ${url}  ^https?:\/\/sandbox\.ea2\.openprocurement\.auction\/auctions\/([0-9A-Fa-f]{32})
+  ${procurementMethodType}=  Отримати дані із тендера  ${viewer}  ${TENDER['TENDER_UAID']}  procurementMethodType
+  Log  ${procurementMethodType}
+  Run Keyword If  '${procurementMethodType}' == 'landLease'


можна привязати до MODE і не треба вичитувати procurementMethodType

ivanka12 · 2019-01-15T13:55:10Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      tender_owner
+  ...      ${USERS.users['${tender_owner}'].broker}
+  ...      modify_auction_rectificationPeriod_endDate
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}


Оновити LAST_MODIFICATION_DATE після кожних змін. Тут і в наступних виправ

ivanka12 · 2019-01-15T13:57:56Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_auction_registrationFee
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${new_guarantee_amount}=  create_fake_guarantee  ${USERS.users['${tender_owner}'].new_amount}


можливо варто використовувати поле зі словника tender_data.data, так ТК буде універсальнішим

ivanka12 · 2019-01-15T13:59:20Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      ${USERS.users['${tender_owner}'].broker}
+  ...      modify_auction_bankAccount
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  ${new_bankAccount_bankName}=  create_fake_bankName


тут враховано, що нове значення може співпасти зі старим і змін не відбудеться?

ivanka12 · 2019-01-15T14:01:58Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_classification_id
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${scheme_id}=  Run As  ${tender_owner}  Отримати інформацію із об'єкта МП  ${TENDER['TENDER_UAID']}  items[0].classification.id


відображення цього поля має йти окремими ТК. Де дальше в ТК використовується змінна scheme_id?

ivanka12 · 2019-01-15T14:03:21Z

op_robot_tests/tests_files/openProcedure.robot

  Неможливість подати цінову попрозицію без кваліфікації користувачем ${provider}


+Можливість подати пропозицію в статусі драфт першим учасником


цей ТК не потрібний, майданчик не може подавати пропозицію і статусі драфт

OrysiaDrabych · 2019-01-16T13:52:51Z

op_robot_tests/tests_files/base_keywords.robot

+
+
+Можливість змінити поле ${field_name} предмета на ${field_value}
+  ${item_id}=  get_id_from_object  ${USERS.users['${tender_owner}'].tender_data.data['items'][0]}


Це в майданчика не спрацює, бо в його словнику ${USERS.users['${tender_owner}'].tender_data} нема items, так як відображення цих полів не робилось для ролі tender_owner, тільки viewer. Тут краще використати словник ${USERS.users['${tender_owner}'].initial_data.data['items'][0]}

треба зробити це ключове слово універсальнішим і додати індекс, бо так получається, що можна змінити лише якщо один предмет

OrysiaDrabych · 2019-01-16T13:55:38Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

+  ${item_index}=  get_object_index_by_id  ${tender.data['items']}  ${item_id}
+  Set_To_Object  ${tender['data']['items'][${item_index}]}  ${fieldname}  ${fieldvalue}
+  Log  ${tender}
+  ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_tender  ${tender}


В клієнті є окремий метод patch_item, краще його використати

OrysiaDrabych · 2019-01-16T13:58:59Z

op_robot_tests/tests_files/awarding.robot

+  [Tags]   ${USERS.users['${tender_owner}'].broker}: Додання документації
+  ...      tender_owner
+  ...      ${USERS.users['${tender_owner}'].broker}
+  ...      add_rejectionProtocol


мабуть варто додати сценарій, де б використовувався цей тест кейс

OrysiaDrabych · 2019-01-16T14:03:30Z

op_robot_tests/tests_files/base_keywords.robot

 ##############################################################################################
 #             BIDDING
 ##############################################################################################
+Можливість подати цінову пропозицію в статусі драфт користувачем ${username}


Цей ківорд зайвий

OrysiaDrabych · 2019-01-16T14:03:49Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

 #             Bid operations
 ##############################################################################

+Подати цінову пропозицію в статусі драфт


Цей ківорд зайвий

OrysiaDrabych · 2019-01-16T15:13:39Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

  ${tender}=  Call Method  ${USERS.users['${username}'].client}  create_tender  ${tender_data}
  Log object data  ${tender}  created_tender
  ${access_token}=  Get Variable Value  ${tender.access.token}
+  ${status}=  Set Variable If  'dgfOtherAssets' in '${MODE}'  active.tendering  ${EMPTY}


Для dgfOtherAssets не потрібно нічого патчити взагалі, бо там процедура одразу створюється в статусі active.tendering, не draft
Тому тут умову якось по-іншому варто зробити. Можливо щось таке

Run Keyword If '${mode}' == 'geb' Run Keywords ... Set To Dictionary ${tender['data']} status=active.rectification ... AND Call Method ${USERS.users['${username}'].client} patch_tender ${tender}

OrysiaDrabych · 2019-01-16T15:21:02Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

+  Log  ${reply}
+
+
+Отримати інформацію із об'єкта МП


Це щось з малої приватизації, тут його не треба) достатньо викликати Отримати інформацію із тендера, а цей ківорд забрати взагалі

ivanka12 · 2019-02-04T15:08:47Z

зміни назву ПРу!

ivanka12 · 2019-02-04T15:13:09Z

op_robot_tests/tests_files/contract_signing.robot

@@ -1,6 +1,5 @@
 *** Settings ***
 Resource        keywords.robot
-Resource        base_keywords.robot


можливо треба було забрати keywords.robot і залишити base_keywords.robot

ivanka12 · 2019-02-04T15:14:02Z

op_robot_tests/tests_files/contract_signing.robot

 *** Settings ***
 Resource        keywords.robot
-Resource        base_keywords.robot
 Resource        resource.robot


для чого тут resource.robot?

ivanka12 · 2019-02-04T15:28:26Z

robot_tests_arguments/landlease_award_desqualification.txt

+-i find_tender
+
+
+


поправ відступи

ivanka12 · 2019-02-04T15:29:02Z

op_robot_tests/tests_files/base_keywords.robot

  ${DIALOGUE_TYPE}=  Get Variable Value  ${DIALOGUE_TYPE}
  Run keyword if  '${DIALOGUE_TYPE}' != '${None}'  Set to dictionary  ${tender_parameters}  dialogue_type=${DIALOGUE_TYPE}
  ${tender_data}=  Підготувати дані для створення тендера  ${tender_parameters}
+  Log  ${tender_data}


тут можна не логувати дані

ivanka12 · 2019-02-05T07:01:39Z

op_robot_tests/tests_files/initial_data.py

+
+
+def create_fake_items_quantity():
+    return round(random.uniform(5, 10), 3)


чому тільки від 5 до 10?

ivanka12 · 2019-02-05T07:02:48Z

op_robot_tests/tests_files/initial_data.py

+    return round(random.uniform(5, 10), 3)
+
+
+def create_fake_scheme_id_test():


можна забрати з назви test

ivanka12 · 2019-02-05T07:13:25Z

op_robot_tests/tests_files/keywords.robot

-  \  ${left}=  Set Variable  ${USERS.users['${tender_owner}'].initial_data.data['items'][${item_index}].${field}}
-  \  ${right}=  Отримати дані із тендера  ${username}  ${TENDER['TENDER_UAID']}  ${field}  ${item_id}
-  \  compare_additionalClassifications_description  ${right}
+  \  Звірити поле тендера із значенням  ${username}  ${TENDER['TENDER_UAID']}  ${USERS.users['${tender_owner}'].initial_data.data['items'][${item_index}].${field}}  ${field}  ${item_id}


ці зміни не вплинуть на тести для інших процедур?

ivanka12 · 2019-02-05T07:15:35Z

op_robot_tests/tests_files/keywords.robot

+  ...      ${tender_uaid}
+  ...      active.tendering
+
+


ці два пусті рядки лишні

ivanka12 · 2019-02-05T07:17:33Z

op_robot_tests/tests_files/op_faker/op_faker_data.json

+        "06111000-3",
+        "06112000-0",
+        "06120000-9"
+    ],


чому тут не всі id, дані для яких розписані нижче?

ivanka12 · 2019-02-05T07:39:24Z

op_robot_tests/tests_files/openProcedure.robot

+  Можливість змінити поле guarantee.amount тендера на ${new_guarantee_amount}
+
+
+Відображення зміненого розміру реєстраційного внеску


цей ТК також лишній

ivanka12 · 2019-02-05T07:41:18Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_auction_budgetSpent
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${new_budgetSpent_amount}=  create_fake_value  ${USERS.users['${tender_owner}'].tender_data.data.value.amount}


даний ТК не пройде від ролі майданчика, звірення поля виконується тільки для viewer, в словнику tender_owner цього поля не буде. Для всіх наступних ТК на редагування budgetSpent ідентично

Є звірення для усіх користувачів.

не зрозуміло, чого змінюється поле budgetSpent.amount, але дані беруться поля value.amount

ivanka12 · 2019-02-05T07:44:48Z

op_robot_tests/tests_files/openProcedure.robot

  ...      modify_auction_description
-  [Setup]  Дочекатись дати закінчення періоду редагування лоту  ${tender_owner}
+  [Setup]  Дочекатись дати закінчення періоду редагування лоту  ${tender_owner}  ${TENDER['TENDER_UAID']}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE


для неможливостей не треба Оновити LAST_MODIFICATION_DATE

ivanka12 · 2019-02-05T07:45:45Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      provider
+  ...      ${USERS.users['${provider}'].broker}
+  ...      add_eligibilityDocuments_to_bid_by_provider
+  [Setup]  Дочекатись дати закінчення прийому пропозицій  ${viewer}  ${TENDER['TENDER_UAID']}


потрібно для provider

ivanka12 · 2019-02-05T07:55:43Z

op_robot_tests/tests_files/openProcedure.robot

+
+
+Можливість кваліфікувати пропозицію першим учасником
+  [Tags]   ${USERS.users['${provider1}'].broker}: Подання пропозиції


виправ теги

ivanka12 · 2019-02-05T08:03:25Z

op_robot_tests/tests_files/base_keywords.robot

+
+Можливість змінити поле ${field_name} предмета ${index} на ${field_value}
+  ${item_id}=  get_id_from_object  ${USERS.users['${tender_owner}'].initial_data.data['items'][${index}]}
+  Set To Dictionary  ${USERS.users['${tender_owner}']}  item_id=${item_id}


для чого зберігати item_id?

ivanka12 · 2019-02-06T08:39:48Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

+  ${item_index}=  get_object_index_by_id  ${tender.data['items']}  ${item_id}
+  ${item}=  Create Dictionary  data=${tender['data']['items'][${item_index}]}
+  Set_To_Object  ${item.data}  ${fieldname}  ${fieldvalue}
+  Log  ${tender}


можливо треба було залогувати item, а не tender

ivanka12 · 2019-02-06T08:42:00Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

  ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
  ${reply}=  Call Method  ${USERS.users['${username}'].client}  create_bid  ${tender}  ${bid}
+  Run Keyword IF  '${tender.data.procurementMethodType}' == 'landLease'
+  ...      Set To Dictionary  ${reply.data}  status=pending


для цієї умови треба додати patch_bid, в інших випадках patch_bid безтолковий

ivanka12 · 2019-02-27T11:09:16Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      ${USERS.users['${viewer}'].broker}
+  ...      tender_view_landLease
+  Звірити відображення поля bankAccount.accountIdentification[0].description тендера для користувача ${viewer}
+


додай відступ, по два між ТК має бути

ivanka12 · 2019-02-27T11:22:02Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_auction_minNumberOfQualifiedBids
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${new_minNumberOfQualifiedBids}=  Set Variable  ${1}


це з врахуванням, що початкове значення поля буде завжди більше 1?

ivanka12 · 2019-02-27T12:42:50Z

op_robot_tests/tests_files/openProcedure.robot

+  Можливість завантажити документ до тендера з типом cancellationDetails
+
+
+


лишній пустий рядок!

OrysiaDrabych · 2019-03-21T12:19:58Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

-  Log  ${reply_active}
  Set To Dictionary  ${USERS.users['${username}']}  bid_id=${reply['data']['id']}
  Log  ${reply}
  [return]  ${reply}


Тут логуємо і повертаємо reply ще в статусі draft, хоча для landlease ще був патч і статус pending

В reply буде записано і залоговано статус pending, оскільки вище виконується Set To Dictionary ${reply.data} status=pending

думаю Орися мала на увазі, що reply логується для того, щоб зберегти відповідь з цбд, а не з того, що ти записав в словник

OrysiaDrabych · 2019-03-21T12:26:43Z

op_robot_tests/tests_files/contract_signing.robot

+  ...      critical
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${award_amount}=  Get From Dictionary  ${USERS.users['${viewer}'].tender_data.data.awards[0].value}  amount


Словник ${USERS.users['${viewer}'].tender_data.data.awards[0].value} буде пустий в майданчика, бо значення туди записується в тк на відображення поля. Потрібно додати такий тест кейс в с'ют awarding
Або ж використати тут не Get From Dictionary а викликати ківорд Отримати дані із тендера

Це був лишній рядок. Прибрав з коду.

OrysiaDrabych · 2019-03-21T12:39:28Z

op_robot_tests/tests_files/initial_data.py

+    return random.choice(banks_name)
+
+
+def create_fake_month(start=1, end=12):


ця функція використовується при створенні і для редагування, тому потрібно передбачити можливість того, що значення співпаде з попереднім

OrysiaDrabych · 2019-03-21T12:43:47Z

op_robot_tests/tests_files/initial_data.py

    period_dict["auctionPeriod"] = {}
    inc_dt += timedelta(minutes=params['intervals']['auction'][0])
    period_dict["auctionPeriod"]["startDate"] = inc_dt.isoformat()
+    print(period_dict["auctionPeriod"]["startDate"])


прінт зайвий

OrysiaDrabych · 2019-03-21T12:54:27Z

op_robot_tests/tests_files/openProcedure.robot

  Звірити відображення поля minNumberOfQualifiedBids тендера для користувача ${viewer}


+Відображення номер лоту замовника


номера

OrysiaDrabych · 2019-03-21T13:20:51Z

op_robot_tests/tests_files/openProcedure.robot

  ...      tender_owner
  ...      ${USERS.users['${tender_owner}'].broker}
-  ...      modify_auction_title
+  ...      modify_auction_title_after_bidding


нагадай, будь ласка, чому тут after_bidding?

Тому що в процедурі other потрібно перевірити можливість редагування після подачі пропозиції.

OrysiaDrabych · 2019-03-21T13:21:34Z

op_robot_tests/tests_files/openProcedure.robot

  ...      ${USERS.users['${tender_owner}'].broker}
-  ...      add_tender_doc
-  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ...      add_tender_doc_after_bidding


чому тут тільки для доків тег змінився, а, наприклад, для add_item і інших ні?

Тому що цей тест викликається після закінчення періоду редагування лоту, і в такому випадку на процедурі geb він впаде оскільки там можна вантажити документи після даного періоду, а в other ні. Тому цей тест і був виділений окремим тегом і включений в усі сценарії для процедури other.

OrysiaDrabych · 2019-03-21T13:29:42Z

op_robot_tests/tests_files/initial_data.py

+    data['procurementMethodType'] = "landLease"
+    data['lotIdentifier'] = fake.dgfID()
+    data['lotHolder'] = fake.procuringEntity()
+    data['registrationFee'] = {


registrationFee не залежить від value.amount?
Бо в test_tender_data ти генеруєш одне значення, з яким створиш процедуру, а вже registrationFee вираховуєш на основі нового значення, яке генерується в цій функції

OrysiaDrabych · 2019-03-21T13:34:28Z

robot_tests_arguments/landlease_award_desqualification.txt

@@ -0,0 +1,51 @@
+-v MODE:geb


Подумай, які ще можна додати сценарії. Наприклад для 1 учасника, або щось ще по авардінгу. Бо в нас по суті тільки базовий для 2 ох учасників і цей по дискваліфікації

OrysiaDrabych · 2019-03-21T13:35:27Z

robot_tests_arguments/landlease_award_desqualification.txt

+-i number_of_awards
+-i add_rejectionProtocol
+-i disqualified_first_award
+-i first_award_status_unsuccessful


тут незаврешений сценарій. Інший ж авард ще активний, з ним можна підписати договір

В процедурі geb тільки один авард.

ivanka12 · 2019-04-03T06:08:46Z

op_robot_tests/tests_files/initial_data.py

-    }
+    # data['rectificationPeriod'] = {
+    #     "endDate": (get_now() + timedelta(minutes=(random.randint(5, 19) * 1440) / accelerator)).isoformat(),
+    # }


для чого цей закоментований код?

Цей код не потрібний для заведення процедур, вже видалив.

ivanka12 · 2019-04-03T06:54:10Z

op_robot_tests/tests_files/initial_data.py

    data = test_tender_data(params, [])

-    data['dgfID'] = fake.dgfID()
+    # data['dgfID'] = fake.dgfID()


тут також

Змінилась назва поля в процедурі, і це вже не потрібно геренрувати, також прибрав з коду.

ivanka12 · 2019-04-03T06:55:23Z

op_robot_tests/tests_files/contract_signing.robot

+  ${max_amount}=  Evaluate  ${starting_price}+${starting_price}
+  ${amount}=  create_fake_amount  ${starting_price}  ${max_amount}
+  Set to dictionary  ${USERS.users['${tender_owner}']}  amount=${amount}
+  Run As  ${tender_owner}  Редагувати угоду  ${TENDER['TENDER_UAID']}  -1  value.amount  ${amount}


краще винести в окреме ключове слово

Виніс в окремий кейворд, і змінив генерацію значення на початкова вартість угорди + мінімальний крок

…auction period

ivanka12 · 2019-04-05T05:41:07Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

+  Set To Dictionary  ${response['data']}  documentType=rejectionProtocol
+  ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_award_document  ${tender}  ${response}  ${award_id}  ${response['data'].id}
+  Log  ${reply}
+  [return]  ${reply}


ключове слово ідентичне до "Завантажити протокол аукціону в авард" відрізняється лише documentType, можливо краще винести окреме ключове слово Завантажити протокол в авард, в аргументах якого передавати documentType і не дублювати код

Виніс в окреме ключове слово з можливістю вказати тип документа в аргументі

ivanka12 · 2019-04-05T05:43:27Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

+  ...      Set To Dictionary  ${tender['data']}   status=active.rectification
+  ...      AND
+  ...      Call Method  ${USERS.users['${username}'].client}  patch_tender  ${tender}
+  Log  ${tender}


лишнє логування для всіх процедур, крім geb

Виніс в умови яка виконується тільки на процедурі geb

ivanka12 · 2019-04-05T05:47:48Z

op_robot_tests/tests_files/brokers/openprocurement_client.robot

-  Log  ${reply_active}
  Set To Dictionary  ${USERS.users['${username}']}  bid_id=${reply['data']['id']}
  Log  ${reply}
  [return]  ${reply}


думаю Орися мала на увазі, що reply логується для того, щоб зберегти відповідь з цбд, а не з того, що ти записав в словник

ivanka12 · 2019-04-05T06:54:54Z

op_robot_tests/tests_files/keywords.robot

  \  ${right}=  Отримати дані із тендера  ${username}  ${TENDER['TENDER_UAID']}  ${field}  ${item_id}
-  \  compare_additionalClassifications_description  ${right}
+  \  Run Keyword If  '${MODE}'=='dgfOtherAssets'  compare_additionalClassifications_description  ${right}
+  \  ...        ELSE  Звірити поле тендера із значенням  ${username}  ${TENDER['TENDER_UAID']}  ${left}  ${field}  ${item_id}


потрібно перейменувати ключове слово 'Звірити поле ${field} тендера усіх предметів для користувача ${username}' на 'звірити відображення опису додаткової класифікаці усіх предметів для користувача ${username}', або щось типу того, в назві треба вказати конкретну дію, яка виконується

Перейменував на Звірити відображення опису додаткової класифікаці усіх предметів для користувача ${viewer}

ivanka12 · 2019-04-05T06:58:48Z

op_robot_tests/tests_files/op_faker/op_faker_data.json

                {
-                    "scheme": "CPVS",
-                    "id": "PA01-7",
-                    "description": "Оренда"


чому цей класифікатор видалено?

бо для landLease потрібно kvtspz і cadastralNumber

ivanka12 · 2019-04-05T07:07:26Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_auction_registrationFee
+  [Setup]  Дочекатись синхронізації з майданчиком  ${viewer}
+  Run Keyword And Ignore Error  Remove From Dictionary  ${USERS.users['${viewer}'].tender_data.data.registrationFee}  amount
+  Звірити відображення поля guarantee.amount тендера із ${USERS.users['${tender_owner}'].new_guarantee_value} для користувача ${viewer}


для чого видаляти registrationFee.amount, якщо звірення йде для поля guarantee.amount?

Звірення повинно було бути для registrationFee.amount. Поправив.

ivanka12 · 2019-04-05T07:10:37Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      modify_auction_budgetSpent
+  [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  ${new_budgetSpent_amount}=  create_fake_value  ${USERS.users['${tender_owner}'].tender_data.data.value.amount}


не зрозуміло, чого змінюється поле budgetSpent.amount, але дані беруться поля value.amount

ivanka12 · 2019-04-05T07:13:51Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      ${USERS.users['${viewer}'].broker}  ${USERS.users['${tender_owner}'].broker}
+  ...      modify_asset_description
+  Run Keyword And Ignore Error  Remove From Dictionary  ${USERS.users['${viewer}'].tender_data.data['items'][0]}  description
+  Звірити відображення поля items[0].description тендера із ${USERS.users['${tender_owner}'].new_description} для користувача ${viewer}


чому тут використовується звірення поля тендера, а не звірення поля зміненого предмета?

Переробив на звірення поля зміненого предмета

ivanka12 · 2019-04-05T07:24:22Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      ${USERS.users['${tender_owner}'].broker}
+  ...      add_tender_cancellationDetails
+  [Teardown]  Оновити LAST_MODIFICATION_DATE
+  Можливість завантажити документ до тендера з типом cancellationDetails


треба додати хоча б по одному ТК на відображення полів доданих документів, щоб переконатися, що вони справді завантажились

ivanka12 · 2019-04-05T07:26:06Z

op_robot_tests/tests_files/openProcedure.robot

+  ...      ask_question_to_tender_after_tendering_period_geb
+  [Teardown]  Оновити LMD і дочекатись синхронізації  ${provider}
+  [Setup]  Дочекатись дати закінчення прийому пропозицій  ${viewer}  ${TENDER['TENDER_UAID']}
+  Можливість задати запитання на тендер користувачем ${provider}


є можливість відповісти на запитання після закінчення прийому пропозицій?

Так, там ще enquiryPeriod триває

тоді треба додати ТК на відповідь і ТК на перевірку полів питання

ivanka12 · 2019-04-08T11:06:41Z

op_robot_tests/tests_files/base_keywords.robot



+Звірити відображення поля ${field} зміненого предмета ${index} із ${data} для користувача ${username}
+  ${item_id}=  get_id_from_object  ${USERS.users['${tender_owner}'].initial_data.data['items'][${index}]}


чи є можливість змінювати description предмета?

ivanka12 · 2019-04-08T11:14:46Z

op_robot_tests/tests_files/initial_data.py

+    return "P{}M".format(random.choice(month))
+
+
+


лишній пустий рядок

Denys Broianovskyi added 2 commits November 16, 2018 17:12

add scenario lendlease procedure

2313138

add landlease tests

c53117b

ivanka12 requested a review from OrysiaDrabych January 15, 2019 13:08

ivanka12 reviewed Jan 15, 2019

View reviewed changes

OrysiaDrabych reviewed Jan 16, 2019

View reviewed changes

fix bugs and add new landlease scenario

4ca6e3f

denis-quintagroup force-pushed the eauction_cdb2 branch from ef880c7 to 4ca6e3f Compare February 4, 2019 14:29

ivanka12 reviewed Feb 4, 2019

View reviewed changes

ivanka12 reviewed Feb 5, 2019

View reviewed changes

denis-quintagroup changed the title ~~Eauction cdb2~~ Landlease procedure Feb 6, 2019

ivanka12 reviewed Feb 6, 2019

View reviewed changes

Denys Broianovskyi added 2 commits February 22, 2019 13:05

fix bugs and improves test

3947530

remove resource from contract_signing.robot

2065713

ivanka12 reviewed Feb 27, 2019

View reviewed changes

remove modify_auction_minNumberOfQualifiedBids test

ad993bb

OrysiaDrabych reviewed Mar 21, 2019

View reviewed changes

Denys Broianovskyi added 2 commits March 29, 2019 13:42

improve function for generation data and fix bugs in tests

1e88b5c

rename field in test from dgfID to lotIdentifie

ad21b61

ivanka12 reviewed Apr 3, 2019

View reviewed changes

add tests in enquiry and auction period, and increase time to expect …

64d3f4f

…auction period

ivanka12 reviewed Apr 5, 2019

View reviewed changes

ivanka12 reviewed Apr 8, 2019

View reviewed changes

op_robot_tests/tests_files/initial_data.py

return "P{}M".format(random.choice(month))

Copy link

Contributor

ivanka12 Apr 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лишній пустий рядок

Denys Broianovskyi added 4 commits April 9, 2019 11:24

remove comments and separate test in another keyword

2971d75

Add test to check display question in enquiry period

355be7a

Add universal keywords to test uploading documents to the award

2671daa

change keywords and function for generation data in edit items tests

d8712e6

fix bugs in change budgetSpent and registrationFee fields tests

a07684f

		# CONTRACT
		##############################################################################################

		Неможливість подати цінову попрозицію без кваліфікації користувачем ${provider}


		Можливість подати пропозицію в статусі драфт першим учасником



		Можливість змінити поле ${field_name} предмета на ${field_value}
		${item_id}= get_id_from_object ${USERS.users['${tender_owner}'].tender_data.data['items'][0]}

		Log ${reply}


		Отримати інформацію із об'єкта МП



		def create_fake_items_quantity():
		return round(random.uniform(5, 10), 3)

		return round(random.uniform(5, 10), 3)


		def create_fake_scheme_id_test():

		Можливість змінити поле guarantee.amount тендера на ${new_guarantee_amount}


		Відображення зміненого розміру реєстраційного внеску



		Можливість кваліфікувати пропозицію першим учасником
		[Tags] ${USERS.users['${provider1}'].broker}: Подання пропозиції

		Можливість завантажити документ до тендера з типом cancellationDetails

		return random.choice(banks_name)


		def create_fake_month(start=1, end=12):

		Звірити відображення поля minNumberOfQualifiedBids тендера для користувача ${viewer}


		Відображення номер лоту замовника



		Звірити відображення поля ${field} зміненого предмета ${index} із ${data} для користувача ${username}
		${item_id}= get_id_from_object ${USERS.users['${tender_owner}'].initial_data.data['items'][${index}]}

Conversation

denis-quintagroup commented Jan 15, 2019 • edited by myroslav Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ivanka12 commented Feb 4, 2019

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

denis-quintagroup commented Jan 15, 2019 •

edited by myroslav

Loading