Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Apply input/output annotations

    • -1
    • +40
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 3 more files in changeset.
Align terminology with Jacoco configuration options

Makes it easier for users to map configuration options to Jacoco documentation. Aligns the configuration options with Maven plugin and Ant tasks.

    • -46
    • +46
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 14 more files in changeset.
Use String data type instead of enum

Allows for better forward and backward capability in case JaCoCo decides to introduce new values or change existing ones.

    • -16
    • +8
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 12 more files in changeset.
Rename field

    • -6
    • +6
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 10 more files in changeset.
Renamed fields to better express intention

    • -12
    • +12
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 15 more files in changeset.
Renamed property

    • -1
    • +1
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 4 more files in changeset.
Rename of method

    • -5
    • +5
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
Use strong typing for enums

    • -8
    • +16
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
Rules can be checked even if all report formats are disabled

    • -0
    • +25
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
Expose configuration option for disabling rules

    • -0
    • +23
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
  1. … 3 more files in changeset.
Break out method

Add test case for multiple rules

    • -0
    • +23
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
Add test cases for multiple report tasks

    • -20
    • +82
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
Basic support for enforcing Jacoco metrics

    • -14
    • +4
    ./plugins/JacocoCachingIntegrationTest.groovy
    • -0
    • +212
    ./plugins/JacocoPluginCheckCoverageIntegrationTest.groovy
    • -19
    • +3
    ./plugins/JacocoPluginIntegrationTest.groovy
    • -14
    • +5
    ./plugins/JacocoReportRelocationIntegrationTest.groovy
    • -16
    • +5
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
    • -24
    • +8
    ./plugins/JacocoVersionIntegTest.groovy
    • -0
    • +71
    ./plugins/fixtures/JavaProjectUnderTest.groovy
  1. … 12 more files in changeset.
Rename LocalTaskCacheFixture to LocalBuildCacheFixture

+review REVIEW-6392

    • -4
    • +4
    ./plugins/JacocoCachingIntegrationTest.groovy
  1. … 14 more files in changeset.
Rename withCache -> withTaskCache

To distinguish the task cache from the many

other caches we have.

+review REVIEW-6338

    • -2
    • +2
    ./plugins/JacocoCachingIntegrationTest.groovy
  1. … 9 more files in changeset.
Make testing the task output cache easier

Instead of relying on subclassing I added a

trait which can be mixed into tests which want

to exercise the cache.

+review REVIEW-6338

    • -4
    • +5
    ./plugins/JacocoCachingIntegrationTest.groovy
  1. … 10 more files in changeset.
Allow 2% difference in Jacoco execution data

+review REVIEW-6301

    • -4
    • +4
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
Ignore Jacoco tests on Java 9

+review REVIEW-6301

    • -0
    • +4
    ./plugins/JacocoCachingIntegrationTest.groovy
    • -0
    • +4
    ./plugins/JacocoReportRelocationIntegrationTest.groovy
    • -0
    • +3
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
Use Levenshtein distance to compare the results

This is actually a much more precise measurement.

+review REVIEW-6301

    • -39
    • +11
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
  1. … 1 more file in changeset.
Hopefully finally fix flaky test here

It appears that execution data can also differ in the length of the header, so we trim it down to size if that happens.

+review REVIEW-6301

    • -11
    • +29
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
Fix test

No need to convert to string here.

+review REVIEW-6301

    • -1
    • +1
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
More robust comparison for Jacoco execution data

Let's say 0.5% difference is not a big deal, and we got rid of differences generated by timestamps and random IDs.

+review REVIEW-6301

    • -3
    • +28
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
  1. … 1 more file in changeset.
Ignore even more of the header of Jacoco execution data

Looks like we have some volatile data at the end of the file as well.

+review REVIEW-6301

    • -3
    • +2
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
Ignore more of the header of Jacoco execution data

TeamCity agents have longer host names. Dropping the first 256 bytes leaves us with quite a bit of data to compare.

+review REVIEW-6301

    • -4
    • +8
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
Better tests for Jacoco, fix execution data path sensitivity

Jacoco doesn't actually care about the name of the execution data files.

+review REVIEW-6301

    • -70
    • +0
    ./plugins/JacocoRelocationIntegrationTest.groovy
    • -0
    • +70
    ./plugins/JacocoReportRelocationIntegrationTest.groovy
    • -0
    • +66
    ./plugins/JacocoTestRelocationIntegrationTest.groovy
  1. … 2 more files in changeset.
    • -0
    • +7
    ./plugins/JacocoRelocationIntegrationTest.groovy
  1. … 1 more file in changeset.
Actually cache Jacoco results

We haven't stored them previously. Now we have an integration test to make sure.

+review REVIEW-6301

    • -0
    • +64
    ./plugins/JacocoCachingIntegrationTest.groovy
  1. … 1 more file in changeset.
Make sure test doesn't reuse previous output

+review REVIEW-6301

    • -0
    • +1
    ./plugins/JacocoRelocationIntegrationTest.groovy
Make Jacoco tasks relocatable

+review REVIEW-6301

    • -0
    • +62
    ./plugins/JacocoRelocationIntegrationTest.groovy
  1. … 2 more files in changeset.