celestia-node icon indicating copy to clipboard operation
celestia-node copied to clipboard

headertest: Consider unifying constructors for header generation and adding opts

Open renaynay opened this issue 11 months ago • 2 comments

We currently have a few different ways to construct header test suite for generating headers whereas we could just unify them into a single constructor NewTestSuite and allow variadic amount of opts, like WithTimestampSpacing to set parameters on the test suite.

Lower priority, but would clean things up even further.

Related to https://github.com/celestiaorg/celestia-node/pull/4032

renaynay avatar Jan 10 '25 14:01 renaynay

Good

Hadijp99 avatar Jan 12 '25 19:01 Hadijp99

@renaynay i've made a PR, could you pls check it out when you'll have a free time

crStiv avatar Jul 17 '25 16:07 crStiv