-
Notifications
You must be signed in to change notification settings - Fork 63
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
unit test runner is failing #312
Comments
Hi @mgstauffer , this might be caused by your local version of unit_test_runner.sh having windows (CLRF) line endings. Try running dos2unix first and see if you get the same error ( If you run the unit tests using |
@hjwilli Interesting! After converting to unix line endings, I'm still getting the I looked in |
Seems like it doesn't see the
This is likely what happened. You shouldn't need to re-commit. |
separate scripts for python and javascript unit test runners python unit test runner no longer uses relative paths ref #312
Hi @mgstauffer . Made some updates related to this in PR #314 , which has been merged to master. You can now use |
Thanks! Updating now
…On Fri, Feb 19, 2021 at 5:13 PM hjwilli ***@***.***> wrote:
Hi @mgstauffer <https://github.com/mgstauffer> . Made some updates
related to this in PR #314
<#314> , which has been merged
to master. You can now use /tests/unit/js_tests.sh to run just the
javascript unit tests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#312 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAC5A6YU4LFEUDZBQMPTKQDS73O65ANCNFSM4XFTWLNQ>
.
|
resolved! |
In current master branch. Attaching a shell to a running docker container and running the unit test runner generates these errors:
The text was updated successfully, but these errors were encountered: