Mock out load_system_paasta_config so tests pass on travis

Yelp · Nov 23, 2020 · 8d6969d · 8d6969d
1 parent 4c52e2f
commit 8d6969d
Showing 1 changed file with 20 additions and 6 deletions.
diff --git a/tests/test_kubernetes_tools.py b/tests/test_kubernetes_tools.py
@@ -1614,9 +1614,16 @@ def test_get_autoscaling_metric_spec_http(self):
             config_dict=config_dict,
             branch_dict=None,
         )
-        return_value = KubernetesDeploymentConfig.get_autoscaling_metric_spec(
-            mock_config, "fake_name", "cluster"
-        )
+        with mock.patch(
+            "paasta_tools.kubernetes_tools.load_system_paasta_config",
+            return_value=mock.Mock(
+                get_hpa_always_uses_external_for_signalfx=lambda: False
+            ),
+            autospec=True,
+        ):
+            return_value = KubernetesDeploymentConfig.get_autoscaling_metric_spec(
+                mock_config, "fake_name", "cluster"
+            )
         annotations = {"signalfx.com.custom.metrics": ""}
         expected_res = V2beta2HorizontalPodAutoscaler(
             kind="HorizontalPodAutoscaler",
@@ -1664,9 +1671,16 @@ def test_get_autoscaling_metric_spec_uwsgi(self):
             config_dict=config_dict,
             branch_dict=None,
         )
-        return_value = KubernetesDeploymentConfig.get_autoscaling_metric_spec(
-            mock_config, "fake_name", "cluster"
-        )
+        with mock.patch(
+            "paasta_tools.kubernetes_tools.load_system_paasta_config",
+            return_value=mock.Mock(
+                get_hpa_always_uses_external_for_signalfx=lambda: False
+            ),
+            autospec=True,
+        ):
+            return_value = KubernetesDeploymentConfig.get_autoscaling_metric_spec(
+                mock_config, "fake_name", "cluster"
+            )
 
         annotations = {"signalfx.com.custom.metrics": ""}
         expected_res = V2beta2HorizontalPodAutoscaler(