TestMain needs an explanation as to why it is needed.

celestiaorg / knuu

Integration Test Framework

Apache License 2.0

39 stars 34 forks source link

TestMain needs an explanation as to why it is needed. #395

Closed MSevey closed 2 months ago

MSevey commented 4 months ago

In the README for writing tests it isn't clear why TestMain is needed. Is it always needed? Can you not just have the test example?

We should add a comment to the test explaining exactly why it is needed in addition to the other test.
Any context or test dependency should be explained clearly.

smuu commented 2 months ago

not needed, as we will not use TestMain anymore