groovy

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

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: