On 06/11/2012, at 1:05 AM, Luke Daley wrote: > > On 01/11/2012, at 7:43 PM, Adam Murdoch wrote: > >> >> On 02/11/2012, at 3:34 AM, Luke Daley wrote: >> >>> Please try and avoid growing classes like AbstractIntegrationSpec with >>> stuff that isn't absolutely core to integration testing. >> >> That are core to integration testing in the build domain. Dependency >> management is a core aspect of this. > > Seems like this would, at least, be better just being on > AbstractDependencyResolutionTest.
Possibly. AbstractDependencyResolutionTest is intended to be used by those tests that test that dependency resolution works as expected, so it does things like set up an isolated user home directory and so on. The methods on AbstractIntegrationSpec are for those tests that just happen to use dependency resolution as part of whatever they are actually testing. -- Adam Murdoch Gradle Co-founder http://www.gradle.org VP of Engineering, Gradleware Inc. - Gradle Training, Support, Consulting http://www.gradleware.com
