Re #8357. Eliminate valgrind invalid write warning.

By making sure to unregister the observer at the end of the test.
Re #8357. Fix leak in test by breaking cyclic reference.

The BoxController & BoxControllerDummyIO objects were each holding a
shared_ptr back to the other.
Re #8357. Fix memory leaks in tests.

By creating object on the stack instead of unnecessarily on the heap.
Re #8357. Remove leak in test by adding components to instrument.

The fact that you have to call two methods in succession to get this
right is probably not great design. Nor is the fact that you can
'mark' something that isn't in the instrument without any error or warning.
Re #8357. Eliminate memory leaks in tests.

By calling delete where appropriate.
Re #8357. Eliminate memory leak in test.

By removing group from within group to eliminate cyclic shared_ptr
reference. Possibly there's a real potential issue here, but would
anyone really ever put a group within a group?
Re #8357. Update to account for change in TestHelpers class.

The BoxControllerDummyIO constructor was changed to accept (and hold
internally) a bare pointer to avoid a cyclic reference.
Added description to both algorithms and summary to ViewBOA re #9191

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>
Re #8357. Add a suppressions file for the API tests.

Valgrind is complaining about an OpenMP call in AlgorithmManagerTest.
I don't know whether this is a real leak, but I suppose that there's not
much we can do about it.
Re #8357. Fix memory leaks in test.

Ensure Property objects returned by loadProperty are deleted after use.
Re #8357. Add suppression for another use of OpenMP in a test.
Merge remote-tracking branch 'origin/bugfix/8357_api_test_memory_leaks'
Merge remote-tracking branch 'origin/feature/9191_wiki_documentation'