[WICKET-4027] Consider dropping MockPageManager and use mock IPageStore instead

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Later
• Affects Version/s: 1.5-RC7
• Component/s: wicket
• Assignee: Unassigned
• Reporter: Martin Tzvetanov Grigorov
• Created: 05/Sep/11 14:46
• Updated: 17/Mar/14 12:11
• Resolved: 17/Mar/14 12:11

Description

Currently WicketTester setups custom IPageManager (MockPageManager) which has several problems:
1) it saves stateless pages - touchPage(statelessPage) shouldn't be stored
2) it calls #detach() for all pages ever stored in the map. it should call it only on the pages stored in the last request.

If we fix these problems then we are going to reimplement the default IPageManager (StorePageManager). So I suggest to drop MockPageManager and setup inmemory IPageStore behind it instead. This way we can test PageStoreBehavior as well.

Issue Links

• is duplicated by: WICKET-4376 MockPageManager touchPage: no guts, no glory