# HG changeset patch
# User iuc
# Date 1521534278 14400
# Node ID f44e93b4529c3f2a0be6085c85a8df82448f2b77
# Parent  988729b728f0bcd015dbb40d503aa37ce77cd7ac
planemo upload for repository https://github.com/galaxyproject/tools-iuc/tree/master/tools/fastp commit 6918a14442b7ebc56950917d668356feefaaaa28

diff -r 988729b728f0 -r f44e93b4529c fastp.xml
--- a/fastp.xml	Fri Mar 09 20:21:51 2018 -0500
+++ b/fastp.xml	Tue Mar 20 04:24:38 2018 -0400
@@ -8,7 +8,6 @@
     </requirements>
     <version_command>fastp --version | tail -n 1</version_command>
     <command detect_errors="exit_code"><![CDATA[
-
 ## Link input files
 
 #if $in1.is_of_type('fastq.gz')
@@ -24,10 +23,19 @@
 #end if
 
 
+## Set filename for output report
+
+#import re
+#set $filename = re.sub('[^\w\-\s]', '_', str($in1.element_identifier))
+
+
 ## Run fastp
 
 fastp
 
+--thread \${GALAXY_SLOTS:-1}
+--report_title 'fastp report for $filename'
+
 #if $in1.is_of_type('fastqillumina', 'fastqsolexa', 'fastqillumina.gz', 'fastqsolexa.gz'):
     --phred64
 #end if
@@ -40,7 +48,8 @@
     -O second.$ext
 #end if
 
-## Adapter trimming options
+
+## Adapter Trimming Options
 
 $single_paired.adapter_trimming_options.disable_adapter_trimming
 
@@ -55,7 +64,7 @@
 #end if
 
 
-## Global trimming options
+## Global Trimming Options
 
 #if str($single_paired.global_trimming_options.trim_front1):
     -f $single_paired.global_trimming_options.trim_front1
@@ -75,76 +84,6 @@
 #end if
 
 
-## PolyG tail trimming, useful for NextSeq/NovaSeq data
-
-#if $polyg_tail_trimming.trimming_select in ['', '-g']:
-    #if str($polyg_tail_trimming.poly_g_min_len):
-        --poly_g_min_len $polyg_tail_trimming.poly_g_min_len
-    #end if
-    $polyg_tail_trimming.trimming_select
-#end if
-
-## Per read cutting by quality options
-
-#if $cutting_by_quality_options.cut_by_quality5 or $cutting_by_quality_options.cut_by_quality3:
-
-    $cutting_by_quality_options.cut_by_quality5
-
-    $cutting_by_quality_options.cut_by_quality3
-
-    #if str($cutting_by_quality_options.cut_window_size):
-        -W $cutting_by_quality_options.cut_window_size
-    #end if
-    #if str($cutting_by_quality_options.cut_mean_quality):
-        -M $cutting_by_quality_options.cut_mean_quality
-    #end if
-#end if
-
-
-## Quality filtering options
-
-$quality_filtering_options.disable_quality_filtering
-
-#if str($quality_filtering_options.qualified_quality_phred):
-    -q $quality_filtering_options.qualified_quality_phred
-#end if
-#if str($quality_filtering_options.unqualified_percent_limit):
-    -u $quality_filtering_options.unqualified_percent_limit
-#end if
-#if str($quality_filtering_options.n_base_limit):
-    -n $quality_filtering_options.n_base_limit
-#end if
-
-
-## Length filtering options
-
-$length_filtering_options.disable_length_filtering
-
-#if str($length_filtering_options.length_required):
-    -l $length_filtering_options.length_required
-#end if
-
-
-## Base correction by overlap analysis options
-
-$base_correction_options.correction
-
-
-## UMI processing
-
-#if $umi_processing.umi:
-    $umi_processing.umi
-    #if str($umi_processing.umi_loc):
-        --umi_loc '$umi_processing.umi_loc'
-    #end if
-    #if str($umi_processing.umi_len):
-        --umi_len $umi_processing.umi_len
-    #end if
-    #if str($umi_processing.umi_prefix):
-        --umi_prefix '$umi_processing.umi_prefix'
-    #end if
-#end if
-
 ## Overrepresented sequence analysis
 
 $overrepresented_sequence_analysis.overrepresentation_analysis
@@ -154,8 +93,94 @@
 #end if
 
 
-## Thread options
---thread \${GALAXY_SLOTS:-1}
+## Filter Options
+
+## Quality filtering options
+
+$filter_options.quality_filtering_options.disable_quality_filtering
+
+#if str($filter_options.quality_filtering_options.qualified_quality_phred):
+    -q $filter_options.quality_filtering_options.qualified_quality_phred
+#end if
+#if str($filter_options.quality_filtering_options.unqualified_percent_limit):
+    -u $filter_options.quality_filtering_options.unqualified_percent_limit
+#end if
+#if str($filter_options.quality_filtering_options.n_base_limit):
+    -n $filter_options.quality_filtering_options.n_base_limit
+#end if
+
+
+## Length filtering options
+
+$filter_options.length_filtering_options.disable_length_filtering
+
+#if str($filter_options.length_filtering_options.length_required):
+    -l $filter_options.length_filtering_options.length_required
+#end if
+
+## Low complexity filtering options
+
+$filter_options.low_complexity_filter.enable_low_complexity_filter
+
+#if str($filter_options.low_complexity_filter.complexity_threshold):
+    -Y $filter_options.low_complexity_filter.complexity_threshold
+#end if
+
+
+## Read Modification Options
+
+## PolyG tail trimming, useful for NextSeq/NovaSeq data
+
+#if $read_mod_options.polyg_tail_trimming.trimming_select in ['', '-g']:
+    #if str($read_mod_options.polyg_tail_trimming.poly_g_min_len):
+        --poly_g_min_len $read_mod_options.polyg_tail_trimming.poly_g_min_len
+    #end if
+    $read_mod_options.polyg_tail_trimming.trimming_select
+#end if
+
+## PolyX tail trimming
+
+#if $read_mod_options.polyx_tail_trimming.polyx_trimming_select == '-x':
+    $read_mod_options.polyx_tail_trimming.polyx_trimming_select
+    #if str($read_mod_options.polyg_tail_trimming.poly_g_min_len):
+        --poly_x_min_len $read_mod_options.polyx_tail_trimming.poly_x_min_len
+    #end if
+#end if
+
+## UMI processing
+
+#if $read_mod_options.umi_processing.umi:
+    $read_mod_options.umi_processing.umi
+    #if str($read_mod_options.umi_processing.umi_loc):
+        --umi_loc '$read_mod_options.umi_processing.umi_loc'
+    #end if
+    #if str($read_mod_options.umi_processing.umi_len):
+        --umi_len $read_mod_options.umi_processing.umi_len
+    #end if
+    #if str($read_mod_options.umi_processing.umi_prefix):
+        --umi_prefix '$read_mod_options.umi_processing.umi_prefix'
+    #end if
+#end if
+
+## Per read cutting by quality options
+
+#if $read_mod_options.cutting_by_quality_options.cut_by_quality5 or $read_mod_options.cutting_by_quality_options.cut_by_quality3:
+
+    $read_mod_options.cutting_by_quality_options.cut_by_quality5
+
+    $read_mod_options.cutting_by_quality_options.cut_by_quality3
+
+    #if str($read_mod_options.cutting_by_quality_options.cut_window_size):
+        -W $read_mod_options.cutting_by_quality_options.cut_window_size
+    #end if
+    #if str($read_mod_options.cutting_by_quality_options.cut_mean_quality):
+        -M $read_mod_options.cutting_by_quality_options.cut_mean_quality
+    #end if
+#end if
+
+## Base correction by overlap analysis options
+
+$read_mod_options.base_correction_options.correction
 
 &&
 
@@ -166,6 +191,7 @@
 #end if
 ]]></command>
     <inputs>
+
         <conditional name="single_paired">
             <param name="single_paired_selector" type="select" label="Single-end or paired reads">
                 <option value="single" selected="true">Single-end</option>
@@ -173,11 +199,11 @@
             </param>
             <when value="single">
                 <expand macro="in1" />
-                <section name="adapter_trimming_options" title="Adapter trimming options" expanded="False">
+                <section name="adapter_trimming_options" title="Adapter Trimming Options" expanded="False">
                     <param name="disable_adapter_trimming" argument="-A" type="boolean" truevalue="-A" falsevalue="" checked="false" label="Disable adapter trimming" help="Adapter trimming is enabled by default. If this option is specified, adapter trimming is disabled."/>
                     <expand macro="adapter_sequence1" />
                 </section>
-                <section name="global_trimming_options" title="Global trimming options" expanded="False">
+                <section name="global_trimming_options" title="Global Trimming Options" expanded="False">
                     <param name="trim_front1" argument="-f" type="integer" optional="true" label="Trim front for input 1" help="Trimming how many bases in front for read1, default is 0."/>
                     <param name="trim_tail1" argument="-t" type="integer" optional="true" label="Trim tail for input 1" help="Trimming how many bases in tail for read1, default is 0."/>
                 </section>
@@ -207,50 +233,81 @@
                 </section>
             </when>
         </conditional>
