Removed empty optimize test//added check for x, y, z_edges all same length

Kully · Kully · commit d13420f7ce07 · 2016-05-25T12:17:54.000-04:00
diff --git a/plotly/tests/test_optional/test_figure_factory.py b/plotly/tests/test_optional/test_figure_factory.py
@@ -753,17 +753,6 @@ def test_trisurf_all_args(self):
         self.assert_dict_equal(test_trisurf_plot['data'][1],
                                exp_trisurf_plot['data'][1])
 
-    def test_optimize_trisurf(self):
-
-      # check that trisurf will give warnings if the plotting will take
-      # a long time
-
-      pass
-
-
-
-
-
 
 class TestScatterPlotMatrix(NumpyTestUtilsMixin, TestCase):
 
diff --git a/plotly/tools.py b/plotly/tools.py
@@ -1589,6 +1589,10 @@ def _trisurf(x, y, z, simplices, colormap=None, dist_func=None,
         y_edge = np.hstack([y_edge, y_edge_pull.reshape([1, -1])[0]])
         z_edge = np.hstack([z_edge, z_edge_pull.reshape([1, -1])[0]])
 
+        if not (len(x_edge) == len(y_edge) == len(z_edge)):
+            raise exceptions.PlotlyError("The lengths of x_edge, y_edge and "
+                                         "z_edge are not the same.")
+
         # define the lines for plotting
         lines = graph_objs.Scatter3d(
             x=x_edge, y=y_edge, z=z_edge, mode='lines',
