add exhaustive option to off target algorithm when searching for cris…

…prs, fix tests for this as well
htgt · May 8, 2015 · 7e34f1a · 7e34f1a
1 parent 67a6cc6
commit 7e34f1a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/lib/LIMS2/WebApp/Controller/User/DesignTargets.pm b/lib/LIMS2/WebApp/Controller/User/DesignTargets.pm
@@ -57,7 +57,7 @@ sub gene_report : Path('/user/design_target_report') {
 
     my %report_parameters = (
         type                 => $c->request->param('report_type') || 'standard',
-        off_target_algorithm => $c->request->param('off_target_algorithm') || 'bwa',
+        off_target_algorithm => $c->request->param('off_target_algorithm') || 'exhaustive',
         crispr_types         => $c->request->param('crispr_types') || 'pair',
         filter               => $c->request->param('filter') || 0,
     );

diff --git a/root/site/user/designtargets/index.tt b/root/site/user/designtargets/index.tt
@@ -19,7 +19,11 @@
 
     <label><strong>Off Target Algorithm Type</strong></label>
     <label class="radio">
-      <input type="radio" name="off_target_algorithm" value="bwa" checked>
+      <input type="radio" name="off_target_algorithm" value="exhaustive" checked>
+      Exhaustive
+    </label>
+    <label class="radio">
+      <input type="radio" name="off_target_algorithm" value="bwa">
       BWA
     </label>
     <label class="radio">

diff --git a/t/lib/LIMS2/t/WebApp/Controller/User/Crisprs.pm b/t/lib/LIMS2/t/WebApp/Controller/User/Crisprs.pm
@@ -47,7 +47,8 @@ sub all_tests  : Test(21)
     ok $res = $mech->submit_form(
             fields => {
                 genes        => 'MGI:1917722',
-                crispr_types => 'group'
+                crispr_types => 'group',
+                off_target_algorithm => 'exhaustive',
             },
             button => 'action',
         );
@@ -75,4 +76,4 @@ sub all_tests  : Test(21)
 
 }
 
-1;
+1;