-
Notifications
You must be signed in to change notification settings - Fork 570
'gopherjs test' always fails tests #624
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
Comments
Is that the only test you have, or are there more? It should definitely work. I'm guessing you don't have system call module installed, do you? https://github.com/gopherjs/gopherjs/blob/master/doc/syscalls.md It's possible something we changed more recently has made it required for tests to run, or perhaps it's always been that way. I don't know, because I always have it installed, and |
For reference, this is what my output is:
(I'm not sure why it's printing the full path instead of import paths though...) |
Correct, just that test. No more.
Ok, I gave that a shot and it fixed it. My output is now: $ gopherjs test
gopherjs: Source maps disabled. Use Node.js 4.x with source-map-support module for nice stack traces.
PASS
ok github.com/gopherjs/vecty/example 0.326s
$ gopherjs test -v
gopherjs: Source maps disabled. Use Node.js 4.x with source-map-support module for nice stack traces.
=== RUN TestExample
--- PASS: TestExample (0.00s)
PASS
ok github.com/gopherjs/vecty/example 0.334s I don't know why I still get the Node.js 4.x warning when my I'm almost certain |
We should document this new requirement. It might be worth it to investigate if it's avoidable. The new requirement might be due to recent enhancements, and so it may be unavoidable. Filed #626 for that.
Well, that's a fail on the warning message text. Filed #627 to improve the text. The issue is you need source-map-support module installed. Here, I've highlighted the important part you're missing:
See the 3rd paragraph of https://github.com/gopherjs/gopherjs/#installation-and-usage for installation instructions of |
sgtm |
With a simple test like:
It will always fail when ran via
gopherjs test
:I'm wondering if this is a known issue / people are not using
gopherjs test
, or if it's potentially something configured incorrectly on my system.gopherjs
version 3496c6fnode --version
reportsv7.9.0
(the latest available viabrew
)The text was updated successfully, but these errors were encountered: