GlobalLoggingManipulationIntegrationTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Change the tooling API provider to route logging to the client stdout and stderr streams in a similar way to the command-line client.

When not producing rich output the tooling API writes errors to the stderr stream (same as when the command-line client is not run from a console), and when producing rich output the errors are written to the stdout stream (same as when the command-line client is run from a console).

    • -8
    • +12
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 14 more files in changeset.
Increase timeout

Test may take longer under heavy load.

    • -1
    • +1
    ./GlobalLoggingManipulationIntegrationTest.groovy
Use an isolated tooling client for integrations to close

Some tests require that the tooling client be closed or reset, either

to verify the behaviour on close or to ensure that certain sticky state

is reset. Previously, these tests were interfering with other,

unrelated tests.

These tests now use an isolated tooling client, so that they don't

impact the static state of the shared tooling client used by the rest

of the integration tests.

Flaky test: gradle/gradle-private#480

    • -7
    • +5
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 4 more files in changeset.
Temporarily ignore another disruptive TAPI test

This test is calling `ConnectorServices.reset()`, which could result in

stop commands being sent to daemons in unrelated tests.

    • -0
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
Temporarily ignore another disruptive TAPI test

This test is calling `ConnectorServices.reset()`, which could result in

stop commands being sent to daemons in unrelated tests.

    • -0
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
Make tests that close the TAPI (via reset()) require an isolated daemon

+review REVIEW-6310

    • -0
    • +1
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 1 more file in changeset.
Reinstated the contract for `LoggingManagerInternal.captureSystemSources()` and `LoggingServiceRegistry.newEmbeddableLogging()`, so that JUL is not touched until `captureSystemSources()` is called.

Reworked the internals to separate the log level for a particular logging system from whether or not the logging system should be generating events or now.

    • -10
    • +85
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 16 more files in changeset.
Disable failing tests temporarily

+review REVIEW-6255

    • -0
    • +2
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 2 more files in changeset.
Changed embedded logging services so that they do not configure Java util logging and added `LoggingManagerInternal.captureSystemSources()` to allow this to be switched on. Changed the TAPI embedded mode to use this, and added some test coverage.

    • -0
    • +6
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 12 more files in changeset.
Changed TAPI embedded mode execution so that System.out and System.err are captured during build execution and restored afterwards. Doesn't deal well with concurrent TAPI operations, but embedded mode is not intended to be used concurrently.

    • -2
    • +0
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 4 more files in changeset.
Temporarily changed tapi to hijack System.out and System.err again, to get logging capture for testkit debug mode working again. Still pending a real fix for this.

    • -1
    • +4
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 3 more files in changeset.
Changed tapi provider so that it does not replace System.out or System.err. This means logging capture is temporarily broken in embedded mode.

    • -3
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 3 more files in changeset.
Check treatment of System.in when tapi is in embedded mode.

    • -0
    • +2
    ./GlobalLoggingManipulationIntegrationTest.groovy
Change logging expectation for tooling api. Standard output/error should now be captured.

+review REVIEW-5615

    • -5
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
Don't have the GradleDistribution be responsible for providing the working space for a test.

    • -2
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 109 more files in changeset.
Added RedirectStdIn test rule, to restore System.in at the end of the test.

    • -1
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 1 more file in changeset.
Remove org.gradle.tooling.model.BuildableProject.

    • -3
    • +3
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 7 more files in changeset.
moved tooling API integration tests from integTest to toolingApi project moved @Requires and related classes from native to internalTesting project

    • -0
    • +83
    ./GlobalLoggingManipulationIntegrationTest.groovy
  1. … 82 more files in changeset.