Fix param name error in batch_update()/batch_animate()

Author: Jon M. Mease

plotly/basedatatypes.py

@@ -1658,8 +1658,8 @@ def plotly_update(self,
         # Send a plotly_update message to the frontend (if any)
         if restyle_changes or relayout_changes:
             self._send_update_msg(
-                style=restyle_changes,
-                layout=relayout_changes,
+                restyle_data=restyle_changes,
+                relayout_data=relayout_changes,
                 trace_indexes=trace_indexes,
                 **msg_kwargs)
 
@@ -1725,13 +1725,17 @@ def _send_relayout_msg(self, layout, source_view_id=None):
         pass
 
     def _send_update_msg(self,
-                         style,
-                         layout,
+                         restyle_data,
+                         relayout_data,
                          trace_indexes=None,
                          source_view_id=None):
         pass
 
-    def _send_animate_msg(self, styles, layout, trace_indexes, animation_opts):
+    def _send_animate_msg(self,
+                          styles_data,
+                          relayout_data,
+                          trace_indexes,
+                          animation_opts):
         pass
 
     # Context managers
@@ -1986,8 +1990,8 @@ def _perform_batch_animate(self, animation_opts):
         # --------------------
         # Sends animate message to the front end (if any)
         self._send_animate_msg(
-            styles=list(animate_styles),
-            layout=animate_layout,
+            styles_data=list(animate_styles),
+            relayout_data=animate_layout,
             trace_indexes=list(animate_trace_indexes),
             animation_opts=animation_opts)
 

plotly/tests/test_core/test_figure_messages/test_batch_animate.py

@@ -47,8 +47,8 @@ def test_batch_animate(self):
 
         # Check that update message was sent
         self.figure._send_animate_msg.assert_called_once_with(
-            styles=[{'marker.color': 'yellow'}, {'marker.opacity': 0.9}],
-            layout={'xaxis.range': [10, 20]},
+            styles_data=[{'marker.color': 'yellow'}, {'marker.opacity': 0.9}],
+            relayout_data={'xaxis.range': [10, 20]},
             trace_indexes=[0, 1],
             animation_opts={'transition': {'easing': 'elastic',
                                            'duration': 1200},

plotly/tests/test_core/test_figure_messages/test_plotly_update.py

@@ -50,9 +50,9 @@ def test_batch_update(self):
 
         # Check that update message was sent
         self.figure._send_update_msg.assert_called_once_with(
-            style={'marker.color': ['yellow', Undefined],
+            restyle_data={'marker.color': ['yellow', Undefined],
                    'marker.opacity': [Undefined, 0.9]},
-            layout={'xaxis.range': [10, 20]},
+            relayout_data={'xaxis.range': [10, 20]},
             trace_indexes=[0, 1])
 
     def test_plotly_update(self):
@@ -70,7 +70,7 @@ def test_plotly_update(self):
 
         # Check that update message was sent
         self.figure._send_update_msg.assert_called_once_with(
-            style={'marker.color': ['yellow', Undefined],
+            restyle_data={'marker.color': ['yellow', Undefined],
                    'marker.opacity': [Undefined, 0.9]},
-            layout={'xaxis.range': [10, 20]},
+            relayout_data={'xaxis.range': [10, 20]},
             trace_indexes=[0, 1])

plotly/tests/test_core/test_graph_objs/test_figure_properties.py

@@ -138,7 +138,7 @@ def test_update_frames(self):
                             [{'layout': {'yaxis': {'title': 'f4'}}}]})
         self.assertEqual(self.figure.frames[0].layout.yaxis.title, 'f4')
 
-    @raises(AttributeError)
+    @raises(ValueError)
     def test_update_invalid_attr(self):
         self.figure.layout.update({'xaxis': {'bogus': 32}})