From b0b119356b130b82c39c6c811369f002cc571614 Mon Sep 17 00:00:00 2001
From: John Chilton <jmchilton@gmail.com>
Date: Thu, 1 Oct 2015 13:06:17 +0100
Subject: [PATCH 1/2] Normalize test output name.

Update toolshed, casperjs, and framework tests to produce reports with the name of run_<type>_tests.html to match functional and API tests.
---
 run_tests.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/run_tests.sh b/run_tests.sh
index 14b38ad0dc5..c9013644bde 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -138,7 +138,7 @@ do
           ;;
       -t|-toolshed|--toolshed)
           test_script="./test/tool_shed/functional_tests.py"
-          report_file="./test/tool_shed/run_functional_tests.html"
+          report_file="run_toolshed_tests.html"
           if [ $# -gt 1 ]; then
               toolshed_script=$2
               shift 2
@@ -162,6 +162,7 @@ do
           fi
           ;;
       -f|-framework|--framework)
+          report_file="run_framework_tests.html"
           framework_test=1;
           shift 1
           ;;
@@ -172,6 +173,7 @@ do
       -j|-casperjs|--casperjs)
           # TODO: Support running casper tests against existing
           # Galaxy instances.
+          report_file="run_casperjs_tests.html"
           casperjs_test=1;
           shift
           ;;

From 49740caaa2cf00e1eeb8bfdef1939638256103a0 Mon Sep 17 00:00:00 2001
From: John Chilton <jmchilton@gmail.com>
Date: Thu, 1 Oct 2015 13:44:20 +0100
Subject: [PATCH 2/2] Add documentation for running casperjs tests through
 run_tests.sh.

Add option to run a specific test. Verified xunit reports work and --dockerize works for casperjs tests.
---
 run_tests.sh                     | 15 ++++++++++++---
 test/casperjs/casperjs_runner.py |  2 +-
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/run_tests.sh b/run_tests.sh
index c9013644bde..cce3eae0a62 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -24,7 +24,7 @@ cat <<EOF
 '${0##*/} -unit (test_path)'        for running all unit tests (doctests in lib and tests in test/unit)
 '${0##*/} -qunit'                   for running qunit JavaScript tests
 '${0##*/} -qunit testname'          for running single JavaScript test with given name
-
+'${0##*/} -casperjs (py_test_path)' for running casperjs JavaScript tests using a Python wrapper for consistency. py_test_path in casperjs_runner.py e.g. 'Test_04_HDAs' or 'Test_04_HDAs.test_00_HDA_states'.
 
 Nose tests will allow specific tests to be selected per the documentation at
 https://nose.readthedocs.org/en/latest/usage.html#selecting-tests.  These are
@@ -173,9 +173,14 @@ do
       -j|-casperjs|--casperjs)
           # TODO: Support running casper tests against existing
           # Galaxy instances.
+          if [ $# -gt 1 ]; then
+              casperjs_test_name=$2
+              shift 2
+          else
+              shift 1
+          fi
           report_file="run_casperjs_tests.html"
           casperjs_test=1;
-          shift
           ;;
       -m|-migrated|--migrated)
           migrated_test=1;
@@ -306,7 +311,11 @@ elif [ -n "$casperjs_test" ]; then
     # TODO: Ensure specific versions of casperjs and phantomjs are
     # available. Some option for leveraging npm to automatically
     # install these dependencies would be nice as well.
-    extra_args="test/casperjs/casperjs_runner.py"
+    if [ -n "$casperjs_test_name" ]; then
+        extra_args="test/casperjs/casperjs_runner.py:$casperjs_test_name"
+    else
+        extra_args="test/casperjs/casperjs_runner.py"
+    fi
 elif [ -n "$section_id" ]; then
     extra_args=`python tool_list.py $section_id` 
 elif [ -n "$test_id" ]; then
diff --git a/test/casperjs/casperjs_runner.py b/test/casperjs/casperjs_runner.py
index 78ebabeb01d..9cb77df04c8 100644
--- a/test/casperjs/casperjs_runner.py
+++ b/test/casperjs/casperjs_runner.py
@@ -160,7 +160,7 @@ class CasperJSTestCase( unittest.TestCase ):
         # as of casperjs 1.1, we always need to use the 'test' command
         command_line_list.append( 'test' )
 
-        # make rel_script_path an absolute path (when this is not run from its dir - i.e. run_functional_tests.sh)
+        # make rel_script_path an absolute path (when this is not run from its dir - i.e. run_tests.sh)
         curr_dir = os.path.dirname( __file__ )
         script_path = os.path.join( curr_dir, rel_script_path )
         command_line_list.append( script_path )