GroovySourceIncrementalCompilationIntegrationTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Rename @ToBeFixedForInstantExecution.value to skip for explicitness

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

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 36 more files in changeset.
Rename @FailsWithInstantExecution to @ToBeFixedForInstantExecution

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

    • -6
    • +6
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 872 more files in changeset.
Move skip reasons to @FailsWithInstantExecution and drop @IgnoreWithInstantExecution

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

    • -2
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 45 more files in changeset.
Annotate test flaky with instant execution

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

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
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>

    • -1
    • +2
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 126 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
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 58 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
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 58 more files in changeset.
Annotate integ tests failing with instant execution in :languageJava

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

    • -0
    • +5
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 34 more files in changeset.
Annotate integ tests failing with instant execution in :languageJava

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

    • -0
    • +5
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 34 more files in changeset.
Document why Groovy recompiles for resource changes

    • -0
    • +5
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
Document why Groovy recompiles for resource changes

    • -0
    • +5
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
Unify messages for non-source file changes

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 4 more files in changeset.
Unify messages for non-source file changes

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 3 more files in changeset.
Fix Groovy compilation tests

Changing a non-Groovy file (resource or .java source) currently causes

a full recompile. The test was wrong before since it changed the

sources of the `javaCompile` task.

    • -0
    • +20
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 3 more files in changeset.
Fail on problems with incremental Groovy compilation

    • -13
    • +6
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 2 more files in changeset.
Avoid merging old mappings on full recompilation (#9964)

This fixes https://github.com/gradle/gradle/issues/9936

Previously even on full recompilation, we merge the old Groovy source-class mappings

into the incremental mappings. This PR avoid doing this on full recompilation.

    • -0
    • +33
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 11 more files in changeset.
Avoid merging old mappings on full recompilation (#9964)

This fixes https://github.com/gradle/gradle/issues/9936

Previously even on full recompilation, we merge the old Groovy source-class mappings

into the incremental mappings. This PR avoid doing this on full recompilation.

    • -0
    • +33
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 11 more files in changeset.
Resolve feedbacks

    • -0
    • +15
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 1 more file in changeset.
Resolve feedbacks

    • -0
    • +16
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 1 more file in changeset.
Resolve feedbacks

    • -0
    • +16
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 1 more file in changeset.
Avoid merging old mappings on full recompilation

This fixes https://github.com/gradle/gradle/issues/9936

Previously even on full recompilation, we merge the old Groovy source-class mappings

into the incremental mappings. This PR avoid doing this on full recompilation.

    • -0
    • +17
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 2 more files in changeset.
Avoid merging old mappings on full recompilation

This fixes https://github.com/gradle/gradle/issues/9936

Previously even on full recompilation, we merge the old Groovy source-class mappings

into the incremental mappings. This PR avoid doing this on full recompilation.

    • -0
    • +17
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 2 more files in changeset.
Full recompilation on Groovy-Java joint compilation (#9890)

Currently, incremental Groovy compilation only works for files compiled by the Groovy compiler. Java files are compiled by the Java compiler.

This PR disables incremental compiler if Java files found because this might cause incorrectness.

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 5 more files in changeset.
Fix failed tests

    • -1
    • +1
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 1 more file in changeset.
Follow-ups of incremental Groovy compilation (#9848)

This PR:

- Closes https://github.com/gradle/gradle/issues/9785 and https://github.com/gradle/gradle/issues/9380

- Adds tests for

- Multiple classes in a same source Groovy file.

- Moving files between source set roots

Previously, removing a source directory from a source set would break the Java (and Groovy) incremental compiler (https://github.com/gradle/gradle/issues/9380). This PR detects this case and runs a full recompilation.

It also records relative path instead of absolute path in Groovy incremental compilation to make it build-cache-friendly.

    • -0
    • +64
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 27 more files in changeset.
Fix deprecation warning

    • -0
    • +24
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 3 more files in changeset.
Fix deprecation warning

    • -0
    • +25
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 3 more files in changeset.
Tests for followups of incremental Groovy compilation

    • -0
    • +39
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 5 more files in changeset.
Test for fixing issue of removing source dirs

    • -0
    • +58
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 2 more files in changeset.
Test for fixing issue of removing source dirs

    • -0
    • +58
    ./GroovySourceIncrementalCompilationIntegrationTest.groovy
  1. … 4 more files in changeset.