SamplesGroovyMultiProjectIntegrationTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Separate samples subproject (#10829)

Also see the discussion in https://github.com/gradle/gradle-private/issues/1663

Previously, `integTest` subproject runs 15 mins, now we split it into 3 jobs: `integTest`/`integTest_2`/`integTest_3`. However, the one containing sample tests is still too slow.

The average build time of that job is 10 minutes. This PR separates the `samples` project so that we can further optimise the build structure (e.g. `samples_2`/`samples_3`).

    • -84
    • +0
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 88 more files in changeset.
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
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 162 more files in changeset.
Use external repository mirrors in build (#6094)

We've been bitten by external repository fluctuation for a long time.

This PR makes most of tests use repository mirrors set up by ourselves

via init script and system property.

There're still some tests not switching to mirrors, which would be fixed

in follow-up commits.

    • -2
    • +2
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 99 more files in changeset.
Remove unnecessary @LeaksFileHandles and fork = true

These were added to work around file leaks in the

in-processor Java/Groovy compilers, which were fixed

in previous commits. This makes the tests faster and

also makes sure that we will spot any future leaks

that could make our user's life harder.

    • -1
    • +1
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 10 more files in changeset.
Use separate output directories for all JVM languages

- Introduce an outputDir on SourceDirectorySet

- Default output directory is now `build/classes/<source directory set name>/<source set name>`

- Example: Java compilation goes to build/classes/java/main instead of build/classes/main

- Adapt JDepend, FindBugs, Test and ValidateTaskProperties tasks to handle multiple class directories

- Deprecate setClassesDir/getClassesDir on SourceSetOutput

- Calling setClassesDir restores old behavior (shared output directory)

- Introduce addClassesDir and getClassesDirs on SourceSetOutput

- OSGi plugin needs a single classes directory, so introduce 'osgiClasses' task that syncs all classes to a single directory

Most of the changes to integration tests are find classes in their new location. Helper methods in AbstractIntegrationSpec

can locate class files vs hardcoding a path.

Squashed commit of sg-split-jvm-classes branch for REVIEW-6502

    • -32
    • +40
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 119 more files in changeset.
Revert "Changing Groovydoc integration tests to run in their own daemon and own gradle user home."

Reverting this for now since there are other issues with this beyond just failed windows CI build.

This reverts commit 94848d9fd76c17517c2104dc86452dd94d330a10.

    • -5
    • +0
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 4 more files in changeset.
Changing Groovydoc integration tests to run in their own daemon and own gradle user home.

+review REVIEW-5677

    • -0
    • +5
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 4 more files in changeset.
Forcefully clean up test temp directories and mark tests for which it fails on Windows with @LeaksFileHandles so that they still pass.

+review REVIEW-5504

    • -0
    • +2
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 97 more files in changeset.
revert accidently changes on integTests

+review REVIEW-5251

    • -3
    • +3
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 7 more files in changeset.
some cleanup on buildcomparison integtests - seperate integtest for nested archives - introduce buildcomparison report fixture

+review REVIEW-5251

    • -3
    • +3
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 57 more files in changeset.
Remove @author tags and names from source code.

- Added checkstyle check for @author

- Added not to CONTRIBUTING.md saying that we don't use names in the codebase

    • -3
    • +0
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 1213 more files in changeset.
Remove RuleHelper.

Use constructor injection instead of reflection.

    • -1
    • +1
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 88 more files in changeset.
Some renaming and cleanup around the test/temp directory used in tests.

    • -1
    • +1
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 308 more files in changeset.
Don't have the GradleDistribution be responsible for providing the working space for a test.

    • -6
    • +3
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 109 more files in changeset.
Rename TestFileContext to TestWorkDirProvider, and don't make TestFile implement it.

    • -2
    • +3
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 63 more files in changeset.
Moved code related to GradleExecuter and GradleDistribution into org.gradle.integtests.fixtures.executer - Affects many, many files, but mostly just imports - Certain files (eg RuleHelper) are not in the correct spot, but were moved to avoid cycles

    • -2
    • +2
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 170 more files in changeset.
Refactoring and restructruization. Moved the samples integ tests into a spearate package for better visibility and runability. Moved test fixture code into the integ test internal subproject.

    • -0
    • +79
    ./SamplesGroovyMultiProjectIntegrationTest.groovy
  1. … 65 more files in changeset.