Fix tests

holoviz · Jun 23, 2020 · fa287c6 · fa287c6
1 parent 350d3c4
commit fa287c6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/holoviews/plotting/bokeh/stats.py b/holoviews/plotting/bokeh/stats.py
@@ -514,7 +514,7 @@ def get_data(self, element, ranges, style):
 
         if split_dim:
             split_name = split_dim.dimension.name
-            if split_name in ranges and not split_dim.ops:
+            if split_name in ranges and not split_dim.ops and 'factors' in ranges[split_name]:
                 split_cats = ranges[split_name].get('factors')
             elif split_dim:
                 split_cats = list(unique_iterator(split_dim.apply(element)))

diff --git a/holoviews/tests/plotting/bokeh/testviolinplot.py b/holoviews/tests/plotting/bokeh/testviolinplot.py
@@ -167,7 +167,8 @@ def test_violin_split_op_multi(self):
         source = plot.handles['patches_1_source']
         glyph = plot.handles['patches_1_glyph']
         cmapper = plot.handles['violin_color_mapper']
-        self.assertEqual(source.data["dim('b')>2"], ['False', 'False', 'False', 'True', 'True'])
+        values = ['False', 'True', 'False', 'True', 'False', 'True', 'False', 'True', 'False', 'True']
+        self.assertEqual(source.data["dim('b')>2"], values)
         self.assertEqual(glyph.fill_color, {'field': "dim('b')>2", 'transform': cmapper})
 
     def test_violin_split_op_single(self):