groovy
  1. groovy
  2. GROOVY-4310

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

    Details

    • Type: Improvement Improvement
    • Status: Closed 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
    • Number of attachments :
      0

      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

        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?
        Hide
        Guillaume Laforge 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 Laforge 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.
        Hide
        Peter Niederwieser added a comment -

        Merged to 1.7 branch.

        Show
        Peter Niederwieser added a comment - Merged to 1.7 branch.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: