groovy

ProxyMetaClass.use method should return the result of the closure call

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.6-rc-2, 1.5.8, 1.7-beta-1
  • Component/s: None
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    2

Description

println "money proxy with method access : ${moneyProxy {1.euro() + 2.euro()}}"

this method prints null where I would expect it printed result of the closure.
moneyProxy is a static method which calls the ProxyMetaClass.use method ...

attached a patch...

  1. Text File
    interceptorTestPatch.txt
    19/Jan/09 2:35 AM
    0.4 kB
    Murat Can ALPAY
  2. Text File
    patchfile.txt
    16/Jan/09 4:18 AM
    1 kB
    Murat Can ALPAY

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Guillaume Laforge added a comment -

These PMC#use() methods are used in MockFor and StubFor.
Shouldn't the patch also affect the use() method available on MockFor and StubFor?
And provide an example test as well.

Show
Guillaume Laforge added a comment - These PMC#use() methods are used in MockFor and StubFor. Shouldn't the patch also affect the use() method available on MockFor and StubFor? And provide an example test as well.
Hide
Permalink
Murat Can ALPAY added a comment -

I have attached a patch InterceptorTest class which checks for the return value of the closure. For MockFor, and StubFor I am not sure how they are used yet...

Show
Murat Can ALPAY added a comment - I have attached a patch InterceptorTest class which checks for the return value of the closure. For MockFor, and StubFor I am not sure how they are used yet...
Hide
Permalink
Guillaume Laforge added a comment -

Thanks for the patch!

Show
Guillaume Laforge added a comment - Thanks for the patch!

People

  • Assignee:
    Guillaume Laforge
    Reporter:
    Murat Can ALPAY
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: