ScalaAnnotationProcessingIntegrationTest.groovy

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge remote-tracking branch 'origin/wolfs/fixit/scala-plugin-zinc-update' into sg/60/zinc-upgrade-2

* origin/wolfs/fixit/scala-plugin-zinc-update:

Use forking when running in parallel

Fix IDE test expectations and mention breaking change

Document Zinc compiler upgrade

Add test case for #8421

Recognize contributor in release notes

Don't use static state

Remove unused `javaHome`

upgraded Scala zinc compiler to version 1.2.5

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 25 more files in changeset.
Let scala tests not use deprecated configurations

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 19 more files in changeset.
Let scala tests not use deprecated configurations

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 19 more files in changeset.
Let scala tests not use deprecated configurations

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 19 more files in changeset.
Let scala tests not use deprecated configurations

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 19 more files in changeset.
upgraded Scala zinc compiler to version 1.2.5

Additionally introduces Scala compiler daemon that remains warm across build, improving further incremental compilation times

Signed-off-by: Predrag Knezevic <pedjak@gmail.com>

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 30 more files in changeset.
Ignore annotation processors on compile classpath

Resolves #6296.

    • -9
    • +5
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 17 more files in changeset.
Always evaluate publishing block eagerly

This commit makes `enableFeaturePreview('STABLE_PUBLISHING')`

meaningless by always using the new behavior. The flag is kept for

backward compatibility.

    • -2
    • +0
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 55 more files in changeset.
Always evaluate publishing block eagerly

This commit makes `enableFeaturePreview('STABLE_PUBLISHING')`

meaningless by always using the new behavior. The flag is kept for

backward compatibility.

    • -2
    • +0
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 55 more files in changeset.
Re-enable Play/Scala tests on Java 9+ (#5593)

Almost all Play/Scala tests were ignored on Java 9/10, which makes it lack coverage. This PR does:

- Update Scala version to Java9-compatible version according to [JDK compatibility](https://docs.scala-lang.org/overviews/jdk-compatibility/overview.html).

- Update default Play version to latest 2.6.15, which supports Java 9/10

- Adjust corresponding samples. Previous default Play version 2.3.10 was too old (release at 2014).

- Ignored the tests which can't work on Java 9+.

- Separate Play test resources with different versions. For example, Play 2.6 uses `build.gradle` and `Play 2.4/2.5` uses `build.gradle.old`.

- Previously `AbstractMultiVersionPlayContinuousBuildIntegrationTest` didn't have `targetPlatform` configuration, which made multi-version tests meaningless

- Fix `play-logback` configuration for `2.5.18`.

    • -2
    • +2
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 170 more files in changeset.
Require distribution in less integration tests

No distribution is required in any of the tests changed here.

Requiring a distribution has several drawbacks:

- you need to run intTestImage before testing any change

- the test can't run in the embedded executer, making it slow

- the test is harder to debug, as it's not run in the embedded executer

There are probably more tests that don't really needs this,

but just changing these few should improve turnarounds a bit.

    • -4
    • +0
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 6 more files in changeset.
Nag users only once about stable_publishing flag

This was an oversight, it should have used the deprecation

logger, not a normal logger to print the warning.

    • -0
    • +2
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 27 more files in changeset.
Nag users only once about stable_publishing flag

This was an oversight, it should have used the deprecation

logger, not a normal logger to print the warning.

    • -0
    • +2
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 28 more files in changeset.
Fix scala integration tests

    • -2
    • +2
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 1 more file in changeset.
Name the processors that prevent incremental compilation

Instead of just informing the user that "processors were present".

This can be expanded once we support different types of incremental

processors.

    • -1
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 20 more files in changeset.
Split processor detection out of path computation

    • -3
    • +3
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 12 more files in changeset.
Add annotationProcessor configurations for each SourceSet

And configure the compileJava.options.annotationProcessorPath

to use the configuration when not empty (and use 'null' when

the configuration is empty to preserve the current behavior).

Part of #2300

Signed-off-by: Thomas Broyer <t.broyer@ltgt.net>

    • -1
    • +51
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 21 more files in changeset.
Use mavenCentral() and jcenter() mirrors in tests where possible

    • -3
    • +1
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 51 more files in changeset.
Escape path for Windows

    • -1
    • +3
    ./ScalaAnnotationProcessingIntegrationTest.groovy
Polish Scala annotation processing and add integration tests

    • -0
    • +246
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 4 more files in changeset.
Add integration tests

    • -21
    • +16
    ./ScalaAnnotationProcessingIntegrationTest.groovy
  1. … 1 more file in changeset.
Initial integration testing

    • -0
    • +208
    ./ScalaAnnotationProcessingIntegrationTest.groovy