refactor: telemetry

[Daanvdplas]

Todo:

• set up integration tests
• Version of pop cli in telemetry payload
• No separation here, so execute the command and get the command executed back. Basically make it work while remove it:

let event = cli.command.to_string();

• If an error occurs, the above should still hold.

[codecov]

Codecov Report

❌ Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.15%. Comparing base (8dd893c) to head (832ee4a).

Files with missing lines	Patch %	Lines
crates/pop-cli/src/main.rs	0.00%	3 Missing ⚠️

@@            Coverage Diff             @@
##             main     #635      +/-   ##
==========================================
- Coverage   78.16%   78.15%   -0.01%     
==========================================
  Files         109      109              
  Lines       25464    25465       +1     
  Branches    25464    25465       +1     
==========================================
  Hits        19903    19903              
- Misses       3481     3482       +1     
  Partials     2080     2080              

Files with missing lines	Coverage Δ
crates/pop-telemetry/src/lib.rs	89.91% <100.00%> (ø)
crates/pop-cli/src/main.rs	87.71% <0.00%> (-0.38%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Daanvdplas]

Tried axum or mockito crate but it was super hard to assert on the payload send to the mockserver. So ended up using wiremock.

[AlexD10S]

Should we get back the variables removed in new?

std::env::remove_var("DO_NOT_TRACK");
std::env::remove_var("CI");

[AlexD10S]

We did this refactor in the tests, once you merge all changes you can take it from: https://github.com/r0gue-io/pop-cli/blob/main/crates/pop-common/src/lib.rs#L94