-        <conditional name="polyg_tail_trimming">
-            <param name="trimming_select" type="select" label="PolyG tail trimming, useful for NextSeq/NovaSeq data">
-                <option value="" selected="true">Automatic trimming for Illumina NextSeq/NovaSeq data</option>
-                <option value="-g">Force polyG tail trimming</option>
-                <option value="-G">Disable polyG tail trimming</option>
-            </param>
-            <when value="-g">
-                <expand macro="poly_g_min_len" />
-            </when>
-            <when value="">
-                <expand macro="poly_g_min_len" />
-            </when>
-            <when value="-G" />
-        </conditional>
-        <section name="cutting_by_quality_options" title="Per read cutting by quality options" expanded="False">
-            <param name="cut_by_quality5" argument="-5" type="boolean" truevalue="-5" falsevalue="" checked="false" label="Cut by quality in front (5')" help="Enable per read cutting by quality in front (5'), default is disabled (WARNING: this will interfere deduplication for both PE/SE data)."/>
-            <param name="cut_by_quality3" argument="-3" type="boolean" truevalue="-3" falsevalue="" checked="false" label="Cut by quality in tail (3')" help="Enable per read cutting by quality in tail (3'), default is disabled (WARNING: this will interfere deduplication for SE data)."/>
-            <param name="cut_window_size" argument="-W" type="integer" optional="true" label="Cutting window size" help="The size of the sliding window for sliding window trimming, default is 4."/>
-            <param name="cut_mean_quality" argument="-M" type="integer" optional="true" label="Cutting mean quality" help="The bases in the sliding window with mean quality below cutting_quality will be cut, default is Q20."/>
-        </section>
-        <section name="quality_filtering_options" title="Quality filtering options" expanded="False">
-            <param name="disable_quality_filtering" argument="-Q" type="boolean" truevalue="-Q" falsevalue="" checked="false" label="Disable quality filtering" help="Quality filtering is enabled by default. If this option is specified, quality filtering is disabled."/>
-            <param name="qualified_quality_phred" argument="-q" type="integer" optional="true" label="Qualified quality phred" help="The quality value that a base is qualified. Default 15 means phred quality >=Q15 is qualified."/>
-            <param name="unqualified_percent_limit" argument="-u" type="integer" optional="true" label="Unqualified percent limit" help="How many percents of bases are allowed to be unqualified (0~100). Default 40 means 40%."/>
-            <param name="n_base_limit" argument="-n" type="integer" optional="true" label="N base limit" help="If one read's number of N base is >n_base_limit, then this read/pair is discarded. Default is 5."/>
-        </section>
-        <section name="length_filtering_options" title="Length filtering options" expanded="False">
-            <param name="disable_length_filtering" argument="-L" type="boolean" truevalue="-L" falsevalue="" checked="false" label="Disable length filtering" help="Length filtering is enabled by default. If this option is specified, length filtering is disabled."/>
-            <param name="length_required" argument="-l" type="integer" optional="true" label="Length required" help="Reads shorter than this value will be discarded, default is 15."/>
-        </section>
-        <section name="base_correction_options" title="Base correction by overlap analysis options" expanded="False">
-            <param name="correction" argument="-c" type="boolean" truevalue="-c" falsevalue="" checked="false" label="Enable base correction" help="Enable base correction in overlapped regions (only for PE data), default is disabled."/>
-        </section>
-        <section name="umi_processing" title="UMI processing" expanded="False">
-            <param name="umi" argument="-U" type="boolean" truevalue="-U" falsevalue="" checked="false" label="Enable unique molecular identifer" help="Enable unique molecular identifer (UMI) preprocessing."/>
-            <param name="umi_loc" argument="--umi_loc" type="text" optional="true" label="UMI location" help="Specify the location of UMI, can be (index1/index2/read1/read2/per_index/per_read, default is none."/>
-            <param name="umi_len" argument="--umi_len" type="integer" optional="true" label="UMI length" help="If the UMI is in read1/read2, its length should be provided."/>
-            <param name="umi_prefix" argument="--umi_prefix" type="text" optional="true" label="UMI prefix" help="If specified, an underline will be used to connect prefix and UMI (i.e. prefix=UMI, UMI=AATTCG, final=UMI_AATTCG). No prefix by default."/>
-        </section>
-        <section name="overrepresented_sequence_analysis" title="Overrepresented sequence analysis" expanded="False">
+
+        <section name="overrepresented_sequence_analysis" title="Overrepresented Sequence Analysis" expanded="False">
             <param name="overrepresentation_analysis" argument="-p" type="boolean" truevalue="-p" falsevalue="" checked="false" label="Enable overrepresented analysis" help="Enable overrepresented sequence analysis."/>
             <param name="overrepresentation_sampling" argument="-P" type="integer" optional="true" label="Overrepresentation sampling" help="One in (--overrepresentation_sampling) reads will be computed for overrepresentation analysis (1~10000), smaller is slower, default is 20."/>
         </section>
