-
Notifications
You must be signed in to change notification settings - Fork 82
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
Uncaught exception in shader execution tests: #3157
Comments
What device are you seeing this on and how are you running the tests in order to provoke it? (In browser, node, etc?) I'm having trouble repro-ing. |
Thanks for looking. I was running with the There's a bunch of changes that I'm going to push for review (likely Monday), which simplifies the usage of this tool. I'd recommend waiting until that's landed, and then to repro should be a one-liner. |
Thanks! I think I see what the problem is: Prior to my refactor a pipeline creation failure would have simply returned an invalid pipeline and the test would catch the unexpected validation failure at the end, but now those errors are only surfaced via promise rejection. I likely just need to catch that and explicitly fail the test when it happens, but I'd like to know that I can repro the problem before I land that as a fix. I'll sketch out the code I think will resolve it and wait till Monday-ish to test it against your changes. |
The text was updated successfully, but these errors were encountered: