Page MenuHomePhabricator

[avalanche] Check that avalanche invalidated items no longer poll in the dont_poll_invalid_item test case
ClosedPublic

Authored by roqqit on Fri, Jan 24, 20:07.

Details

Summary

Technically the case of avalanche-triggered invalidation is already tested but it makes sense to re-test that behavior here to ensure avalanche invalidation is treated similarly as items being made invalid for other reasons.

This patch also improves the test coverage to actually check that the item is no longer being polled, not just that unexpected responses are handled correctly.

Depends on D17589

Test Plan
ninja check-avalanche-processor_tests

Diff Detail

Event Timeline

roqqit requested review of this revision.Fri, Jan 24, 20:07
Fabien requested changes to this revision.Sat, Jan 25, 15:09
Fabien added a subscriber: Fabien.

This is not testing what you think, as it's not testing the node stops polling the invalidated item. It checks that it rejects a unexpected vote, which is the correct behavior but will happen whatever the vote status of the item is.

This revision now requires changes to proceed.Sat, Jan 25, 15:09

Add comments to make it more clear that the test is checking the item is not being polled for

This revision is now accepted and ready to land.Wed, Jan 29, 19:58