Fixes #1536: Removed unnecessary check for n >= 0

[ganeshmurthy]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.5%. Comparing base (1b1d4b9) to head (033d808).
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##            main   #1537      +/-   ##
========================================
+ Coverage   63.0%   77.5%   +14.5%     
========================================
  Files        214      65     -149     
  Lines      51857   16389   -35468     
  Branches    5986       0    -5986     
========================================
- Hits       32681   12705   -19976     
+ Misses     16966    3684   -13282     
+ Partials    2210       0    -2210     

Flag	Coverage Δ
pysystemtests	77.5% <ø> (-0.8%)	⬇️
systemtests	?

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

Components	Coverage Δ
unittests	77.5% <31.5%> (+14.5%)	⬆️
systemtests	77.5% <31.5%> (-0.8%)	⬇️

[jiridanek]

Took a look at the failing CI and reported

• KeyError: 'owningAddr' in link['owningAddr'].endswith(service_address), links) #1538

[jiridanek]

superseded by

• Fixes #1536: Replace unnecessary check for n >= 0 with assertion that n is unsigned long #1539