Refactor ActionManagementAuditLogger

[ashanthamara]

Proposed changes in this pull request

1. Remove printAuditLog(Operation operation, String actionId,ActionDTO actionDTO) overloaded method as we can utilize printAuditLog(Operation operation, ActionDTO actionDTO) for all usecases.
2. Improve tests for ActionManagementAuditLogger for both Add and Update operations

Related Issue

• Internal Server Error when creating an Action with Audit logs enabled product-is#21931

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 41.03%. Comparing base (3b6ada1) to head (c39855e).
Report is 14 commits behind head on master.

Files with missing lines	Patch %	Lines
...n/management/util/ActionManagementAuditLogger.java	90.90%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #6180   +/-   ##
=========================================
  Coverage     41.02%   41.03%           
- Complexity    15391    15398    +7     
=========================================
  Files          1796     1796           
  Lines        123037   122999   -38     
  Branches      21408    21403    -5     
=========================================
- Hits          50480    50475    -5     
+ Misses        65058    65034   -24     
+ Partials       7499     7490    -9     

Flag	Coverage Δ
unit	27.67% <91.66%> (+<0.01%)	⬆️

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

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

[ashanthamara]

Not running the product-is PR builder as the changes in this PR are related to audit logs. Since we have not enabledV2audit logs in the integration tests, these changes won't be tested in the PR builder it self.

We have covered all the uses cases for the audit logs using the unit tests.
Hence merging the PR without running the pr builder.