Compare coordinates

Leits
1 parent 366e53be
Showing 9 changed files with 99 additions and 71 deletions
op_robot_tests/tests_files/keywords.robot
op_robot_tests/tests_files/limited_keywords.robot
op_robot_tests/tests_files/multiItemTender.robot
op_robot_tests/tests_files/negotiation.quick.robot
op_robot_tests/tests_files/negotiation.robot
op_robot_tests/tests_files/reporting.robot
op_robot_tests/tests_files/service_keywords.py
op_robot_tests/tests_files/singleItemTender.robot
setup.py
--- a/op_robot_tests/tests_files/keywords.robot
View file @3277487
+++ b/op_robot_tests/tests_files/keywords.robot
View file @3277487
@@ -405,7 +405,33 @@ Get Broker Property By Username
   Should Not Be Equal  ${left}  ${None}
   Should Not Be Equal  ${right}  ${None}
   ${status}=  compare_date  ${left}  ${right}  accuracy=${accuracy}  absolute_delta=${absolute_delta}
-  Should Be True  ${status}  msg=Dates are not equal: ${left} != ${right}
+  Should Be True  ${status}  msg=Dates differ: ${left} != ${right}
+
+
+Звірити координати доставки тендера
+  [Arguments]  ${username}  ${tender_data}  ${field}  ${object_id}=${None}
+  ${left_lat}=  Get_From_Object  ${tender_data.data}  ${field}.deliveryLocation.latitude
+  ${left_lon}=  Get_From_Object  ${tender_data.data}  ${field}.deliveryLocation.longitude
+  ${right_lat}=  Отримати дані із тендера  ${username}  ${field}.deliveryLocation.latitude  ${object_id}
+  ${right_lon}=  Отримати дані із тендера  ${username}  ${field}.deliveryLocation.longitude  ${object_id}
+  Порівняти координати  ${left_lat}  ${left_lon}  ${right_lat}  ${right_lon}
+
+
+Порівняти координати
+  [Documentation]
+  ...      Compare coordinates with specified ``accuracy`` (in km).
+  ...      Default is `0.1`.
+  ...
+  ...      The keyword will fail if the difference between
+  ...      ``left`` and ``right`` is more than ``accuracy``,
+  ...      otherwise it will pass.
+  [Arguments]  ${left_lat}  ${left_lon}  ${right_lat}  ${right_lon}  ${accuracy}=0.1
+  Should Not Be Equal  ${left_lat}  ${None}
+  Should Not Be Equal  ${left_lon}  ${None}
+  Should Not Be Equal  ${right_lat}  ${None}
+  Should Not Be Equal  ${right_lon}  ${None}
+  ${status}=  compare_coordinates  ${left_lat}  ${left_lon}  ${right_lat}  ${right_lon}  ${accuracy}
+  Should Be True  ${status}  msg=Coordinates differ: (${left_lat}, ${left_lon}) != (${right_lat}, ${right_lon})
 Звірити поля предметів закупівлі багатопредметного тендера
@@ -420,12 +446,19 @@ Get Broker Property By Username
 Звірити дату предметів закупівлі багатопредметного тендера
   [Arguments]  ${username}  ${tender_data}  ${field}  ${accuracy}=60  ${absolute_delta}=${False}
   @{items}=  Get_From_Object  ${tender_data.data}  items
-  ${len_of_items}=  Get Length  ${items}
-  :FOR  ${index}  IN RANGE  ${len_of_items}
+  :FOR  ${index}  ${_}  IN ENUMERATE  @{items}
   \  Log  ${index}
   \  Звірити дату тендера  ${viewer}  ${tender_data}  items[${index}].${field}  accuracy=${accuracy}  absolute_delta=${absolute_delta}
