WrapperHttpIntegrationTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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. … 162 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 286 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 283 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 286 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 284 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 285 more files in changeset.
Use 'implementation' dependencies everywhere

  1. … 285 more files in changeset.
Remove flaky tests on release

Rework `BlockingHttpServer` API to separate expected HTTP request from the action that should be performed when the request is received. Also remove usages of `HttpExchange` from tests (but not fixtures) as this is an internal detail of the fixture.

    • -12
    • +11
    ./WrapperHttpIntegrationTest.groovy
  1. … 22 more files in changeset.
Remove dependency on Jetty from `WrapperHttpIntegrationTest`

    • -47
    • +26
    ./WrapperHttpIntegrationTest.groovy
Ignore flaky WrapperHttpIntegrationTest on Java 10 for now

Ignore flaky test on Java 10 for now

Use a separated TestPrecondition.OLD_JETTY_COMPATIBLE to mark Jetty issue

See https://github.com/gradle/gradle-private/issues/1537

Old Jetty version seems to be incompatible with Java 10. This commit uses a

separated annotation to mark the affected tests.

  1. … 3 more files in changeset.
Ignore some tests on Java 10 for now

See https://github.com/gradle/gradle-private/issues/1537

Java 10 seems not to be quite compatible with Jetty, which fails frequently recently.

Now we ignore it temporarily before we can upgrade Jetty to higher version.

  1. … 1 more file in changeset.
Preliminary support for building Gradle on Java 9 (#5811)

This is a follow-up of https://github.com/gradle/gradle/pull/5749 . #5749 introduced Java 9 compiler but broke IDE import. This PR aims at fixing IDE import. With this PR, we can run `./gradlew idea` and set project SDK to JDK 9 to run integration tests.

The changes are:

- Upgrade javaassist to JDK9-compatible version

- Adds empty implementation to `AnnotationProcessingCompileTask` and `ResourceCleaningCompilationTask`

- Remove support of Java 5 because Java 9 can't generate Java 5 bytecode any more.

- Slightly increased wrapper size limitation because two extra classes are added.

    • -20
    • +0
    ./WrapperHttpIntegrationTest.groovy
  1. … 21 more files in changeset.
Fix wrapper usages of output test fixtures

    • -11
    • +11
    ./WrapperHttpIntegrationTest.groovy
  1. … 1 more file in changeset.
Fix int test for changes to test fixtures.

Change int tests to use relevant fixture methods to express their expectations about the build log output.

  1. … 28 more files in changeset.
Upgrade Jetty to 6.1.26 to fix file handle leaks

Tests using both HttpServer and TestKeyStore fixtures to setup a TLS

enabled HTTP server leak file handles because Jetty is not releasing

the keystore files.

This minor Jetty version includes the proper fix:

- http://dev.jetty.codehaus.narkive.com/rDyeZ8st/jetty-dev-jira-created-jetty-1292-sslsocketconnector-createfactory-leaks-file-handles-when-reading

- http://jetty.4.x6.nabble.com/Jetty-release-6-1-26-td45150.html

  1. … 8 more files in changeset.
Use doLast instead of left shift operator

Made this change in preparation for deprecating the left shift operator.

+review REVIEW-6236

  1. … 163 more files in changeset.
Refine WrapperHttpIntegrationTest

+review REVIEW-6222

Fix and re-enable Wrapper HTTP Basic Auth tests for Java 9

JAXB DatatypeConverter is not available in Java 9 Jigsaw without adding

a module.

Starting from this commit, we use it only on Java 6 and 7.

And we use java.util.Base64 starting from Java 8.

+review REVIEW-6222

  1. … 2 more files in changeset.
Set SSL properties on the wrapper client JVM explicitly so that tests pass in daemon mode

+review REVIEW-6222

Ignore a bunch of tests on Java 9 for now

+review REVIEW-6222

Fix Wrapper HTTP Basic Auth on Java 5 integration test

No request expected on test server.

Disable stacktrace checks in the executer.

Assert on stderr.

+review REVIEW-6222

Wrapper downloads distributions using HTTP Basic Authentication

+review REVIEW-6222

    • -0
    • +149
    ./WrapperHttpIntegrationTest.groovy
  1. … 5 more files in changeset.
Fixed all wrapper tests not to leak file handles by killing daemons

  1. … 9 more files in changeset.
Extracted all the various ways that int tests set up and run the wrapper into a single `AbstractWrapperIntegrationSpec` superclass.

    • -17
    • +4
    ./WrapperHttpIntegrationTest.groovy
  1. … 9 more files in changeset.
Fix proxy tests on Java 1.6

+review REVIEW-5736

  1. … 1 more file in changeset.
Fix wrapper download proxy tests

+review REVIEW-5736