dough icon indicating copy to clipboard operation
dough copied to clipboard

Add unit tests for dough-data

Open md5sha256 opened this issue 4 years ago • 2 comments

Waiting on #62 to be merged as this includes the tests for CopyUtils.

Whilst the PR has enough tests, there is room for improvement in the quality and (potentially) necessary documentation of the tests.

This PR adds tests for the blocks and collections packages in dough data alongside data.TriStateOptional. ~~That being said, there aren't any tests for the persistence package. ~~

I'm hoping once https://github.com/MockBukkit/MockBukkit/pull/244 is merged I can start working on writing tests for PersistentDataAPI.

I haven't added @Displayname annotations to the tests consistently but there are a few which do have them.

This brings us up to 88% class coverage and 75% lines covered.

md5sha256 avatar Aug 08 '21 10:08 md5sha256

Marking this as stale The repo is no merged to the Slimefun org so it should be reviewed faster in the future.

J3fftw1 avatar Feb 25 '24 16:02 J3fftw1