DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Revert to ungrouped plain output

Only the stderr output event listener is getting the

EndOutputEvent which caused JVMs to be left open (as evidenced

by the failure of ToolingApiIntegrationTest).

Continue use of ungrouped ProgressLogEventGenerator and find a

better way to do this in a future version of Gradle.

    • -0
    • +199
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 13 more files in changeset.
Rename network request build operation and drop unused attributes.

    • -199
    • +0
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 7 more files in changeset.
Removed unused `ExternalResource.isLocal()`

    • -6
    • +1
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 6 more files in changeset.
Further reworking external resource APIs to remove `ExternalResourceRepository.getResource()` and corresponding `ExternalResource.close()`.

Now, all `ExternalResource` implementations are live, rather than some implementations being live and some being an awkward one shot instance.

    • -6
    • +6
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 20 more files in changeset.
Started reworking `ExternalResourceRepository` so that no network request spans more than one method call on `ExternalResourceRepository` or `ExternalResource`.

This is work in progress and this change leaves `ExternalResourceRepository` in an intermediate state between the old and new behaviours. A subsequent change will remove the old behaviours entirely.

The semantics of the `ExternalResourceDownloadBuildOperationType` have changed in several ways:

- The build operation wraps the entire network request, rather than the "download" portion of the request.

- The build operation events are fired regardless of whether the network request was successful or not.

- No content type or content length are included in the operation's detail.

This build operation type is yet to be refactored or documented to reflect these changes.

    • -5
    • +15
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 10 more files in changeset.
Avoid filesystem in test.

    • -8
    • +2
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
Reorganise the rich build operation details/results type to further decouple from producer side semantics.

    • -0
    • +200
    ./DownloadBuildOperationFiringExternalResourceDecoratorTest.groovy
  1. … 82 more files in changeset.