refactor(node): split run from main, as it will be used by many binaries

[matan-starkware]

Also moved build_network_manager function.

---

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 #885
• 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.51852% with 22 lines in your changes missing coverage. Please review.

Project coverage is 74.27%. Comparing base (bdb18d7) to head (7c9e727).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/papyrus_node/src/main.rs	18.51%	21 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #873      +/-   ##
==========================================
- Coverage   74.30%   74.27%   -0.03%     
==========================================
  Files         357      357              
  Lines       36275    36285      +10     
  Branches    36275    36285      +10     
==========================================
- Hits        26953    26951       -2     
- Misses       7184     7195      +11     
- Partials     2138     2139       +1     

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

[dan-starkware]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @matan-starkware)

---

crates/papyrus_node/src/main.rs line 523 at r1 (raw file):

}

pub async fn run(

Where is the call to run?

Code quote:

pub async fn run(

[matan-starkware]

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @dan-starkware)

---

crates/papyrus_node/src/main.rs line 523 at r1 (raw file):

Previously, dan-starkware wrote…

Where is the call to run?

Whoops, that was supposed to be in fn main. It's correct in the next PR so must have messed up when splitting the diffs.

[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 19, 7:54 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.
• Sep 19, 7:55 AM EDT: @matan-starkware merged this pull request with Graphite.