Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Software Model @UnsupportedWithInstantExecution

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -3
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./ModelDslCreationIntegrationTest.groovy
    • -2
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -8
    • +2
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelRuleDslDetectionIntegrationTest.groovy
  1. … 69 more files in changeset.
Mark software model tests as @UnsupportedWithInstantExecution

to reduce CI time and noise when making changes

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -3
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./ModelDslCreationIntegrationTest.groovy
    • -2
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -8
    • +2
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelRuleDslDetectionIntegrationTest.groovy
  1. … 70 more files in changeset.
Mark software model tests as @UnsupportedWithInstantExecution

to reduce CI time and noise when making changes

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -3
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./ModelDslCreationIntegrationTest.groovy
    • -2
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -8
    • +2
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelRuleDslDetectionIntegrationTest.groovy
  1. … 70 more files in changeset.
Mark software model tests as @UnsupportedWithInstantExecution

to reduce CI time and noise when making changes

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -3
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./ModelDslCreationIntegrationTest.groovy
    • -2
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -8
    • +2
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
    • -0
    • +2
    ./internal/transform/NestedModelRuleDslDetectionIntegrationTest.groovy
  1. … 70 more files in changeset.
Rename @FailsWithInstantExecution to @ToBeFixedForInstantExecution

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -3
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -2
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -8
    • +8
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
  1. … 869 more files in changeset.
Annotate integ tests failing with instant execution in :modelGroovy

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -0
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -0
    • +8
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
Annotate integ tests failing with instant execution in :modelGroovy

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -0
    • +3
    ./ManagedTypeDslIntegrationTest.groovy
    • -0
    • +2
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -0
    • +8
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
Fix ModelDslRuleInputDetectionIntegrationSpec

    • -0
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
Replace 'Matchers' with 'CoreMatchers'

So we do not require 'org.hamcrest:hamcrest-library' as additional

dependency anymore. Which was only available for most of the tests

because it leaked onto the test compile classpath.

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 160 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 284 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 281 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 284 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 282 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 283 more files in changeset.
Use 'implementation' dependencies everywhere

    • -1
    • +1
    ./internal/transform/ModelDslRuleDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -1
    • +1
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
  1. … 283 more files in changeset.
Make software model tasks accessible using abbreviated names again

This was broken by a refactoring towards lazy task configuration,

because we had no test coverage for this case.

    • -1
    • +1
    ./ModelDslCreationIntegrationTest.groovy
  1. … 4 more files in changeset.
Change `DefaultNamedDomainObjectCollection` to better handle elements added using a provider, in particular when locating elements by name.

Change `DefaultTaskContainer` to use this to implement `createLater()` rather than the existing placeholder actions, to make deferred task creation less of a special case.

    • -1
    • +1
    ./ModelDslCreationIntegrationTest.groovy
  1. … 9 more files in changeset.
Fix model Groovy usages of output test fixtures

    • -12
    • +12
    ./ModelMapDslIntegrationTest.groovy
    • -3
    • +3
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
GRADLE-3373: Fixed model DSL transformation to allow input references as method parameters when the last method parameter is a closure.

    • -1
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
    • -2
    • +51
    ./internal/transform/NestedModelRuleDslDetectionIntegrationTest.groovy
  1. … 1 more file in changeset.
Changed the 'set' method that is mixed in to the generated view objects for a property of scalar type so that it now applies the convenience type conversion for the value.

Also generate a dummy 'configure' method for all properties that don't require a real one, which fails with a MissingMethodException. This avoids the setter being invoked in this case and type conversion being attempted on the closure. In theory this will result in a more informative error message when the user attempts to configure a property using a closure, when the property does not support this.

+review REVIEW-5708

    • -4
    • +39
    ./ManagedTypeDslIntegrationTest.groovy
  1. … 2 more files in changeset.
Added some test coverage to verify that configure methods are not available for various types of properties of views of managed types.

+review REVIEW-5708

    • -0
    • +110
    ./ManagedTypeDslIntegrationTest.groovy
Generate a 'configure' method for each property of a `@Managed` type.

This configure method has the following signature:

void $propName(Closure<?> cl);

For now, the closure is executed immediately. Later, this will add a rule that will apply the action to the target property.

+review REVIEW-5708

    • -0
    • +59
    ./ManagedTypeDslIntegrationTest.groovy
  1. … 4 more files in changeset.
Moved int test into a package to live alongside some other tests with similar responsibilities.

    • -0
    • +466
    ./ModelMapDslIntegrationTest.groovy
    • -466
    • +0
    ./internal/ModelMapDslIntegrationTest.groovy
Added some test coverage for using `$.p` expressions in closures passed to `Project.model { }`.

These closures are not transformed and this usage should fail. The `Project.model { }` will be removed once the model DSL supports use cases such as 'apply these rules to all projects in the build'.

    • -5
    • +39
    ./internal/transform/NestedModelDslUsageIntegrationSpec.groovy
Replaced a bunch of usages of `$('p')` with `$.p`

    • -8
    • +8
    ./ModelDslCreationIntegrationTest.groovy
  1. … 31 more files in changeset.
Changed the contract of `ExecutionResult.output` and `error` to return the output with line separators normalized to \n.

    • -3
    • +2
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
  1. … 5 more files in changeset.
Don't need to mess with line separators in this test.

Added some test coverage to verify that nested rule DSL closures cannot see the delegate of outer closures.

    • -0
    • +50
    ./internal/ModelMapDslIntegrationTest.groovy
    • -0
    • +1
    ./internal/transform/ModelDslRuleInputDetectionIntegrationSpec.groovy
Enabled nested DSL rules for the API methods of `ModelMap`.

For example, `all { ... }` or `beforeEach(Type) { ... }`. This means that the closures passed to these methods will be treated as nested rules, with their own inputs, descriptor and so on.

    • -0
    • +143
    ./internal/ModelMapDslIntegrationTest.groovy
  1. … 2 more files in changeset.
Fixed merge problem.

    • -3
    • +1
    ./ModelDslCreationIntegrationTest.groovy