TarTaskOutputPackerTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Do not restore file dates when unpacking task outputs (#2667)

  1. … 5 more files in changeset.
Use UTF-8 when packing TAR

+review REVIEW-6559

  1. … 1 more file in changeset.
Polish TAR archiving of task outputs

    • -65
    • +34
    ./TarTaskOutputPackerTest.groovy
  1. … 8 more files in changeset.
Resolve the output files for the cacheable properties of a task once

Calling `getOutputFile` on a CacheableTaskOutputFilePropertySpec is potentially expensive

since we'll go through layers of convention mapping, providers and callbacks.

Since we were calling this in the TaskOutputPacker for each file in an output directory,

we were resolving the output properties hundreds of thousands of times in large builds.

+review REVIEW-6532

    • -26
    • +19
    ./TarTaskOutputPackerTest.groovy
  1. … 9 more files in changeset.
Simplify TaskOutputPacker interface

This allows tests to avoid having to construct a TaskOutputs mock

+review REVIEW-6500

    • -67
    • +44
    ./TarTaskOutputPackerTest.groovy
  1. … 5 more files in changeset.
Fix tests even more

+review REVIEW-6477

    • -14
    • +14
    ./TarTaskOutputPackerTest.groovy
Fix tests

+review REVIEW-6477

    • -14
    • +15
    ./TarTaskOutputPackerTest.groovy
  1. … 1 more file in changeset.
Polish TarTaskOutputPacker

+review REVIEW-6447

  1. … 1 more file in changeset.
Remove previously present output files when unpacking cached result

We currently remove any pre-existing output when unpacking a missing @OutputDirectory. This commit makes sure we do the same with missing @OutputFile properties, too.

+review REVIEW-6447

    • -14
    • +20
    ./TarTaskOutputPackerTest.groovy
  1. … 1 more file in changeset.
Do not create output directory for cached result when there was none

We used to blindly create output directories for every @OutputDirectory, even if after the task’s original execution the directory did not exist. We now restore the original state in these cases.

+review REVIEW-6447

  1. … 4 more files in changeset.
Move types back to `org.gradle.api.internal.tasks`

We moved some types to `org.gradle.api.internal.tasks` and it broke binary compatibility, because they leaked through to public APIs.

+review REVIEW-6403

  1. … 53 more files in changeset.
Move everything build cache related to `org.gradle.caching`

+review REVIEW-6396

    • -0
    • +225
    ./TarTaskOutputPackerTest.groovy
  1. … 89 more files in changeset.