--- a/plot.xml	Wed May 20 16:08:18 2020 -0400
+++ b/plot.xml	Tue Mar 16 13:01:04 2021 +0000
@@ -45,7 +45,7 @@
     projection='$method.plot.projection',
     legend_loc='$method.plot.legend_loc',
     #if $method.plot.palette != 'default'
-    palette='$method.plot.palette',
+    palette=['$method.plot.palette'],
     #end if
     #if $method.plot.color_map != 'None'
     color_map='$method.plot.color_map',
@@ -239,9 +239,7 @@
     gene_symbols='$method.gene_symbols',
     #end if
     use_raw=$method.use_raw,
-    #if str($method.layout) != 'None'
     layout='$method.layout',
-    #end if
     edges=$method.edges.edges,
     #if str($method.edges.edges) == 'True'
     edges_width=$method.edges.edges_width,
@@ -404,7 +402,9 @@
                 <option value="pl.paga_compare">Branching trajectories and pseudotime, clustering: Scatter and PAGA graph side-by-side, using 'pl.paga_compare'</option>
                 <option value="pl.paga_path">Branching trajectories and pseudotime, clustering: Gene expression and annotation changes along paths, using 'pl.paga_path'</option>
                 <option value="pl.rank_genes_groups">Marker genes: Plot ranking of genes using dotplot plot, using 'pl.rank_genes_groups'</option>
+                <!-- doesn't currenty work: https://github.com/galaxyproject/tools-iuc/pull/3485#issuecomment-800055967
                 <option value="pl.rank_genes_groups_violin">Marker genes: Plot ranking of genes as violin plot, using 'pl.rank_genes_groups_violin'</option>
+                -->
                 <option value="pl.rank_genes_groups_dotplot">Marker genes: Plot ranking of genes as dotplot plot, using 'pl.rank_genes_groups_dotplot'</option>
                 <option value="pl.rank_genes_groups_heatmap">Marker genes: Plot ranking of genes as heatmap plot, using 'pl.rank_genes_groups_heatmap'</option>
                 <option value="pl.rank_genes_groups_matrixplot">Marker genes: Plot ranking of genes as matrixplot plot, using 'pl.rank_genes_groups_matrixplot'</option>
@@ -639,7 +639,7 @@
                 <expand macro="param_color"/>
                 <expand macro="gene_symbols"/>
                 <expand macro="param_use_raw"/>
-                <param argument="layout" type="select" optional="true" label="Plotting layout" help="">
+                <param argument="layout" type="select" optional="false" label="Plotting layout" help="">
                     <expand macro="options_layout"/>
                 </param>
                 <expand macro="pl_edges"/>
@@ -814,7 +814,6 @@
                     <has_text_matching expression="legend_fontsize=1"/>
                     <has_text_matching expression="legend_fontweight='normal'"/>
                     <has_text_matching expression="color_map='inferno'"/>
-                    <has_text_matching expression="palette='inferno'"/>
                     <has_text_matching expression="frameon=True"/>
                     <has_text_matching expression="size=1.0"/>
                 </assert_contents>
@@ -862,7 +861,6 @@
                     <has_text_matching expression="legend_loc='right margin'"/>
                     <has_text_matching expression="legend_fontsize=1"/>
                     <has_text_matching expression="legend_fontweight='normal'"/>
-                    <has_text_matching expression="palette='bwr'"/>
                     <has_text_matching expression="frameon=False"/>
                     <has_text_matching expression="size=1.0"/>
                     <has_text_matching expression="title='A title'"/>
@@ -1029,7 +1027,6 @@
                     <has_text_matching expression="scale='width'"/>
                     <has_text_matching expression="linewidth=0.0"/>
                     <has_text_matching expression="color='AliceBlue'"/>
-                    <has_text_matching expression="palette='viridis'"/>
                     <has_text_matching expression="saturation=0.75"/>
                 </assert_contents>
             </output>
@@ -1067,7 +1064,6 @@
                     <param name="bw" value="scott"/>
                     <param name="linewidth" value="0"/>
                     <param name="color" value="AliceBlue"/>
-                    <param name="palette" value="viridis"/>
                     <param name="saturation" value="0.75"/>
                 </section>
             </conditional>
@@ -1091,7 +1087,6 @@
                     <has_text_matching expression="scale='width'"/>
                     <has_text_matching expression="linewidth=0.0"/>
                     <has_text_matching expression="color='AliceBlue'"/>
-                    <has_text_matching expression="palette='viridis'"/>
                     <has_text_matching expression="saturation=0.75"/>
                 </assert_contents>
             </output>
@@ -1539,6 +1534,7 @@
             <conditional name="method">
                 <param name="method" value="pl.draw_graph"/>
                 <param name="use_raw" value="false"/>
+                <param name="layout" value="fr"/>
                 <conditional name="edges">
                     <param name="edges" value="True"/>
                     <param name="edges_width" value="0.1"/>
@@ -1679,8 +1675,8 @@
             </output>
             <output name="out_png" file="pl.rank_genes_groups.rank_genes_groups.krumsiek11.png" ftype="png" compare="sim_size"/>
         </test>
+        <!-- test 20: pl.rank_genes_groups_violin, currently broken
         <test>
-            <!-- test 20: pl.rank_genes_groups_violin !-->
             <param name="adata" value="tl.rank_genes_groups.krumsiek11.h5ad" />
             <param name="format" value="png"/>
             <conditional name="method">
@@ -1723,6 +1719,7 @@
                 <element name="cell_type_progenitor" file="pl.rank_genes_groups_violin.progenitor.png" ftype="png" compare="sim_size"/>
             </output_collection>
         </test>
+        -->
         <test>
             <!-- test 21: pl.rank_genes_groups_dotplot !-->
             <param name="adata" value="tl.rank_genes_groups.krumsiek11.h5ad" />
@@ -1875,7 +1872,6 @@
                     <has_text_matching expression="scale='width'"/>
                     <has_text_matching expression="linewidth=0.0"/>
                     <has_text_matching expression="color='AliceBlue'"/>
-                    <has_text_matching expression="palette='viridis'"/>
                     <has_text_matching expression="saturation=0.75"/>
                 </assert_contents>
             </output>