Add more metric integration tests

[cijothomas]

Add few more tests to metrics.
Tokio::Current is failing, will investigate that separately. -- This is fixed.#2583 (comment)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.5%. Comparing base (a1860eb) to head (7e66a54).
Report is 2 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #2582   +/-   ##
=====================================
  Coverage   79.5%   79.5%           
=====================================
  Files        118     118           
  Lines      22490   22490           
=====================================
  Hits       17888   17888           
  Misses      4602    4602           

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

[cijothomas]

@lalitb Did some refactoring. In a follow up, I plan to split the metrics test into two - one testing the full payload, and another one just checking presence of the guid. The latter one is where I plan to add all testing permutations (tokio vs other vs etc...)

[lalitb]

@cijothomas Seems tokio current test is still failing. If so, should we keep it ignored, and add as separate PR., to unblock this?

[cijothomas]

@cijothomas Seems tokio current test is still failing. If so, should we keep it ignored, and add as separate PR., to unblock this?

It is expected to pass. There seems to be occasional instability in the test, not related to tokio::current, but the general tension of running things in parallel. We'll need to make it more robust in future.

[lalitb]

Thanks.