refactor(node): spawned tasks should all return anyhow result

[matan-starkware]

This fairly generic return type should allow for flexibility with test overrides.

---

This change is 

[matan-starkware]

• refactor(node): remove test code from production code #876 : 2 dependent PRs (#877 , #880 )
• refactor(node): move main to its own file and keep run infrastructure reusable #874
• refactor(node): split run from main, as it will be used by many binaries #873
• refactor(node): introduce PapyrusTaskHandles to hold task overrides #872
• refactor(node): refactor how monitoring server is spawned #871
• refactor(node): spawned tasks should all return anyhow result #870 👈
• refactor(node): miscellaneous clippy suggestions #867
• refactor(node): create papyrus resources for generating top level tasks #865
• refactor(node): network channel creation is done by users #863
• refactor(node): move building the p2p sync server to its own function #862
• refactor(node): move building the sync client to its own function #861
• refactor(node): convert storage metrics handle to the desired task type #860
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 18.91892% with 30 lines in your changes missing coverage. Please review.

Project coverage is 74.31%. Comparing base (c36ed0d) to head (98a4868).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/papyrus_node/src/main.rs	18.91%	27 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #870      +/-   ##
==========================================
- Coverage   74.33%   74.31%   -0.02%     
==========================================
  Files         357      357              
  Lines       36240    36255      +15     
  Branches    36240    36255      +15     
==========================================
+ Hits        26940    26944       +4     
- Misses       7169     7179      +10     
- Partials     2131     2132       +1     

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

[dan-starkware]

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @matan-starkware)

[matan-starkware]

Merge activity

• Sep 19, 6:16 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.
• Sep 19, 6:33 AM EDT: @matan-starkware merged this pull request with Graphite.