-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add unit testing #13
base: master
Are you sure you want to change the base?
Add unit testing #13
Conversation
Busted expects a structure that looks like this:
(any Should we use this instead of the current structure? All this would do is make it that we do |
f6397f0
to
93e8b31
Compare
@truemedian Should be good enough for review! Do note the I changed how you run the tests, now all you need to do is Also frankly, reviewing those tests is probably harder than writing them, may just want to give them a fast glance to assure nothing is severely out of place. |
Add unit testing for most methods. This is still working in progress.
This uses the Busted unit testing framework, running on Lua 5.4 (though it should also run on any Lua version).
The tests are ran from the main tree with
busted -p .-%.lua tests/
. The exact details may change in later commits.Progress: