This will need a few changes in GradleConfiguration to map the Scope to a GradleConfiguration.

Something like:

TEST_AND_DEVELOPMENT_ONLY("testAndDevelopmentOnly", 2),

if (scope.getPhases().contains(Phase.DEVELOPMENT_ONLY)) {
    if(scope == Scope.DEVELOPMENT_ONLY) {
        return Optional.of(GradleConfiguration.DEVELOPMENT_ONLY);
    }
    else {
        return Optional.of(GradleConfiguration.TEST_AND_DEVELOPMENT_ONLY);
    }
}

The failure is unrelated to this change:

* What went wrong:
Could not determine the dependencies of task ':bootWar'.
> Could not create task ':assetCompile'.
   > Could not set unknown property 'destinationDirectory' for task ':assetCompile' of type asset.pipeline.gradle.AssetCompile.

Also seeing the assetCompile error on: #596

Merging since we need part of this for #596

I noticed that the tests are failing and this code exists in the asset pipeline: https://github.com/wondrify/asset-pipeline/blob/00db3ef6b137e7fab84b5b9f5a7d687fa844283f/asset-pipeline-gradle/src/main/groovy/asset/pipeline/gradle/AssetPipelinePlugin.groovy#L116

Doesn't that mean we should revert this change? Since the developmentOnly assets will be added to the test run classpath? @matrei @jamesfredley

Doesn't that mean we should revert this change? Since the developmentOnly assets will be added to the test run classpath?

No, this change puts the actual webjars on the testRuntimeClasspath, and they are also needed.

But according to that link, the testRuntimeClasspath extends the development runtimeOnly. It looks like this is now broken for dev too because of wondrify/asset-pipeline#380

But according to that link, the testRuntimeClasspath extends the development runtimeOnly. It looks like this is now broken for dev too because of wondrify/asset-pipeline#380

I think testRuntimeOnly extends an asset-pipeline custom configuration named assetDevelopmentRuntime. This is used to make asset-pipeline-gradle available when running bootRun, bootTestRun and on the testRuntimeOnly classpath.

I completely read that configuration wrong. Yes, this is the right change. thank you!