lf-lang / rust-rti

Other
3 stars 1 forks source link

Adding basic unit tests #24

Closed hokeun closed 7 months ago

chanijjani commented 9 months ago

It is related to https://github.com/hokeun/lf-rust-rti/issues/13.

Suggestion: How about completing easier unit tests of sources (federate_info.rs / lib.rs / net_common.rs / rti_remote.rs / tag.rs) in this PR?

hokeun commented 9 months ago

It is related to #13.

Suggestion: How about completing easier unit tests of sources (federate_info.rs / lib.rs / net_common.rs / rti_remote.rs / tag.rs) in this PR?

Sounds good to me! Let's merge this PR cnce we're done with these .rs files,

chanijjani commented 9 months ago

image html_20240207.zip

chanijjani commented 9 months ago

@the-systematic-chaos Are you implementing and uploading the remaining unit test cases (TCs) of tag.rs within a few days? Before I add commits for unit TCs of sources from up-to-down directions in the above code coverage graph soon, I'll wait for you to finish your first commit.

the-systematic-chaos commented 9 months ago

@chanijjani

Hi Dr. Lee, yes I think I will be able to get it done in a few days. I am progressing with the test cases.

chanijjani commented 8 months ago

image

chanijjani commented 8 months ago

image

chanijjani commented 8 months ago

It is necessary to rebase the main branch to enable CI After https://github.com/hokeun/lf-rust-rti/pull/42. --> [Update] rebased!

chanijjani commented 8 months ago

@the-systematic-chaos Great to update unit tests of tag.rs! Let me submit a PR to resolve current conflicts in some files.