Re #7216. Tidy up, removing unused code.
Re #7216. Only do the work creating a workspace when it's needed.

The setUp method would be getting called before every single test
whether they needed it or not. This just does this relatively expensive
work where it's actually used.
Re #7216. Add test for output workspace same as input one.

testExec_Events_inplace fails right now of course.
Re #7216. Fix for inplace normalisation of event workspaces.
Re #7216. Best practice is to pass shared_ptr by const ref if poss.
Merge branch 'feature/7216_normalisetomonitor_crash' into develop