Uploaded image for project: 'Groovy'
  1. Groovy
  2. GROOVY-8114

Include correct time unit in timeout error message

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4.9
    • Fix Version/s: 2.5.0-beta-1
    • Component/s: None
    • Labels:
      None
    • Flags:
      Patch

      Description

      When a class is annotated with @TimedInterrupt(value = 5L, unit = TimeUnit.SECONDS), and the execution is aborted, the error message reads "Execution timed out after 5 units". It should read "Execution timed out after 5 seconds".

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user paulk-asert opened a pull request:

          https://github.com/apache/groovy/pull/549

          GROOVY-8114: Include correct time unit in timeout error message

          PR for patch as supplied by Jochen Kemnade.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/paulk-asert/groovy groovy8114

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/groovy/pull/549.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #549


          commit 04cacc9b61a12730b6692b5c94b2d0981d6e9874
          Author: paulk <paulk@asert.com.au>
          Date: 2017-05-25T06:32:11Z

          GROOVY-8114: Include correct time unit in timeout error message


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user paulk-asert opened a pull request: https://github.com/apache/groovy/pull/549 GROOVY-8114 : Include correct time unit in timeout error message PR for patch as supplied by Jochen Kemnade. You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert/groovy groovy8114 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/groovy/pull/549.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #549 commit 04cacc9b61a12730b6692b5c94b2d0981d6e9874 Author: paulk <paulk@asert.com.au> Date: 2017-05-25T06:32:11Z GROOVY-8114 : Include correct time unit in timeout error message
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/groovy/pull/549

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/groovy/pull/549
          Hide
          paulk Paul King added a comment -

          Patch applied. Thanks for the improvement!

          Show
          paulk Paul King added a comment - Patch applied. Thanks for the improvement!

            People

            • Assignee:
              paulk Paul King
              Reporter:
              jkemnade Jochen Kemnade
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development