-
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 documentation for tests #5
Comments
Would you be open to changing up how they're formatted? I think I see how they're working, but having them in interpreted log files makes debugging a bit harder. Happy to take the work on and document as I go |
The reason of how the files are formatted is because it is super easy to create these test cases.
@skybloo What would you change to make them better? What makes debugging harder; is it the lack of being able to set breakpoints? Is there something I can do to make debugging with the log replay easier? I made it so that if a chat replay message is wrong, that both the test log location and the code location where the message was emitted is shown: |
|
@skybloo with the merge of #22 into master, I moved the code for setting up a test case into https://github.com/ToransuShoujo/quesoqueue_plus/blob/master/tests/simulation.js. |
This is something I plan on doing eventually:
The text was updated successfully, but these errors were encountered: