Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Make it possible to use an ecosystem describer in more cases

Before this commit the describer would only be used if the same set of attributes

was found. This means that if the consumer added, or removed, one attribute, we

would lose the benefit of better user error messages. With this change, we try

to find the _best matching_ describer, if any.

    • -1
    • +1
    ./api/reporting/model/ReportNode.groovy
  1. … 20 more files in changeset.
Make it possible to use an ecosystem describer in more cases

Before this commit the describer would only be used if the same set of attributes

was found. This means that if the consumer added, or removed, one attribute, we

would lose the benefit of better user error messages. With this change, we try

to find the _best matching_ describer, if any.

    • -1
    • +1
    ./api/reporting/model/ReportNode.groovy
  1. … 19 more files in changeset.
Change a few Groovy HTTP links in docs to HTTPS

    • -1
    • +1
    ./api/reporting/model/ReportNode.groovy
  1. … 17 more files in changeset.
Fix samples tests for Java 10

Illegal access warnings get grouped, causing extra lines in the

actual samples output. Furthermore, play component report was not

properly normalized. This change adds normalization for the Java

version string.

  1. … 6 more files in changeset.
Remove download message in play tests to avoid flakiness (#5267)

This is an attempt to fix https://github.com/gradle/gradle-private/issues/1238.

This flaky test can be reproduced by removing `<projectDir>/intTestHomeDir/worker-1/caches`.

If caches is empty when executing this test, extra `Downloading http://xxx.jar` will appear in test log, failing the test with unmatched log failure.

This PR removes the downloading message from test log.

In addition, this PR fixes an issue in `GroupedOutputFixture.java`: previously, if the test output is `\n\t\tOutput\n`, it will be trimmed to `Output` - this is not right. Only empty lines should be trimmed, so the correct trimmed output should be `\t\tOutput`.

  1. … 1 more file in changeset.
Remove spurious newline from tests that check output

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 54 more files in changeset.
Change int tests to use relevant fixture methods to express their expectations about the build log output.

  1. … 22 more files in changeset.
Introduce several methods on `ExecutionResult` and `ExecutionFailure` to allow a test to express various expectations about the logging output of the build, rather than scraping the `output` and `errorOutput` properties in the test.

Change several tests to use these fixture methods, plus fix several other tests for changes to the logging output.

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 20 more files in changeset.
Fix some tests that check output

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 8 more files in changeset.
Use common output chain for grouping and header generation

This allows us to group logs for print streams like stdout and stderr.

Use PrettyPrefix header for all output chains, making our output

consistent between rich and plain console modes.

Adjust functional test coverage to accomodate for new task header

  1. … 111 more files in changeset.
Fix some integration tests

  1. … 2 more files in changeset.
Group logs in all environments

Allow renderer to specify header formatter and whether all

log headers should be rendered and use that for standard streams

output (e.g. plain console, piped output, etc.)

Set build operation display status for completed tasks

Build Operations can set a status through the build operation

context. Status is forwarded on through the build operation

executor upon completion.

Replace TaskExecutionLogger with progress logs generated from

build operation events through the task graph executer.

Issue: #1985

  1. … 26 more files in changeset.
Report task outcome statistics after build finishes

This changes the format of the Build Result to have consistent

duration display to the console.

- Move TerseDurationFormatter and reuse it for BuildResultLogger.

Inject a DurationFormatter into the BuildResultLogger with a

concrete TersePrettyDurationFormatter impl.

- Remove cache stats reporting

- Fix tests that scrape build result message

Issue: #1639

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 37 more files in changeset.
Changed a number of int tests to make assertions using the normalized build output rather than the full build output.

    • -3
    • +8
    ./api/reporting/model/ModelReportParser.groovy
  1. … 10 more files in changeset.
Introduce data only model report

Story: gradle/langos#153

    • -0
    • +11
    ./api/reporting/model/ModelReportParser.groovy
  1. … 3 more files in changeset.
Remove unused import

Fixes component report tests on Windows when dependencies are resolved during the test

Removed a bunch of duplication from various component report int tests.

  1. … 6 more files in changeset.
Fixed component report fixtures to deal with the new binary locations. Could be reimplemented to be reliable later.

Fixed `AbstractComponentReportIntegrationTest` for changes to normalization of build output.

Allow LanguageSourceSet instances to be attached to a managed type

- Fix model report parsing issue

+review REVIEW-5673

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 4 more files in changeset.
Removed the `+ model` node from model report.

    • -11
    • +8
    ./api/reporting/model/ModelReportParser.groovy
  1. … 3 more files in changeset.
Show the includes for the source set's filter in the component report.

  1. … 4 more files in changeset.
Show play platform as `@Variant` in component report

+review REVIEW-5596

  1. … 6 more files in changeset.
Fix output samples for component reports

  1. … 3 more files in changeset.
Show native binary variants in component report

+review REVIEW-5596

    • -0
    • +23
    ./api/reporting/components/AbstractNativeComponentReportIntegrationTest.groovy
  1. … 10 more files in changeset.
Fixing broken build due to JVM component version mismatch

+review REVIEW-5567

  1. … 1 more file in changeset.
Model report displays a relative path to the build script for DSL based rules - Moves RelativeFileNameTransformer to core

+review

    • -1
    • +1
    ./api/reporting/model/ModelReportParser.groovy
  1. … 19 more files in changeset.
Shows more concise names for rule source method rules

+review

    • -0
    • +3
    ./api/reporting/model/ModelReportOutput.groovy
    • -2
    • +20
    ./api/reporting/model/ModelReportParser.groovy
  1. … 31 more files in changeset.
Changes test utility to do a full text assertion on model nodes

    • -1
    • +1
    ./api/reporting/model/ModelReportOutput.groovy