Wicket
  1. Wicket
  2. WICKET-4027

Consider dropping MockPageManager and use mock IPageStore instead

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Later
    • Affects Version/s: 1.5-RC7
    • Fix Version/s: None
    • Component/s: wicket
    • Labels:
      None

      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.

      1. WICKET-4027-2.patch
        2 kB
        Martin Grigorov

        Issue Links

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              Martin Grigorov
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development