SLIDER-5 improve functional testing docs, get tests to recompile after groovy compiler rollback

git-svn-id: https://svn.apache.org/repos/asf/incubator/slider/trunk@1593104 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestAppsThroughAgent.groovy b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestAppsThroughAgent.groovy
index 694641e..836d770 100644
--- a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestAppsThroughAgent.groovy
+++ b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestAppsThroughAgent.groovy
@@ -20,14 +20,17 @@
 
 import groovy.transform.CompileStatic
 import groovy.util.logging.Slf4j
+import org.apache.slider.common.SliderExitCodes
+import org.apache.slider.common.params.Arguments
+import org.apache.slider.common.params.SliderActions
+import org.apache.slider.funtest.framework.FuntestProperties
 import org.apache.slider.funtest.framework.SliderShell
 import org.junit.Test
 
 @CompileStatic
 @Slf4j
 public class TestAppsThroughAgent extends AgentCommandTestBase
-//    implements FuntestProperties, Arguments, SliderExitCodes, SliderActions 
-{
+    implements FuntestProperties, Arguments, SliderExitCodes, SliderActions {
 
   private static String COMMAND_LOGGER = "COMMAND_LOGGER"
   private static String APPLICATION_NAME = "agenttst"
diff --git a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestClusterBuildDestroy.groovy b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestClusterBuildDestroy.groovy
index 5c1743e..0c3924f 100644
--- a/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestClusterBuildDestroy.groovy
+++ b/slider-funtest/src/test/groovy/org/apache/slider/funtest/lifecycle/TestClusterBuildDestroy.groovy
@@ -33,7 +33,8 @@
 
 @CompileStatic
 @Slf4j
-public class TestClusterBuildDestroy extends AgentCommandTestBase {
+public class TestClusterBuildDestroy extends AgentCommandTestBase
+    implements FuntestProperties, Arguments, SliderExitCodes, SliderActions {
 
 
   static String CLUSTER = "test_cluster_build_destroy"
diff --git a/src/site/markdown/developing/functional_tests.md b/src/site/markdown/developing/functional_tests.md
index cf8ef11..b4a1d76 100644
--- a/src/site/markdown/developing/functional_tests.md
+++ b/src/site/markdown/developing/functional_tests.md
@@ -28,6 +28,7 @@
 a live cluster. 
 
 For these to work you need
+
 1. A YARN Cluster -secure or insecure
 1. A `slider-client.xml` file configured to interact with the cluster
 1. Agent 
@@ -38,7 +39,8 @@
 
 ## Configuration of functional tests
 
-Maven needs to be given 
+Maven needs to be given
+
 1. A path to the expanded test archive
 1. A path to a slider configuration directory for the cluster
 
@@ -109,8 +111,7 @@
       There are some maven properties which also define limits and may need adjusting</description>
       <value>180000</value>
     </property>
-    
-    
+
     
 Note that while the same properties need to be set in
 `slider-core/src/test/resources/slider-client.xml`, those tests take a file in the local
@@ -159,7 +160,6 @@
         
 Mandatory test parameters must be added to `slider-client.xml`
 
-  
     <property>
       <name>slider.test.hbase.tar</name>
       <description>Path to the HBase Tar file in HDFS</description>
@@ -180,7 +180,6 @@
       <value>180000</value>
     </property>  
 
-
 #### Accumulo configuration options
 
 Enable/disable the tests
@@ -191,7 +190,6 @@
       <value>true</value>
      </property>
          
-         
 Optional parameters
          
      <property>
@@ -200,11 +198,8 @@
       <value>180000</value>
      </property>
 
-
-
 ### Configuring the YARN cluster for tests
 
-
 Here are the configuration options we use in `yarn-site.xml` for testing:
 
 These tell YARN to ignore memory requirements in allocating VMs, and
@@ -240,7 +235,7 @@
       </property>
 
 
-### Testing against a secure cluster
+## Testing against a secure cluster
 
 To test against a secure cluster