-
Notifications
You must be signed in to change notification settings - Fork 7
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
Needs tests #4
Comments
I think the first test should be |
You mean the only test |
You can also do |
not to mention battalions of functional test cases if an assertion fails in a functional test case it might even uncover a bug in the upstream implementation, maybe, although there should definitely be a fine-toothed review combing over the entire codebase locally before assuming something like that |
that would be crazy, import a bunch of known failing func test cases from phantom/casper/blink to make variable assignment is so unreliable, guys, just look at these test scores |
hahaha we're already getting these: #7 |
Fixed by #10 |
Requires #3 and a .travis.yml
The text was updated successfully, but these errors were encountered: