Merge pull request #125 from Leits/id

Support for question ids

Merge pull request #125 from Leits/id
Support for question ids
Myroslav Opyr
2 parents df29e887 66039b2a
Showing 5 changed files with 104 additions and 42 deletions
op_robot_tests/tests_files/brokers/openprocurement_client.robot
op_robot_tests/tests_files/keywords.robot
op_robot_tests/tests_files/multiLotTender.robot
op_robot_tests/tests_files/service_keywords.py
op_robot_tests/tests_files/singleItemTender.robot
--- a/op_robot_tests/tests_files/brokers/openprocurement_client.robot
View file @9ef3350
+++ b/op_robot_tests/tests_files/brokers/openprocurement_client.robot
View file @9ef3350
@@ -78,6 +78,12 @@ Library  openprocurement_client_helper.py
   Fail  Field not found: ${field_name}
+Отримати інформацію із запитання
+  [Arguments]  ${username}  ${question_id}  ${field_name}
+  ${field_name}=  Отримати шлях до поля об’єкта  ${username}  ${field_name}  ${question_id}
+  Run Keyword And Return  openprocurement_client.Отримати інформацію із тендера  ${username}  ${field_name}
+
+
 Внести зміни в тендер
   [Arguments]  ${username}  ${tender_uaid}  ${fieldname}  ${fieldvalue}
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
@@ -136,10 +142,10 @@ Library  openprocurement_client_helper.py
 Відповісти на питання
-  [Arguments]  ${username}  ${tender_uaid}  ${question}  ${answer_data}
+  [Arguments]  ${username}  ${tender_uaid}  ${question}  ${answer_data}  ${question_id}
   ${tender}=  openprocurement_client.Пошук тендера по ідентифікатору  ${username}  ${tender_uaid}
   ${tender}=  set_access_key  ${tender}  ${USERS.users['${username}'].access_token}
-  ${answer_data.data.id}=  Set Variable   ${question.data.id}
+  ${answer_data.data.id}=  openprocurement_client.Отримати інформацію із запитання  ${username}  ${question_id}  id
   ${question_with_answer}=  Call Method  ${USERS.users['${username}'].client}  patch_question  ${tender}  ${answer_data}
   Log object data   ${question_with_answer}  question_with_answer
   [return]  ${question_with_answer}
--- a/op_robot_tests/tests_files/keywords.robot
View file @9ef3350
+++ b/op_robot_tests/tests_files/keywords.robot
View file @9ef3350
@@ -311,10 +311,8 @@ Get Broker Property By Username
 Звірити поле тендера із значенням
-  [Arguments]  ${username}  ${left}  ${field}
-  ${right}=  Отримати дані із тендера  ${username}  ${field}
-  Log  ${left}
-  Log  ${right}
+  [Arguments]  ${username}  ${left}  ${field}  ${object_id}=${None}
+  ${right}=  Отримати дані із тендера  ${username}  ${field}  ${object_id}
   Порівняти об'єкти  ${left}  ${right}
@@ -334,8 +332,8 @@ Get Broker Property By Username
 Звірити дату тендера із значенням
-  [Arguments]  ${username}  ${left}  ${field}
-  ${right}=  Отримати дані із тендера  ${username}  ${field}
+  [Arguments]  ${username}  ${left}  ${field}  ${object_id}=${None}
+  ${right}=  Отримати дані із тендера  ${username}  ${field}  ${object_id}
   Порівняти дати  ${left}  ${right}
@@ -374,6 +372,44 @@ Get Broker Property By Username
   \  Звірити дату тендера  ${viewer}  ${tender_data}  items[${index}].${field}
+Отримати дані із тендера
+  [Arguments]  ${username}  ${field_name}  ${object_id}=${None}
+  Log  ${username}
+  Log  ${field_name}
+  ${field}=  Run Keyword If  '${object_id}'=='${None}'  Set Variable  ${field_name}
+  ...             ELSE  Отримати шлях до поля об’єкта  ${username}  ${field_name}  ${object_id}
+  ${status}  ${field_value}=  Run keyword and ignore error
+  ...      Get from object
+  ...      ${USERS.users['${username}'].tender_data.data}
+  ...      ${field}
+  # If field in cache, return its value
+  Run Keyword if  '${status}' == 'PASS'  Return from keyword   ${field_value}
+  # Else call broker to find field
+  ${field_value}=  Run Keyword IF  '${object_id}'=='${None}'  Run As  ${username}  Отримати інформацію із тендера  ${field}
+  ...                          ELSE  Отримати дані із об’єкта тендера  ${username}  ${object_id}  ${field_name}
+  # And caching its value before return
+  Set_To_Object  ${USERS.users['${username}'].tender_data.data}  ${field}  ${field_value}
+  [return]  ${field_value}
+
+
+Отримати шлях до поля об’єкта
+  [Arguments]  ${username}  ${field_name}  ${object_id}
+  ${object_type}=  get_object_type_by_id  ${object_id}
+  ${objects}=  Get Variable Value  ${USERS.users['${username}'].tender_data.data['${object_type}']}  ${empty}
+  ${object_index}=  get_object_index_by_id  ${objects}  ${object_id}
+  [return]  ${object_type}[${object_index}].${field_name}
+
+
+Отримати дані із об’єкта тендера
+  [Arguments]  ${username}  ${object_id}  ${field_name}
+  ${object_type}=   get_object_type_by_id  ${object_id}
+  ${status}  ${value}=  Run Keyword And Ignore Error  Run As  ${username}  Отримати інформацію із запитання  ${object_id}  ${field_name}
+  ${field}=  Отримати шлях до поля об’єкта  ${username}  ${field_name}  ${object_id}
+  ${field_value}=  Run Keyword IF  '${status}'=='PASS'  Set Variable  ${value}
+  ...                          ELSE  Run As  ${username}  Отримати інформацію із тендера  ${field}
+  [return]  ${field_value}
+
+
 Викликати для учасника
   [Arguments]  ${username}  ${command}  @{arguments}
   Run keyword unless  '${WARN_RUN_AS}' == '${True}'
@@ -388,24 +424,6 @@ Get Broker Property By Username
   Run Keyword And Return  Run As  ${username}  ${command}  @{arguments}
-Отримати дані із тендера
-  [Arguments]  ${username}  ${field_name}
-  Log  ${username}
-  Log  ${field_name}
-
-  ${status}  ${field_value}=  Run keyword and ignore error
-  ...      Get from object
-  ...      ${USERS.users['${username}'].tender_data.data}
-  ...      ${field_name}
-  # If field in cache, return its value
-  Run Keyword if  '${status}' == 'PASS'  Return from keyword   ${field_value}
-  # Else call broker to find field
-  ${field_value}=  Викликати для учасника  ${username}  Отримати інформацію із тендера  ${field_name}
-  # And caching its value before return
-  Set_To_Object  ${USERS.users['${username}'].tender_data.data}  ${field_name}  ${field_value}
-  [return]  ${field_value}
-
-
 Run As
   [Arguments]  ${username}  ${command}  @{arguments}
   [Documentation]
@@ -415,7 +433,12 @@ Run As
   Log  ${command}
   Log Many  @{arguments}
   ${keywords_file}=  Get Broker Property By Username  ${username}  keywords_file
-  Run Keyword And Return  ${keywords_file}.${command}  ${username}  @{arguments}
+  ${status}  ${value}=  Run keyword and ignore keyword definitions  ${keywords_file}.${command}  ${username}  @{arguments}
+  ${unexpected_args}=  Get Regexp Matches  '${value}'  expected [0-9] arguments, got [0-9]
+  ${status}  ${value}=  Run Keyword If  "${unexpected_args}"=="[]"  Set Variable  ${status}  ${value}
+  ...      ELSE  Run keyword and ignore keyword definitions  ${keywords_file}.${command}  ${username}  @{arguments[:-1]}
+  Run Keyword If  '${status}' == 'FAIL'  Fail  ${value}
+  [return]  ${value}
 Require Failure
--- a/op_robot_tests/tests_files/multiLotTender.robot
View file @9ef3350
+++ b/op_robot_tests/tests_files/multiLotTender.robot
View file @9ef3350
@@ -15,7 +15,6 @@ ${mode}         multi
 ${role}         viewer
 ${broker}       Quinta
