Remove return statements where were redundant

mykhaly
1 parent 91964465
Showing 3 changed files with 11 additions and 40 deletions
op_robot_tests/tests_files/aboveThreshold_keywords.robot
op_robot_tests/tests_files/base_keywords.robot
op_robot_tests/tests_files/brokers/openprocurement_client.robot
--- a/op_robot_tests/tests_files/aboveThreshold_keywords.robot
View file @c8ac67a
+++ b/op_robot_tests/tests_files/aboveThreshold_keywords.robot
View file @c8ac67a
@@ -32,9 +32,7 @@ Resource           base_keywords.robot
 Можливість підтвердити цінову пропозицію учасником ${username}
   ${status}=  Run Keyword IF  '${MODE}'=='openeu'  Set Variable  pending
   ...                     ELSE IF  '${MODE}'=='openua'  Set Variable  active
-  ${activestatusresp}=  Run As  ${username}  Змінити цінову пропозицію  ${TENDER['TENDER_UAID']}  status  ${status}
-  Set To Dictionary  ${USERS.users['${username}'].bidresponses}  activestatusresp=${activestatusresp}
-  log  ${activestatusresp}
+  Run As  ${username}  Змінити цінову пропозицію  ${TENDER['TENDER_UAID']}  status  ${status}
 ##############################################################################################
 #             OPENEU  Bid documentation
@@ -45,8 +43,7 @@ Resource           base_keywords.robot
   ${privat_doc}=  create_data_dict  data.confidentialityRationale  ${confidentialityRationale}
   Set To Dictionary  ${privat_doc.data}  confidentiality=buyerOnly
   ${docid}=  Get Variable Value  ${USERS.users['${username}'].bidresponses['bid_doc_upload']['upload_response'].data.id}
-  ${bid_doc_modified}=  Run As  ${username}  Змінити документацію в ставці  ${TENDER['TENDER_UAID']}  ${privat_doc}  ${docid}
-  Set To Dictionary  ${USERS.users['${username}'].bidresponses}  bid_doc_modified=${bid_doc_modified}
+  Run As  ${username}  Змінити документацію в ставці  ${TENDER['TENDER_UAID']}  ${privat_doc}  ${docid}
 Можливість завантажити ${doc_type} документ до пропозиції учасником ${username}
--- a/op_robot_tests/tests_files/base_keywords.robot
View file @c8ac67a
+++ b/op_robot_tests/tests_files/base_keywords.robot
View file @c8ac67a
@@ -235,7 +235,7 @@ Resource           resource.robot
 Можливість створення лоту із прив’язаним предметом закупівлі
   ${lot}=  Підготувати дані для створення лоту  ${USERS.users['${tender_owner}'].tender_data.data.value.amount}
   ${item}=  Підготувати дані для створення предмету закупівлі  ${USERS.users['${tender_owner}'].initial_data.data['items'][0]['classification']['id']}
-  ${lot_resp}=  Run As  ${tender_owner}  Створити лот із предметом закупівлі  ${TENDER['TENDER_UAID']}  ${lot}  ${item}
+  Run As  ${tender_owner}  Створити лот із предметом закупівлі  ${TENDER['TENDER_UAID']}  ${lot}  ${item}
   ${item_id}=  get_id_from_object  ${item}
   ${item_data}=  Create Dictionary
   ...      item=${item}
@@ -244,7 +244,6 @@ Resource           resource.robot
   ${lot_id}=  get_id_from_object  ${lot.data}
   ${lot_data}=  Create Dictionary
   ...      lot=${lot}
-  ...      lot_resp=${lot_resp}
   ...      lot_id=${lot_id}
   ${lot_data}=  munch_dict  arg=${lot_data}
   Set To Dictionary  ${USERS.users['${tender_owner}']}  item_data=${item_data}  lot_data=${lot_data}
@@ -408,13 +407,12 @@ Resource           resource.robot
 Можливість задати запитання на тендер користувачем ${username}
   ${question}=  Підготувати дані для запитання
-  ${question_resp}=  Run As  ${username}  Задати запитання на тендер  ${TENDER['TENDER_UAID']}  ${question}
+  Run As  ${username}  Задати запитання на тендер  ${TENDER['TENDER_UAID']}  ${question}
   ${now}=  Get Current TZdate
   ${question.data.date}=  Set variable  ${now}
   ${question_id}=  get_id_from_object  ${question.data}
   ${question_data}=  Create Dictionary
   ...      question=${question}
-  ...      question_resp=${question_resp}
   ...      question_id=${question_id}
   ${question_data}=  munch_dict  arg=${question_data}
   Set To Dictionary  ${USERS.users['${username}']}  tender_question_data=${question_data}
@@ -423,13 +421,12 @@ Resource           resource.robot
 Можливість задати запитання на ${lot_index} лот користувачем ${username}
   ${lot_id}=  get_id_from_object  ${USERS.users['${tender_owner}'].tender_data.data.lots[${lot_index}]}
   ${question}=  Підготувати дані для запитання
-  ${question_resp}=  Run As  ${username}  Задати запитання на лот  ${TENDER['TENDER_UAID']}  ${lot_id}  ${question}
+  Run As  ${username}  Задати запитання на лот  ${TENDER['TENDER_UAID']}  ${lot_id}  ${question}
   ${now}=  Get Current TZdate
   ${question.data.date}=  Set variable  ${now}
   ${question_id}=  get_id_from_object  ${question.data}
   ${question_data}=  Create Dictionary
   ...      question=${question}
-  ...      question_resp=${question_resp}
   ...      question_id=${question_id}
   ${question_data}=  munch_dict  arg=${question_data}
   Set To Dictionary  ${USERS.users['${username}']}  lots_${lot_index}_question_data=${question_data}
@@ -438,13 +435,12 @@ Resource           resource.robot
 Можливість задати запитання на ${item_index} предмет користувачем ${username}
   ${item_id}=  get_id_from_object  ${USERS.users['${tender_owner}'].tender_data.data['items'][${item_index}]}
   ${question}=  Підготувати дані для запитання
-  ${question_resp}=  Run As  ${username}  Задати запитання на предмет  ${TENDER['TENDER_UAID']}  ${item_id}  ${question}
+  Run As  ${username}  Задати запитання на предмет  ${TENDER['TENDER_UAID']}  ${item_id}  ${question}
   ${now}=  Get Current TZdate
   ${question.data.date}=  Set variable  ${now}
   ${question_id}=  get_id_from_object  ${question.data}
   ${question_data}=  Create Dictionary
   ...      question=${question}
-  ...      question_resp=${question_resp}
   ...      question_id=${question_id}
   ${question_data}=  munch_dict  arg=${question_data}
   Set To Dictionary  ${USERS.users['${username}']}  items_${item_index}_question_data=${question_data}
@@ -932,8 +928,7 @@ Resource           resource.robot
   ${features_ids}=  Run Keyword IF  ${features}
   ...     Отримати ідентифікатори об’єктів  ${username}  features
   ...     ELSE  Set Variable  ${None}
-  ${resp}=  Run As  ${username}  Подати цінову пропозицію  ${TENDER['TENDER_UAID']}  ${bid}  ${lots_ids}  ${features_ids}
-  Set To Dictionary  ${USERS.users['${username}'].bidresponses}  resp=${resp}
+  Run As  ${username}  Подати цінову пропозицію  ${TENDER['TENDER_UAID']}  ${bid}  ${lots_ids}  ${features_ids}
 Можливість подати цінову пропозицію на другий етап ${index} користувачем ${username}
@@ -948,8 +943,7 @@ Resource           resource.robot
   ${features_ids}=  Run Keyword IF  ${features}
   ...     Отримати ідентифікатори об’єктів  ${username}  features
   ...     ELSE  Set Variable  ${None}
-  ${resp}=  Run As  ${username}  Подати цінову пропозицію  ${TENDER['TENDER_UAID']}  ${bid}  ${lots_ids}  ${features_ids}
-  Set To Dictionary  ${USERS.users['${username}'].bidresponses}  resp=${resp}
+  Run As  ${username}  Подати цінову пропозицію  ${TENDER['TENDER_UAID']}  ${bid}  ${lots_ids}  ${features_ids}
 Неможливість подати цінову пропозицію без прив’язки до лоту користувачем ${username}
   ${bid}=  Підготувати дані для подання пропозиції  ${username}
@@ -984,8 +978,7 @@ Resource           resource.robot
 Можливість змінити документацію цінової пропозиції користувачем ${username}
   ${file_path}  ${file_name}  ${file_content}=  create_fake_doc
   ${docid}=  Get Variable Value  ${USERS.users['${username}'].bidresponses['bid_doc_upload']['upload_response'].data.id}
-  ${bid_doc_modified}=  Run As  ${username}  Змінити документ в ставці  ${TENDER['TENDER_UAID']}  ${file_path}  ${docid}
-  Set To Dictionary  ${USERS.users['${username}'].bidresponses}  bid_doc_modified=${bid_doc_modified}
+  Run As  ${username}  Змінити документ в ставці  ${TENDER['TENDER_UAID']}  ${file_path}  ${docid}
   Remove File  ${file_path}
 ##############################################################################################
--- a/op_robot_tests/tests_files/brokers/openprocurement_client.robot
View file @c8ac67a
+++ b/op_robot_tests/tests_files/brokers/openprocurement_client.robot
View file @c8ac67a
@@ -38,6 +38,7 @@ Library  openprocurement_client_helper.py
   ${tender}=  set_access_key  ${tender}   ${USERS.users['${username}'].access_token}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  upload_document  ${filepath}  ${tender}
   Log object data   ${reply}  reply
+  #return here is needed to have uploaded doc data in `Завантажити документ в лот` keyword
   [return]   ${reply}
@@ -175,6 +176,7 @@ Library  openprocurement_client_helper.py
   [Arguments]  ${username}  ${tender_uaid}  ${lot}
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}   create_lot   ${tender}    ${lot}
+  #return here is needed to have created lot id in `Створити лот з предметом закупівлі` keyword
   [return]  ${reply}
@@ -183,7 +185,6 @@ Library  openprocurement_client_helper.py
   ${reply}=  openprocurement_client.Створити лот  ${username}  ${tender_uaid}  ${lot}
   ${lot_id}=  get_id_from_object  ${lot.data}
   openprocurement_client.Додати предмет закупівлі в лот  ${username}  ${tender_uaid}  ${lot_id}  ${item}
-  [return]  ${reply}
 Отримати інформацію із лоту
@@ -199,7 +200,6 @@ Library  openprocurement_client_helper.py
   ${lot}=  Create Dictionary  data=${tender.data.lots[${lot_index}]}
   Set_To_Object   ${lot.data}   ${fieldname}   ${fieldvalue}
   ${reply}=  Call Method   ${USERS.users['${username}'].client}   patch_lot   ${tender}   ${lot}
-  [return]  ${reply}
 Додати предмет закупівлі в лот
