Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
TestKit must use a user home for the reduced distribution it runs with

If a test that uses TestKit does not have the 'full' distribution

available, it needs to use the user home dedicated to the distribution

it runs on. And not the default shared test-kit home in the system

temp folder. Otherwise, multiple TestKit tests based on different

distributions might end up sharing a daemon.

  1. … 3 more files in changeset.
Remove ability to run real distributions in embedded mode

This way the full distributions no longer need to be input to

the embedded test tasks.

    • -0
    • +6
    ./dsl/TaskContainerDslIntegrationTest.kt
  1. … 9 more files in changeset.
Never execute a real Gradle distribution in embedded test mode

This will become unsupported such that we do not need to assemble

the distribution for embedded mode anymore. The tests that

are affected by this were checked and are adjusted:

- If the test can run embedded, the restriction that the test requires

a distribution is removed. This is the case for some older tests

that were not checked for a long time and tests that involve TestKit

or KotlinDSL, for which the embedded support is improved in the next

commit.

- If the test only requires a forked process, but not a complete

distribution, it can run a new process based on the current

classpath using 'requiresDaemon().requiresIsolatedDaemon()'.

- If the test needs a real distribution, it is ignored in embedded mode.

    • -2
    • +1
    ./dsl/integration/PluginDependenciesSpecAccessorsIntegrationTest.kt
    • -1
    • +13
    ./dsl/resolver/KotlinScriptDependenciesResolverTest.kt
    • -7
    • +2
    ./dsl/support/ClassBytesRepositoryTest.kt
  1. … 102 more files in changeset.
Remove @ToBeFixed in :kotlinDsl

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

Upgrade JUnit version (#12924)

Upgrade JUnit to 4.13, JUnit platform to 5.6.2

    • -15
    • +15
    ./dsl/TaskContainerDslIntegrationTest.kt
    • -1
    • +1
    ./dsl/integration/LocationAwareScriptEvaluationIntegrationTest.kt
    • -1
    • +1
    ./dsl/resolver/KotlinScriptDependenciesResolverTest.kt
    • -1
    • +1
    ./dsl/support/ClassBytesRepositoryTest.kt
  1. … 322 more files in changeset.
@ToBeFixedForInstantExecution(because = "Kotlin Gradle Plugin")

Kotlin 1.3.61 still reaches out to the project in task actions

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

    • -0
    • +2
    ./dsl/integration/PluginDependenciesSpecAccessorsIntegrationTest.kt
  1. … 3 more files in changeset.
@ToBeFixedForInstantExecution(because = ":reportingTasks use Task.getProject()")

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

  1. … 19 more files in changeset.
Rename @FailsWithInstantExecution to @ToBeFixedForInstantExecution

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

  1. … 870 more files in changeset.
Annotate new tests failing with instant execution after merging master

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

  1. … 1 more file in changeset.
Merge branch 'master' into eskatos/ie/instantIntegTest-enable

  1. … 8 more files in changeset.
Polish `KotlinBuildScriptIntegrationTest`

- Replace `iterable.map(f).joinToString()` by `iterable.joinToString(f)`

Preserve `fileTree(mapOf(...))` behaviour for backward compatibility

See #11335

  1. … 1 more file in changeset.
Annotate integ tests failing with instant execution in various projects

after third round of CI feedback

much better, no more OOMEs, no more hung builds, no more console

replacement weirdness

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

    • -2
    • +0
    ./dsl/integration/PluginDependenciesSpecAccessorsIntegrationTest.kt
  1. … 37 more files in changeset.
Annotate integ tests failing with instant execution in various projects

after first round of CI feedback

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

    • -0
    • +2
    ./dsl/integration/PluginDependenciesSpecAccessorsIntegrationTest.kt
  1. … 58 more files in changeset.
Annotate integ tests failing with instant execution in :kotlinDsl

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

Polish `KotlinSettingsScriptIntegrationTest`

Let compiled Kotlin script templates implement `PluginAware` by delegation

In order to enable `ObjectConfigurationAction` syntax (`apply {... }`)

and to preserve file resolving semantics when using a `PluginAware`

extension such as `PluginAware.apply(from: String? = null, ...)` -

init and settings scripts resolve the applied script file relative to

the applying script.

  1. … 5 more files in changeset.
Replace `Project` interface delegation by `Project` implicit receiver

  1. … 12 more files in changeset.
Remove `Gradle` and `Settings` interface delegation from Kotlin scripts

And take advantage of Kotlin script _implicit receivers_ instead.

    • -24
    • +45
    ./dsl/resolver/KotlinScriptDependenciesResolverTest.kt
  1. … 14 more files in changeset.
Add coverage for conflict with pinned embedded kotlin

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

Polish EmbeddedKotlinProviderTest

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

Pin the whole subgraph of embedded kotlin dependencies

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

  1. … 3 more files in changeset.
Remove the embedded-kotlin-repository

It was based on ClientModule dependencies that caused more troubles than

anything. The main one being that the artificial dependency graph needed

to be kept up to date with each Kotlin version. Moreover since several

Kotlin versions, declaring a repository is necessary in any case. This

change simplifies the code and should make the runtime a bit faster.

However, the Kotlin dependencies when applying the `kotlin-dsl` plugin

are now required to be downloaded. This isn't a big change and should

only impact people using the `kotlin-dsl` plugin without depending on

Kotlin in other parts of their build. This could be alleviated in the

future if dependency resolution considers the Gradle install/distro as a

source of artifacts in the same way it uses maven local.

Along the way, the pinning of Kotlin dependencies to the embedded

version for the build scripts classpath has been moved from a resolution

rule to proper, faster, dependency constraints.

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

  1. … 6 more files in changeset.
Add missing dependency to embedded kotlin

* kotlin-stdlib now depends on kotlin-stdlib-common, this was not

reflected in the setup of the embedded kotlin repo.

* A way to experience this issue was with locking on a kotlin-dsl

project.

Fixes #10697

  1. … 1 more file in changeset.
Build buildSrc after applying the settings file (#10305)

Fixes #9094 and #5333

    • -20
    • +12
    ./dsl/integration/KotlinSettingsScriptIntegrationTest.kt
  1. … 54 more files in changeset.
Merge pull request #10351 from gradle/eskatos/kotlin-dsl/resolver-env-variables

Let Kotlin DSL IJ script resolver pass environment variables

    • -2
    • +17
    ./dsl/resolver/KotlinScriptDependenciesResolverTest.kt
  1. … 1 more file in changeset.
Attempt to make plugins {} work in settings scripts in Kotlin

This is currently not quite working

  1. … 2 more files in changeset.
Attempt to make plugins {} work in settings scripts in Kotlin

This is currently not quite working

  1. … 2 more files in changeset.
Make `SourceDistributionResolver` compatible with an yet to published major.0 version

    • -21
    • +4
    ./dsl/resolver/SourceDistributionResolverIntegrationTest.kt
  1. … 1 more file in changeset.
Adjust `SourceDistributionResolverIntegrationTest` not fail before a major release

By explicitly setting the minimum version to use for resolving the

source distribution to the previous major Gradle version in the test.

    • -1
    • +20
    ./dsl/resolver/SourceDistributionResolverIntegrationTest.kt
  1. … 1 more file in changeset.