From ed2e87f4ca2fac07879172eef2be08bfdd6e58e5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Juan=20Ignacio=20S=C3=A1nchez=20Lara?= Date: Tue, 14 Aug 2018 09:27:59 +0200 Subject: [PATCH 1/2] Fix test fixture --- .../test/test_mapboxrouting.py | 21 ++++++++++--------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/server/lib/python/cartodb_services/test/test_mapboxrouting.py b/server/lib/python/cartodb_services/test/test_mapboxrouting.py index 2101535..160b2a5 100644 --- a/server/lib/python/cartodb_services/test/test_mapboxrouting.py +++ b/server/lib/python/cartodb_services/test/test_mapboxrouting.py @@ -17,16 +17,17 @@ VALID_PROFILE = DEFAULT_PROFILE INVALID_PROFILE = 'invalid_profile' WELL_KNOWN_SHAPE = [(40.73312, -73.98891), (40.73353, -73.98987), - (40.73398, -73.99095), (40.73321, -73.99111), - (40.73245, -73.99129), (40.7333, -73.99332), - (40.7338, -73.99449), (40.73403, -73.99505), - (40.73344, -73.99549), (40.73286, -73.9959), - (40.73226, -73.99635), (40.73186, -73.99664), - (40.73147, -73.99693), (40.73141, -73.99698), - (40.73147, -73.99707), (40.73219, -73.99856), - (40.73222, -73.99861), (40.73225, -73.99868), - (40.73293, -74.00007), (40.733, -74.00001)] -WELL_KNOWN_LENGTH = 1384.8 + (40.73398, -73.99095), (40.73453, -73.99227), + (40.73531, -73.99412), (40.73467, -73.99459), + (40.73442, -73.99477), (40.73435, -73.99482), + (40.73403, -73.99505), (40.73344, -73.99549), + (40.73286, -73.9959), (40.73226, -73.99635), + (40.73186, -73.99664), (40.73147, -73.99693), + (40.73141, -73.99698), (40.73147, -73.99707), + (40.73219, -73.99856), (40.73222, -73.99861), + (40.73225, -73.99868), (40.73293, -74.00007), + (40.733, -74.00001)] +WELL_KNOWN_LENGTH = 1317.9 class MapboxRoutingTestCase(unittest.TestCase): From d9f647504a014b5b320eaa9a6d52afa852cb1cb1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Juan=20Ignacio=20S=C3=A1nchez=20Lara?= Date: Tue, 14 Aug 2018 09:39:42 +0200 Subject: [PATCH 2/2] Disable Mapbox Matrix API usage at tests --- .../python/cartodb_services/test/metrics/test_config.py | 7 ++++--- .../lib/python/cartodb_services/test/test_mapboxisoline.py | 4 +--- .../lib/python/cartodb_services/test/test_mapboxmatrix.py | 1 + 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/server/lib/python/cartodb_services/test/metrics/test_config.py b/server/lib/python/cartodb_services/test/metrics/test_config.py index 9ab9f34..4a14a8a 100644 --- a/server/lib/python/cartodb_services/test/metrics/test_config.py +++ b/server/lib/python/cartodb_services/test/metrics/test_config.py @@ -173,7 +173,8 @@ class TestGeocoderOrgConfig(TestCase): class TestIsolinesUserConfig(TestCase): - ISOLINES_PROVIDERS = ['heremaps', 'mapzen', 'mapbox', 'tomtom'] + # Don't test mapbox. See CartoDB/cartodb-management/issues/5199" + ISOLINES_PROVIDERS = ['heremaps', 'mapzen', 'tomtom'] def setUp(self): self.redis_conn = MockRedis() @@ -233,8 +234,8 @@ class TestIsolinesUserConfig(TestCase): class TestIsolinesOrgConfig(TestCase): - - ISOLINES_PROVIDERS = ['heremaps', 'mapzen', 'mapbox', 'tomtom'] + # Don't test mapbox. See CartoDB/cartodb-management/issues/5199" + ISOLINES_PROVIDERS = ['heremaps', 'mapzen', 'tomtom'] def setUp(self): self.redis_conn = MockRedis() diff --git a/server/lib/python/cartodb_services/test/test_mapboxisoline.py b/server/lib/python/cartodb_services/test/test_mapboxisoline.py index ce80d0e..091ac7c 100644 --- a/server/lib/python/cartodb_services/test/test_mapboxisoline.py +++ b/server/lib/python/cartodb_services/test/test_mapboxisoline.py @@ -3,15 +3,13 @@ from mock import Mock from cartodb_services.mapbox.isolines import MapboxIsolines from cartodb_services.mapbox.matrix_client import DEFAULT_PROFILE from cartodb_services.mapbox.matrix_client import MapboxMatrixClient -from cartodb_services.mapbox.routing import MapboxRouting from cartodb_services.tools import Coordinate -from cartodb_services.tools.coordinates import (validate_coordinates, - marshall_coordinates) from credentials import mapbox_api_key VALID_ORIGIN = Coordinate(-73.989, 40.733) +@unittest.skip("Stop using Matrix API. CartoDB/cartodb-management/issues/5199") class MapboxIsolinesTestCase(unittest.TestCase): def setUp(self): diff --git a/server/lib/python/cartodb_services/test/test_mapboxmatrix.py b/server/lib/python/cartodb_services/test/test_mapboxmatrix.py index 579810a..b1ce42e 100644 --- a/server/lib/python/cartodb_services/test/test_mapboxmatrix.py +++ b/server/lib/python/cartodb_services/test/test_mapboxmatrix.py @@ -20,6 +20,7 @@ VALID_PROFILE = DEFAULT_PROFILE INVALID_PROFILE = 'invalid_profile' +@unittest.skip("Stop using Matrix API. CartoDB/cartodb-management/issues/5199") class MapboxMatrixTestCase(unittest.TestCase): def setUp(self): self.matrix_client = MapboxMatrixClient(token=mapbox_api_key(),