-        <section name="output_options" title="Output options" expanded="False">
+
+        <!-- Filter Options -->
+        <section name="filter_options" title="Filter Options">
+            <section name="quality_filtering_options" title="Quality filtering options" expanded="True">
+                <param name="disable_quality_filtering" argument="-Q" type="boolean" truevalue="-Q" falsevalue="" checked="false" label="Disable quality filtering" help="Quality filtering is enabled by default. If this option is specified, quality filtering is disabled."/>
+                <param name="qualified_quality_phred" argument="-q" type="integer" optional="true" label="Qualified quality phred" help="The quality value that a base is qualified. Default 15 means phred quality >=Q15 is qualified."/>
+                <param name="unqualified_percent_limit" argument="-u" type="integer" optional="true" label="Unqualified percent limit" help="How many percents of bases are allowed to be unqualified (0~100). Default 40 means 40%."/>
+                <param name="n_base_limit" argument="-n" type="integer" optional="true" label="N base limit" help="If one read's number of N base is >n_base_limit, then this read/pair is discarded. Default is 5."/>
+            </section>
+
+            <section name="length_filtering_options" title="Length filtering options" expanded="True">
+                <param name="disable_length_filtering" argument="-L" type="boolean" truevalue="-L" falsevalue="" checked="false" label="Disable length filtering" help="Length filtering is enabled by default. If this option is specified, length filtering is disabled."/>
+                <param name="length_required" argument="-l" type="integer" optional="true" label="Length required" help="Reads shorter than this value will be discarded. Default is 15."/>
+            </section>
+
+            <section name="low_complexity_filter" title="Low complexity filtering options" expanded="True">
+                <param name="enable_low_complexity_filter" argument="-y" type="boolean" truevalue="-y" falsevalue="" checked="false" label="Enable low complexity filter" help="The complexity is defined as the percentage of base that is different from its next base, default is No"/>
+                <param name="complexity_threshold" argument="-Y" type="integer" optional="true" label="Complexity threshold" help="Threshold for low complexity filter (0~100). Default is 30, which means 30% complexity is required."/>
+            </section>
+        </section>
+
+        <!-- Read Modification Options -->
+         <section name="read_mod_options" title="Read Modification Options">
+            <conditional name="polyg_tail_trimming">
+                <param name="trimming_select" type="select" label="PolyG tail trimming" help="Useful for NextSeq/NovaSeq data">
+                    <option value="" selected="true">Automatic trimming for Illumina NextSeq/NovaSeq data</option>
+                    <option value="-g">Force polyG tail trimming</option>
+                    <option value="-G">Disable polyG tail trimming</option>
+                </param>
+                <when value="-g">
+                    <expand macro="poly_g_min_len" />
+                </when>
+                <when value="">
+                    <expand macro="poly_g_min_len" />
+                </when>
+                <when value="-G" />
+            </conditional>
+
+            <conditional name="polyx_tail_trimming">
+                <param name="polyx_trimming_select" type="select" label="PolyX tail trimming" help="Similar to polyG tail trimming. When polyG tail trimming and polyX tail trimming are both enabled, fastp will perform polyG trimming first, then perform polyX trimming. Disabled by default.">
+                    <option value="" selected="true">Disable polyX trimming</option>
+                    <option value="-x">Enable polyX tail trimming</option>
+                </param>
+                <when value="-x">
+                    <param name="poly_x_min_len" argument="--poly_x_min_len" type="integer" optional="true" label="PolyX minimum length"
+                    help="The minimum length to detect polyX in the read tail. 10 by default."/>
+                </when>
+                <when value="" />
+            </conditional>
+
+            <section name="umi_processing" title="UMI processing" expanded="True">
+                <param name="umi" argument="-U" type="boolean" truevalue="-U" falsevalue="" checked="false" label="Enable unique molecular identifer" help="Enable unique molecular identifer (UMI) preprocessing."/>
+                <param name="umi_loc" argument="--umi_loc" type="text" optional="true" label="UMI location" help="Specify the location of UMI, can be (index1/index2/read1/read2/per_index/per_read, default is none."/>
+                <param name="umi_len" argument="--umi_len" type="integer" optional="true" label="UMI length" help="If the UMI is in read1/read2, its length should be provided."/>
+                <param name="umi_prefix" argument="--umi_prefix" type="text" optional="true" label="UMI prefix" help="If specified, an underline will be used to connect prefix and UMI (i.e. prefix=UMI, UMI=AATTCG, final=UMI_AATTCG). No prefix by default."/>
+            </section>
+
+            <section name="cutting_by_quality_options" title="Per read cutting by quality options" expanded="True">
+                <param name="cut_by_quality5" argument="-5" type="boolean" truevalue="-5" falsevalue="" checked="false" label="Cut by quality in front (5')" help="Enable per read cutting by quality in front (5'), default is disabled (WARNING: this will interfere deduplication for both PE/SE data)."/>
+                <param name="cut_by_quality3" argument="-3" type="boolean" truevalue="-3" falsevalue="" checked="false" label="Cut by quality in tail (3')" help="Enable per read cutting by quality in tail (3'), default is disabled (WARNING: this will interfere deduplication for SE data)."/>
+                <param name="cut_window_size" argument="-W" type="integer" optional="true" label="Cutting window size" help="The size of the sliding window for sliding window trimming, default is 4."/>
+                <param name="cut_mean_quality" argument="-M" type="integer" optional="true" label="Cutting mean quality" help="The bases in the sliding window with mean quality below cutting_quality will be cut, default is Q20."/>
+            </section>
+
+            <section name="base_correction_options" title="Base correction by overlap analysis options" expanded="True">
+                <param name="correction" argument="-c" type="boolean" truevalue="-c" falsevalue="" checked="false" label="Enable base correction" help="Enable base correction in overlapped regions (only for PE data), default is disabled."/>
+            </section>
+        </section>
+
+        <section name="output_options" title="Output Options" expanded="False">
             <param name="report_html" type="boolean" truevalue="True" falsevalue="False" checked="True" label="Output HTML report" help="fastp provides a QC report for the data Before and After filtering within a single HTML page, which enables comparison of the quality statistics changed by the preprocessing step directly"/>
             <param name="report_json" type="boolean" truevalue="True" falsevalue="False" checked="False" label="Output JSON report" help="The JSON report contains all the data visualized in the HTML report. The format of the JSON report is manually optimized to be easily readable by humans"/>
         </section>
