--- a/seqtk_seq.xml	Sat Dec 09 11:14:21 2023 +0000
+++ b/seqtk_seq.xml	Wed Oct 16 09:08:52 2024 +0000
@@ -1,5 +1,5 @@
 <?xml version="1.0"?>
-<tool id="seqtk_seq" name="seqtk_seq" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" profile="22.05">
+<tool id="seqtk_seq" name="seqtk_seq" version="@TOOL_VERSION@+galaxy1" profile="22.05">
     <description>common transformation of FASTA/Q</description>
     <macros>
         <import>macros.xml</import>
@@ -34,13 +34,25 @@
 '$in_file'
 @CONDITIONAL_GZIP_OUT@
     ]]></command>
+    <configfiles>
+        <configfile filename="outputs.json">
+#if $A and $in_file.is_of_type('fasta.gz', 'fastq.gz')
+    #set $ext = "fasta.gz"
+#elif $A
+    #set $ext = "fasta"
+#else
+    #set $ext = $in_file.ext
+#end if
+{"default": {"ext": "$ext"}}
+        </configfile>
+    </configfiles>
     <inputs>
         <expand macro="in_faq"/>
         <param argument="-q" type="integer" value="0" label="Mask bases with quality lower than INT" />
         <param argument="-X" type="integer" value="255" label="Mask bases with quality higher than INT" />
-        <param argument="-n" type="text" value="0" label="Masked bases converted to CHAR; 0 for lowercase" />
+        <param argument="-n" type="text" value="" label="Masked bases converted to CHAR; leave empty for lowercase masking" />
         <param argument="-l" type="integer" value="0" label="Number of residues per line; 0 for 2^32-1" />
-        <param argument="-Q" type="integer" value="33" label="Quality shift: ASCII-INT gives base quality" />
+        <param argument="-Q" type="integer" value="33" label="Quality shift: ASCII-INT gives base quality" help="Only applied during comparison to quality thresholds for masking" />
         <param argument="-s" type="integer" value="11" label="Random seed" help="Effective with -f" />
         <param argument="-f" type="float" value="1" label="Sample fraction of sequences" />
         <param argument="-M" type="data" format="bed,txt" optional="true" label="Mask regions in BED or name list file" />
@@ -53,26 +65,37 @@
         <param name="x1" argument="-1" type="boolean" truevalue="-1" falsevalue="" checked="false" label="Output the 2n-1 reads only" />
         <param name="x2" argument="-2" type="boolean" truevalue="-2" falsevalue="" checked="false" label="Output the 2n reads only" />
     </inputs>
-    <outputs>
-        <data name="default" format_source="in_file" label="${tool.name} on ${on_string}" />
+    <outputs provided_metadata_file="outputs.json">
+        <data name="default" format="auto" label="${tool.name} on ${on_string}" />
     </outputs>
+
     <tests>
         <!-- This is a sorry excuse for a test for a tool which does way more
              than it should, but upstream decided to put a TON of functionality
              into a single tool rather than using the single responsibility
              principle. -->
-        <test>
+        <test expect_num_outputs="1">
             <param name="in_file" value="seqtk_seq.fa"/>
             <param name="r" value="True"/>
             <param name="n" value=""/>
             <output name="default" file="seqtk_seq_revcom.fa" ftype="fasta"/>
         </test>
-        <test>
+        <test expect_num_outputs="1">
             <param name="in_file" value="seqtk_seq.fa.gz" ftype="fasta.gz"/>
             <param name="r" value="True"/>
             <param name="n" value=""/>
             <output name="default" file="seqtk_seq_revcom.fa.gz" ftype="fasta.gz"/>
         </test>
+        <test expect_num_outputs="1">
+            <param name="in_file" value="seqtk_trimfq.fq" ftype="fastq"/>
+            <param name="A" value="True" />
+            <output name="default" file="seqtk_seq_A.fasta" ftype="fasta"/>
+        </test>
+        <test expect_num_outputs="1">
+            <param name="in_file" value="seqtk_trimfq.fq.gz" ftype="fastq.gz"/>
+            <param name="A" value="True" />
+            <output name="default" file="seqtk_seq_A.fasta.gz" ftype="fasta.gz"/>
+        </test>
     </tests>
     <help><![CDATA[
 **What it does**