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
test/functions/implicit_parameters.mag fails #27
Comments
That failure is really strange. That isn't a part of the codebase that I would expect to have platform-specific behavior. There must be some bug in there. If you get a chance to look into that would be great. Otherwise, I'll have to wait until I have time to try Magpie out on a Windows machine myself. That test runs fine on my Mac. Skipped tests indicate tests that exercise something that's known to not be working right now. For example, there are a bunch of tests that rely on field initializers, which are not implemented in the C++ VM yet. Those tests are skipped. Marking them skip gives us a way to distinguish between new unexpected test failures versus ones that we know aren't working right now. If you run As the implementation matures, those skipped tests should be re-enabled once they can pass. I just pushed a change that associates a reason with each skipped test. Now when you run test.py, you get some feedback about why various tests are skipped. |
Can you file a separate ticket for |
Thanks for the explanation! About the test failure, I can look into it, can you tell me where should I start looking? |
I wrote a little guide here that may help a bit. For this specific issue, I'd try to see where it's actually exiting since that exit code looks like such an odd number. That will help us narrow down what's going wrong. Since this is a platform-specific error, it's likely something either in libuv or where we're using libuv incorrectly. |
I think your last comment is for the other issue. And by my last comment I meant where is the implicit argument handling code :D |
Oh, heh. It's just a parser transform. See here. |
The order of implicit parameters is messed up in some cases (Windows 8 x64 Release build):
(btw, why are some tests skipped?)
The text was updated successfully, but these errors were encountered: