-
Notifications
You must be signed in to change notification settings - Fork 14
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
Set up automatic testing #8
Comments
There is indeed no systematic test suite; I have tested it via existing irc logs and I used it for the various groups I work with the past few days. When some error occurred, I used the real life irc logs to chase down the problem. Some test irc logs and configuration files are present in the
(Run at the top level, that is) 'rocks' the script in many respect. And there are some other txt files in the same directory that should produce proper results, too. I am not sure how to set up travis, though. |
Good thing there are sample files already. I'll explain to you how to automate this, or I will submit a PR myself, @iherman. |
On 15 Jun 2017, at 10:36, tripu ***@***.*** ***@***.***>> wrote:
Good thing there are sample files already.
I can add, if needed, several full logs a week:-)
I'll explain to you how to automate this, or I will submit a PR myself, @iherman <https://github.com/iherman>.
Thanks!
|
A comprehensive test suite would be ideal, of course.
But for now, we can make do with the simplest test: feed scribejs a minimal, fixed IRC log, and check that there's an output and that the output contains certain expected strings.
Test it both as a CLI tool and as a CGI script, if feasible.
That'd allow us to use Travis CI and have it running this test automatically for each new commit or PR, which is useful to quickly identify critical bugs, of the kind that completely break the tool.
The text was updated successfully, but these errors were encountered: