ContinuousBuildChangeReportingIntegrationTest.groovy

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fixed output when using continuous build and quiet logging together, and added some test coverage for such a thing.

    • -1
    • +15
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 3 more files in changeset.
Ensure that we wait for quiet period in integTest

For continuous build tests that verify detection of changes _during_

build execution, we need to ensure that the build waits long enough

for the changes to be notified. This wait is based on the quiet-period

for file change detection.

Flaky test: gradle/gradle-private#529

    • -8
    • +1
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 2 more files in changeset.
Further attempt to fix flaky test

gradle/gradle-private#447

    • -4
    • +4
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Use an extended quiet period for test, to avoid race

    • -0
    • +6
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 1 more file in changeset.
Attempt to fix flaky test

If the build doesn't take long enough, it's possible that it will

finish without changed files being detected. This change makes the

build take a little longer on Linux, to avoid this race condition.

    • -8
    • +8
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Make continuous integration tests work on OSX using sleeps

This is until we have a proper native implementation of the file watcher service

    • -1
    • +9
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 1 more file in changeset.
Add test for change reporting for changes during the build

+review REVIEW-5788

    • -0
    • +24
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 1 more file in changeset.
Revise test method names

+review REVIEW-5788

    • -4
    • +4
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Simplify test

- use result field instead of local variable

+review REVIEW-5788

    • -20
    • +19
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Fix ContinuousBuildChangeReportingIntegrationTest on MacOSX

- allow order of reported events to vary

- order is different on MacOSX

- don't ignore CREATE events on MacOSX in ChangeReporter

+review REVIEW-5788

    • -8
    • +12
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 1 more file in changeset.
Don't report the number of additional events

+review REVIEW-5788

    • -45
    • +18
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 3 more files in changeset.
Fix codenarc

+review REVIEW-5788

    • -17
    • +17
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Simplify test case

+review REVIEW-5788

    • -14
    • +8
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Refactor test: simplify assertions

+review REVIEW-5788

    • -73
    • +113
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Test only with single task

+review REVIEW-5788

    • -104
    • +62
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Ignore failing tests for continuous build reporting changes

    • -0
    • +2
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Use regexp pattern to check integration test output

+review REVIEW-5788

    • -21
    • +51
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Refactor change reporting to use FileWatcherEventListener

- improve efficiency of change reporting

- use callback approach instead of returning all events that occurred

+review REVIEW-5788

    • -1
    • +1
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 8 more files in changeset.
Refactor test

- make changes after the first build has finished

+review REVIEW-5788

    • -70
    • +21
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
Implement reporting changes for continuous build

+review REVIEW-5788

    • -0
    • +264
    ./ContinuousBuildChangeReportingIntegrationTest.groovy
  1. … 6 more files in changeset.