-
Notifications
You must be signed in to change notification settings - Fork 15
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
[Refactor] Clean up Tests #2
Comments
I've made enough progress here that I feel comfortable to remove this from the "Initial Release" milestone. Issue will remain open as there is still lots of room for improving the legacy tests. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The tests currently use a lot of parameterized tests which makes them difficult to read. We should refactor them and consider using BDD tests as throughout most of the PlanGrid codebase (though this means taking on Quick+Nimble as development and CI dependency).
The text was updated successfully, but these errors were encountered: