babylonchain / babylon-finality-gadget

2 stars 2 forks source link

Refactor: decouple config, sdk, and client #43

Closed bap2pecs closed 1 month ago

bap2pecs commented 1 month ago

context

https://github.com/babylonchain/babylon-da-sdk/pull/38#pullrequestreview-2155195317

LGTM! In general, I think we can start to refactor the project a bit in terms of architecture in subsequent prs. For example, config is now mixed with client. Many structs are mixed with sdk along with client methods

https://github.com/babylonchain/babylon-da-sdk/pull/45#discussion_r1666279613

Suggest we formalise the mock procedure for unit tests, like what we did in finality provider

tasks

bap2pecs commented 1 month ago

let's also rewrite all tests to not depend on anything external. so we will decouple from:

right now make test and make run are both broken. but since we are already covering this SDK in the e2e test in FP repo, I suggest we only do mocked unit test here

cc @lesterli @parketh wdyt? (example: #45)