[ 
https://issues.apache.org/jira/browse/WICKET-2090?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12678054#action_12678054
 ] 

Clint Popetz commented on WICKET-2090:
--------------------------------------

Ugh, I spoke to soon.  Running our full test suite shows that setRequestTarget 
isn't always called.  For example, if one does formTester.submit(), the trace 
is:

submit -> RequestCycle.processRequestCycle -> request -> steps() -> step() 

which does this on RESOLVE_TARGET:

                                        // Add (inserting at the bottom) in 
case before or during
                                        // target resolving one or more request 
targets were pushed
                                        // on the stack before this. If that is 
the case, they
                                        // should be handled before this
                                        requestTargets.add(0, target);

and never calls setRequestTarget.  I'm willing to offer a patch on this, but 
this code looks rather important and I hesitated to do so without a better 
grasp of the ways the request cycle is used.  

> Need reliable hook for storing/restoring data to/from page metadata that is 
> tes compatbile
> ------------------------------------------------------------------------------------------
>
>                 Key: WICKET-2090
>                 URL: https://issues.apache.org/jira/browse/WICKET-2090
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.3.5, 1.4-RC1
>            Reporter: Clint Popetz
>         Attachments: wicket-2090.txt
>
>
> Requesting a hook to reliable store/restore data to/from page metadata 
> before/after requests that works with WicketTester.  Currently I'm using 
> WebRequestCycleProcessor.resolve() and and respond(), but 
> BaseWicketTester.clickLink() doesn't run processRequestCycle before executing 
> the onClick() handler, so this doesnt' work for testing ajax components.
> Original post:  
> http://www.nabble.com/Proper-hook-for-conversational-state-that-also-works-with-tests-td21876375.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to