[avalanche] Cleanup unnecessary setArgs in compute_staking_rewards test
ClosedPublic
Actions

Authored by roqqit on Mon, Nov 18, 18:28.

Details

Reviewers

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rABC59076f7b8f83: [avalanche] Cleanup unnecessary setArgs in compute_staking_rewards test

Summary

These args are already set to 0 in the fixture. I could have set these to be useful values, but that test coverage is already handled in other tests so it is unnecessarily verbose.

Test Plan

ninja check-avalanche-processor_tests

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

roqqit created this revision.Mon, Nov 18, 18:28

Owners added a reviewer: Restricted Owners Package.Mon, Nov 18, 18:28

Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Nov 18, 18:28

roqqit requested review of this revision.Mon, Nov 18, 18:28

Harbormaster completed remote builds in B31195: Diff 50940.Mon, Nov 18, 18:50

Fabien accepted this revision.Mon, Nov 18, 19:31

This revision is now accepted and ready to land.Mon, Nov 18, 19:31

Closed by commit rABC59076f7b8f83: [avalanche] Cleanup unnecessary setArgs in compute_staking_rewards test (authored by roqqit). · Explain WhyMon, Nov 18, 21:46

This revision was automatically updated to reflect the committed changes.

roqqit added a commit: rABC59076f7b8f83: [avalanche] Cleanup unnecessary setArgs in compute_staking_rewards test.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

processor_tests.cpp

4 lines

Diff 50948

View Options

src/avalanche/test/processor_tests.cpp

[avalanche] Cleanup unnecessary setArgs in compute_staking_rewards testClosedPublicActions