[YUNIKORN-2971] Use FakeRecorder and MockedRecorder properly in the tests

[pbacsko]

What is this PR for?

Remove FakeRecorder type check from the unit tests where it's present.
Also make sure that we always restore the default MockedRecorder in a deferred call wherever it's needed.

What type of PR is it?

• - Bug Fix
• - Improvement
• - Feature
• - Documentation
• - Hot Fix
• - Refactoring

Todos

• - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-2971

How should this be tested?

Screenshots (if appropriate)

Questions:

• - The licenses files need update.
• - There is breaking changes for older versions.
• - It needs documentation.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.03%. Comparing base (8fa300f) to head (1b81428).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #937   +/-   ##
=======================================
  Coverage   68.03%   68.03%           
=======================================
  Files          70       70           
  Lines        9194     9194           
=======================================
  Hits         6255     6255           
  Misses       2732     2732           
  Partials      207      207           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[craigcondit]

+1 LGTM.