Looks good to me

We needn't check the pointers here as `loadRun` would have thrown an
exception on failure.

Refs #10472

Refs #10472

Refs #10472

Refs #10472

Refs #10306

Refs #10169

There are some methods we just don't care about, so let's not bother
setting expectations and just provide stub implementations.

* Rename fake workspaces to fake run numbers
* Get rid of superfluous assertions

This new helper function can created fake workspaces to use for
processing in the tests, instead of loading runs like INTER1346{0,2,3,4}
from disk.

It won't play nicely with the coming changes. Autofill will probably
need to be moved to its own action for better testing in future.

Refs #10169

Refs #10169

Refs #10169

Refs #10169

Refs #10347

Conflicts:
	Code/Mantid/MantidQt/CustomInterfaces/src/ReflMainViewPresenter.cpp
	Code/Mantid/MantidQt/CustomInterfaces/test/ReflMainViewPresenterTest.h

It brings the unit test time down to an acceptable place.
Refs #10169

Refs #10169