Where are the test cases? #34

Closed
opened 2 months ago by melroy89 · 2 comments

I do not see any unit tests or integration tests on the client library?

https://git.nextgraph.org/NextGraph/nextgraph-rs/src/branch/master/nextgraph

Nor on the server/broker for that matter: https://git.nextgraph.org/NextGraph/nextgraph-rs/src/branch/master/ng-broker/

Please, implement test-cases to avoid regression impact now and in the future. It helps you to create software that is testable, find potential bugs early and fast and prevent regression impact in the future.

Regards,
Melroy van den Berg

I do not see any unit tests or integration tests on the client library? https://git.nextgraph.org/NextGraph/nextgraph-rs/src/branch/master/nextgraph Nor on the server/broker for that matter: https://git.nextgraph.org/NextGraph/nextgraph-rs/src/branch/master/ng-broker/ Please, implement test-cases to avoid regression impact now and in the future. It helps you to create software that is testable, find potential bugs early and fast and prevent regression impact in the future. Regards, Melroy van den Berg
Owner

Hello there,
I am not sure it is appropriate to remind me that unit tests are important.
In any case, each crate contains its own unit tests, as stated in the main README.
For now, the test suite is minimal, as I am in a rush to finish the first MVP (alpha release in early september).
More test cases will be added before the first stable release.
And anyway, you are more than welcome to contribute test cases.
I am closing all your issues as they are not issues.
The issues here are for developers/contributors to organize their work, or to report bugs.
Please use the forum instead if you have more questions.
Thank you

Hello there, I am not sure it is appropriate to remind me that unit tests are important. In any case, each crate contains its own unit tests, as stated in the main README. For now, the test suite is minimal, as I am in a rush to finish the first MVP (alpha release in early september). More test cases will be added before the first stable release. And anyway, you are more than welcome to contribute test cases. I am closing all your issues as they are not issues. The issues here are for developers/contributors to organize their work, or to report bugs. Please use the forum instead if you have more questions. Thank you
niko closed this issue 2 months ago
Poster

It wasn't my intention to point it out to you that way. Sorry if it came across that way.

I was just curious where the test cases were. Luckily you do know about unit tests. So that is great!

I'm interested in how each crate already contains tests tho. Like. Is this local_broker::test::gen_wallet_for_test for example written somewhere? Or is that generated? I'm still kinda new to Rust.

It wasn't my intention to point it out to you that way. Sorry if it came across that way. I was just curious where the test cases were. Luckily you do know about unit tests. So that is great! I'm interested in how each crate already contains tests tho. Like. Is this local_broker::test::gen_wallet_for_test for example written somewhere? Or is that generated? I'm still kinda new to Rust.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: NextGraph/nextgraph-rs#34
Loading…
There is no content yet.