[YUNIKORN-2844] Inject event recorder externally

[pbacsko]

What is this PR for?

Don't create the event recorder inside GetRecorder() because it has to call IsTestMode(). Afthis this PR, we can remove that method.
Instead, use a mock recorder by default, then inject the real one when YuniKorn starts.

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-2844

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

Attention: Patch coverage is 28.57143% with 10 lines in your changes missing coverage. Please review.

Project coverage is 68.21%. Comparing base (f79d10c) to head (c61c9b4).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
pkg/shim/scheduler.go	0.00%	9 Missing ⚠️
pkg/common/events/recorder.go	80.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #922      +/-   ##
==========================================
+ Coverage   68.19%   68.21%   +0.02%     
==========================================
  Files          70       70              
  Lines        7622     7621       -1     
==========================================
+ Hits         5198     5199       +1     
+ Misses       2215     2214       -1     
+ Partials      209      208       -1     

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

[craigcondit]

+1 LGTM.