[VIRTS-2717] Operations API Tests

[bleepbop]

Description

Pytest unittests for the v2 Operations API. Note: the DELETE endpoint is currently untested, as there seems to be an underlying aiohttp issue with v3.6.2, causing a failure when running with tox.

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Verified that tests work for various python versions using tox.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

[codecov]

Codecov Report

Merging #2219 (06f155c) into master (b51b51a) will increase coverage by 2.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2219      +/-   ##
==========================================
+ Coverage   68.71%   71.04%   +2.33%     
==========================================
  Files          94       94              
  Lines        7419     7419              
==========================================
+ Hits         5098     5271     +173     
+ Misses       2321     2148     -173     

Impacted Files	Coverage Δ
app/objects/secondclass/c_link.py	87.90% <0.00%> (+0.46%)	⬆️
app/objects/c_operation.py	72.26% <0.00%> (+0.84%)	⬆️
app/api/v2/managers/base_api_manager.py	79.82% <0.00%> (+0.87%)	⬆️
app/contacts/contact_websocket.py	81.48% <0.00%> (+7.40%)	⬆️
app/service/event_svc.py	67.27% <0.00%> (+9.09%)	⬆️
app/api/v2/handlers/operation_api.py	98.52% <0.00%> (+36.76%)	⬆️
app/api/v2/managers/operation_api_manager.py	88.77% <0.00%> (+59.35%)	⬆️

---

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 b51b51a...06f155c. Read the comment docs.

[uruwhy]

Looking great! Couple other addition's that would help with completeness:

• Testing behavior for nonexistent operation IDs / link IDs
• Testing certain edge cases, like trying to update or create a finished operation.

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
1 Security Hotspot
5 Code Smells

No Coverage information
0.0% Duplication

[uruwhy]

Few small changes here and there, but overall looks great!

[uruwhy]

Looks good to me!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
5 Code Smells

No Coverage information
0.0% Duplication