-
Notifications
You must be signed in to change notification settings - Fork 69
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
Write some tests! #11
Comments
👍 I'd love to give some help (if you need it). |
Lovely idea. Any thoughts on the best way to test a command line tool like this? |
If anyone fancies taking a stab at this please feel free. Let me know and I'll assign the case over. Otherwise I will take a look hopefully this evening. |
What about cucumber with aruba? |
Ah, I was wondering whether cucumber would be the right tool. Aruba looks like this is exactly what it was for. I would probably prefer something a bit more unit test style, but I haven't found anything yet. |
I just managed to got cucumber with aruba working on my branch. But I had to do quite some changes in the naming of classes, etc. Have a look at franciscoj/powder@384faa13a90625fdd953 |
I took a stab at writing a test-suite with test/unit. Some refactoring was necessary, and during testing I changed some stuff around. Anyway, was a nice learning experience for me, maybe you can utilize some of my findings. You can find the code here: https://github.com/fschwahn/powder/tree/test |
To prevent regressions, write a friggin test suit!
The text was updated successfully, but these errors were encountered: