Groovy
  1. Groovy
  2. GROOVY-4310

Power asserts should use DefaultGroovyMethods.toString() instead of 'plain' toString()

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7.4, 1.8-beta-1
    • Fix Version/s: 1.7.4, 1.8-beta-1
    • Component/s: Compiler
    • Labels:
      None

      Description

      DefaultGroovyMethods.toString() would lead to Groovyier output; e.g. maps would be printed as "[a:1, b:2]" instead of "

      {a=1, b=2}

      ".

        Activity

        Peter Niederwieser created issue -
        Peter Niederwieser made changes -
        Field Original Value New Value
        Priority Major [ 3 ] Minor [ 4 ]
        Hide
        Peter Niederwieser added a comment -

        I'm assuming everyone is happy with this change and hence have committed it (rev. 20477). Should this also go into the 1.7 branch?

        Show
        Peter Niederwieser added a comment - I'm assuming everyone is happy with this change and hence have committed it (rev. 20477). Should this also go into the 1.7 branch?
        Peter Niederwieser made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Guillaume Delcroix added a comment -

        Unless you think people check the content of the assert messages (which I hope they don't), I think it's safe to havet he change in 1.7 as well.

        Show
        Guillaume Delcroix added a comment - Unless you think people check the content of the assert messages (which I hope they don't), I think it's safe to havet he change in 1.7 as well.
        Peter Niederwieser made changes -
        Fix Version/s 1.7.4 [ 16563 ]
        Affects Version/s 1.7.4 [ 16563 ]
        Hide
        Peter Niederwieser added a comment -

        Merged to 1.7 branch.

        Show
        Peter Niederwieser added a comment - Merged to 1.7 branch.
        Paul King made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
        Mark Thomas made changes -
        Workflow jira [ 12733253 ] Default workflow, editable Closed status [ 12745084 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
        Mark Thomas made changes -
        Workflow jira [ 12970881 ] Default workflow, editable Closed status [ 12978650 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        33m 24s 1 Peter Niederwieser 19/Jul/10 15:27
        Resolved Resolved Closed Closed
        3d 3h 4m 1 Paul King 22/Jul/10 18:32

          People

          • Assignee:
            Peter Niederwieser
            Reporter:
            Peter Niederwieser
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development