fix:sealing:check index out of bounds against correct param length not return length

[ZenGround0]

Related Issues

Proposed Changes

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

• All commits have a clear commit message.
• The PR title is in the form of of <PR type>: <area>: <change being made>
  • example: fix: mempool: Introduce a cache for valid signatures
  • PR type: fix, feat, INTERFACE BREAKING CHANGE, CONSENSUS BREAKING, build, chore, ci, docs,perf, refactor, revert, style, test
  • area: api, chain, state, vm, data transfer, market, mempool, message, block production, multisig, networking, paychan, proving, sealing, wallet, deps
• This PR has tests for new functionality or change in behaviour
• If new user-facing features are introduced, clear usage guidelines and / or documentation updates should be included in https://lotus.filecoin.io or Discussion Tutorials.
• CI is green

[magik6k]

needs make gen; The failing test may have a good reason to be failing

=== RUN   TestGetCurrentDealInfo/deal_lookup_succeeds_when_theres_a_separate_deal_failure
nv: 14
unserialized deal ids: [8 10]
    currentdealinfo_test.go:295: 
        	Error Trace:	currentdealinfo_test.go:295
        	Error:      	Not equal: 
        	            	expected: 0xa
        	            	actual  : 0x0
        	Test:       	TestGetCurrentDealInfo/deal_lookup_succeeds_when_theres_a_separate_deal_failure
    --- FAIL: TestGetCurrentDealInfo/deal_lookup_succeeds_when_theres_a_separate_deal_failure (0.00s)

[ZenGround0]

Test failure is because tests use this to sort deals which doesn't work which ruins order invariants which causes flake.

[codecov]

Codecov Report

Merging #8475 (5bdcc7d) into master (aca9496) will decrease coverage by 0.05%.
The diff coverage is 53.48%.

@@            Coverage Diff             @@
##           master    #8475      +/-   ##
==========================================
- Coverage   40.77%   40.72%   -0.06%     
==========================================
  Files         686      686              
  Lines       75495    75524      +29     
==========================================
- Hits        30782    30755      -27     
- Misses      39404    39441      +37     
- Partials     5309     5328      +19     

Impacted Files	Coverage Δ
chain/actors/builtin/market/market.go	66.98% <ø> (ø)
chain/actors/builtin/market/v5.go	16.12% <0.00%> (ø)
chain/actors/builtin/market/v7.go	45.98% <46.66%> (+0.82%)	⬆️
markets/storageadapter/client.go	40.55% <50.00%> (ø)
chain/actors/builtin/market/v6.go	23.35% <60.00%> (+4.80%)	⬆️
extern/storage-sealing/currentdealinfo.go	70.17% <71.42%> (-4.60%)	⬇️
chain/events/state/fastapi.go	75.00% <0.00%> (-25.00%)	⬇️
chain/actors/builtin/miner/diff.go	52.94% <0.00%> (-5.89%)	⬇️
chain/types/tipset_key.go	85.96% <0.00%> (-3.51%)	⬇️
... and 21 more