fix: Unable to select None for Alert's log retention

[michael-s-molina]

SUMMARY

There was a bug when changing log retention to None in the alerts modal. This PR fixes this bug.
This PR also creates a new test for this case in a new test file using RTL. Previous tests are written with Enzyme.

@rusackas @junlincc

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-10-28.at.10.17.07.AM.mov

Screen.Recording.2021-10-28.at.11.54.58.AM.mov

TESTING INSTRUCTIONS

Check the videos for instructions.

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #17268 (713ea81) into master (be6cca5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #17268   +/-   ##
=======================================
  Coverage   77.02%   77.03%           
=======================================
  Files        1037     1037           
  Lines       55629    55631    +2     
  Branches     7594     7596    +2     
=======================================
+ Hits        42851    42855    +4     
+ Misses      12528    12526    -2     
  Partials      250      250           

Flag	Coverage Δ
javascript	71.15% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...frontend/src/views/CRUD/alert/AlertReportModal.tsx	61.79% <100.00%> (+0.65%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be6cca5...713ea81. Read the comment docs.

[geido]

LGTM!