fix for GRAILS-10956 “Forked functional testing with war throwing exceptions”

Author: graemerocher

grails-test/src/main/groovy/org/codehaus/groovy/grails/test/runner/phase/FunctionalTestPhaseConfigurer.groovy

@@ -16,6 +16,7 @@
 package org.codehaus.groovy.grails.test.runner.phase
 
 import grails.util.BuildSettings
+import grails.util.Environment
 import grails.util.Holders
 import groovy.transform.CompileStatic
 
@@ -90,7 +91,9 @@ class FunctionalTestPhaseConfigurer extends DefaultTestPhaseConfigurer {
 
                 // need to swap out the args map so any test phase/targetting patterns
                 // aren't intepreted as the war name.
-                projectRunner.warCreator.packageWar()
+                if( !Environment.isFork() ) {
+                    projectRunner.warCreator.packageWar()
+                }
 
                 if (https) {
                     projectRunner.runWarHttps()

scripts/_GrailsTest.groovy

@@ -86,6 +86,9 @@ target(allTests: "Runs the project's tests.") {
     depends(compile, startLogging, packagePlugins, configureServerContextPath)
     Integer exitCode
     if(grailsSettings.forkSettings.test) {
+        if(argsMap?.war) {
+            projectRunner.warCreator.packageWar()
+        }
         def forkedTestRunner = new ForkedGrailsTestRunner(grailsSettings)
         if(grailsSettings.forkSettings.test instanceof Map) {
             forkedTestRunner.configure(grailsSettings.forkSettings.test)