fix: increment and refactor number of hops for routed messages

[stedfn]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.49%. Comparing base (2e8dcea) to head (a9cbfe1).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #12188      +/-   ##
==========================================
+ Coverage   70.47%   70.49%   +0.02%     
==========================================
  Files         845      845              
  Lines      172255   172251       -4     
  Branches   172255   172251       -4     
==========================================
+ Hits       121389   121422      +33     
+ Misses      45766    45733      -33     
+ Partials     5100     5096       -4     

Flag	Coverage Δ
backward-compatibility	0.16% <0.00%> (+<0.01%)	⬆️
db-migration	0.16% <0.00%> (+<0.01%)	⬆️
genesis-check	1.36% <0.00%> (+<0.01%)	⬆️
linux	69.35% <100.00%> (+0.01%)	⬆️
linux-nightly	70.07% <100.00%> (-0.01%)	⬇️
pytests	1.66% <0.00%> (+<0.01%)	⬆️
sanity-checks	1.47% <0.00%> (+<0.01%)	⬆️
unittests	70.32% <100.00%> (+0.02%)	⬆️
upgradability	0.20% <0.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[saketh-are]

Thanks!

Could be worth adding a couple of basic unit tests covering the 0-hop and 1-hop cases.

[stedfn]

@saketh-are I cleaned it up a bit more and added an assert in an integration test to confirm it gets incremented. Let me know if you are happy with the changes

[saketh-are]

Still LGTM, left a couple of small comments