-
Notifications
You must be signed in to change notification settings - Fork 356
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 suite work #395
Test suite work #395
Conversation
Panic handling on Windows is unhappy:
|
We can merge without windows working. Or just ignore the tests on windows for now. Once my const panic PR is merged we can copy the logic and do something similar and OS independent |
Ah, good point. I was going to try and add some more shims for these syscalls (or whatever they are), but if this will all change soon anyway, I'll just make it ignore the test. I'd prefer to keep AppVeyor green. |
Fix most compile-fail tests. Some are left ignored, but at least the rest is tested against regressions.
Also move some tests out of fullmir that no longer need it.