switch to Sub::Override from Test::MockModule

This is for consistency as Sub::Override is used for another test and
also so we minimise the number of modules installed just for testing