@@ -270,89 +327,100 @@
     </outputs>
 
     <tests>
-        <test>
-            <param name="in1" value="R1.fq" ftype="fastqsanger"/>
+        <!-- Ensure default output works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastqsanger" value="R1.fq"/>
             <param name="single_paired_selector" value="single"/>
-            <output name="out1" file="out1.fq" ftype="fastqsanger"/>
+            <output name="out1" ftype="fastqsanger" file="out1.fq"/>
+            <output name="report_html">
+                <assert_contents>
+                    <has_text text="fastp report"/>
+                </assert_contents>
+            </output>
         </test>
-        <test>
-            <param name="in1" value="R1.fq" ftype="fastq"/>
+        <!-- Ensure custom adapter works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastq" value="R1.fq"/>
             <param name="single_paired_selector" value="single"/>
             <param name="adapter_sequence1" value="ATCG"/>
-            <output name="out1" file="out_a.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out_a.fq"/>
         </test>
-        <test>
-            <param name="in1" value="R1.fq" ftype="fastq"/>
+        <!-- Ensure UMI processing works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastq" value="R1.fq"/>
             <param name="single_paired_selector" value="single"/>
             <section name="umi_processing">
                 <param name="umi" value="true"/>
                 <param name="umi_loc" value="read1"/>
                 <param name="umi_len" value="8"/>
             </section>
-            <output name="out1" file="out2.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out2.fq"/>
         </test>
-        <test>
-            <param name="in1" value="R1.fq" ftype="fastq"/>
+        <!-- Ensure UMI processing with different lengths works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastq" value="R1.fq"/>
             <param name="single_paired_selector" value="single"/>
             <section name="umi_processing">
                 <param name="umi" value="true"/>
                 <param name="umi_loc" value="read1"/>
                 <param name="umi_len" value="12"/>
             </section>
-            <output name="out1" file="out3.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out3.fq"/>
         </test>
-        <test>
-            <param name="in1" value="bwa-mem-fastq1.fq" ftype="fastq"/>
-            <param name="in2" value="bwa-mem-fastq2.fq" ftype="fastq"/>
+        <!-- Ensure paired-end fastq works -->
+        <test expect_num_outputs="3">
+            <param name="in1" ftype="fastq" value="bwa-mem-fastq1.fq"/>
+            <param name="in2" ftype="fastq" value="bwa-mem-fastq2.fq"/>
             <param name="single_paired_selector" value="paired"/>
-            <output name="out1" file="out_bwa1.fq" ftype="fastq"/>
-            <output name="out2" file="out_bwa2.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out_bwa1.fq"/>
+            <output name="out2" ftype="fastq" file="out_bwa2.fq"/>
         </test>
-        <test>
-            <param name="in1" value="bwa-mem-fastq1.fq" ftype="fastq"/>
-            <param name="in2" value="bwa-mem-fastq2.fq" ftype="fastq"/>
+        <!-- Ensure paired-end UMI processing of Read 1 works -->
+        <test expect_num_outputs="3">
+            <param name="in1" ftype="fastq" value="bwa-mem-fastq1.fq"/>
+            <param name="in2" ftype="fastq" value="bwa-mem-fastq2.fq"/>
             <param name="single_paired_selector" value="paired"/>
             <section name="umi_processing">
                 <param name="umi" value="true"/>
                 <param name="umi_loc" value="read1"/>
                 <param name="umi_len" value="8"/>
             </section>
-            <output name="out1" file="out_bwa_umi_read1_1.fq" ftype="fastq"/>
-            <output name="out2" file="out_bwa_umi_read1_2.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out_bwa_umi_read1_1.fq"/>
+            <output name="out2" ftype="fastq" file="out_bwa_umi_read1_2.fq"/>
         </test>
-        <test>
-            <param name="in1" value="bwa-mem-fastq1.fq" ftype="fastq"/>
-            <param name="in2" value="bwa-mem-fastq2.fq" ftype="fastq"/>
+        <!-- Ensure paired-end UMI processing of Read 2 works -->
+        <test expect_num_outputs="3">
+            <param name="in1" ftype="fastq" value="bwa-mem-fastq1.fq"/>
+            <param name="in2" ftype="fastq" value="bwa-mem-fastq2.fq"/>
             <param name="single_paired_selector" value="paired"/>
             <section name="umi_processing">
                 <param name="umi" value="true"/>
                 <param name="umi_loc" value="read2"/>
                 <param name="umi_len" value="8"/>
             </section>
-            <output name="out1" file="out_bwa_umi_read2_1.fq" ftype="fastq"/>
-            <output name="out2" file="out_bwa_umi_read2_2.fq" ftype="fastq"/>
+            <output name="out1" ftype="fastq" file="out_bwa_umi_read2_1.fq"/>
+            <output name="out2" ftype="fastq" file="out_bwa_umi_read2_2.fq"/>
         </test>
-        <test>
-            <param name="in1" value="R1.fq" ftype="fastq"/>
+        <!-- Ensure JSON report output works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastqsanger" value="R1.fq"/>
             <param name="single_paired_selector" value="single"/>
-            <output name="out1" file="out1.fq" ftype="fastq"/>
-            <output name="report_html">
+            <param name="report_html" value="False"/>
+            <param name="report_json" value="True"/>
+            <output name="out1" ftype="fastqsanger" file="out1.fq"/>
+            <output name="report_json">
                 <assert_contents>
                     <has_text text="fastp report"/>
                 </assert_contents>
             </output>
         </test>
-        <test>
-            <param name="in1" value="R1.fq.gz" ftype="fastq.gz"/>
+        <!-- Ensure polyG trimming works -->
+        <test expect_num_outputs="2">
+            <param name="in1" ftype="fastq.gz" value="R1.fq.gz"/>
             <param name="single_paired_selector" value="single"/>
             <param name="trimming_select" value="-g"/>
             <param name="poly_g_min_len" value="10"/>
-            <output name="out1" file="out1.fq.gz" ftype="fastq.gz" compare="sim_size"/>
-            <output name="report_html">
-                <assert_contents>
-                    <has_text text="fastp report"/>
-                </assert_contents>
-            </output>
+            <output name="out1" ftype="fastq.gz" decompress="True" file="out1.fq.gz"/>
         </test>
     </tests>
     <help><![CDATA[
@@ -366,23 +434,25 @@
 
 1. Filter out bad reads (too low quality, too short, or too many N...)
 
-2. Cut low quality bases for per read in its 5' and 3' by evaluating the mean quality from a sliding window (like Trimmomatic but faster).
+2. Cut low quality bases for per read in its 5' and 3' by evaluating the mean quality from a sliding window (like Trimmomatic but faster)
 
 3. Trim all reads in front and tail
 
-4. Cut adapters. Adapter sequences can be automatically detected,which means you don't have to input the adapter sequences to trim them.
+4. Cut adapters. Adapter sequences can be automatically detected, which means you don't have to input the adapter sequences to trim them.
+
+5. Correct mismatched base pairs in overlapped regions of paired end reads, if one base is with high quality while the other is with ultra-low quality
 
-5. Correct mismatched base pairs in overlapped regions of paired end reads, if one base is with high quality while the other is with ultra low quality
+6. Trim polyG in 3' ends, which is commonly seen in NovaSeq/NextSeq data. Trim polyX in 3' ends to remove unwanted polyX tailing (i.e. polyA tailing for mRNA-Seq data)
 
-6. Preprocess unique molecular identifer (UMI) enabled data, shift UMI to sequence name.
+7. Preprocess unique molecular identifer (UMI) enabled data, shift UMI to sequence name
 
-7. Report JSON format result for further interpreting.
+8. Report JSON format result for further interpreting
 
-8. Visualize quality control and filtering results on a single HTML page (like FASTQC but faster and more informative).
+9. Visualize quality control and filtering results on a single HTML page (like FASTQC but faster and more informative)
 
-9. Split the output to multiple files (0001.R1.gz, 0002.R1.gz...) to support parallel processing. Two modes can be used, limiting the total split file number, or limitting the lines of each split file.
+10. Split the output to multiple files (0001.R1.gz, 0002.R1.gz...) to support parallel processing. Two modes can be used, limiting the total split file number, or limitting the lines of each split file (*Not enabled in this Galaxy tool*)
 
-10. Support long reads (data from PacBio / Nanopore devices).
+11. Support long reads (data from PacBio / Nanopore devices)
 
 -----
 
diff -r 988729b728f0 -r f44e93b4529c macros.xml
--- a/macros.xml	Fri Mar 09 20:21:51 2018 -0500
+++ b/macros.xml	Tue Mar 20 04:24:38 2018 -0400
@@ -1,5 +1,5 @@
 <macros>
-    <token name="@WRAPPER_VERSION@">0.12.4</token>
+    <token name="@WRAPPER_VERSION@">0.12.5</token>
 
     <xml name="adapter_sequence1">
         <param name="adapter_sequence1" argument="--adapter_sequence" type="text" optional="true" label="Adapter sequence for input 1"
@@ -21,5 +21,4 @@
         <param name="poly_g_min_len" argument="--poly_g_min_len" type="integer" optional="true" label="PolyG minimum length"
             help="The minimum length to detect polyG in the read tail. 10 by default."/>
     </xml>
-
 </macros>