evoke icon indicating copy to clipboard operation
evoke copied to clipboard

Add unit tests everywhere

Open dascandy opened this issue 7 years ago • 4 comments

dascandy avatar Jun 10 '18 20:06 dascandy

Note to this effect in DESIGN.md.

dascandy avatar Nov 12 '18 09:11 dascandy

Target percentage for this issue is 50% as measured and reported by coveralls (or some other online service), and tested on both Appveyor and Travis.

dascandy avatar Nov 30 '18 20:11 dascandy

Hi, I would be happy to help by adding some more unit tests, if that's fine for you.

Also, in the case in-code documentation is wanted (e.g. // Method A checks this and outputs that, using doxygen and the likes) it would make sense to do it along the way, since writing tests will require understanding the intent of the units. Documentation could be included in a separate issue.

Let me know what you think! :)

PenguinLemma avatar Jun 05 '19 06:06 PenguinLemma

We need some form of documentation, but I have no huge preference for the format. If you think using doxygen comments will generate good documentation, please go right ahead!

dascandy avatar Jun 05 '19 08:06 dascandy