конечно вопросу уже неделя, но все же отвечу :)

в принципе вас не особо должно волновать как реализована функция базового класса. Того, что имеется вполне достаточно.
Выше дело протестировать UpdateMoney()

если ничего не оговорено в спецификации (например про обработку ошибок внутри GetData), то я б поступила сл.образом:

- дописала бы тело функции GetData для разных id (их вы можете сами выбирать), т.о. чтобы протестировать различные нормальные значения, которые может возвращать GetData внутри диапазона типа double.

например, если id = 1, то GetData вернет 1.0 , если id = 0 , то GetData вернет 0.0 и т.д.

- дополнить базовую функцию GetData так, чтобы она возвращала значения приграничные и граничные значения типа double

- на случай, когда GetData не может вернуть информацию по id (нет такого id): тут конечно есть место для маневра. Можете попробовать вернуть какое-нибудь нормальное значение, например тот же 0.0 или 10.0.

- проверяем, что MoneySun не больше -1, а равно тому значению, которое вы ождидаете получить от GetData в случае конкретного id

- дописать тест, если объект ClientInfo не существует (тут у вас должна быть ошибка)