Add missing action logging to api/v1/admin/reports#update

[mjankowski]

A few things here:

• Add coverage for the previous actions in api/v1/admin/reports which were creating logs, to verify they are created
• Add coverage for the non-api admin/reports controller along same line
• Add (initially failing) coverage around the update action in api/v1/admin/reports
• Add the log_action line to make that last spec pass

@ThisIsMissEm - is that what you were looking at?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1726085) 85.00% compared to head (f8511eb) 85.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #29044   +/-   ##
=======================================
  Coverage   85.00%   85.00%           
=======================================
  Files        1058     1058           
  Lines       28281    28282    +1     
  Branches     4538     4538           
=======================================
+ Hits        24040    24041    +1     
  Misses       3078     3078           
  Partials     1163     1163           

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

[ThisIsMissEm]

Yes! This looks good to me. Whilst it won't capture "what" changed, at least it'll capture that a certain user made that change, and when.