refactor(node): move building the sync client to its own function

[matan-starkware]

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 59.72222% with 29 lines in your changes missing coverage. Please review.

Project coverage is 74.49%. Comparing base (fa8057c) to head (044d791).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/papyrus_node/src/main.rs	59.72%	29 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #861      +/-   ##
==========================================
+ Coverage   74.42%   74.49%   +0.07%     
==========================================
  Files         366      366              
  Lines       38550    38555       +5     
  Branches    38550    38555       +5     
==========================================
+ Hits        28690    28723      +33     
+ Misses       7534     7508      -26     
+ Partials     2326     2324       -2     

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

[dan-starkware]

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

[matan-starkware]

Merge activity

• Sep 18, 10:03 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.
• Sep 18, 10:03 AM EDT: @matan-starkware merged this pull request with Graphite.