Fix a memory leak in the ViewTester

The following discussion from !729 (merged) should be addressed:

• @graven started a discussion: (+3 comments)

  please avoid 'naked new', and be explicit about ownership, eg. why not auto viewContext = std::make_unique<EventContext>( getContext().evt(), getContext().slot() ); and on transfer of ownership either std::move or .release() explicitly...