@@ -220,7 +220,6 @@ Library  openprocurement_client_helper.py
   ${doc}=  openprocurement_client.Завантажити документ  ${username}  ${filepath}  ${tender_uaid}
   ${lot_doc}=  test_lot_document_data  ${doc}  ${lot_id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_document   ${tender}   ${lot_doc}
-  [return]   ${reply}
 Видалити лот
@@ -233,7 +232,6 @@ Library  openprocurement_client_helper.py
   \  Run Keyword If  '${item.relatedLot}'=='${lot.data.id}'
   \  ...     openprocurement_client.Видалити предмет закупівлі  ${username}  ${tender_uaid}  ${item_id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}   delete_lot   ${tender}    ${lot}
-  [return]  ${reply}
 Скасувати лот
@@ -321,7 +319,6 @@ Library  openprocurement_client_helper.py
   ${item_id}=  Get Variable Value  ${tender.data['items'][${item_index}].id}
   ${question}=  test_related_question  ${question}  item  ${item_id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  create_question  ${tender}  ${question}
-  [return]  ${reply}
 Задати запитання на лот
@@ -331,14 +328,12 @@ Library  openprocurement_client_helper.py
   ${lot_id}=  Get Variable Value  ${tender.data.lots[${lot_index}].id}
   ${question}=  test_related_question  ${question}  lot  ${lot_id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  create_question  ${tender}  ${question}
-  [return]  ${reply}
 Задати запитання на тендер
   [Arguments]  ${username}  ${tender_uaid}  ${question}
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  create_question  ${tender}  ${question}
-  [return]  ${reply}
 Отримати інформацію із запитання
@@ -353,7 +348,6 @@ Library  openprocurement_client_helper.py
   ${tender}=  set_access_key  ${tender}  ${USERS.users['${username}'].access_token}
   ${answer_data.data.id}=  openprocurement_client.Отримати інформацію із запитання  ${username}  ${tender_uaid}  ${question_id}  id
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_question  ${tender}  ${answer_data}
-  [return]  ${reply}
 ##############################################################################
 #             Claims
@@ -724,7 +718,6 @@ Library  openprocurement_client_helper.py
   Set To Dictionary  ${USERS.users['${username}']}  access_token=${reply['access']['token']}
   Set To Dictionary   ${USERS.users['${username}'].bidresponses['bid'].data}  id=${reply['data']['id']}
   Log  ${reply_active}
-  [return]  ${reply}
 Змінити цінову пропозицію
@@ -735,7 +728,6 @@ Library  openprocurement_client_helper.py
   ${tender}=  set_access_key  ${tender}  ${USERS.users['${username}'].bidresponses['resp'].access.token}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_bid  ${tender}  ${bid}
   Log  ${reply}
-  [return]   ${reply}
 Скасувати цінову пропозицію
@@ -744,7 +736,6 @@ Library  openprocurement_client_helper.py
   ${bid_id}=  openprocurement_client.Отримати інформацію із пропозиції  ${username}  ${tender_uaid}  id
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  delete_bid   ${tender}  ${bid_id}  ${USERS.users['${username}'].bidresponses['resp'].access.token}
   Log  ${reply}
-  [return]   ${reply}
 Завантажити документ в ставку
@@ -779,7 +770,6 @@ Library  openprocurement_client_helper.py
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${tender}=  set_access_key  ${tender}  ${USERS.users['${username}'].bidresponses['resp'].access.token}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_bid_document   ${tender}   ${doc_data}   ${bid_id}   ${docid}
-  [return]  ${reply}
 Отримати пропозицію
@@ -818,7 +808,6 @@ Library  openprocurement_client_helper.py
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${doc}=  Call Method  ${USERS.users['${username}'].client}  upload_award_document  ${document}  ${tender}  ${tender.data.awards[${award_num}].id}
   Log  ${doc}
-  [Return]  ${doc}
 Підтвердити постачальника
@@ -863,7 +852,6 @@ Library  openprocurement_client_helper.py
   Set To Dictionary  ${award.data}  id=${tender.data.awards[${award_num}].id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_award  ${tender}  ${award}
   Log  ${reply}
-  [Return]  ${reply}
 ##############################################################################
 #             Limited procurement
@@ -989,7 +977,6 @@ Library  openprocurement_client_helper.py
   Set To Dictionary  ${qualification.data}  id=${tender.data.qualifications[${qualification_num}].id}  eligible=${True}  qualified=${True}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_qualification  ${tender}  ${qualification}
   Log  ${reply}
-  [Return]  ${reply}
 Відхилити кваліфікацію
@@ -1003,7 +990,6 @@ Library  openprocurement_client_helper.py
   Set To Dictionary  ${qualification.data}  id=${tender.data.qualifications[${qualification_num}].id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_qualification  ${tender}  ${qualification}
   Log  ${reply}
-  [Return]  ${reply}
 Завантажити документ у кваліфікацію
@@ -1015,7 +1001,6 @@ Library  openprocurement_client_helper.py
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${doc_reply}=  Call Method  ${USERS.users['${username}'].client}  upload_qualification_document  ${document}  ${tender}  ${tender.data.qualifications[${qualification_num}].id}
   Log  ${doc_reply}
-  [Return]  ${doc_reply}
 Скасувати кваліфікацію
@@ -1029,7 +1014,6 @@ Library  openprocurement_client_helper.py
   Set To Dictionary  ${qualification.data}  id=${tender.data.qualifications[${qualification_num}].id}
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_qualification  ${tender}  ${qualification}
   Log  ${reply}
-  [Return]  ${reply}
 Затвердити остаточне рішення кваліфікації
@@ -1046,7 +1030,6 @@ Library  openprocurement_client_helper.py
   set_to_object  ${tender}  data.status  active.pre-qualification.stand-still
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_tender  ${tender}
   Log  ${reply}
-  [Return]  ${reply}
 Перевести тендер на статус очікування обробки мостом
@@ -1063,7 +1046,6 @@ Library  openprocurement_client_helper.py
   set_to_object  ${tender}  data.status  active.stage2.waiting
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_tender  ${tender}
   Log  ${reply}
-  [Return]  ${reply}
 Активувати другий етап
@@ -1080,4 +1062,3 @@ Library  openprocurement_client_helper.py
   set_to_object  ${tender}  data.status  active.tendering
   ${reply}=  Call Method  ${USERS.users['${username}'].client}  patch_tender  ${tender}
   Log  ${reply}
-  [Return]  ${reply}