-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
Tests are out of sync with Node.js tests #177
Comments
I'm not a maintainer, but I'll say "go for it" anyway 😀 |
This commit uses the `test-buffer-alloc.js` from upstream Node.js as of 2018-01-20. This is part of feross#177.
This commit uses the `test-buffer-alloc.js` from upstream Node.js as of 2018-01-20. This is part of feross#177.
Merged @vmx's PR and the tests are closer to node's now, but there's still a lot of differences. |
I just spent like 3 hours merging in more changes from Node.js, but there's still more changes to be done... wow. Would appreciate help from other folks, if there's interest. @vmx, any interest? I've started adding the new test files that were added. Once they're all added in, then we can delete EDIT: Here's the changes I've made (committed to master already): e0adc7a...f5aae10 |
Thanks for the review @feross. I still have interest and would spend some time next week. I'll start from the top (alphabetically) and work on the differences. I'll push again in small increments to prevent double efforts. |
Great, looking forward to a PR next week, @vmx! |
Note to self: When all these tests and code changes are merged in, publish a new major version. |
Hey all. It's been some time since this issue was opened and buffer was updated. Has the herd moved to another implementation/fork? Or is there still a roadmap for this one to be actively developed? Just curious. |
@superguineapig When I was talking with @feross he said he's happy to see those fixes. So my currnet workflow is doing some of those fixes, waiting for a review (wow, it's been a long time for #190 being open) and merge and then doing some more whenever I find time. |
Thanks for the info @vmx Glad to hear the project is still alive. |
I'm in grad school as of the past year, so I was off GitHub for a while. Still around and still plan to maintain this as good as ever. :) |
When running
./bin/download-node-tests.js
it's apparent that the tests are heavily out of sync with the Node tests.I plan to go over them an resync things. I open this issue to prevent double work, so a "go for it" from the maintainers would be appreciated.
The text was updated successfully, but these errors were encountered: