added mock client and consensus state

[dhruvja]

I have added mock client and consensus state so that we can test it using mocks.

But this would require the mocks and std feature to be used.

[mina86]

FYI,

cargo +nightly fmt

For formatting. Dunno, maybe using nightly formatter features wasn’t the best idea, but for now that’s what we’re using I guess. ;)

[dhruvja]

FYI,

cargo +nightly fmt

For formatting.

haha, was just wondering why cargo fmt wasnt working lol

[mina86]

LGTM, I just wonder, how convoluted it would be to add mocks feature to the code. Adding #[cfg(feature = "mocks")] everywhere is a bit noisy but on the other hand having mock code in production program isn’t the best idea either.

[dhruvja]

LGTM, I just wonder, how convoluted it would be to add mocks feature to the code. Adding #[cfg(feature = "mocks")] everywhere is a bit noisy but on the other hand having mock code in production program isn’t the best idea either.

We could add it as a feature ( dont know how complicated it might be ), but i think we should focus on having a working program first. What do u think?

[mina86]

It shouldn't really be complicated. Just add mocks feature to Cargo.toml and then bunch of conditional compilation in the code. Though I admit that I don’t really understand how programs ids work and that could be an issue especially in the Anchor.toml file I suppose.

But yes, overall LGTM.

[dhruvja]

It shouldn't really be complicated. Just add mocks feature to Cargo.toml and then bunch of conditional compilation in the code. Though I admit that I don’t really understand how programs ids work and that could be an issue especially in the Anchor.toml file I suppose.

But yes, overall LGTM.

Program ids are just identifiers.
I am getting some vague errors atm. Once i fix that i can try out the mocks feature.