+Звірити координати доставки предметів закупівлі багатопредметного тендера
+  [Arguments]  ${username}  ${tender_data}
+  @{items}=  Get_From_Object  ${tender_data.data}  items
+  :FOR  ${index}  ${_}  IN ENUMERATE  @{items}
+  \  Log  ${index}
+  \  Звірити координати тендера  ${viewer}  ${tender_data}  items[${index}]
+
+
 Отримати дані із тендера
   [Arguments]  ${username}  ${field_name}  ${object_id}=${None}
   Log  ${username}
--- a/op_robot_tests/tests_files/limited_keywords.robot
View file @3277487
+++ b/op_robot_tests/tests_files/limited_keywords.robot
View file @3277487
@@ -314,16 +314,10 @@ Resource           resource.robot
   ...      absolute_delta=${True}
-Відображення координат широти доставки номенклатури закупівлі
-  Звірити поле тендера  ${viewer}
-  ...      ${USERS.users['${tender_owner}'].initial_data}
-  ...      items[${ITEMS_NUM}].deliveryLocation.latitude
-
-
-Відображення координат довготи доставки номенклатури закупівлі
-  Звірити поле тендера  ${viewer}
+Відображення координат доставки номенклатури закупівлі
+  Звірити координати доставки тендера  ${viewer}
   ...      ${USERS.users['${tender_owner}'].initial_data}
-  ...      items[${ITEMS_NUM}].deliveryLocation.longitude
+  ...      items[${ITEMS_NUM}]
 Відображення назви нас. пункту доставки номенклатури закупівлі
--- a/op_robot_tests/tests_files/multiItemTender.robot
View file @3277487
+++ b/op_robot_tests/tests_files/multiItemTender.robot
View file @3277487
@@ -52,17 +52,11 @@ ${mode}         multiItem
   ...      level3
   Звірити дату предметів закупівлі багатопредметного тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  deliveryDate.endDate  day  absolute_delta=${True}
-Відображення координат широти доставки позицій закупівлі багатопредметного тендера
+Відображення координат доставки позицій закупівлі багатопредметного тендера
   [Tags]   ${USERS.users['${viewer}'].broker}: Відображення полів предметів багатопредметного тендера
   ...      viewer
   ...      ${USERS.users['${viewer}'].broker}
-  Звірити поля предметів закупівлі багатопредметного тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  deliveryLocation.latitude
-
-Відображення координат довготи доставки позицій закупівлі багатопредметного тендера
-  [Tags]   ${USERS.users['${viewer}'].broker}: Відображення полів предметів багатопредметного тендера
-  ...      viewer
-  ...      ${USERS.users['${viewer}'].broker}
-  Звірити поля предметів закупівлі багатопредметного тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  deliveryLocation.longitude
+  Звірити координати доставки предметів закупівлі багатопредметного тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}
 Відображення назви нас. пункту доставки позицій закупівлі багатопредметного тендера
   [Tags]   ${USERS.users['${viewer}'].broker}: Відображення полів предметів багатопредметного тендера
--- a/op_robot_tests/tests_files/negotiation.quick.robot
View file @3277487
+++ b/op_robot_tests/tests_files/negotiation.quick.robot
View file @3277487
@@ -357,18 +357,11 @@ ${mode}         negotiation.quick
   Відображення дати доставки номенклатури закупівлі