-${question_id}  1
 ${complaint_id}  1
 *** Test Cases ***
@@ -177,8 +176,12 @@ ${complaint_id}  1
   ${question_resp}=  Викликати для учасника   ${provider}   Задати питання  ${TENDER['TENDER_UAID']}   ${question}
   ${now}=  Get Current TZdate
   ${question.data.date}=  Set variable  ${now}
-  ${question_data}=  Create Dictionary  question=${question}  question_resp=${question_resp}
+  ${question_id}=  get_id_from_field  ${question.data.description}
+  ${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['${provider}']}  question_data  ${question_data}
+
+
 Можливість відповісти на запитання
   [Tags]   ${USERS.users['${tender_owner}'].broker}: Можливість відповісти на запитання
   ...      tender_owner
@@ -186,7 +189,10 @@ ${complaint_id}  1
   [Setup]  Дочекатись синхронізації з майданчиком    ${tender_owner}
   [Teardown]  Оновити LAST_MODIFICATION_DATE
   ${answer}=  Підготовка даних для відповіді на запитання
-  ${answer_resp}=  Викликати для учасника   ${tender_owner}   Відповісти на питання    ${TENDER['TENDER_UAID']}  ${USERS.users['${provider}']['question_data']['question_resp']}  ${answer}
+  ${answer_resp}=  Викликати для учасника  ${tender_owner}
+  ...      Відповісти на питання  ${TENDER['TENDER_UAID']}
+  ...      ${USERS.users['${provider}']['question_data']['question_resp']}  ${answer}
+  ...      question_id=${USERS.users['${provider}'].question_data.question_id}
   ${now}=  Get Current TZdate
   ${answer.data.date}=  Set variable  ${now}
   ${answer_data}=  Create Dictionary  answer=${answer}  answer_resp=${answer_resp}
--- a/op_robot_tests/tests_files/service_keywords.py
View file @9ef3350
+++ b/op_robot_tests/tests_files/service_keywords.py
View file @9ef3350
@@ -8,7 +8,7 @@ from json import load
 from jsonpath_rw import parse as parse_path
 from munch import fromYAML, Munch, munchify
 from restkit import request
-from robot.errors import HandlerExecutionFailed
+from robot.errors import ExecutionFailed
 from robot.libraries.BuiltIn import BuiltIn
 from robot.output import LOGGER
 from robot.output.loggerhelper import Message
