1. Refactored and thus fixed some issues;
2. Added  more unit tests;
3. Change input property. Algorithm can work with 1 spectrum only.

Looks like this is due to lack of language support against 2.7.

rename the run function and always have it return the OutputWorkspace value. This saves lots of lines of code and makes it clearer.

This prevents orphaned workspaces from outliving the algorithm.

This is done by getting the property specifying the file like in other similar unit tests.

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

The cppcheck warning was a false positive as the varaible wasn't allocated before.

I think the reason cppcheck didn't like it is because i allocate it in a function.

Refs #8940

Refs #8940

The scientits said they are interested in some of the temporary workspaces
needed for the reduction.