Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Test fixes for strictness.

    • -0
    • +1
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 13 more files in changeset.
Test fixes for strictness.

    • -0
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 17 more files in changeset.
Test fixes for strictness.

    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
  1. … 10 more files in changeset.
Test fixes for strictness.

    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
  1. … 10 more files in changeset.
Test fixes for strictness.

    • -0
    • +2
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 17 more files in changeset.
Test fixes for strictness.

    • -0
    • +2
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 17 more files in changeset.
Change `AbstractIntegrationSpec` to fail when a test runs a build that fails with more than one exception and does not verify the number of exceptions in the failure using `assertHasFailures()`.

This is to avoid additional exceptions unintentionally being introduced, for example when a failure starts being thrown because of configuration cache problems alongside the expected failure.

    • -0
    • +3
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
    • -0
    • +2
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 79 more files in changeset.
Change `AbstractIntegrationSpec` to fail when a test runs a build that fails with more than one exception and does not verify the number of exceptions in the failure using `assertHasFailures()`.

This is to avoid additional exceptions unintentionally being introduced, for example when a failure starts being thrown because of configuration cache problems alongside the expected failure.

    • -0
    • +3
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
    • -0
    • +2
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 79 more files in changeset.
Change `AbstractIntegrationSpec` to fail when a test runs a build that fails with more than one exception and does not verify the number of exceptions in the failure using `assertHasFailures()`.

This is to avoid additional exceptions unintentionally being introduced, for example when a failure starts being thrown because of configuration cache problems alongside the expected failure.

    • -0
    • +3
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
    • -0
    • +2
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 79 more files in changeset.
Add @ToBeFixedForInstantExecution to MetadataSourcesResolveIntegrationTest

because of the GenerateGraphTask in ResolveTestFixture that has a Configuration field

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

    • -0
    • +5
    ./MetadataSourcesResolveIntegrationTest.groovy
Add @ToBeFixedForInstantExecution to ArtifactLookupResolveIntegrationTest

because of the GenerateGraphTask in ResolveTestFixture that has a Configuration field

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

    • -0
    • +2
    ./ArtifactLookupResolveIntegrationTest.groovy
Remove `@ToBeFixedForInstantExecution` from `HttpAuthenticationDependencyResolutionIntegrationTest`

    • -2
    • +0
    ./HttpAuthenticationDependencyResolutionIntegrationTest.groovy
Fix for previous commit.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 15 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 36 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 42 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 36 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 36 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 42 more files in changeset.
Do not write the source paths for a `CopySpec` multiple times to the instant execution cache.

Previously, these were written once when the `CopySpec` was serialized and once when the associated input property spec was serialized.

This commit adds some special case handling for this case. It also preserves the identity of each `FileTree` and `FileCollection` instance for each task to enable short circuiting when the same file collections are referenced via multiple paths, such as in the `CopySpec` case.

    • -3
    • +0
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 42 more files in changeset.
@RequiredFeature can be used as a repeated annotation

If used for a single feature, avoid annotation noise by not using the

composite annotation. This also avoids the confusion that the

@RequiredFeature annotation cannot be used independently

(no compile error but does not work).

I made the @RequiredFeatures annotation package-private as it is

only required by the compiler and the runner now.

Signed-off-by: Benjamin Muskalla <bmuskalla@gradle.com>

    • -4
    • +1
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 43 more files in changeset.
@RequiredFeature can be used as a repeated annotation

If used for a single feature, avoid annotation noise by not using the

composite annotation. This also avoids the confusion that the

@RequiredFeature annotation cannot be used independently

(no compile error but does not work).

I made the @RequiredFeatures annotation package-private as it is

only required by the compiler and the runner now.

Signed-off-by: Benjamin Muskalla <bmuskalla@gradle.com>

    • -4
    • +1
    ./MetadataSourcesResolveIntegrationTest.groovy
  1. … 43 more files in changeset.
Add documentation links for insecure url deprecations

    • -2
    • +2
    ./AbstractRedirectResolveIntegrationTest.groovy
  1. … 10 more files in changeset.
Rename @FailsWithInstantExecution to @ToBeFixedForInstantExecution

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

    • -3
    • +3
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
    • -2
    • +2
    ./HttpAuthenticationDependencyResolutionIntegrationTest.groovy
  1. … 871 more files in changeset.
Annotate integ tests failing with instant execution in various projects

removing most of @IgnoreWithInstantExecution annotations

after fixing the @FailsWithInstantExecution rule

and more ci feedback

also make @IgnoreWithInstantExecution require a reason from a fixed set

and add it to the remaining ignores

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

    • -6
    • +0
    ./AbstractProxyResolveIntegrationTest.groovy
    • -4
    • +0
    ./AbstractRedirectResolveIntegrationTest.groovy
  1. … 125 more files 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>

    • -5
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 36 more files 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>

    • -5
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 38 more files 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>

    • -5
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 37 more files 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>

    • -5
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  1. … 37 more files 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>

    • -5
    • +2
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
  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
    • +3
    ./AbstractHttpsRepoResolveIntegrationTest.groovy
    • -0
    • +6
    ./AbstractProxyResolveIntegrationTest.groovy
    • -0
    • +4
    ./AbstractRedirectResolveIntegrationTest.groovy
  1. … 56 more files in changeset.