@@ -202,9 +202,8 @@ def run_keyword_and_ignore_keyword_definitions(name, *args, **kwargs):
     """
     try:
         status, _ = BuiltIn().run_keyword_and_ignore_error(name, *args, **kwargs)
-    except HandlerExecutionFailed:
-        LOGGER.log_message(Message("Keyword is not implemented: {}".format(name), "ERROR"))
-        status, _ = "FAIL", None
+    except ExecutionFailed as e:
+        status, _ = "FAIL", e.message
     return status, _
@@ -339,6 +338,24 @@ def munch_dict(arg=None, data=False):
     return munchify(arg)
+def get_id_from_field(field):
+    return re.match(r'(^[filq]-[0-9a-fA-F]{8}): ', field).group(1)
+
+
+def get_object_type_by_id(object_id):
+    prefixes = {'q': 'questions', 'f': 'features', 'i': 'items', 'l': 'lots'}
+    return prefixes.get(object_id[0])
+
+
+def get_object_index_by_id(data, object_id):
+    for index, element in enumerate(data):
+        element_id = get_id_from_field(element['description'])
+        if element_id == object_id:
+            break
+    else:
+        index = 0
+    return index
+
 # GUI Frontends common
 def add_data_for_gui_frontends(tender_data):
     now = get_now()
--- a/op_robot_tests/tests_files/singleItemTender.robot
View file @9ef3350
+++ b/op_robot_tests/tests_files/singleItemTender.robot
View file @9ef3350
@@ -15,8 +15,6 @@ ${mode}         single
 ${role}         viewer
 ${broker}       Quinta
-${question_id}  0
-
 *** Test Cases ***
 Можливість оголосити однопредметний тендер
   [Tags]   ${USERS.users['${tender_owner}'].broker}: Можливість оголосити тендер
@@ -331,7 +329,8 @@ ${question_id}  0
   ${question_resp}=  Викликати для учасника  ${provider}  Задати питання  ${TENDER['TENDER_UAID']}  ${question}
   ${now}=  Get Current TZdate
   ${question.data.date}=  Set variable  ${now}
-  ${question_data}=  Create Dictionary  question=${question}  question_resp=${question_resp}
+  ${question_id}=  get_id_from_field  ${question.data.description}
+  ${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['${provider}']}  question_data=${question_data}
@@ -346,7 +345,9 @@ ${question_id}  0
   ...      level2
   [Setup]  Дочекатись синхронізації з майданчиком  ${viewer}
   Викликати для учасника  ${viewer}  Оновити сторінку з тендером  ${TENDER['TENDER_UAID']}
-  Звірити поле тендера із значенням  ${viewer}  ${USERS.users['${provider}'].question_data.question.data.title}  questions[${question_id}].title
+  Звірити поле тендера із значенням  ${viewer}
+  ...      ${USERS.users['${provider}'].question_data.question.data.title}  title
+  ...      object_id=${USERS.users['${provider}'].question_data.question_id}
 Відображення опису анонімного питання без відповіді
@@ -354,14 +355,18 @@ ${question_id}  0
   ...      viewer
   ...      ${USERS.users['${viewer}'].broker}
   ...      level2
-  Звірити поле тендера із значенням  ${viewer}  ${USERS.users['${provider}'].question_data.question.data.description}  questions[${question_id}].description
+  Звірити поле тендера із значенням  ${viewer}
+  ...      ${USERS.users['${provider}'].question_data.question.data.description}  description
+  ...      object_id=${USERS.users['${provider}'].question_data.question_id}
 Відображення дати анонімного питання без відповіді
   [Tags]   ${USERS.users['${viewer}'].broker}: Відображення анонімного питання без відповідей
   ...      viewer
   ...      ${USERS.users['${viewer}'].broker}
-  Звірити дату тендера із значенням  ${viewer}  ${USERS.users['${provider}'].question_data.question.data.date}  questions[${question_id}].date
+  Звірити дату тендера із значенням  ${viewer}
+  ...      ${USERS.users['${provider}'].question_data.question.data.date}  date
+  ...      object_id=${USERS.users['${provider}'].question_data.question_id}
 ##############################################################################################
 #             МОЖЛИВІСТЬ
@@ -389,7 +394,10 @@ ${question_id}  0
   [Setup]  Дочекатись синхронізації з майданчиком  ${tender_owner}
   [Teardown]  Оновити LAST_MODIFICATION_DATE
   ${answer}=  Підготовка даних для відповіді на запитання
-  ${answer_resp}=  Викликати для учасника  ${tender_owner}  Відповісти на питання  ${TENDER['TENDER_UAID']}  ${USERS.users['${provider}']['question_data']['question_resp']}  ${answer}
+  ${answer_resp}=  Викликати для учасника  ${tender_owner}
+  ...      Відповісти на питання  ${TENDER['TENDER_UAID']}
+  ...      ${USERS.users['${provider}']['question_data']['question_resp']}  ${answer}
+  ...      question_id=${USERS.users['${provider}'].question_data.question_id}
   ${now}=  Get Current TZdate
   ${answer.data.date}=  Set variable  ${now}
   ${answer_data}=  Create Dictionary  answer=${answer}  answer_resp=${answer_resp}
@@ -406,7 +414,9 @@ ${question_id}  0
   ...      level2
   [Setup]  Дочекатись синхронізації з майданчиком  ${viewer}
   Викликати для учасника  ${viewer}  Оновити сторінку з тендером  ${TENDER['TENDER_UAID']}
-  Звірити поле тендера із значенням  ${viewer}  ${USERS.users['${provider}']['answer_data']['answer'].data.answer}  questions[${question_id}].answer
+  Звірити поле тендера із значенням  ${viewer}
+  ...      ${USERS.users['${provider}']['answer_data']['answer'].data.answer}  answer
+  ...      object_id=${USERS.users['${provider}'].question_data.question_id}
 ##############################################################################################
 #             МОЖЛИВІСТЬ