Without that line changed when I was running `./grailsw test-app`, the tests were failing with `No such property: controller for class: com.netlix.asgard.LaunchConfigurationControllerTests`.   

Hmm, what is "the" correct way to run the build?

I've just noticed I have different results when running
`./gradlew test` or `./grailsw test-app`.  (or maybe it's the same, but just the tests are flaky, possibly due to my change)

At the moment it looks like 1 of the tests (controller.protocols) I've added is flaky:

```
 Failure:  testSaveSuccessfully(com.netflix.asgard.LoadBalancerControllerTests)
|  java.lang.NullPointerException: Cannot invoke method getBean() on null object
	at com.netflix.asgard.LoadBalancerController.save(LoadBalancerController.groovy:163)
	at com.netflix.asgard.LoadBalancerControllerTests.testSaveSuccessfully(LoadBalancerControllerTests.groovy:88)
| Running 200 unit tests... 98 of 200
| Running 200 unit tests... 99 of 200
| Failure:  testSaveWithoutMostParams(com.netflix.asgard.LoadBalancerControllerTests)
|  java.lang.NullPointerException: Cannot invoke method getBean() on null object
	at com.netflix.asgard.LoadBalancerController.save(LoadBalancerController.groovy:132)
	at com.netflix.asgard.LoadBalancerControllerTests.testSaveWithoutMostParams(LoadBalancerControllerTests.groovy:62)
| Running 200 unit tests... 100 of 200
| Failure:  testProtocols(com.netflix.asgard.LoadBalancerControllerTests)
|  Assertion failed:

assert ['HTTP', 'TCP'] == controller.protocols
                       |  |          |
                       |  |          [HTTP, HTTPS, TCP]
                       |  com.netflix.asgard.LoadBalancerController@494ce1be
                       false

	at com.netflix.asgard.LoadBalancerControllerTests.testProtocols(LoadBalancerControllerTests.groovy:94)
```
I will investigate tomorrow. Just quick questions, should I be adding JUnit tests at all, or should I only use Spock?