-Відображення координат широти доставки номенклатури переговорної процедури за нагальною потребою
+Відображення координат доставки номенклатури переговорної процедури за нагальною потребою
   [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
   ...  viewer
   ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат широти доставки номенклатури закупівлі
-
-
-Відображення координат довготи доставки номенклатури переговорної процедури за нагальною потребою
-  [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
-  ...  viewer
-  ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат довготи доставки номенклатури закупівлі
+  Відображення координат доставки номенклатури закупівлі
 Відображення назви нас. пункту доставки номенклатури переговорної процедури за нагальною потребою
--- a/op_robot_tests/tests_files/negotiation.robot
View file @3277487
+++ b/op_robot_tests/tests_files/negotiation.robot
View file @3277487
@@ -357,18 +357,11 @@ ${mode}         negotiation
   Відображення дати доставки номенклатури закупівлі
-Відображення координат широти доставки номенклатури переговорної процедури
+Відображення координат доставки номенклатури переговорної процедури
   [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
   ...  viewer
   ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат широти доставки номенклатури закупівлі
-
-
-Відображення координат довготи доставки номенклатури переговорної процедури
-  [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
-  ...  viewer
-  ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат довготи доставки номенклатури закупівлі
+  Відображення координат доставки номенклатури закупівлі
 Відображення назви нас. пункту доставки номенклатури переговорної процедури
--- a/op_robot_tests/tests_files/reporting.robot
View file @3277487
+++ b/op_robot_tests/tests_files/reporting.robot
View file @3277487
@@ -363,18 +363,11 @@ ${mode}         reporting
   Відображення дати доставки номенклатури закупівлі
-Відображення координат широти доставки номенклатури звіту про укладений договір
+Відображення координат доставки номенклатури звіту про укладений договір
   [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
   ...  viewer
   ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат широти доставки номенклатури закупівлі
-
-
-Відображення координат довготи доставки номенклатури звіту про укладений договір
-  [Tags]  ${USERS.users['${viewer}'].broker}: Відображення номенклатури процедури
-  ...  viewer
-  ...  ${USERS.users['${viewer}'].broker}
-  Відображення координат довготи доставки номенклатури закупівлі
+  Відображення координат доставки номенклатури закупівлі
 Відображення назви нас. пункту доставки номенклатури звіту про укладений договір
--- a/op_robot_tests/tests_files/service_keywords.py
View file @3277487
+++ b/op_robot_tests/tests_files/service_keywords.py
View file @3277487
@@ -12,6 +12,8 @@ from robot.errors import ExecutionFailed
 from robot.libraries.BuiltIn import BuiltIn
 from robot.output import LOGGER
 from robot.output.loggerhelper import Message
+from haversine import haversine
+from math import radians, cos, sin, asin, sqrt
 # These imports are not pointless. Robot's resource and testsuite files
 # can access them by simply importing library "service_keywords".
 # Please ignore the warning given by Flake8 or other linter.
@@ -54,6 +56,8 @@ import os
 from barbecue import chef
 import re
+NUM_TYPES = (int, long, float)
+
 def get_current_tzdate():
     return get_now().strftime('%Y-%m-%d %H:%M:%S.%f')
@@ -68,20 +72,20 @@ def get_file_contents(path):
         return unicode(f.read()) or u''
-def compare_date(date1, date2, accuracy="minute", absolute_delta=True):
+def compare_date(left, right, accuracy="minute", absolute_delta=True):
     '''Compares dates with specified accuracy
     Before comparison dates are parsed into datetime.datetime format
     and localized.
-    :param date1:           First date
-    :param date2:           Second date
+    :param left:            First date
+    :param right:           Second date
     :param accuracy:        Max difference between dates to consider them equal
                             Default value   - "minute"
                             Possible values - "day", "hour", "minute" or float value
                             of seconds
     :param absolute_delta:  Type of comparison. If set to True, then no matter which date order. If set to
-                            False then date2 must be lower then date1 for accuracy value.
+                            False then right must be lower then left for accuracy value.
                             Default value   - True
                             Possible values - True and False or something what can be casted into them
     :returns:               Boolean value
@@ -91,15 +95,15 @@ def compare_date(date1, date2, accuracy="minute", absolute_delta=True):
                             given and accuracy will be set to 60.
     '''
-    date1 = parse(date1)
-    date2 = parse(date2)
+    left = parse(left)
+    right = parse(right)
-    if date1.tzinfo is None:
-        date1 = TZ.localize(date1)
-    if date2.tzinfo is None:
-        date2 = TZ.localize(date2)
+    if left.tzinfo is None:
+        left = TZ.localize(left)
+    if right.tzinfo is None:
+        right = TZ.localize(right)
-    delta = (date1 - date2).total_seconds()
+    delta = (left - right).total_seconds()
     if accuracy == "day":
         accuracy = 24 * 60 * 60 - 1
@@ -120,6 +124,38 @@ def compare_date(date1, date2, accuracy="minute", absolute_delta=True):
     return True
+def compare_coordinates(left_lat, left_lon, right_lat, right_lon, accuracy=0.1):
+    '''Compares dates with specified accuracy
+
+    Before comparison dates are parsed into datetime.datetime format
+    and localized.
+
+    :param left_lat:        First coordinate latitude
+    :param left_lon:        First coordinate longitude
+    :param right_lat:       Second coordinate latitude
+    :param right_lon:       Second coordinate longitude
+    :param accuracy:        Max difference between coordinates to consider them equal
+                            Default value   - 0.1
+                            Possible values - float or integer value of kilometers
+
+    :returns:               Boolean value
+
+    :error:                 ValueError when there is problem with converting accuracy
+                            into float value. When it will be catched warning will be
+                            given and accuracy will be set to 0.1.
+
+    '''
+    for key, value in {'left_lat': left_lat, 'left_lon': left_lon, 'right_lat': right_lat, 'right_lon': right_lon}.iteritems():
+        if not isinstance(value, NUM_TYPES):
+            raise TypeError("Invalid type for coordinate '{0}'. "
+                            "Expected one of {1}, got {2}".format(
+                                key, str(NUM_TYPES), str(type(value))))
+    distance = haversine((left_lat, left_lon), (right_lat, right_lon))
+    if distance > accuracy:
+        return False
+    return True
+
+
 def log_object_data(data, file_name=None, format="yaml", update=False):
     """Log object data in pretty format (JSON or YAML)
@@ -201,8 +237,6 @@ def compute_intrs(brokers_data, used_brokers):
     does not contain ``Default`` entry.
     Using `load_data_from` with ``mode='brokers'`` is recommended.
     """
-    num_types = (int, long, float)
-
     def recur(l, r):
         l, r = deepcopy(l), deepcopy(r)
         if isinstance(l, list) and isinstance(r, list) and len(l) == len(r):
@@ -210,7 +244,7 @@ def compute_intrs(brokers_data, used_brokers):
             for ll, rr in zip(l, r):
                 lst.append(recur(ll, rr))
             return lst
-        elif isinstance(l, num_types) and isinstance(r, num_types):
+        elif isinstance(l, NUM_TYPES) and isinstance(r, NUM_TYPES):
             if l == r:
                 return l
             if l > r:
--- a/op_robot_tests/tests_files/singleItemTender.robot
View file @3277487
+++ b/op_robot_tests/tests_files/singleItemTender.robot
View file @3277487
@@ -169,18 +169,11 @@ ${mode}         single
   Звірити дату тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  items[0].deliveryDate.endDate  day  absolute_delta=${True}
-Відображення координат широти доставки номенклатури однопредметного тендера
+Відображення координати доставки позицій закупівлі однопредметного тендера
   [Tags]   ${USERS.users['${viewer}'].broker}: Відображення номенклатури тендера
   ...      viewer
   ...      ${USERS.users['${viewer}'].broker}
-  Звірити поле тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  items[0].deliveryLocation.latitude
-
-
-Відображення координат довготи доставки номенклатури однопредметного тендера
-  [Tags]   ${USERS.users['${viewer}'].broker}: Відображення номенклатури тендера
-  ...      viewer
-  ...      ${USERS.users['${viewer}'].broker}
-  Звірити поле тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  items[0].deliveryLocation.longitude
+  Звірити координати доставки тендера  ${viewer}  ${USERS.users['${tender_owner}'].initial_data}  items[0]
 Відображення назви нас. пункту доставки номенклатури однопредметного тендера
--- a/setup.py
View file @3277487
+++ b/setup.py
View file @3277487
@@ -33,7 +33,8 @@ setup(name='op_robot_tests',
           'pytz',
           'parse',
           'chromedriver',
-          'barbecue'
+          'barbecue',
+          'haversine'
       ],
       entry_points={
           'console_scripts': [